https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34784 --- Comment #11 from Kyle M Hall <kyle@bywatersolutions.com> ---
Most of these are cosmetic and should be quick to follow-up:
1) Needs a follow-up to fix documentation.
+=head3 populate_empty_callnumbers + +Controller function that handles deleting a biblio object
Fixed!
2) Spelling
In the item form and the holding table we are using 'call number', but the patch uses 'callnumber'. I believe we should follow the GUI.
Fixed!
3) I feel we should include the call number that would be used in the confirmation question.
if ( confirm(__('Are you sure you want to populate callnumbers for all items without a callnumber on this record?')) )
Wow, that was more work than I expected but I was able to make that happen!
4) Please no line breaks in translatable strings!
This is a known issue with Pootle, not sure about Weblate, but translators often might not know what it is and accidentally remove it.
msg += __('\nReload the page?'); let msg = __('Item populated with the callnumber "%s"\nReload the page?').format(callnumber);
I think I've fixed this based on your description. We should add a coding guideline for this.
4) Button in the holdings table
The button for every single item in the holdings table takes up quite a bit of space. I wonder if we could find a way to make this work a little better. Maybe a split button with Edit as the main action?
Done!
5) Button in the detail page's toolbar (not blocker)
Why not move this into the edit menu?
Done!
4) Maintenance script? (not blocker)
I wonder about the use case to require such a one by one change. If you have itemcallnumber set, this would only apply to items that have been created before, that have been imported or otherwise added from outside of Koha.
Are there plans for a maintenance script to do this in batch?
There are no plans for a maintenance script but I'd be happy to make one on a followup bug report if you can provide the parameters you'd like to see! -- You are receiving this mail because: You are watching all bug changes.