Attachment 20320 Details for Bug 10720 – Bug 10720 : substitution term in overdue notices shouldn't be removed if they don't match. It prevents use of html in notices to use with css

[patch] Bug 10720 : substitution term in overdue notices shouldn't be removed if they don't match. It prevents use of html in notices to use with css

Bug-10720--substitution-term-in-overdue-notices-sh.patch (text/plain), 1.95 KB, created by Sophie MEYNIEUX on 2013-08-13 16:22:12 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Sophie MEYNIEUX

Created: 2013-08-13 16:22:12 UTC

Size: 1.95 KB

patch

obsolete

>From a19890297703f06d043ef20d2a21d83e2d0ed4b7 Mon Sep 17 00:00:00 2001
>From: Sophie Meynieux <sophie.meynieux@biblibre.com>
>Date: Tue, 13 Aug 2013 18:14:21 +0200
>Subject: [PATCH] Bug 10720 : substitution term in overdue notices shouldn't
> be removed if they don't match.         It prevents use of
> html in notices to use with css
>
>        Test plan :
>        - define an overdue notice containing some valid substitution terms + some valid html tags (<strong> or <div class="my class" to use with a css
>        - find a borrower with overdues that trigger this notice
>        - run overdue_notices.pl -html <directory>
>
>        Without patch, your html tags are missing in your html file
>        With the patch, they are present and can be used with a css to generate a formatted pdf file.
>
>        If you add some invalid substitution term, with the patch, they are still in the final letter, but the warn in the log helps you to correct your notice template.
>---
> misc/cronjobs/overdue_notices.pl |    4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/misc/cronjobs/overdue_notices.pl b/misc/cronjobs/overdue_notices.pl
>index 5f285c0..3c8ff4b 100755
>--- a/misc/cronjobs/overdue_notices.pl
>+++ b/misc/cronjobs/overdue_notices.pl
>@@ -568,8 +568,8 @@ END_SQL
>                 if (@misses) {
>                     $verbose and warn "The following terms were not matched and replaced: \n\t" . join "\n\t", @misses;
>                 }
>-                $letter->{'content'} =~ s/\<[^<>]*?\>//g;    # Now that we've warned about them, remove them.
>-                $letter->{'content'} =~ s/\<[^<>]*?\>//g;    # 2nd pass for the double nesting.
>+                #$letter->{'content'} =~ s/\<[^<>]*?\>//g;    # Now that we've warned about them, remove them.
>+                #$letter->{'content'} =~ s/\<[^<>]*?\>//g;    # 2nd pass for the double nesting.
> 
>                 if ( !$nomail && scalar @emails_to_use ) {
>                     C4::Letters::EnqueueLetter(
>-- 
>1.7.10.4

Actions: View | Diff | Splinter Review

Attachments on bug 10720: 20320 | 20756 | 20829