Minor typo in a hash key.
YAHOO.widget.Button didn't call right function, other minor fixes was to make sure tag and subfield were both checked, since one was 'tag_mandatory' and the other was 'mandatory'. Also added *. Display and JS check should now work for any framework.
Appears the smart matching was not correctly being evaluated as 'W'aiting and 'T'ransit items were still being suspended. The croak error is also now being caught by Try::Tiny.
13 03 pt17425127 4 08
Not that these two exit() calls were reachable, anyway, since an unconditional return is called just prior to each of them, but exit() should never be called from a module. It's safe to call it from a .pl script, but calling it from within a .pm causes the Plack engine itself to exit, which is bad news.
15 06 pt18149087 4 08
15 04 pt14999551 4 08
15 03 pt17119619 4 08
Pt13645205 4 08
14 05 pt17944127 4 08
14 04 pt17807355 4 08
holds, needed to be transferred back to Home branch but no prompt for transfer, only the lost+link to patron patron. fix was to check for both transfer and needstransfer, which are ambiguous distinctions anyway. Also tested case when no patron is attached but item was lost and found and needed transfer to item Home.
…d=0: this is now done in 4 places: when the cronjob runs, on the bib's Items page for the item's otherstatus, on the add/edit item page, and batch item modify NOTE: outstanding is warning prompts/confirmations not implemented
The ok field (pos3) in the SIP2 renewal response message (30) was a little ambiguous despite the renewal_ok field (pos4) being less so. In addition, a renewal denial because of patron conditions is now better handled. It was previously allowing attempted work on an undefined object.
…f enabled If items.permanent_location is NULL/empty, save the contents of items.location into it before setting current location to shelf.
Additional variable scoping issues manifested in the Plack environment.
Error was being thrown on a close STDERR. Removed this line of code. I suspect this was Plack related although I not necessarily sure why.
This reverts commit eb7bd6c. The change as implemented breaks the leader, causing encoding problems.
Also clean up useless debugging noise
13 01 pt14554159 4 08
13 02 pt16905729 4 08
The meat of this is the changes to C4/Biblio.pm, using a safe caching function instead of storing it in a package global. The changes to C4/Items.pm are preventative cleanup to keep this variable from being set at some point in the future.
syspref EnableOverdueAccruedAmount was ON: pass to UpdateFine() the new accounttype, from 'FU' to 'F' so patron can pay on it
deprecate C4::Members::GetMemberLostItems() and use instead previously fixed C4::LostItems::GetLostItems()