From f5114fbd426e1f9aed4680dbde6bab55fa5ad5c2 Mon Sep 17 00:00:00 2001
From: Alex Buckley <alexbuckley@catalyst.net.nz>
Date: Wed, 6 Apr 2022 12:40:01 +0000
Subject: [PATCH] Bug 23538: (follow-up) Fix up notices

- Add toolkit template (TT) syntax to notices
- Add TT conditionals to notices

Sponsored-by: Catalyst IT

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
---
 ...lAddressForPatronRegistrations_sysprefs.pl | 21 ++++++++++---------
 .../mysql/en/mandatory/sample_notices.yml     | 21 ++++++++++---------
 2 files changed, 22 insertions(+), 20 deletions(-)

diff --git a/installer/data/mysql/atomicupdate/bug23538-add_EmailPatronRegistrations_and_EmailAddressForPatronRegistrations_sysprefs.pl b/installer/data/mysql/atomicupdate/bug23538-add_EmailPatronRegistrations_and_EmailAddressForPatronRegistrations_sysprefs.pl
index 55082df006..daa80ba933 100755
--- a/installer/data/mysql/atomicupdate/bug23538-add_EmailPatronRegistrations_and_EmailAddressForPatronRegistrations_sysprefs.pl
+++ b/installer/data/mysql/atomicupdate/bug23538-add_EmailPatronRegistrations_and_EmailAddressForPatronRegistrations_sysprefs.pl
@@ -11,17 +11,18 @@ return {
 
         $dbh->do(q{INSERT IGNORE INTO letter (`module`, `code`, `branchcode`, `name`, `is_html`, `title`, `content`, `message_transport_type`, `lang`) VALUES ( 'members', 'OPAC_REG', '', 'New OPAC self-registration submitted', 1, 'New OPAC self-registration',
             '<h3>New OPAC self-registration</h3>
-            <p><h4>Self-registration made by</h4>
+            <p><h4>Self-registration made:</h4>
             <ul>
-            <li><<borrowers.firstname>> <<borrowers.surname>></li>
-            <li>Email: <<borrowers.email>></li>
-            <li>Phone: <<borrowers.phone>></li>
-            <li>Mobile: <<borrowers.mobile>></li>
-            <li>Fax: <<borrowers.fax>></li>
-            <li>Secondary email: <<borrowers.emailpro>></li>
-            <li>Secondary phone:<<borrowers.phonepro>></li>
-            <li>Home library: <<borrowers.branchcode>></li>
-            <li>Patron category: <<borrowers.categorycode>></li>
+            <li>[% borrower.firstname %] [% borrower.surname %]</li>
+            [% IF borrower.cardnumber %]<li>Cardnumber: [% borrower.cardnumber %]</li>[% END %]
+            [% IF borrower.email %]<li>Email: [% borrower.email %]</li>[% END %]
+            [% IF borrower.phone %]<li>Phone: [% borrower.phone %]</li>[% END %]
+            [% IF borrower.mobile %]<li>Mobile: [% borrower.mobile %]</li>[% END %]
+            [% IF borrower.fax %]<li>Fax: [% borrower.fax %]</li>[% END %]
+            [% IF borrower.emailpro %]<li>Secondary email: [% borrower.emailpro %]</li>[% END %]
+            [% IF borrower.phonepro %]<li>Secondary phone:[% borrower.phonepro %]</li>[% END %]
+            [% IF borrower.branchcode %]<li>Home library: [% borrower.branchcode %]</li>[% END %]
+            [% IF borrower.categorycode %]<li>Patron category: [% borrower.categorycode %]</li>[% END %]
             </ul>
             </p>', 'email', 'default') });
     },
diff --git a/installer/data/mysql/en/mandatory/sample_notices.yml b/installer/data/mysql/en/mandatory/sample_notices.yml
index 53a7e2a068..8b8cf3aee1 100644
--- a/installer/data/mysql/en/mandatory/sample_notices.yml
+++ b/installer/data/mysql/en/mandatory/sample_notices.yml
@@ -1901,16 +1901,17 @@ tables:
           lang: default
           content:
             - "<h3>New OPAC self-registration</h3>"
-            - "<p><h4>Self-registration made by</h4>"
+            - "<p><h4>Self-registration made:</h4>"
             - "<ul>"
-            - "<li><<borrowers.firstname>> <<borrowers.surname>></li>"
-            - "<li>Email: <<borrowers.email>></li>"
-            - "<li>Phone: <<borrowers.phone>></li>"
-            - "<li>Mobile: <<borrowers.mobile>></li>"
-            - "<li>Fax: <<borrowers.fax>></li>"
-            - "<li>Secondary email: <<borrowers.emailpro>></li>"
-            - "<li>Secondary phone:<<borrowers.phonepro>></li>"
-            - "<li>Home library: <<borrowers.branchcode>></li>"
-            - "<li>Temporary patron category: <<borrowers.categorycode>></li>"
+            - "<li> [% borrower.firstname %] [% borrower.surname %]</li>"
+            - "[% IF borrower.cardnumber %]<li>Cardnumber: [% borrower.cardnumber %]</li>[% END %]"
+            - "[% IF borrower.email %]<li>Email: [% borrower.email %]</li>[% END %]"
+            - "[% IF borrower.phone %]<li>Phone: [% borrower.phone %]</li>[% END %]"
+            - "[% IF borrower.mobile %]<li>Mobile: [% borrower.mobile %]</li>[% END %]"
+            - "[% IF borrower.fax %]<li>Fax: [% borrower.fax %]</li>[% END %]"
+            - "[% IF borrower.emailpro %]<li>Secondary email: [% borrower.emailpro %]</li>[% END %]"
+            - "[% IF borrower.phonepro %]<li>Secondary phone: [% borrower.phonepro %]</li>[% END %]"
+            - "[% IF borrower.branchcode %]<li>Home library: [% borrower.branchcode %]</li>[% END %]"
+            - "[% IF borrower.categorycode %]<li>Temporary patron category: [% borrower.categorycode %]</li>[% END %]"
             - "</ul>"
             - "</p>"
-- 
2.39.5