http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6390

Katrin Fischer <katrin.fisc...@bsz-bw.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Attachment #5963|0                           |1
        is obsolete|                            |

--- Comment #34 from Katrin Fischer <katrin.fisc...@bsz-bw.de> 2011-10-18 
20:51:39 UTC ---
Created attachment 5966
  --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=5966
[SIGNED-OFF] Bug 6390 Followup: Basket's authorisedby might not be a
borrowernumber

aqbasket.authorisedby is not foreign key constrainted to
borrowers.borrowernumber,
so if it's anything other than that, vendor search breaks by using an undefined
value
(GetMember() on an invalid borrowernumber) as a HASH ref.

This patch sets the branchcode (the desired value from GetMember) to a blank
value, then only
changes it if GetMember is defined.  In the case where the authorisedby
librarian has been deleted,
or the value is otherwise invalid, and AcqViewBaskets is set to 'branch', the
basket will not appear
except to superlibrarians

Signed-off-by: Katrin Fischer <katrin.fischer...@web.de>
Invalid borrowernumber in aqbasket.authorisedby no longer breaks search.

-- 
Configure bugmail: 
http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA Contact for the bug.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to