Uploaded attachments not showing with paged nodes [#180845]

The problem is that paging is overwriting the $node->body variable on lines 136-137 (_paging_nodeapi function):

          $node->body = check_markup($node->pages[$page], $node->format, false);
          $node->body .= theme('pager', NULL, 1, $element);

I changed the lines to:

          $node->content['body']['#value'] = check_markup($node->pages[$page], $node->format, false);
          $node->content['body']['#value'] .= theme('pager', NULL, 1, $element);

and the attachments show. Don't know if this can cause other problems though...

Comment	File	Size	Author
#2	paging.180845_1.patch	750 bytes	baldwinlouie
#1	paging.180845.patch	634 bytes	baldwinlouie

Comments

Comment #1

baldwinlouie commented 6 March 2008 at 03:42

Status:

Active

» Needs review

Status	File	Size
new	paging.180845.patch	634 bytes

I've been running this change in production for a while now. It shouldn't cause any problems. I took your changes above and made a patch for it.

Comment #2

baldwinlouie commented 13 March 2008 at 19:39

Status	File	Size
new	paging.180845_1.patch	750 bytes

I made a slight tweak to the patch. Instead of concatenating the pager into the body, I put it in its own array element. This way, it will be easier to override.