hi

first off thanks for this great module it's very handy

just a quick note, the latest version doesn't correctly display pages using the paging module to split large articles into multiple viewable pages. It just shows the current page that you were on. The previous release works fine for this, so I reverted back to it.

thank you

Comments

jcnventura’s picture

jcnventura
Primary language English
commented
Status: Active » Fixed

Thanks for the report.

A user recently complained (http://drupal.org/node/213280) that it wasn't working for him, so I changed the name of one variable. It seems that the dev version of the paging module is using a new variable name. I have added both variable names to the code now so that the print module works properly with both the dev and official releases of paging.

This fix is now included in the latest dev release.

João

--- print.module        25 Jan 2008 11:27:35 -0000      1.15.2.5.2.29.2.14
+++ print.module        28 Jan 2008 16:40:00 -0000
@@ -704,6 +704,7 @@
   $node->printing = true;
   // Turn off Pagination by the Paging module
   unset($node->pages);
+  unset($node->pages_count);
   unset($node->page_count);

   unset($node->teaser);
Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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