| Summary: | Suggestions filter by fund displays inactive budgets | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Nick Clemens (kidclamp) <nick> | 
| Component: | Acquisitions | Assignee: | Nick Clemens (kidclamp) <nick> | 
| Status: | Needs documenting --- | QA Contact: | Pedro Amorim <pedro.amorim> | 
| Severity: | normal | ||
| Priority: | P5 - low | CC: | fridolin.somers, lucas, pedro.amorim, wainuiwitikapark | 
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Crowdfunding goal: | 0 | 
| Patch complexity: | --- | Documentation contact: | |
| Documentation submission: | Text to go in the release notes: | ||
| Version(s) released in: | 
       24.11.00,24.05.06 
 | 
        
        Circulation function: | |
| Bug Depends on: | |||
| Bug Blocks: | 38986 | ||
| Attachments: | 
              Bug 37246: Use the same dropdown for filtering and editing/adding budgets
               Bug 37246: Use the same dropdown for filtering and editing/adding budgets Bug 37246: Use the same dropdown for filtering and editing/adding budgets Bug 37246: (QA follow-up) DRY Dropdown  | 
      ||
| 
 
        
          Description
        
        
          Nick Clemens (kidclamp)
        
        
        
        
          2024-07-03 14:07:06 UTC
        
       
    Created attachment 168459 [details] [review] Bug 37246: Use the same dropdown for filtering and editing/adding budgets This patch removes a second fetch of budgets form the db, and clones the dropdown in the template as we will only ever be displaying one of them at a time. The JS is moved to fire for both operations. To test: 1 - Create an active and inactive budget in acquisitions 2 - Add funds to both 3 - Visit - Acquisitions -> Suggestions 4 - Click 'Acquisition information' under filters on the left 5 - Note all funds are showing 6 - Apply patch 7 - Reload 8 - Note you only see active budgets 9 - Check the box to show inactive, verify they show 10 - Filter by an inactive budget 11 - Confirm selection remains 12 - Add a suggestion and verify budget dropdown works 13 - Edit the suggestion and verify budget dropdown works Created attachment 168464 [details] [review] Bug 37246: Use the same dropdown for filtering and editing/adding budgets This patch removes a second fetch of budgets form the db, and clones the dropdown in the template as we will only ever be displaying one of them at a time. The JS is moved to fire for both operations. To test: 1 - Create an active and inactive budget in acquisitions 2 - Add funs to both 3 - Visit - Acquisitions -> Suggestions 4 - Click 'Acquisition information' under filters on the left 5 - Note all funds are showing 6 - Apply patch 7 - Reload 8 - Note you only see active budgets 9 - Check the box to show inactive, verify they show 10 - Filter by an inactive budget 11 - Confirm selection remains 12 - Add a suggestion and verify budget dropdown works 13 - Edit the suggestion and verify budget dropdown works Signed-off-by: Sam Lau <samalau@gmail.com> Created attachment 168556 [details] [review] Bug 37246: Use the same dropdown for filtering and editing/adding budgets This patch removes a second fetch of budgets form the db, and clones the dropdown in the template as we will only ever be displaying one of them at a time. The JS is moved to fire for both operations. To test: 1 - Create an active and inactive budget in acquisitions 2 - Add funs to both 3 - Visit - Acquisitions -> Suggestions 4 - Click 'Acquisition information' under filters on the left 5 - Note all funds are showing 6 - Apply patch 7 - Reload 8 - Note you only see active budgets 9 - Check the box to show inactive, verify they show 10 - Filter by an inactive budget 11 - Confirm selection remains 12 - Add a suggestion and verify budget dropdown works 13 - Edit the suggestion and verify budget dropdown works Signed-off-by: Sam Lau <samalau@gmail.com> Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com> Created attachment 168557 [details] [review] Bug 37246: (QA follow-up) DRY Dropdown If we're using the same dropdown it should be defined once. Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com> Pushed for 24.11! Well done everyone, thank you! Backported to 24.05.x for upcoming 24.05.06 Does not apply easy on 23.11.x Please provide rebased patches if needed Not backporting to 22.11  |