render_cache_node_node_show() is said to be replacing node_show(), but actually replaces node_page_view() the page callback from node_menu().
And it's not fully compatible since node_page_view() has the following code:
// If there is a menu link to this node, the link becomes the last part
// of the active trail, and the link name becomes the page title.
// Thus, we must explicitly set the page title to be the node title.
drupal_set_title($node->title);
$uri = entity_uri('node', $node);
// Set the node path as the canonical URL to prevent duplicate content.
drupal_add_html_head_link(array('rel' => 'canonical', 'href' => url($uri['path'], $uri['options'])), TRUE);
// Set the non-aliased path as a default shortlink.
drupal_add_html_head_link(array('rel' => 'shortlink', 'href' => url($uri['path'], array_merge($uri['options'], array('alias' => TRUE)))), TRUE);
return node_show($node);
and the page title is not set.
Comments
Comment #1
mfernea commentedHere is the patch for the 7.x-2.x branch.
Comment #2
mfernea commentedHere is the patch for the 7.x-1.x branch.
Comment #3
mfernea commentedHere is the correct patch for 7.x-1.x branch.
Comment #4
fabianx commentedWhoops, that is a definite oversight from the original code base. Thanks for the patches!
Comment #5
pribeh commentedTested #3 and it works!