addbiblio.pl should use GetFrameWorksLoop instead of getframeworks
Created attachment 52799 [details] [review] Bug 16807 - Frameworks unordered in dropdown when adding/editing a biblio To test: Start to add or edit a biblio Open the dropdown to change framework Notice the order Refresh the page - notice the order changes Apply patch Order is alphabetical Refresh the page Order does not change
Created attachment 52808 [details] [review] [SIGNED-OFF]Bug 16807: Frameworks unordered in dropdown when adding/editing a biblio To test: Start to add or edit a biblio Open the dropdown to change framework Notice the order Refresh the page - notice the order changes Apply patch Order is alphabetical Refresh the page Order does not change Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com> Works as described
I removes GetFrameWorksLoop in bug 15801 (needs QA), could you please avoid to introduce a new call to it?
s/removes/removed
Created attachment 52828 [details] [review] Bug 16807 - Use Koha::BiblioFrameworks instead of GetFrameWorkLoop Refactor code to use work from 14889
Created attachment 52833 [details] [review] Bug 16807: Frameworks unordered in dropdown when adding/editing a biblio To test: Start to add or edit a biblio Open the dropdown to change framework Notice the order Refresh the page - notice the order changes Apply patch Order is alphabetical Refresh the page Order does not change Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com> Works as described Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 52834 [details] [review] Bug 16807 - Use Koha::BiblioFrameworks instead of GetFrameWorkLoop Refactor code to use work from 14889 Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Pushed to master for 16.11, thanks Nick!
Can this be back-ported? It looks like a fairly small patch.
Pushed in 16.05. Will be in 16.05.02.