https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29173 --- Comment #46 from Thibaud Guillot <thibaud.guillot@biblibre.com> --- (In reply to Martin Renvoize from comment #45)
This all works well and the QA script is happy.
I would have loved to see some consistency in the implementation between the two buttons however.. At first I thought this approach made the most logical sense, at the controller level, but as I worked through the code to understand how it works on the edit page I concluded that I think that's the cleaner approach pre-populating the search fields using the data already on display in the page rather than having to query for it again in the search controller.
Anyway.. I don't think that should hold this up.. but it might be nice to clean up as a follow-up bug.
Passing QA, thanks Thibaud, and welcome!
Thanks to you Martin :) -- You are receiving this mail because: You are watching all bug changes.