There are a few places in the transport cost matrix template where event attributes (onclick, onblur) are used. These should be rewritten so that JS functionality can be separate from the markup.
Created attachment 50450 [details] [review] Bug 16305 - Remove the use of "onclick" from transport cost matrix This patch updates the transport cost matrix template to eliminate the use of event attributes (onclick and onblur) in favor of JS separate from the markup. To test, apply the patch and go to Administration -> Transport cost matrix. - Click in any table cell. Form fields in the cell should activate. - Check and uncheck the "disable" checkbox. The form field should be correctly disabled and enabled. - Fill various enabled fields and submit the form to confirm that data is saved correctly.
Created attachment 50455 [details] [review] Bug 16305 - Remove the use of "onclick" from transport cost matrix This patch updates the transport cost matrix template to eliminate the use of event attributes (onclick and onblur) in favor of JS separate from the markup. To test, apply the patch and go to Administration -> Transport cost matrix. - Click in any table cell. Form fields in the cell should activate. - Check and uncheck the "disable" checkbox. The form field should be correctly disabled and enabled. - Fill various enabled fields and submit the form to confirm that data is saved correctly. Signed-off-by: Marc Véron <veron@veron.ch>
Created attachment 50624 [details] [review] [PASSED QA] Bug 16305 - Remove the use of "onclick" from transport cost matrix This patch updates the transport cost matrix template to eliminate the use of event attributes (onclick and onblur) in favor of JS separate from the markup. To test, apply the patch and go to Administration -> Transport cost matrix. - Click in any table cell. Form fields in the cell should activate. - Check and uncheck the "disable" checkbox. The form field should be correctly disabled and enabled. - Fill various enabled fields and submit the form to confirm that data is saved correctly. Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Pushed to master for Koha 16.05. Thanks Owen!