Bug 19334

Summary: circulation history doesn't set biblionumber so left navigation is broken
Product: Koha Reporter: Dobrica Pavlinusic <dpavlin>
Component: CirculationAssignee: Dobrica Pavlinusic <dpavlin>
Status: CLOSED FIXED QA Contact: Julian Maurice <julian.maurice>
Severity: major    
Priority: P5 - low CC: fridolin.somers, gmcharlt, jonathan.druart, josef.moravec, julian.maurice, kyle.m.hall, ldjamison, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 18260    
Bug Blocks:    
Attachments: Bug 19334 - circulation history doesn't set biblionumber so left navigation is broken
Bug 19334 - circulation history doesn't set biblionumber so left navigation is broken
Bug 19334 - circulation history doesn't set biblionumber so left navigation is broken

Description Dobrica Pavlinusic 2017-09-18 17:15:38 UTC

    
Comment 1 Dobrica Pavlinusic 2017-09-18 17:20:48 UTC
Created attachment 67188 [details] [review]
Bug 19334 - circulation history doesn't set biblionumber so left navigation is broken

Navigation on the left (Normal, MARC, etc...) needs biblionumber in
template variables to work.

Test:
1. go to checkout history for any biblio
2. verify that normal, MARC, etc links on the left no longer work
   due to missing biblionumber in URL
3. apply patch and test it again
Comment 2 Jonathan Druart 2017-09-18 19:01:20 UTC
Caused by bug 18260.
Comment 3 Lee Jamison 2017-09-18 20:56:57 UTC
Patch does not address the bug. The problem still exists after the patch is applied.
Comment 4 Jonathan Druart 2017-09-19 11:47:29 UTC
(In reply to Lee Jamison from comment #3)
> Patch does not address the bug. The problem still exists after the patch is
> applied.

It works for me, could you detail what does not work?
Comment 5 Josef Moravec 2017-09-22 09:06:03 UTC
Created attachment 67312 [details] [review]
Bug 19334 - circulation history doesn't set biblionumber so left navigation is broken

Navigation on the left (Normal, MARC, etc...) needs biblionumber in
template variables to work.

Test:
1. go to checkout history for any biblio
2. verify that normal, MARC, etc links on the left no longer work
   due to missing biblionumber in URL
3. apply patch and test it again

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Comment 6 Julian Maurice 2017-09-29 11:59:25 UTC
Created attachment 67439 [details] [review]
Bug 19334 - circulation history doesn't set biblionumber so left navigation is broken

Navigation on the left (Normal, MARC, etc...) needs biblionumber in
template variables to work.

Test:
1. go to checkout history for any biblio
2. verify that normal, MARC, etc links on the left no longer work
   due to missing biblionumber in URL
3. apply patch and test it again

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Comment 7 Jonathan Druart 2017-09-29 15:49:01 UTC
Pushed to master for 17.11, thanks to everybody involved!
Comment 8 Fridolin Somers 2017-10-17 08:06:20 UTC
Depends on Bug 18260 that is not in 17.05.x