Created attachment 191127 [details] [review] Bug 41570: Tidy kohaTable - course_reserves/course-details.tt Patch from commit 78c174e
Created attachment 191128 [details] [review] Bug 41570: Tidy kohaTable - course_reserves/course-reserves.tt Patch from commit 4cb6d9d
Created attachment 191129 [details] [review] Bug 41570: course_reserves - auto tidy Patch from commit 8f0d9c9
Created attachment 191389 [details] [review] Bug 41570: Tidy kohaTable - course_reserves/course-details.tt Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 191390 [details] [review] Bug 41570: Tidy kohaTable - course_reserves/course-reserves.tt Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 191391 [details] [review] Bug 41570: course_reserves - auto tidy Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 191392 [details] [review] Bug 41570: (follow-up) i18n correction, course deletion fix The template's i18n function, I18N.tnx, doesn't work with JavaScript variables, so I've updated it to use the template variable. I also found that I couldn't properly test the course deletion function without fixing a problem which exists in main: The delete course button didn't trigger the confirmation, and the event which was supposed to trigger the confirmation was attached to the whole form. The confirmation is now triggered on form submit. Signed-off-by: Owen Leonard <oleonard@myacpl.org>