Bug 19180

Summary: Vendor name is missing from breadcrumbs when closing an order
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: AcquisitionsAssignee: Aleisha Amohia <aleisha>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: aleisha, caroline.cyr-la-rose, fridolin.somers, jonathan.druart, mtompset
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19118
Change sponsored?: Sponsored Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 19180: Add vendor name to breadcrumbs when closing an order
Bug 19180: Add vendor name to breadcrumbs when closing an order
Bug 19180: [FOLLOW-UP] Renaming all instances of 'name' variable to 'booksellername'
Bug 19180: Add vendor name to breadcrumbs when closing an order
Bug 19180: [FOLLOW-UP] Renaming all instances of 'name' variable to 'booksellername'
Bug 19180: [FOLLOW-UP] Renaming all instances of 'name' variable to 'booksellername'
Bug 19180: [FOLLOW-UP] Renaming all instances of 'name' variable to 'booksellername'
Bug 19180: [FOLLOW-UP] Renaming all instances of 'name' variable to 'booksellername'
Bug 19180: Add vendor name to breadcrumbs when closing an order
Bug 19180: [FOLLOW-UP] Renaming all instances of 'name' variable to 'booksellername'

Description Katrin Fischer 2017-08-26 22:23:08 UTC
When an open basket/order is closed, the breadcrumbs on the page asking about adding the order to a basket group is missing the vendor name in the breadcrumbs in 2 spots:

Home › Acquisitions › › Basket Test bug 15347 (1) for
Comment 1 Aleisha Amohia 2017-08-27 23:04:31 UTC
Created attachment 66517 [details] [review]
Bug 19180: Add vendor name to breadcrumbs when closing an order

To test:
1) Go to Acquisitions
2) Find a vendor and a basket
3) Click 'Close basket' button
4) Notice that on confirmation page, breadcrumbs are missing vendor
5) Apply patch and refresh page
6) Vendor name should now show
7) Confirm link to vendor works as expected

Sponsored-by: Catalyst IT
Comment 2 Mark Tompsett 2017-08-28 18:32:50 UTC
*grumbles about having to set up budgets, funds, AND a vendor* ;)
Before:
Home › Acquisitions › › Basket test basket (1) for
After:
Home › Acquisitions › Test › Basket test basket (1) for Test
Link works. :)
Comment 3 Mark Tompsett 2017-08-28 18:34:26 UTC
Created attachment 66562 [details] [review]
Bug 19180: Add vendor name to breadcrumbs when closing an order

To test:
1) Go to Acquisitions
2) Find a vendor and a basket
3) Click 'Close basket' button
4) Notice that on confirmation page, breadcrumbs are missing vendor
5) Apply patch and refresh page
6) Vendor name should now show
7) Confirm link to vendor works as expected

Sponsored-by: Catalyst IT

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Comment 4 Jonathan Druart 2017-08-29 16:49:00 UTC
We are on basket.pl, sending a "name" variable to the template that is actually the "vendor name" can lead to issue.
Let's rename it with "bookseller_name" or whatever more meaningful.
Comment 5 Aleisha Amohia 2017-08-29 21:28:52 UTC
Created attachment 66603 [details] [review]
Bug 19180: [FOLLOW-UP] Renaming all instances of 'name' variable to 'booksellername'

... when referring to the name of the vendor.

To test:
1) Confirm vendor shows on webpage title (tab name)
2) Confirm vendor shows in breadcrumbs
3) Confirm vendor shows in heading when viewing basket ('Basket x (1) for
vendor')

Sponsored-by: Catalyst IT
Comment 6 Caroline Cyr La Rose 2017-09-08 20:02:30 UTC
Error message :

fatal: sha1 information is lacking or useless (acqui/basket.pl).
Comment 7 Mark Tompsett 2017-09-09 03:51:32 UTC
Created attachment 67029 [details] [review]
Bug 19180: Add vendor name to breadcrumbs when closing an order

To test:
1) Go to Acquisitions
2) Find a vendor and a basket
3) Click 'Close basket' button
4) Notice that on confirmation page, breadcrumbs are missing vendor
5) Apply patch and refresh page
6) Vendor name should now show
7) Confirm link to vendor works as expected

Sponsored-by: Catalyst IT

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Comment 8 Mark Tompsett 2017-09-09 03:51:39 UTC
Created attachment 67030 [details] [review]
Bug 19180: [FOLLOW-UP] Renaming all instances of 'name' variable to 'booksellername'

... when referring to the name of the vendor.

To test:
1) Confirm vendor shows on webpage title (tab name)
2) Confirm vendor shows in breadcrumbs
3) Confirm vendor shows in heading when viewing basket ('Basket x (1) for
vendor')

Sponsored-by: Catalyst IT
Comment 9 Mark Tompsett 2017-09-09 03:55:34 UTC
Created attachment 67031 [details] [review]
Bug 19180: [FOLLOW-UP] Renaming all instances of 'name' variable to 'booksellername'

... when referring to the name of the vendor.

To test:
1) Confirm vendor shows on webpage title (tab name)
2) Confirm vendor shows in breadcrumbs
3) Confirm vendor shows in heading when viewing basket ('Basket x (1) for
vendor')

Sponsored-by: Catalyst IT
Comment 10 Mark Tompsett 2017-09-09 04:00:40 UTC
Created attachment 67032 [details] [review]
Bug 19180: [FOLLOW-UP] Renaming all instances of 'name' variable to 'booksellername'

... when referring to the name of the vendor.

To test:
1) Confirm vendor shows on webpage title (tab name)
2) Confirm vendor shows in breadcrumbs
3) Confirm vendor shows in heading when viewing basket ('Basket x (1) for
vendor')

Sponsored-by: Catalyst IT

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Comment 11 Mark Tompsett 2017-09-09 04:01:40 UTC
(In reply to Caroline Cyr La Rose from comment #6)
> Error message :
> 
> fatal: sha1 information is lacking or useless (acqui/basket.pl).

Because someone did a 'basketno | html' tweak to the template file. Easy enough to rebase and compare.
Comment 12 Caroline Cyr La Rose 2017-09-15 19:30:01 UTC
Created attachment 67156 [details] [review]
Bug 19180: [FOLLOW-UP] Renaming all instances of 'name' variable to 'booksellername'

... when referring to the name of the vendor.

To test:
1) Confirm vendor shows on webpage title (tab name)
2) Confirm vendor shows in breadcrumbs
3) Confirm vendor shows in heading when viewing basket ('Basket x (1) for
vendor')

Sponsored-by: Catalyst IT

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com>
Comment 13 Jonathan Druart 2017-09-19 14:28:16 UTC
Created attachment 67213 [details] [review]
Bug 19180: Add vendor name to breadcrumbs when closing an order

To test:
1) Go to Acquisitions
2) Find a vendor and a basket
3) Click 'Close basket' button
4) Notice that on confirmation page, breadcrumbs are missing vendor
5) Apply patch and refresh page
6) Vendor name should now show
7) Confirm link to vendor works as expected

Sponsored-by: Catalyst IT

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 14 Jonathan Druart 2017-09-19 14:28:21 UTC
Created attachment 67214 [details] [review]
Bug 19180: [FOLLOW-UP] Renaming all instances of 'name' variable to 'booksellername'

... when referring to the name of the vendor.

To test:
1) Confirm vendor shows on webpage title (tab name)
2) Confirm vendor shows in breadcrumbs
3) Confirm vendor shows in heading when viewing basket ('Basket x (1) for
vendor')

Sponsored-by: Catalyst IT

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 15 Jonathan Druart 2017-09-19 14:51:43 UTC
Pushed to master for 17.11, thanks to everybody involved!
Comment 16 Fridolin Somers 2017-10-23 15:04:56 UTC
Pushed to 17.05.x, will be in 17.05.06.
Comment 17 Katrin Fischer 2017-10-29 18:07:55 UTC
Doesn't apply cleanly to 16.11.x (second patch conflicts), please rebase if you want this to be included.
Comment 18 Katrin Fischer 2017-11-15 20:38:06 UTC
I checked - the bug exists in 16.11.x.