Issue #1410260 by swentel, mojzis, droplet, pingers, drupdan3: Fixed rss.xml/whatever...
authorwebchick
Thu, 20 Sep 2012 06:33:35 +0000 (23:33 -0700)
committerwebchick
Thu, 20 Sep 2012 06:33:35 +0000 (23:33 -0700)
modules/node/node.module
modules/node/node.test

index 65a4224..d3d1c85 100644 (file)
@@ -1984,6 +1984,9 @@ function node_menu() {
     'page callback' => 'node_feed',
     'access arguments' => array('access content'),
     'type' => MENU_CALLBACK,
+    // Pass a FALSE and array argument to ensure that additional path components
+    // are not passed to node_feed().
+    'page arguments' => array(FALSE, array()),
   );
   // @todo Remove this loop when we have a 'description callback' property.
   // Reset internal static cache of _node_types_build(), forces to rebuild the
index 7080ce7..dc21daf 100644 (file)
@@ -832,7 +832,11 @@ class NodeRSSContentTestCase extends DrupalWebTestCase {
     // viewing node.
     $this->drupalGet("node/$node->nid");
     $this->assertNoText($rss_only_content, t('Node content designed for RSS doesn\'t appear when viewing node.'));
-
+    
+    // Check that the node feed page does not try to interpret additional path
+    // components as arguments for node_feed() and returns default content.
+    $this->drupalGet('rss.xml/' . $this->randomName() . '/' . $this->randomName());
+    $this->assertText($rss_only_content, t('Ignore page arguments when delivering rss.xml.'));
   }
 }