The translation processing issue no error (translate install fr-FR), but the resulting page for almost one template is incorrect (I don't speak about translated string, but structure of resulting file). In 'koha-tmpl/opac-tmpl/prog/en/modules/sco/sco-main.tt', line 135: <input type="button" value="Renew Item" [% UNLESS ( renew ) %] name="confirm"[% END %] class="renew" onclick="this.form.confirmed.value='1';this.form.submit();" /> The result, 'koha-tmpl/opac-tmpl/prog/fr-FR/modules/sco/sco-main.tt', line 88: <input value="Renouveler document" name="confirm" unless="UNLESS" %]="%]" renew="renew" end="END" )=")" type="button" onclick="this.form.confirmed.value='1';this.form.submit();" class="renew" (="(" [%="[%" /> This kind of error arrived two times in 'sco/sco-main.tt'. Note: The result is in reality more complex: 7 lines of 'en' template are mixed in 1 line in 'fr-FR' result template.
It seems the problem is in 'misc/translator/TTParser.pm', in the processing of tag (function 'start', for handling opening html tags). It isn't expected to have "template text" at the tag level: <tagname attr1="value1" templateText attr2="value2" ...> So TTParser interprete this as: <tagname attr1="value1" templateText="templateText" attr2="value2" ...> In the reported example: <input type="button" value="Renew Item" [% UNLESS ( renew ) %] name="confirm"[% END %] class="renew" onclick="this.form.confirmed.value='1';this.form.submit();" /> TTParser read as: tagname=input attributes = { In template, the two only occurrences I found in this scheme are in 'modules/sco/sco-main.tt'...
sorry for comments before: tabulation don't work well in textarea, and the next focused item is 'Save Changes' button... Redo : <input type="button" value="Renew Item" [% UNLESS ( renew ) %] name="confirm"[% END %] class="renew" onclick="this.form.confirmed.value='1';this.form.submit();" /> TTParser read as: tagname=input attributes = { type = "button", value = "Renew Item", [% = "[%", unless = "UNLESS", %] = "%]", ... } The re-ordoring at the end is due to order in hashtable...
I have found few others cases with Template Directive in tag, but where there are not problem with transformation. - in 'opac-user.tt' <tr[% IF ( overdue ) %] class="overdue"[% END %]> - in 'opac-detail.tt' <li[% IF ( defaulttab == 'subscriptions' ) %] class="ui-tabs-selected"[% END %]> The difference seems to be that here, there are not string to be translated. So problem may be corrected in 'tmpl_process3.pl': function 'text_replace_tag'. Currently, if translation is found, the HTML element is "rewrited" based on attributes parsed (where directives are not tagged as DIRECTIVE, and in HASHTABLE [so order is lost]). So two solutions: - work in 'TTParser.pm' for correctly tag DIRECTIVE in tag-element, and processing accordingly in 'text_replace_tag' - work in 'tmpl_process3.pl' for another method (like regex, but it not a very strong method...)
here, a discussion about his problem: http://lists.koha-community.org/pipermail/koha-devel/2011-July/035917.html
(In reply to comment #4) > here, a discussion about his problem: > http://lists.koha-community.org/pipermail/koha-devel/2011-July/035917.html Could list html tags within which TT directives can be found? I would like to port your test case, bug I'm not very fluent in haskell...
Currently, the parser search the string "[%" in *every* HTML tag. But it isn't a HTML parser. It just track when the string "[%" in placed after the string "<" (and not ">" between "<" and "[%"), and not between two string '"'. (but need to track also CDATA and COMMENT because of false-positive else). It follow a *very simplified* HTML syntax for parsing. It is relatively simple (and commented), and based on a "context" concept: it just defined rules for enter in a "context", and to change the "context", when a specific string is encounter. For the INITIAL context (the default one): - when "<![CDATA[" string is encounter: go in CDATA context - when "<!--" string is encounter: go in COMMENT context - when "<" string encounter: go in TAG context For the TAG context: - when encounter the string '"' : enter in STRING context - when encounter the string "[%" : mark a problem, as a template directive is found in tag, but not in an attribute (because an attribute is a STRING context) - when encounter the string ">": go in INITIAL context For the STRING context: - when encounter the string '\"' : keep the STRING context - when encounter only the string '"' : go in TAG context (in CDATA context, how to go in INITIAL context : "]]>" encounter ; or in COMMENT context: "-->" ) That's all... (but the parser keep track the line number for reporting). I hope I answer your question, and that I have clarified how work the parser...
Created attachment 4695 [details] [review] Proposed patch Could you try this test case and check that it detects the appropriate TT issues?
(In reply to comment #7) > Created attachment 4695 [details] [review] > Proposed patch > > Could you try this test case and check that it detects the appropriate TT > issues? Issues are incomplete: haskell version report 269 issues in 45 files, versus perl 34 issues in 14 files. The list could be consulted here: http://lists.koha-community.org/pipermail/koha-devel/2011-July/035922.html , in the attachment (but *.inc are not included, and "269 in 45 files" is the last count on master repo) There are multiples problems with this perl version: - it is line oriented versus stream oriented. So in case of opening tag on one line, and inclusion of directive in other line, the issue is not reported. (for examples: see ./koha-tmpl/opac-tmpl/prog/en/modules/opac-messaging.tt, line 74 see ./koha-tmpl/opac-tmpl/prog/en/includes/doc-head-close.inc, line 42 ) - it only check directive near the tagname (like <li[%IF ), and not more general construct (like <option value="10"[%IF ) (for examples: see ./koha-tmpl/opac-tmpl/prog/en/modules/sco/sco-main.tt, line 135 see ./koha-tmpl/intranet-tmpl/prog/en/modules/admin/systempreferences.tt, line 91 )
> There are multiples problems with this perl version: You're correct. I have no solution in Perl, neither time. We have TTParser module. So we should use it to parse properly templates and identify invalid constructions.
Created attachment 4697 [details] [review] Amended proposed patch Please try this new version of test case. It uses TTParser. Tell me if more invalid constructions are found.
Created attachment 4699 [details] [review] Proposed patch (2)
(In reply to comment #11) > Created attachment 4699 [details] [review] > Proposed patch (2) This version works very well. There are some differences between the haskell version and the perl one, but there are not generally problematic: - problems not related of TT template - haskell version report also problems with CDATA section, partial quoting attribute (and a false-positive for use of single-quoting in attributes) - differences but not a problem - haskell version report multiple errors for a tag (if multiple directive inclusion) - haskell version report line number of the directive, although perl version report line number of the beginning of tag with directive. The only amelioration to do, is to include the *.inc for parsing: some of them included directive in tag... else perfect ! So in don't sign-off, waiting the corrected version; and I also need more time to check for side-effect due to move of TTParser files in C4. Thanks.
Created attachment 4729 [details] [review] Proposed patch for testing and signing off With the last proposed patch, .inc files are also validated. I agree that moving translation modules into C4 requires some testing. I've tested using standard translation tools: ./translate create | update | install.
Created attachment 4968 [details] [review] [SIGNED-OFF] Bug 6458 Template Toolkit files test case This test validate Template Toolkit (TT) Koha files. For the time being an unique validation is done: Test if TT files contain TT directive within HTML tag. For example: <li[% IF This kind of constuction MUST be avoided because it break Koha translation process. This patch transform also translation specific modules into C4 modules in order to be able to use them in test case: C4::TTPaser C4::TmplToken C4::TmplTokenType This patch is a Perl adaptation of a Haskell script from Frère Sébastien Marie. Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Notes on testing: - translate install de-DE - worked ok - translate update de-DE > translate install de-DE - worked ok - running the test xt/tt_valid.t - worked ok and pointed out lots of problems. Found no problems. Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Pushed.
Created attachment 5070 [details] [review] Bug 6458: Incorrect parsing result in translation processing Patch removes template directives from within HTML tags from OPAC detail view. Problems were related to the items tabs for holdings, subscriptions, and serial collection.
Created attachment 5071 [details] [review] Bug 6458: Incorrect translation processing / sco-main.tt Patch removes templates directives from HTML tags in sco-main.tt Problems were related to buttons for renewing items.
Created attachment 5072 [details] [review] Bug 6458: Incorrect translation processing - masthead.inc Patch removes templates directives from HTML tags in masthead.inc Problem was related to library pull down shown when OpacAddMastheadLibraryPulldown is activated.
Created attachment 5074 [details] [review] Bug 6458: Incorrect translation processing - doc-head-close/open.inc Patch removes templates directives from HTML tags in masthead.inc
Created attachment 5078 [details] [review] Bug 6458: Incorrect parsing result in translation processing Patch removes template directives from within HTML tags from OPAC detail view. Problems were related to the items tabs for holdings, subscriptions, and serial collection. Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Before patch opac-tmpl/prog/en/modules/opac-detail.tt: 325, 353, 369 After, no errors for opac-detail.tt
Created attachment 5079 [details] [review] Bug 6458: Incorrect translation processing / sco-main.tt Patch removes templates directives from HTML tags in sco-main.tt Problems were related to buttons for renewing items. Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Before opac-tmpl/prog/en/modules/sco/sco-main.tt: 135, 222 After, no errors
Created attachment 5080 [details] [review] Bug 6458: Incorrect translation processing - masthead.inc Patch removes templates directives from HTML tags in masthead.inc Problem was related to library pull down shown when OpacAddMastheadLibraryPulldown is activated. Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> Before opac-tmpl/prog/en/includes/masthead.inc: 83 After no errors
I have signed off on 3 of these patches, but the doc-head-close and open still needs a bit more work Before # opac-tmpl/prog/en/includes/doc-head-close.inc: 41, 77 # opac-tmpl/prog/en/includes/doc-head-open.inc: 3 After # opac-tmpl/prog/en/includes/doc-head-close.inc: 80 So still needs some work
Comment on attachment 5078 [details] [review] Bug 6458: Incorrect parsing result in translation processing Hi, The patch on 'opac-detail.tt' don't preserve the semantic, of original IF/ELSE clause: in the correction the IF clause and the ELSE clause are the same... The code is currently: +[% IF ( defaulttab == 'holdings' ) %]<li class="ui-tabs-selected"> +[% ELSE %]<li class="ui-tabs-selected">[% END %] But it should be: +[% IF ( defaulttab == 'holdings' ) %]<li class="ui-tabs-selected"> +[% ELSE %]<li>[% END %] The same problem occurs for all corrections in opac-detail.tt (but others patchs seems good).
Created attachment 5081 [details] [review] Bug 6458: Incorrect translation processing - doc-head-close/open.inc Patch removes templates directives from HTML tags in masthead.inc Fixes all 3 problems now.
Created attachment 5082 [details] [review] Bug 6458: Incorrect translation processing / opac-detail.tt Patch removes template directives from within HTML tags from OPAC detail view. Problems were related to the items tabs for holdings, subscriptions, and serial collection. Fixed the problem spotted by Frère Sébastien Marie - thx!
Created attachment 5090 [details] [review] Bug 6458: Incorrect translation processing / overdue.tt Patch removes template directives from within HTML tags from circulation > overdues. Problems were related to patron flags pull down and sort by loop for patron attributes.
Created attachment 5091 [details] [review] Bug 6458: Incorrect translation processing / review.tt Patch removes template directives from within HTML tags from Tools > Tags.
Created attachment 5092 [details] [review] Bug 6458: Incorrect translation processing / authorities.tt Patch removes template directives from within HTML tags from authorities > modify authority.
Created attachment 5093 [details] [review] Bug 6458: Incorrect translation processing / 3x reports/... Patch removes template directives from within HTML tags from - Reports > Guided Reports - Reports > Items with no checkouts - Reports > Patron statistics
Created attachment 5094 [details] [review] Bug 6458: Incorrect translation processing / acqui/... Patch removes template directives from within HTML tags from - acquisitions > new vendor / modify vendor / vendor details - acquisitions > late orders - acquisitons > vendor x > basket groups
Created attachment 5095 [details] [review] Bug 6458: Incorrect translation processing / cataloguing/... Patch removes template directives from within HTML tags from - cataloging > new record - cataloging - UNIMARC plugin for 4XX There is one problem remaining in the UNIMARC plugin: <option value="[% frameworkcodeloo.value %]" [% frameworkcodeloo.selected %]> [% frameworkcodeloo.frameworktext %] </option>
Created attachment 5096 [details] [review] Bug 6458: Incorrect translation processing / cataloguing/... Patch removes template directives from within HTML tags from - cataloging > new record - cataloging - UNIMARC plugin for 4XX There is one problem remaining in addbiblio: <option value="[% frameworkcodeloo.value %]" [% frameworkcodeloo.selected %]> [% frameworkcodeloo.frameworktext %] </option>
Created attachment 5121 [details] [review] Bug 6458: Incorrect translation processing - rotating_collections/... Patch removes templates directives from HTML tags in the rotating collections module. The module is currently not interated into Koha's staff interface. Fixing templats to make Jenkins happy.
Created attachment 5125 [details] [review] Signed off patch correcting doc-head-close.inc and doc-head-open.inc
Created attachment 5126 [details] [review] Signed off patch correcting overdue.tt
Created attachment 5127 [details] [review] Signed-off patch correcting review.tt
Created attachment 5128 [details] [review] Signed-off patch correcting authorities.tt
Created attachment 5129 [details] [review] Signed off patch correcting 3x reports
Created attachment 5130 [details] [review] Signed off patch correcting masthead.inc
Created attachment 5131 [details] [review] Signed off patch correcting acqui templates
Created attachment 5132 [details] [review] Signed off patch correcting cataloging templates
Created attachment 5133 [details] [review] Bug 6458: Incorrect translation processing / serials/... Fixes problems with template toolkit directives within HTML tags for the serials module. Serials > add/modify subsciption serial-issues-full.tt seems to be unused? I couldn't find the page in the serials module.
Created attachment 5147 [details] [review] Signed off patch correcting opac-detail.tt
Created attachment 5148 [details] [review] Signed off patch correcting rotating collections
Created attachment 5149 [details] [review] Signed off patch correcting serials templates
Created attachment 5150 [details] [review] Signed-off patch correcting sco-main.tt
Working on memberentrygen.tt. Somebody please stop me if they are too!
Created attachment 5152 [details] [review] Bug 6458: Incorrect translation processing / suggestion Removes template toolkit directives from within HTML tags. Acquisitions > Suggestions
Created attachment 5153 [details] [review] Bug 6458: Incorrect translation processing / results Removed template toolkit directives from within HTML tags. Advanced Search > Scan Index > Search results Additional changes: - removes entry 'Series Title' from the index pull down as it was the same index as 'Series'
Created attachment 5155 [details] [review] Proposed fix for opac-reserve.tt Correction in opac holds template. Corrections made are logically consistent, but I couldn't reproduce the combination of factors to trigger the "disabled" attribute on the branch selector. In that respect this patch hasn't been tested 100%.
Created attachment 5156 [details] [review] Signed off patch correcting suggestions
Created attachment 5157 [details] [review] Signed off patch correcting results.tt
Created attachment 5158 [details] [review] Proposed fix for opac-messaing.tt Correction for opac-messaging.tt tops tag attribute markup from being passed to the template directly in favor of using template logic.
Created attachment 5159 [details] [review] Proposed fix for memberentrygen.tt Fixing improperly nested template logic inside HTML tags in member entry form.
Created attachment 5160 [details] [review] Proposed fix for patron-attr-types.tt Correction for patron-attr-types.tt stops tag attribute markup from being passed to the template directly in favor of using template logic.
Created attachment 5168 [details] [review] Proposed fix for admin/clone-rules.tt Fixing improperly nested template logic inside HTML tags in clone circulation rules form.
Created attachment 5169 [details] [review] Proposed fix for admin/branches.tt Correction for branch admin template stops tag attribute markup from being passed to the template directly in favor of using template logic.
Created attachment 5170 [details] [review] Proposed fix for admin/branch_transfer_limits.tt Fixing improperly nested template logic inside HTML tags in branch transfer limits template.
Comment on attachment 5156 [details] [review] Signed off patch correcting suggestions In the patch 'suggestion/suggestion.tt', in the second block of corrections, the semantic seems to not be preserved: Here, just an example, for the first line (of second block): Orign: [% IF ( suggestion.statusselected_ASKED ) %] Patch: [% IF (statusselected_ASKED ) %]
Created attachment 5258 [details] [review] Proposed fix for header.inc Fixing improperly nested template logic inside HTML tags in header.inc.
Created attachment 5259 [details] [review] Proposed fix for admin/aqplan.tt Fixing improperly nested template logic inside HTML tags in aqplan template.
Created attachment 5260 [details] [review] Proposed fix for moremember.tt Fixing improperly nested template logic inside HTML tags in member details template.
Created attachment 5261 [details] [review] Proposed fix for labels/spinelabel-print.tt Fixing improperly nested template logic inside HTML tags in spine label print template.
Created attachment 5262 [details] [review] Proposed fix for admin/systempreferences.tt Fixing improperly nested template logic inside HTML tags in system preferences editor.
Comment on attachment 5158 [details] [review] Proposed fix for opac-messaing.tt This patch should be combined with one for includes/messaging-preference-form.inc. I'll submit a new patch.
Created attachment 5263 [details] [review] Proposed fix for addbiblio.tt Fixing improperly nested template logic inside HTML tags in MARC editor. Modifying script to stop tag attribute markup from being passed to the template directly in favor of using template logic.
Whoops, forgot that some fixes had already been applied to addbiblio.tt. That last one may not apply in conjunction with attachment 5132 [details] [review].
STATUS UPDATE Assuming all patches proposed patches get signed off, this is the current output of xt/tt_valid.t: # Files list: # opac-tmpl/prog/en/modules/opac-messaging.tt: 62, 71, 81 # intranet-tmpl/prog/en/modules/tools/csv-profiles.tt: 104 # intranet-tmpl/prog/en/modules/tools/manage-marc-import.tt: 160 # intranet-tmpl/prog/en/includes/messaging-preference-form.inc: 57, 62, 75, 80, 93, 98 # intranet-tmpl/prog/en/includes/doc-head-open.inc: 3 # intranet-tmpl/prog/en/includes/budgets-admin-search.inc: 64
Created attachment 5273 [details] [review] [SIGNED-OFF] Fix for Bug 6458 - incorrect parsing result in translation processing / addbiblio Fixing improperly nested template logic inside HTML tags in MARC editor. Modifying script to stop tag attribute markup from being passed to the template directly in favor of using template logic. Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> I fixed the conflict with previous patch for addbiblio.tt.
Created attachment 5274 [details] [review] [SIGNED-OFF] Fix for Bug 6458 - incorrect parsing result in translation processing / systempreferences Fixing improperly nested template logic inside HTML tags in the system preferences editor. Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Created attachment 5286 [details] [review] [SIGNED-OFF] Fix for Bug 6458 - incorrect parsing result in translation processing / memberentrygen Fixing improperly nested template logic inside HTML tags in member entry form. Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de> Fixes lots of problems. Spotted one small problem. I am sending a follow-up.
Created attachment 5287 [details] [review] Fix for Bug 6458 Follow-up for memberentrygen.tt The registration date field would show up twice after applying the first patch for memberentrygen.tt. This patch fixes the problem.
Created attachment 5288 [details] [review] Fix for Bug 6458 - incorrect parsing result in translation processing Correction for patron-attr-types.tt stops tag attribute markup from being passed to the template directly in favor of using template logic. Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de> Read the patch and tested the page, didn't find any problems.
Created attachment 5289 [details] [review] [SIGNED-OFF] Fix for Bug 6458 - incorrect parsing result in translation processing / patron-attr-types Correction for patron-attr-types.tt stops tag attribute markup from being passed to the template directly in favor of using template logic. Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de> Read the patch and tested the page, didn't find any problems.
(In reply to comment #60) > In the patch 'suggestion/suggestion.tt', in the second block of corrections, > the semantic seems to not be preserved: > > Here, just an example, for the first line (of second block): > Orign: [% IF ( suggestion.statusselected_ASKED ) %] > Patch: [% IF (statusselected_ASKED ) %] I think this modification is correct. Line 305 of suggestion.pl: $template->param( "statusselected_$$suggestion_ref{'STATUS'}" =>1); It doesn't look like that variable needs any scope. My search of the source indicates that there isn't any other place where a variable with that name is created. The behavior of the page seems to be the same with our without the variable scope, and I think this is because the variable never evaluates as true. I think the logic around that <option> is unused. I can't see what circumstances would lead to that status having a preselected value.
Created attachment 5292 [details] [review] [SIGNED-OFF] Fix for Bug 6458 - incorrect parsing result in translation processing / opac-reserve Correction in opac holds template. Corrections made are logically consistent, but I couldn't reproduce the combination of factors to trigger the "disabled" attribute on the branch selector. In that respect this hasn't been tested 100%. Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Created attachment 5293 [details] [review] Proposed fix for tools/manage-marc-import.tt There is template logic in manage-marc-import.tt which is intended to set the "selected" option for the framework chooser ("Add new bibliographic records into this framework"). However, I can't see any place in the code which sends the "selected" variable to the template, and I can't see any circumstance in which a value would be passed to the template. This patch removes the "selected" variable from the template.
Created attachment 5299 [details] [review] [SIGNED-OFF] Fix for Bug 6458 - incorrect parsing result in translation processing / clone-rules Fixing improperly nested template logic inside HTML tags in clone circulation rules form. Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Created attachment 5300 [details] [review] [SIGNED-OFF] Fix for Bug 6458 - incorrect parsing result in translation processing / branches Correction for branch admin template stops tag attribute markup from being passed to the template directly in favor of using template logic. Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
I have pushed up to Signed off patch correcting results.tt, please sign off the others and I will push the rest
Created attachment 5416 [details] [review] Proposed fix for budgets-admin-search.inc Fixing improperly nested template logic inside HTML tags in budgets admin search form.
Created attachment 5417 [details] [review] Proposed fix for tools/csv-profiles.tt Fixing improperly nested template logic inside HTML tags in CSV profiles template Also fixing improperly scoped template variable which prevented the current profile from being preselected under the edit tab.
Created attachment 5418 [details] [review] Follow-up fix for addbiblio.tt Somehow some markup which should have been deleted by one of this bug's pushed patches has reappeared, messing up the display of the MARC editor. This patch removes that section again.
Created attachment 5419 [details] [review] Follow-up fix for Bug 6458, incorrect parsing result in translation processing Somehow some markup which should have been deleted by a previous patch has reappeared, messing up the display of the MARC editor. This patch removes that section again. Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Created attachment 5420 [details] Follow-up fix for Bug 6458, incorrect parsing result in translation processing Somehow some markup which should have been deleted by a previous patch has reappeared, messing up the display of the MARC editor. This patch removes that section again. Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
The content of attachment 5420 [details] has been deleted by Chris Cormack <chris@bigballofwax.co.nz> without providing any reason. The token used to delete this attachment was generated at 2011-09-14 10:13:19 NZST.
I have pushed all the patches attached to this, apart from the one left which does not apply.
not ok 1 - TT syntax: not using TT directive within HTML tag # Failed test 'TT syntax: not using TT directive within HTML tag' # at xt/tt_valid.t line 55. # Files list: # opac-tmpl/prog/en/modules/opac-messaging.tt: 62, 71, 81 # intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt: 782 # intranet-tmpl/prog/en/includes/messaging-preference-form.inc: 57, 62, 75, 80, 93, 98 # intranet-tmpl/prog/en/includes/doc-head-open.inc: 3 Here are the things left failing
I have patches ready to submit for the last few. Will do so ASAP.
Comment on attachment 5158 [details] [review] Proposed fix for opac-messaing.tt I'm removing obsolete status from this patch because it doesn't look like it got pushed.
Created attachment 5425 [details] [review] Proposed fix for doc-head-open.inc Correcting doc-head-open.inc (again?)
Created attachment 5426 [details] [review] Revised fix for addbiblio.tt
Created attachment 5427 [details] [review] Proposed fix for messaging preference form Fixing improperly nested template logic inside HTML tags in messaging preferences include. NOTE: This patch requires the patch for opac-messaging.tt, which includes a change to C4/Form/MessagingPreferences.pm
Pushed, all TT tests now passing WOOT!
The fix for this bug was published in the 3.4.5 release. If you were the reporter of this bug, please take time to verify the fix and update the status of this bug report accordingly. If the bug is fixed to your satisfaction, please close this report.
(In reply to comment #96) > The fix for this bug was published in the 3.4.5 release. If you were the > reporter of this bug, please take time to verify the fix and update the status > of this bug report accordingly. If the bug is fixed to your satisfaction, > please close this report. Thanks.