Summary: | Insert runtime parameter button is not working in Reports | ||
---|---|---|---|
Product: | Koha | Reporter: | Brendan Lawlor <blawlor> |
Component: | Reports | Assignee: | Brendan Lawlor <blawlor> |
Status: | RESOLVED FIXED | QA Contact: | Emily Lamancusa (emlam) <emily.lamancusa> |
Severity: | major | ||
Priority: | P5 - low | CC: | david, emily.lamancusa, lucas |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: |
This fixes the 'Insert runtime parameter' when creating reports from SQL - nothing was happening when selecting any of the parameters to insert. (This is related to the OPAC and staff interface Bootstrap 5 upgrade in Koha 24.11.)
|
Version(s) released in: |
24.11.00
|
Circulation function: | |||
Bug Depends on: | 35402 | ||
Bug Blocks: | |||
Attachments: |
Bug 37734: Fix insert runtime parameter button in reports
Bug 37734: Fix insert runtime parameter button in reports Bug 37734: Fix insert runtime parameter button in reports Bug 37734: Fix insert runtime parameter button in reports Bug 37734: Fix insert runtime parameter button in reports Bug 37734: Fix insert runtime parameter button in reports |
Description
Brendan Lawlor
2024-08-26 13:47:07 UTC
Created attachment 170728 [details] [review] Bug 37734: Fix insert runtime parameter button in reports To test: 1. Go to Reports > Create from SQL 2. Click the Insert runtime parameter button 3. Click on any of the drop down options 4. Nothing happens :( 5. Apply patch 6. Repeat steps 2 and 3 for all different parameters 7. Notice the modal launches :) 8. Confirm the parameters are inserted as expected Created attachment 170734 [details] [review] Bug 37734: Fix insert runtime parameter button in reports To test: 1. Go to Reports > Create from SQL 2. Click the Insert runtime parameter button 3. Click on any of the drop down options 4. Nothing happens :( 5. Apply patch 6. Repeat steps 2 and 3 for all different parameters 7. Notice the modal launches :) 8. Confirm the parameters are inserted as expected Signed-off-by: David Nind <david@davidnind.com> Could you please add a description to the commit message, as per the commit message guidelines? https://wiki.koha-community.org/wiki/Commit_messages#Bug.2Ffeature_description Created attachment 170739 [details] [review] Bug 37734: Fix insert runtime parameter button in reports The drop down item links had two class attributes which prevented javascript from opening the modal. This patch combines the classes into one class attribute. To test: 1. Go to Reports > Create from SQL 2. Click the Insert runtime parameter button 3. Click on any of the drop down options 4. Nothing happens :( 5. Apply patch 6. Repeat steps 2 and 3 for all different parameters 7. Notice the modal launches :) 8. Confirm the parameters are inserted as expected Created attachment 170740 [details] [review] Bug 37734: Fix insert runtime parameter button in reports To test: 1. Go to Reports > Create from SQL 2. Click the Insert runtime parameter button 3. Click on any of the drop down options 4. Nothing happens :( 5. Apply patch 6. Repeat steps 2 and 3 for all different parameters 7. Notice the modal launches :) 8. Confirm the parameters are inserted as expected Signed-off-by: David Nind <david@davidnind.com> Created attachment 170741 [details] [review] Bug 37734: Fix insert runtime parameter button in reports The drop down item links had two class attributes which prevented javascript from opening the modal. This patch combines the classes into one class attribute. To test: 1. Go to Reports > Create from SQL 2. Click the Insert runtime parameter button 3. Click on any of the drop down options 4. Nothing happens :( 5. Apply patch 6. Repeat steps 2 and 3 for all different parameters 7. Notice the modal launches :) 8. Confirm the parameters are inserted as expected Signed-off-by: David Nind <david@davidnind.com> Thanks Brendan! (In reply to David Nind from comment #3) > Could you please add a description to the commit message, as per the commit > message guidelines? > https://wiki.koha-community.org/wiki/Commit_messages#Bug. > 2Ffeature_description Thanks for the reminder. I amended the commit and reattached, but I messed something up because it took off your sign off, so I tried again and think I did it correctly the second time around :) Created attachment 170775 [details] [review] Bug 37734: Fix insert runtime parameter button in reports The drop down item links had two class attributes which prevented javascript from opening the modal. This patch combines the classes into one class attribute. To test: 1. Go to Reports > Create from SQL 2. Click the Insert runtime parameter button 3. Click on any of the drop down options 4. Nothing happens :( 5. Apply patch 6. Repeat steps 2 and 3 for all different parameters 7. Notice the modal launches :) 8. Confirm the parameters are inserted as expected Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov> Looks good - thanks for the fix! Passing QA Pushed for 24.11! Well done everyone, thank you! |