Bug 7545

Summary: can't edit items
Product: Koha Reporter: Nicole C. Engard <nengard>
Component: CatalogingAssignee: Nicole C. Engard <nengard>
Status: CLOSED FIXED QA Contact: Ian Walls <koha.sekjal>
Severity: blocker    
Priority: P1 - high CC: jonathan.druart, paul.poulain, wizzyrea
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 6875    
Bug Blocks:    
Attachments: Bug 7545: Can't edit items
[Signed Off] Bug 7545: Can't edit items

Description Nicole C. Engard 2012-02-15 19:48:03 UTC
when trying to edit items you get the following:


Bareword "C4::Search::enabled_staff_search_views" not allowed while "strict subs" in use at /home/nengard/kohaclone/cataloguing/additem.pl line 715.
Bareword "C4::Search::enabled_staff_search_views" not allowed while "strict subs" in use at /home/nengard/kohaclone/cataloguing/additem.pl line 485.
Execution of /home/nengard/kohaclone/cataloguing/additem.pl aborted due to compilation errors.
Comment 1 Nicole C. Engard 2012-02-15 19:49:31 UTC Comment hidden (obsolete)
Comment 2 Liz Rea 2012-02-15 19:53:51 UTC
Created attachment 7664 [details] [review]
[Signed Off] Bug 7545: Can't edit items

This patch makes it so that you can click 'edit items' and actually
edit items.

Signed-off-by: Liz Rea <wizzyrea@gmail.com>
Works as advertised.

Signed-off-by: Liz Rea <wizzyrea@gmail.com>
Comment 3 Jonathan Druart 2012-02-16 09:08:03 UTC
I don't have this error whitout this patch. My perl version is v5.14.2
Comment 4 Liz Rea 2012-02-16 16:08:16 UTC
This was on master pulled as of yesterday (15-2-12). I was able to replicate.
Comment 5 Ian Walls 2012-02-16 16:25:04 UTC
Re-enables lost dependency on C4::Search in additems.pl.  Marking as Passed QA
Comment 6 Paul Poulain 2012-02-16 17:28:20 UTC
The problem was a consequence of de-nesting bug.
Removing use C4::Search from C4::Heading resulted in C4::Search not being loaded at all on additem.pl :
additem use C4::Biblio that use C4::Heading that used C4::Search.

This patch fixes the problem
Comment 7 Paul Poulain 2012-02-16 17:30:20 UTC
Note that this bug has been pushed on new/bug_6875 branch as it can be considered as a follow-up