From: Dieter S. <dsi...@sq...> - 2007-12-03 23:17:23
|
On Mon, 3 Dec 2007, Trevor Hennion wrote: > Dieter Simader wrote: >> On Mon, 3 Dec 2007, Trevor Hennion wrote: >> >>> Colin, >>> >>> This bit me this afternoon and it is due to an upgrade of Perl on >>> Fedora7 and Fedora8. >>> >>> I tried some fix suggested in the support forum but it didn't work. >>> >>> To make sure I could get invoices out this afternoon I had to install >>> SQL-Ledger on another box running Fedora5, without the updated Perl >>> version, and that worked OK. >>> >>> If this doesn't get fixed soon a lot more people will feel the pain. >> >> change the code in OE.pm on line 1375 to >> >> # adjust taxes for lineitems >> my $total = 0; >> for my $ref (@{ $form->{lineitems} }) { >> $total += $ref->{tax}; >> } >> >> report back if perl still barfs after the change. >> > > Dieter, > > Thanks for the reply, however I'm sorry to say it didn't work. > Still get: > > Error! > Not a HASH reference at SL/IS.pm line 395. > Attempt to free unreferenced scalar: SV 0x944bfb8, Perl interpreter: > 0x8d45008 at SL/IS.pm line 395. OE.pm and IS.pm are two different things. This particular code construct also exists in other modules, so let's not test something you have not made changes to. Of course there will still be an error if you haven't made the change yet. So, please edit IS.pm and make the change and report back or try to print an order with the changed OE.pm file. If printing an order works we change all the other occurances where this code construct exists in the code. clear as mud? |