Fixed? : open_basedir error under php5(?) in print.module line 260

I believe I may have fixed my first bit of Drupal here! :)

Referring to a problem I had (reported identically here) with print.module breaking under php5 (solely? not sure... worked ok as is, on a php4 install not long ago)with an open_basedir error, and a blank page upon clicking "print friendly page" on a node, I did a bit (read: ages) of research and found this about urls.

Hence, I changed line 260 of print.module (v4.7.x-1.x, 13/11/06) from:

include_once('print.node.tpl.php');

to:

include_once(drupal_get_path('module', 'print') .'/print.node.tpl.php');

and this seems to work! Hopefully as I test this further it won't break anything...! I am assuming that this will also work fine under php4 - if anyone wants to test that and let me know here, I would be grateful. I would patch this but I haven't got a clue and I'm just leaving work, so don't have time to learn (sorry) - best just get the info up ASAP for some other poor soul, I figure.

Nik