Galen,
>From my testing, no warning is visible in the OPAC for items where itemcallnumber is undefined. The value of $itemcallnumber prints as a null string, and thus fails gracefully. Will check the staff client tomorrow. Am I missing a logged warning somewhere? This test was performed on the latest code, synced today. -Ian On Tue, May 4, 2010 at 4:45 PM, Galen Charlton <gmcha...@gmail.com> wrote: > Hi, > > Why take out the test on whether $item->{itemcallnumber} was defined? > Doing so will cause a warning if the call number is null. > > Regards, > > Galen > > On Tue, May 4, 2010 at 4:36 PM, Ian Walls > <ian.wa...@bywatersolutions.com> wrote: > > From: Brendan A. Gallagher <bren...@bywatersolutions.com> > > > > Signed-off-by: Ian Walls <ian.wa...@bywatersolutions.com> > > --- > > C4/XSLT.pm | 5 +++-- > > 1 files changed, 3 insertions(+), 2 deletions(-) > > > > diff --git a/C4/XSLT.pm b/C4/XSLT.pm > > index f1fc9db..ccf6fc2 100644 > > --- a/C4/XSLT.pm > > +++ b/C4/XSLT.pm > > @@ -206,10 +206,11 @@ sub buildKohaItemsNamespace { > > $status = "available"; > > } > > my $homebranch = > $branches->{$item->{homebranch}}->{'branchname'}; > > + my $itemcallnumber = $item->{itemcallnumber}; > > + $itemcallnumber =~ s/\&/\&\;/g; > > $xml.= "<item><homebranch>$homebranch</homebranch>". > > "<status>$status</status>". > > - (defined $item->{'itemcallnumber'} ? > "<itemcallnumber>".$item->{'itemcallnumber'}."</itemcallnumber>" > > - : "<itemcallnumber />") > > + "<itemcallnumber>".$itemcallnumber."</itemcallnumber>" > > . "</item>"; > > > > } > > -- > > 1.5.6.5 > > > > _______________________________________________ > > Koha-patches mailing list > > Koha-patches@lists.koha.org > > http://lists.koha.org/mailman/listinfo/koha-patches > > > > > > -- > Galen Charlton > gmcha...@gmail.com > -- Ian Walls Lead Development Specialist ByWater Solutions ALA Booth # 817 Phone # (888) 900-8944 http://bywatersolutions.com ian.wa...@bywatersolutions.com Twitter: @sekjal
_______________________________________________ Koha-patches mailing list Koha-patches@lists.koha.org http://lists.koha.org/mailman/listinfo/koha-patches