Le Mercredi 7 Juin 2006 02:33, bob@katipo.co.nz a écrit :

Hi,

I am looking at committing katipo koha changes back to cvs and i noticed this bit of code variation

katipo (older) version:

my ($resfound, $resrec) = CheckReserves($iteminformation->{'itemnumber'}); if ($resfound and not $ignoreRs) { $resrec->{'ResFound'} = $resfound; $messages->{'ResFound'} = $resrec; $dotransfer = 0; } #actually do the transfer.... if ($dotransfer) { dotransfer($iteminformation->{'itemnumber'}, $fbr, $tbr); $messages->{'WasTransfered'} = 1; }

newer version on Savannah cvs:

my ($resfound, $resrec) = CheckReserves($iteminformation->{'itemnumber'}); if ($resfound and not $ignoreRs) { $resrec->{'ResFound'} = $resfound; # $messages->{'ResFound'} = $resrec; $dotransfer = 1; }

if ($dotransfer and not $resfound) { dotransfer($iteminformation->{'itemnumber'}, $fbr, $tbr); $messages->{'WasTransfered'} = 1; }

i'm confused as to what the newer version of code is trying to achieve, the way i read it (and i might be wrong) is that it is saying that if a reserve is found, and the ignore reserve is false (ie. do not ignore reserve), allow dotransfer but then the next bit says if dotransfer and not reserve found then allow transfer which seems to be contrary to the first bit

any help to understanding the change will be most appreciated thanks,

Robert Lyon

I think this new function, is for return the document to his homebranch, if the document is returned in an another library, the system check if there is no reserves linked to this document, and if the document is not on his homebranch, we return the document (by dotransfer function...) to his homebranch ..... .