update to 5.0

Can this be put into the HEAD so it can be downloaded as a 5.0 module?

Thanks,
Maria
(also commented on http://drupal.org/node/106261)

subscribing.

Here's another iteration of the patch. I adjusted the capitalization as per http://drupal.org/node/64279#sentence_capitalization

Otherwise, it functions well.

+1 for DRUPAL5 branch 5.x-1.x-dev release :)

I have successfully patched 4.7 version of print module with print_4_0.patch above into my Drupal 5 and can confirm that it works.

I guess new release is imminent.
Thanks!

Who maintains this module? It'd be nice to get it released officially for drupal 5.

I've just applied patch_4 to the last 4.7.x version of print.module.

some minor bugs?:

1. The patch should remove the closing PHP tag ?> from the last line (385) of the 5.x print.module ... after I did this it stopped giving me errors like this:
   

   warning: Cannot modify header information - headers already sent by (output started at /home/username/public_html/drupal5/sites/all/modules/print/print.module:385) in /home/username/public_html/drupal5/includes/common.inc on line 141.
   

2. Couldn't find the print module settings in the /admin/by-module page. (missing a D5 hook?)
3. 'Source URL' links are corrupted : eg http://example.com/drupal5/drupal5/node/4 [= Drupal Page Not Found ]
   It may have something to do with my using the 'drupal5/sites/all/modules' directory rather than the more common 'drupal5/modules' ?
   It may have to do with other $base_url issues with Drupal that I'm now becoming dimly aware of ... ?

     /* Grab and format the src URL */
     $node->source_url = $base_url . url("node/$node->nid");
   ...
     $source_url = $base_url . substr($_GET['q'], 5);
   ...
   

Otherwise it seems to be working well enough.

Any feedback / suggestions welcome :)