Summary: | Wrong reference to table_borrowers in circulation.tt | ||
---|---|---|---|
Product: | Koha | Reporter: | Jonathan Druart <jonathan.druart> |
Component: | Architecture, internals, and plumbing | Assignee: | Jonathan Druart <jonathan.druart> |
Status: | RESOLVED FIXED | QA Contact: | Martin Renvoize (ashimema) <martin.renvoize> |
Severity: | normal | ||
Priority: | P5 - low | CC: | caroline.cyr-la-rose, fridolin.somers, lucas, martin.renvoize, nick, oleonard |
Version: | unspecified | ||
Hardware: | All | ||
OS: | All | ||
See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36614 | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
24.05.00,23.11.06
|
|
Circulation function: | |||
Bug Depends on: | 17353, 21034 | ||
Bug Blocks: | |||
Attachments: |
Bug 30068: Remove table_settings_borrowers_table from circulation.tt
Bug 30068: Remove table_settings_borrowers_table from circulation.tt Bug 30068: Remove table_settings_borrowers_table from circulation.tt Bug 30068: Remove table_settings_borrowers_table from circulation.tt |
Description
Jonathan Druart
2022-02-10 10:20:57 UTC
olumns_settings_borrowers_table = [% TablesSettings.GetColumns( 'circ', 'circulation', 'table_borrowers', 'json' ) | $raw %] This is just fetching the settings from the yaml, they are only applied to "#table_borrowers" in circ-patron-search-results.js The table_borrowers that shows items is simply a class on the itemSearchFallBack table - this class is not referenced anywhere so can simply be removed I'm not sure if this is the bug I'm looking for, but I think table_borrowers doesn't refer to any table anymore? It used to be the patron search when searching for a patron for a checkout, but now this search simply falls back to the regular patron search (table is memberresultst). Just to make sure we're talking about the same thing: 1) In the main page of the staff interface, in the search bar at the top, with the tab 'Check out' selected, search for part of a name (e.g. "mar" should give several results with the sample patrons) --> This table used to be table_borrowers, in the past, we could configure it under Administration > Table settings > Circulation --> Note that the tab selected in the search bar at the top is now 'Search patrons' 2) In the main staff interface, in the search bar at the top, with the tab 'Search patrons' selected, search for part of a name --> This table is memberresultst, we can configure it under Administration > Table settings > Patrons Something was consolidated under 'Search patrons', which is fine, but I think the reference to that table under Administration > Table settings > Circulation was forgotten. Oh I think I found where table_borrowers is used: when searching for a patron to place a hold. So ignore my previous comment! Created attachment 164989 [details] [review] Bug 30068: Remove table_settings_borrowers_table from circulation.tt The history is a mess, but here we init a JS variable that is never used. Test plan: git grep table_settings_borrowers_table should not return anything after this patch applied Created attachment 165015 [details] [review] Bug 30068: Remove table_settings_borrowers_table from circulation.tt The history is a mess, but here we init a JS variable that is never used. Test plan: git grep table_settings_borrowers_table should not return anything after this patch applied oops, forgot the so line... Created attachment 165016 [details] [review] Bug 30068: Remove table_settings_borrowers_table from circulation.tt The history is a mess, but here we init a JS variable that is never used. Test plan: git grep table_settings_borrowers_table should not return anything after this patch applied Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> I also tried a couple of things like searching for patrons and placing a hold and nothing exploded. Created attachment 165027 [details] [review] Bug 30068: Remove table_settings_borrowers_table from circulation.tt The history is a mess, but here we init a JS variable that is never used. Test plan: git grep table_settings_borrowers_table should not return anything after this patch applied Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Pushed for 24.05! Well done everyone, thank you! Pushed to 23.11.x for 23.11.06 Doesn't apply cleanly to 23.05.x, no backport. |