[Koha-patches] [PATCH] Bug Fixing : Bullet Proofing

Colin Campbell colin.campbell at ptfs-europe.com
Wed Aug 26 10:27:49 CEST 2009


On 08/26/2009 12:40 AM, Henri-Damien LAURENT wrote:

>   my ($issues) = GetPendingIssues($borrowernumber);
> -my @issue_list = sort { $b->{'date_due'} cmp $a->{'date_due'} } @$issues;
> +my @issue_list = sort { $b->{'date_due'} cmp $a->{'date_due'} } @$issues if ($issues);
>
Would it not be preferable to declare @issue_list outside of the 
conditional?
C.

-- 
Colin Campbell
Software Engineer, PTFS Europe Limited
Content Management and Library Solutions
+44 (0) 208 366 1295 (phone)
+44 (0) 7759 633626  (mobile)
colin.campbell at ptfs-europe.com
skype: colin_campbell2

http://www.ptfs-europe.com



More information about the Koha-patches mailing list