Bug 8221

Summary: Adding item to patroncard Batch causes error in Plack
Product: Koha Reporter: Jared Camins-Esakov <jcamins>
Component: Label/patron card printingAssignee: Dobrica Pavlinusic <dpavlin>
Status: CLOSED WONTFIX QA Contact:
Severity: minor    
Priority: P5 - low CC: chris, dpavlin, paul.poulain
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Bug Depends on: 9008    
Bug Blocks:    

Description Jared Camins-Esakov 2012-06-10 14:46:24 UTC
Can't call method "get_attr" without a package or object reference at /home/jcamins/kohaclone/patroncards/edit-batch.pl line 126.
 at /home/jcamins/kohaclone/patroncards/edit-batch.pl line 145
Comment 1 Dobrica Pavlinusic 2012-06-14 15:49:49 UTC
I can't reproduce that error. Do you get that error in pop-up window which "Add item(s)" opens?

I'm not very good in using Koha, so I would appreciate step-by-step test scenario to reproduce it :-)

I did notice that "Remove duplicates" doesn't work for me under plack and always leave just first item in batch and it seems that Export items always exports single item, so something is fishy there.
Comment 2 Dobrica Pavlinusic 2012-06-15 18:20:12 UTC
I manage to replicate this problem, but and it happen when you are logged in with koha admin account which doesn't have library (branch) assigned.

Adding batch dies because of referential integrity error (since branch is NULL) and editing dies because there is no row created to edit.

It does work for CGI version, more research needs to be done.
Comment 3 Chris Cormack 2012-06-15 20:02:40 UTC
(In reply to comment #2)
> I manage to replicate this problem, but and it happen when you are logged in
> with koha admin account which doesn't have library (branch) assigned.
> 
Ahh yeah, we really really should stop people doing this, or at least strongly recommend they don't. Should only ever use the kohaadmin user for upgrades and setting up your first real user.


> Adding batch dies because of referential integrity error (since branch is
> NULL) and editing dies because there is no row created to edit.
> 
> It does work for CGI version, more research needs to be done.
Comment 4 Paul Poulain 2012-11-02 11:50:31 UTC
I've submitted bug 9008 as an answer to chris comment 3