Summary: | Statistic 1/2 not saving when updating fund after receipt | ||
---|---|---|---|
Product: | Koha | Reporter: | Katrin Fischer <katrin.fischer> |
Component: | Acquisitions | Assignee: | Nick Clemens (kidclamp) <nick> |
Status: | Pushed to oldoldstable --- | QA Contact: | Martin Renvoize (ashimema) <martin.renvoize> |
Severity: | major | ||
Priority: | P5 - low | CC: | dch, fridolin.somers, lucas, michaela.sieber, nick, victor, wainuiwitikapark |
Version: | Main | Keywords: | rel_22_11_candidate, rel_23_05_candidate, rel_23_11_candidate |
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
24.11.00,24.05.02,23.11.07,23.05.15
|
|
Circulation function: | |||
Bug Depends on: | 25763 | ||
Bug Blocks: | |||
Attachments: |
Bug 34444: [22.11.x] Correct handling of sort1 and sort2 values
Bug 34444: Correct handling of sort1 and sort2 values Bug 34444: Correct handling of sort1 and sort2 values Bug 34444: Correct handling of sort1 and sort2 values |
Description
Katrin Fischer
2023-07-30 11:53:46 UTC
22.11 is the LTS... it would be great to get some help here. Created attachment 166417 [details] [review] Bug 34444: [22.11.x] Correct handling of sort1 and sort2 values Removed extraneous sort_1 data elements Update selectors to use field names for statistics field Updated code to set the value after finding the correct selector To test: * Make sure you have at least 2 funds with different stat settings, using AV and not * Create a basket with an order line * Close it and receive shipment * Create an invoice and receive the order line * Finish receiving * Click "Modify fund" * Switch fund, verify the stat fields are updated accordingly * Change values for statistical values * Update fund * Edit fund again, pull downs are correct * Change values in form and close, do not update * Click 'Modify fund' - confrim form is filled with the saved values Created attachment 166419 [details] [review] Bug 34444: Correct handling of sort1 and sort2 values Removed extraneous sort_1 data elements Update selectors to use field names for statistics field Updated code to set the value after finding the correct selector To test: * Make sure you have at least 2 funds with different stat settings, using AV and not * Create a basket with an order line * Close it and receive shipment * Create an invoice and receive the order line * Finish receiving * Click "Modify fund" * Switch fund, verify the stat fields are updated accordingly * Change values for statistical values * Update fund * Edit fund again, pull downs are correct * Change values in form and close, do not update * Click 'Modify fund' - confrim form is filled with the saved values I recreated on master - the same patch applied and works so reattached here - there is more cleanup that can be done, but this should fix the issue Created attachment 166481 [details] [review] Bug 34444: Correct handling of sort1 and sort2 values Removed extraneous sort_1 data elements Update selectors to use field names for statistics field Updated code to set the value after finding the correct selector To test: * Make sure you have at least 2 funds with different stat settings, using AV and not * Create a basket with an order line * Close it and receive shipment * Create an invoice and receive the order line * Finish receiving * Click "Modify fund" * Switch fund, verify the stat fields are updated accordingly * Change values for statistical values * Update fund * Edit fund again, pull downs are correct * Change values in form and close, do not update * Click 'Modify fund' - confrim form is filled with the saved values Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> Hm, I see there was a patch for 22.11 and the current patch doesn't apply on 22.11. I am restoring it for some testing with our 22.11 version. Hm, still doesn't apply on 22.11 - Nick, could you have a look? (In reply to Katrin Fischer from comment #7) > Hm, still doesn't apply on 22.11 - Nick, could you have a look? Both apply clean to 22.11.x for me (In reply to Nick Clemens (kidclamp) from comment #8) > (In reply to Katrin Fischer from comment #7) > > Hm, still doesn't apply on 22.11 - Nick, could you have a look? > > Both apply clean to 22.11.x for me I might need to take a closer look and maybe rebase our branch. Sorry for the noise and thanks! Just to be sure, is it expected that when - switching from a fund with AV stats - to one with free text stat - then the free field contains the raw AV values This fix is not just about when - going back and forth between funds with AV stats and free values - going back to the AV one - then it will remember a changed AV not saved yet Created attachment 167451 [details] [review] Bug 34444: Correct handling of sort1 and sort2 values Removed extraneous sort_1 data elements Update selectors to use field names for statistics field Updated code to set the value after finding the correct selector To test: * Make sure you have at least 2 funds with different stat settings, using AV and not * Create a basket with an order line * Close it and receive shipment * Create an invoice and receive the order line * Finish receiving * Click "Modify fund" * Switch fund, verify the stat fields are updated accordingly * Change values for statistical values * Update fund * Edit fund again, pull downs are correct * Change values in form and close, do not update * Click 'Modify fund' - confrim form is filled with the saved values Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Thanks for all the hard work! Pushed to main for the next 24.11.00 release as RM Assistant Backported to 24.05.x for upcoming 24.05.02 Pushed to 23.11.x for 23.11.07 The test plan is not working for me on 23.05.x - if required for 23.05.x please provide a specific patch :) Maybe the 22.11 patch works for 23.05 as well? As far as we know it was broken all the way down, I can confirm the bug for 22.11. (In reply to Katrin Fischer from comment #16) > Maybe the 22.11 patch works for 23.05 as well? As far as we know it was > broken all the way down, I can confirm the bug for 22.11. Thanks Katrin, I will try that next month (string freeze already announced for this month) :) There are no string changes on this one so string freeze isn't an issue. (though it might still be too late to have the time to test and wait for CI so no worries) Would someone please be able to provide a more detailed test plan for this for 23.05? Hi Wainui, the test plan is: Setup: * Go to administration budgets and make sure you have an active budget added * Create at least 2 funds, with different stat settings: Fund 1: Stat 1 = Any AV from the list, Stat 2 = none Fund 2: Stat 1 = none, Stat 2 = Any AV but the one from Fund 1 * Create a basket with an order line, it doesn't matter how it's created * Close the basket and receive shipment for this vendor * Add the invoice details and receive the order line * Click Finish receiving at the bottom of the page Finally on the right page: Invoice summary * Click "Modify fund" link in the table of received order lines * Switch fund, verify the stat fields are updated accordingly * Change the values for the statistical values * Save the changes with "Update fund" * Edit fund again = pull downs will jump back to first entry instead of showing the previously picked value. Changes haven't been saved. I have added some steps and explanations, please let me know if you get stuck at a certain step. (In reply to Katrin Fischer from comment #20) > I have added some steps and explanations, please let me know if you get > stuck at a certain step. Thanks Katrin!! Patch applied to 23.05.x for 23.05.15 |