Bug 25611

Summary: Changing the vendor when creating the basket does not keep that new vendor
Product: Koha Reporter: Kelly McElligott <kelly>
Component: AcquisitionsAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: aleisha, jonathan.druart, julian.maurice, lucas, rkuiper, victor
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:
20.11.00, 20.05.02, 19.11.08, 19.05.14
Bug Depends on: 7757    
Bug Blocks:    
Attachments: Bug 25611: Use selected vendor when creating a basket
Bug 25611: Use selected vendor when creating a basket
Bug 25611: Use selected vendor when creating a basket
Bug 25611: Use selected vendor when creating a basket

Description Kelly McElligott 2020-05-27 18:00:47 UTC
When creating a basket, the vendor has the ability to be altered from the original chosen vendor.

Currently:
1. Choose Vendor
2. Create Basket
3. Assign a new vendor
4. This new vendor is not changed, it is still on the first vendor that was chosen.

If the vendor can not be changed after initially choosing the vendor, the field should not be a dropdown, where it would appear that a change in vendor can be made.

If the vendor can be changed during the creation of the basket, then this is not currently working.
Comment 1 Jonathan Druart 2020-06-05 09:14:34 UTC
Looks like a super old bug, good catch!
Comment 2 Jonathan Druart 2020-06-05 09:18:20 UTC
Created attachment 105572 [details] [review]
Bug 25611: Use selected vendor when creating a basket

We did not take into account the vendor selected.

Test plan:
1. Choose Vendor
2. Create Basket
3. Assign a new vendor
Without the patch this new vendor is not changed
With the patch applied the vendor is changed

4. Edit the basket, change the vendor
Comment 3 ByWater Sandboxes 2020-06-15 20:06:28 UTC
Created attachment 105891 [details] [review]
Bug 25611: Use selected vendor when creating a basket

We did not take into account the vendor selected.

Test plan:
1. Choose Vendor
2. Create Basket
3. Assign a new vendor
Without the patch this new vendor is not changed
With the patch applied the vendor is changed

4. Edit the basket, change the vendor

Signed-off-by: Rhonda Kuiper <rkuiper@roundrocktexas.gov>
Comment 4 ByWater Sandboxes 2020-06-16 16:42:06 UTC
Created attachment 105929 [details] [review]
Bug 25611: Use selected vendor when creating a basket

We did not take into account the vendor selected.

Test plan:
1. Choose Vendor
2. Create Basket
3. Assign a new vendor
Without the patch this new vendor is not changed
With the patch applied the vendor is changed

4. Edit the basket, change the vendor

Signed-off-by: Rhonda Kuiper <rkuiper@roundrocktexas.gov>

Signed-off-by: Rhonda Kuiper <rkuiper@roundrocktexas.gov>
Comment 5 Julian Maurice 2020-06-19 14:14:31 UTC
Created attachment 106100 [details] [review]
Bug 25611: Use selected vendor when creating a basket

We did not take into account the vendor selected.

Test plan:
1. Choose Vendor
2. Create Basket
3. Assign a new vendor
Without the patch this new vendor is not changed
With the patch applied the vendor is changed

4. Edit the basket, change the vendor

Reported-by: Kelly McElligott <kelly@bywatersolutions.com>
Signed-off-by: Rhonda Kuiper <rkuiper@roundrocktexas.gov>
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Comment 6 Julian Maurice 2020-06-19 14:16:34 UTC
It's strange that we can change the vendor when the page title says "Create new basket for <Vendor>" (and the title is not updated when we select another vendor), but the patch does fix a bug.
Passed QA
Comment 7 Jonathan Druart 2020-06-24 13:32:56 UTC
Pushed to master for 20.11, thanks to everybody involved!
Comment 8 Lucas Gass 2020-07-10 16:10:19 UTC
backported to 20.05.x for 20.05.02
Comment 9 Aleisha Amohia 2020-07-13 02:23:45 UTC
backported to 19.11.x for 19.11.08
Comment 10 Victor Grousset/tuxayo 2020-07-23 23:43:07 UTC
Backported to 19.05.x branch for 19.05.14