Closed (fixed)
Project:
Printer, email and PDF versions
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
27 Jan 2009 at 16:38 UTC
Updated:
16 Feb 2009 at 21:40 UTC
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.
| Comment | File | Size | Author |
|---|---|---|---|
| print.patch | 2.04 KB | pletcher |
Comments
Comment #1
jcnventuraThanks 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