https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42451 --- Comment #1 from Owen Leonard <oleonard@myacpl.org> --- (In reply to David Cook from comment #0)

Uncaught TypeError: Cannot read properties of undefined (reading 'backdrop')

This is an error related to Bootstrap modals which should not affect any functionality. It shows up on several staff interface pages. I'm not sure when this might have gotten broken, but it looks to me like the 'data-bs-toggle="modal"' might be overriding the click handler for "#update_sql_button". Since the button should act as a link which reloads the page I don't think we need the modal toggle action at all. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42451 --- Comment #3 from David Cook <dcook@prosentient.com.au> --- (In reply to Owen Leonard from comment #1)

(In reply to David Cook from comment #0)

...

Uncaught TypeError: Cannot read properties of undefined (reading 'backdrop')

This is an error related to Bootstrap modals which should not affect any functionality. It shows up on several staff interface pages.

I'm not sure when this might have gotten broken, but it looks to me like the 'data-bs-toggle="modal"' might be overriding the click handler for "#update_sql_button". Since the button should act as a link which reloads the page I don't think we need the modal toggle action at all.

I didn't have time to dig in as I was working on something else, but thanks for that insight. You're probably right. Happy for you to make the change if you want, or I'll get to it at some point -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42451 Thibaud Guillot (thibaud_g) <thibaud.guillot@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |42548 Depends on|42548 | Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42548 [Bug 42548] Add possibility to list multiple barcodes or one biblionumber on attach item action -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42451 --- Comment #4 from Jonathan Druart <jonathan.druart@gmail.com> --- Created attachment 199001 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=199001&action=edit Bug 42451: Remove JS error when converting a report Test plan: Create a report with: SELECT marcxml from biblioitems Hit /reports/guided_reports.pl?op=list Notice the "Update SQL" button in the "Update" column, Click "Update" => JS error "Uncaught TypeError: can't access property "backdrop", this._config is undefined" (in Firefox) And nothing happens Apply this patch, reload, click the button => Success, the report is updated -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42451 --- Comment #6 from Jonathan Druart <jonathan.druart@gmail.com> --- (In reply to Jonathan Druart from comment #5)

Created attachment 199002 [details] [review] Bug 42451: Remove duplicate success message

Patch from commit 65f47a8

Caused by commit cb01b253308060bad942723413feef1fd70825fd Bug 17047: SQL reports management with Mana-KB -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42451 Owen Leonard <oleonard@myacpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #199001|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42451 --- Comment #7 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 199006 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=199006&action=edit Bug 42451: Remove JS error when converting a report Test plan: Create a report with: SELECT marcxml from biblioitems Hit /reports/guided_reports.pl?op=list Notice the "Update SQL" button in the "Update" column, Click "Update" => JS error "Uncaught TypeError: can't access property "backdrop", this._config is undefined" (in Firefox) And nothing happens Apply this patch, reload, click the button => Success, the report is updated Signed-off-by: Owen Leonard <oleonard@myacpl.org> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42451 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lucas@bywatersolutions.com --- Comment #9 from Lucas Gass (lukeg) <lucas@bywatersolutions.com> --- (In reply to Owen Leonard from comment #7)

Created attachment 199006 [details] [review] Bug 42451: Remove JS error when converting a report

Test plan: Create a report with: SELECT marcxml from biblioitems Hit /reports/guided_reports.pl?op=list Notice the "Update SQL" button in the "Update" column, Click "Update" => JS error "Uncaught TypeError: can't access property "backdrop", this._config is undefined" (in Firefox) And nothing happens Apply this patch, reload, click the button => Success, the report is updated

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Is this still valid? there is no Update button in the Update column for me. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42451 --- Comment #11 from David Cook <dcook@prosentient.com.au> --- (In reply to Owen Leonard from comment #10)

(In reply to Lucas Gass (lukeg) from comment #9)

...

Is this still valid? there is no Update button in the Update column for me.

I see the update button if I create a new report with the SQL in the test plan, "SELECT marcxml from biblioitems." It says "This report seems obsolete, it uses biblioitems.marcxml field."

Yeah the key part is having old school fields in the SQL -- You are receiving this mail because: You are watching all bug changes.