To have the links in the body i have changed this

Line 149:
  
  if ($node->links) {
    $body .= "<hr /><div style=\"text-align: right;\">[ " . theme("links", $node->links) . " ]</div>";
  }

  in:
  
  if ($links = link_node($node, $main)) {
    $body .= "<hr /><div style=\"text-align: right;\">[ " . theme("links", array_merge($submitted, $links)) . " ]</div>";
  }

i do not now if this is also right for version 4.6

MM

Comments

TDobes’s picture

TDobes commented

I'm not sure where the $submitted is coming from that you are merging $links with. There is no $submitted anywhere in goofy.theme.

The use of link_node is for 4.5 and earlier. $node->links is used in 4.6 and later. Read the updating 4.5 themes to HEAD (soon to be 4.6) for more details. (the "Node links" section)

In short, the change you made is necessary if you want to backport the 4.6 theme to 4.5. However, it will break the theme in 4.6. Also, the "array_merge($submitted, $links)" can be simplified to just "$links" unless you've made other changes.

Mad Maks’s picture

Mad Maks commented

Ok i allready tought so that is was changed in 4.6 (my site with the goofy theme can not be upgrade to 4.6 because of the php version :-( ).

Also, the "array_merge($submitted, $links)" can be simplified to just "$links" unless you've made other changes.

you are right, it was a left over from something i tried weeks ago. thanks

MM

TDobes’s picture

TDobes commented

FYI: For backporting the 4.6 theme to 4.5, you'll probably also want to reverse this change. (You'll want to add the lines prefixed with a - and remove the line prefixed with a +.)

In 4.5, modules can pass a title and breadcrumb to the theme_page function, but in 4.6 this was moved to the drupal_set_breadcrumb/title functions.