Just a patch to remove notices on new pages. I assumed the empty string ('') would be a preferable default here, but I wasn't really sure.

CommentFileSizeAuthor
print.patch2.04 KBpletcher
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jcnventura’s picture

Status: Active » Fixed

Thanks for the report. It was even worse than simple notices, as the fact that those variables weren't initialized meant that the per-node settings are set to 0, thus making it the default not to show the PF links.

Anyway, I have just committed some code similar to yours to fix this issue and it will be in the next dev in a couple of hours.
Just for you to know, the empty string is not the best one as it doesn't fix the above problem. The correct fix is to set it to the default setting for those fields.

João

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.