Bug 13885

Summary: Cataloging search header should include more options
Product: Koha Reporter: Nick Clemens <nick>
Component: CatalogingAssignee: Nick Clemens <nick>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: frederic, gwilliams, m.de.rooy, nengard, tomascohen
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: 13934    
Bug Blocks:    
Attachments: Bug 13885 - Cataloging search header should include more options
Bug 13885 - Cataloging search header should include more options
Bug 13885 - Cataloging search header should include more options
Bug 13885 - Cataloging search header should include more options
Bug 13885 [QA Followup] - Remove tab, remove unnecessary permission check
[PASSED QA] Bug 13885 - Cataloging search header should include more options
[PASSED QA] Bug 13885 [QA Followup] - Remove tab, remove unnecessary permission check

Description Nick Clemens 2015-03-21 14:54:18 UTC
When cataloging it would be nice to have the option of moving easily to check-in or regular catalog search
Comment 1 Nick Clemens 2015-03-21 14:58:59 UTC Comment hidden (obsolete)
Comment 2 Nicole C. Engard 2015-03-29 20:24:58 UTC
Nick I think 'Standard search' Should be Search the Catalog like on all other pages.  I know it can be confusing to have that word in 2 searches, but I think we need to be consistent.
Comment 3 Nick Clemens 2015-03-30 20:19:37 UTC Comment hidden (obsolete)
Comment 4 Nicole C. Engard 2015-03-30 21:34:08 UTC
All tabs work - except check in which gives me this error:

Software error:

Can't bless non-reference value at /home/nengard/kohaclone/C4/ItemType.pm line 64.

For help, please send mail to the webmaster (webmaster@koha-vm), giving this error message and the time and date of the error.
Comment 5 Nick Clemens 2015-03-31 18:11:19 UTC
Not my bug causing it, problem is on master (bug 13943) , but can't test until that is cleared
Comment 6 Nicole C. Engard 2015-03-31 19:15:24 UTC
What is the bug number? that one doesn't exist, I was going to add it as a dependency here.

Nicole
Comment 7 Nick Clemens 2015-03-31 19:21:17 UTC
(In reply to Nicole C. Engard from comment #6)
> What is the bug number? that one doesn't exist, I was going to add it as a
> dependency here.
> 
> Nicole

13934, stupid fingers
Comment 8 Frédéric Demians 2015-04-11 10:12:32 UTC
Bug 13934 is in master. It solves check-in error.
Comment 9 Frédéric Demians 2015-04-11 10:14:56 UTC Comment hidden (obsolete)
Comment 10 Kyle M Hall 2015-04-17 11:32:31 UTC Comment hidden (obsolete)
Comment 11 Kyle M Hall 2015-04-17 11:32:39 UTC Comment hidden (obsolete)
Comment 12 Kyle M Hall 2015-04-17 11:32:55 UTC
Created attachment 38039 [details] [review]
[PASSED QA] Bug 13885 - Cataloging search header should include more options

This patch adds the check in and standard catalog search to cataloging-search.inc

To test:
1 - Apply patch
2 - Go to More->Cataloging
3 - Search in each tab (Cataloging search / Check out / Check in / Standard
    search) and ensure that all work as expected

Signed-off-by: Frederic Demians <f.demians@tamil.fr>
  - Added search options work properly, and are consistently named.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 13 Kyle M Hall 2015-04-17 11:33:06 UTC
Created attachment 38040 [details] [review]
[PASSED QA] Bug 13885 [QA Followup] - Remove tab, remove unnecessary permission check

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 14 Tomás Cohen Arazi 2015-04-29 18:43:39 UTC
Patches pushed to master.

Thanks Nick!