If you save the form nothing is saved (and all silent)
Created attachment 172335 [details] [review] Bug 38069: (bug 37955 follow-up) Fix edition of table settings The form cannot longer be saved, and no appear appear (nothing in log, nothing on the UI) This is coming from bug 37955 where we modify the table_id to fix anchors, but we didn't care of the rest of the code using this TT variable. This patch suggests to replace # and _ with | Test plan: Confirm that the form can be edited, saved, and that it is taken correctly into account.
Created attachment 172364 [details] [review] Bug 38069: (bug 37955 follow-up) Fix edition of table settings The form cannot longer be saved, and no appear appear (nothing in log, nothing on the UI) This is coming from bug 37955 where we modify the table_id to fix anchors, but we didn't care of the rest of the code using this TT variable. This patch suggests to replace # and _ with | Test plan: Confirm that the form can be edited, saved, and that it is taken correctly into account. Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Trivial, PQA
Pushed for 24.11! Well done everyone, thank you!
Created attachment 174462 [details] [review] Bug 38069: Fix jump to the correct table
Picked follow-up "Fix jump to correct table" for main.
Created attachment 174681 [details] [review] Bug 38069: (follow-up) Fix jump to the correct table
(In reply to Jonathan Druart from comment #7) > Created attachment 174681 [details] [review] [review] > Bug 38069: (follow-up) Fix jump to the correct table Fixes JS error "Uncaught Error: Syntax error, unrecognized expression: #detail|holdings_table"
Picked the last patch "(follow-up) Fix jump to correct table" for main.
Not needed in 24.05.x