Summary: | Adding a patron shows javascript on screen under login section | ||
---|---|---|---|
Product: | Koha | Reporter: | Marcel de Rooy <m.de.rooy> |
Component: | Templates | Assignee: | Owen Leonard <oleonard> |
Status: | CLOSED FIXED | QA Contact: | Jonathan Druart <jonathan.druart> |
Severity: | major | ||
Priority: | P5 - low | CC: | bgkriegel, gmcharlt, jonathan.druart, kyle.m.hall |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | Trivial patch | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Bug Depends on: | |||
Bug Blocks: | 9556 | ||
Attachments: |
Bug 9529 - Adding a patron shows javascript on screen under login section
Bug 9529 - Adding a patron shows javascript on screen under login section Bug 9529 - Adding a patron shows javascript on screen under login section |
Description
Marcel de Rooy
2013-02-01 11:16:35 UTC
This appears with EnhancedMessagingPreferences set to Allow. The code is in koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt The JS code can be hidden with <!-- ... -->. The confirm message needs to be between " (double quotes), can't produce a situation to trigger it. Don't know if this code is used. Created attachment 15104 [details] [review] Bug 9529 - Adding a patron shows javascript on screen under login section A typo in my patch for Bug 9265 (HTML5 switch) left an extra quotation mark in a <script> tag on the patron entry form, causing JavaScript code to be output as text instead of interpreted. This patch corrects it. To test, apply the patch, enable EnhancedMessagingPreferences, and load the patron entry form. You should see no JavaScript on screen around the 'Patron messaging preferences' fieldset. Created attachment 15106 [details] [review] Bug 9529 - Adding a patron shows javascript on screen under login section A typo in my patch for Bug 9265 (HTML5 switch) left an extra quotation mark in a <script> tag on the patron entry form, causing JavaScript code to be output as text instead of interpreted. This patch corrects it. To test, apply the patch, enable EnhancedMessagingPreferences, and load the patron entry form. You should see no JavaScript on screen around the 'Patron messaging preferences' fieldset. Signed-off-by: Galen Charlton <gmc@esilibrary.com> Created attachment 15130 [details] [review] Bug 9529 - Adding a patron shows javascript on screen under login section A typo in my patch for Bug 9265 (HTML5 switch) left an extra quotation mark in a <script> tag on the patron entry form, causing JavaScript code to be output as text instead of interpreted. This patch corrects it. To test, apply the patch, enable EnhancedMessagingPreferences, and load the patron entry form. You should see no JavaScript on screen around the 'Patron messaging preferences' fieldset. Signed-off-by: Galen Charlton <gmc@esilibrary.com> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> This patch has been pushed to master. |