The template for adding/editing patron clubs has several instances of "onclick," in violation of coding guidelines. https://wiki.koha-community.org/wiki/Coding_Guidelines#JS9:_Avoid_the_use_of_event_attributes_like_.22onclick.22_to_attach_events They should be removed in favor of defining events in JavaScript instead of markup.
Created attachment 97128 [details] [review] Bug 24391: Remove event attributes from patron clubs templates This patch removes "onclick" attributes from the template for adding and editing patron club templates. To test, apply the patch and go to Tools -> Patron clubs -> New club template. Test the process of adding and deleting both club and enrollment fields. Test during the template add process and when editing an existing template.
Created attachment 97140 [details] [review] Bug 24391: Remove event attributes from patron clubs edit template This patch removes "onclick" attributes from the template for adding and editing patron club templates. To test, apply the patch and go to Tools -> Patron clubs -> New club template. Test the process of adding and deleting both club and enrollment fields. Test during the template add process and when editing an existing template.
Created attachment 97270 [details] [review] Bug 24391: Remove event attributes from patron clubs edit template This patch removes "onclick" attributes from the template for adding and editing patron club templates. To test, apply the patch and go to Tools -> Patron clubs -> New club template. Test the process of adding and deleting both club and enrollment fields. Test during the template add process and when editing an existing template. Signed-off-by: David Nind <david@davidnind.com>
Hi Owen, is the dependency correct here? It looks like there were patches on bug 18530 once, but there aren't now.
Created attachment 97373 [details] [review] Bug 24391: Remove event attributes from patron clubs edit template This patch removes "onclick" attributes from the template for adding and editing patron club templates. To test, apply the patch and go to Tools -> Patron clubs -> New club template. Test the process of adding and deleting both club and enrollment fields. Test during the template add process and when editing an existing template. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
(In reply to Katrin Fischer from comment #4) > Hi Owen, is the dependency correct here? Sorry, it should block, not depend on, 18530
Nice work everyone! Pushed to master for 20.05
Pushed to 19.11.x branch for 19.11.03
backported to 19.05.x for 19.05.08
Enhancement will not be backported to 18.11.x series.