"Print view include" option not displaying side content in printer-friendly page

Jim Kirkpatrick - April 26, 2009 - 22:39
Project:Side Content
Version:6.x-1.0-beta1
Component:User interface
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

Hi,

I'm using D6 with the print module and have so far been unable to get the side content to appear on the print-friendly page as advertised.

Checking the "Print view include" option, [which says: "Show content in print view (requires Print Friendly Pages module)."] in the block's configuration page does nothing - going to the print version of a page with side content shows only the normal content, not the relevant side content.

#1

Jim Kirkpatrick - April 26, 2009 - 22:47
Status:active» needs review

Ok, after some digging I worked out that there's an error on line 246 whereby the wrong argument is checked:

<?php
 
if (arg(2) == 'print' && !empty($node->sidecontent)) { 
?>

That arg(2) should be arg(0):
<?php
 
if (arg(0) == 'print' && !empty($node->sidecontent)) {
?>

Once the above is changed the print page shows the side content correctly.

#2

Robert Castelo - May 24, 2009 - 14:47
Status:needs review» fixed

Thanks for spotting that.

Fixed in 6.x-2.0-RC1

#3

System Message - June 7, 2009 - 14:50
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.