Hi all... I'm new to the list, and I've been searching through the archives for information on the following bug, and didn't see anything, so I decided to post it. Please don't flame me. In many of the perl files, the following code exists... print mktablerow(5,$toggle,$results->[$i]{'printername'},$results->[$i]{'printqueue'},$results->[$i]{'printtype'}, mklink("$script_name?op=add_form&searchfield=".$results->[$i]{'printername'},'Edit'), mklink("$script_name?op=delete_confirm&searchfield=".$results->[$i]{'printername'},'Delete','')); Now, when I ran these perl functions "out of the box" I was gettig 500 server errors... digging through the apache logs, I see the following. Too many arguments for C4::Output::mklink at /home/httpd/cgi-bin/koha/admin/printers.pl line 227, near "'')" Execution of /home/httpd/cgi-bin/koha/admin/printers.pl aborted due to compilation errors. [Thu Jun 27 23:44:22 2002] [error] [client 66.92.231.155] Premature end of script headers: /home/httpd/cgi-bin/koha/admin/printers.pl Odd, I think to myself, so I look at the two calls to "mklink" and I see that they do, in fact, have a different number of parameters. Now, since the final parameter in the "delete" call is blank, I tried deleting the last parameter, and it works (well, at least it displays). Is this a legitimate bug, or did I totally screw something up? Tom Albrecht III West Chester, PA United States of America
On Fri, 2002-06-28 at 11:51, Tom Albrecht wrote:
Hi all... I'm new to the list, and I've been searching through the archives for information on the following bug, and didn't see anything, so I decided to post it. Please don't flame me.
In many of the perl files, the following code exists...
print mktablerow(5,$toggle,$results->[$i]{'printername'},$results->[$i]{'printqueue'},$results->[$i]{'printtype'},
mklink("$script_name?op=add_form&searchfield=".$results->[$i]{'printername'},'Edit'),
mklink("$script_name?op=delete_confirm&searchfield=".$results->[$i]{'printername'},'Delete',''));
Hi Tom Yep, this is a legitimate bug in the 1.2.0 code. It has been fixed in cvs, and is working fine in the latest release candidate for 1.2.1. We are currently testing 1.2.1 and hope to have a release in the near future. Basically the fix you made (deleting the last parameter) is the right thing to do. Chris -- Chris Cormack chris@katipo.co.nz Programmer 025 500 789 Katipo Communications www.katipo.co.nz
participants (2)
-
Chris Cormack -
Tom Albrecht