https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29173
Martin Renvoize <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |martin.renvoize@ptfs-europe | |.com Status|Signed Off |Passed QA --- Comment #45 from Martin Renvoize <[email protected]> --- 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! -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
