From 3ad5dbeb5714907be5db8150994062e70aaf72df Mon Sep 17 00:00:00 2001
From: Bill Erickson <berickxx@gmail.com>
Date: Fri, 19 Apr 2019 11:02:18 -0400
Subject: [PATCH] server templates continued

Signed-off-by: Bill Erickson <berickxx@gmail.com>
---
 Open-ILS/examples/fm_IDL.xml                                     | 1 +
 .../eg2/src/app/staff/admin/server/print-template.component.html | 9 +++++++--
 .../eg2/src/app/staff/admin/server/print-template.component.ts   | 2 +-
 Open-ILS/src/perlmods/lib/OpenILS/WWW/PrintTemplate.pm           | 5 +++--
 .../src/sql/Pg/upgrade/XXXX.schema.server-print-templates.sql    | 3 ++-
 Open-ILS/web/js/ui/default/staff/circ/patron/app.js              | 3 +--
 6 files changed, 15 insertions(+), 8 deletions(-)

diff --git a/Open-ILS/examples/fm_IDL.xml b/Open-ILS/examples/fm_IDL.xml
index d70568ee21..cd78d61251 100644
--- a/Open-ILS/examples/fm_IDL.xml
+++ b/Open-ILS/examples/fm_IDL.xml
@@ -12822,6 +12822,7 @@ SELECT  usr,
 			<field name="name" reporter:datatype="text" oils_obj:required="true"/>
 			<field name="label" reporter:datatype="text" oils_obj:required="true" oils_persist:i18n="true"/>
 			<field reporter:label="Owner" name="owner" oils_obj:required="true" reporter:datatype="link"/>
+			<field reporter:label="Active" name="active" reporter:datatype="bool"/>
 			<field reporter:label="Locale" name="locale" reporter:datatype="link"/>
 			<field name="content_type" reporter:datatype="text"/>
 			<field name="template" reporter:datatype="text" oils_obj:required="true"/>
diff --git a/Open-ILS/src/eg2/src/app/staff/admin/server/print-template.component.html b/Open-ILS/src/eg2/src/app/staff/admin/server/print-template.component.html
index 32cba625a0..9a95f61234 100644
--- a/Open-ILS/src/eg2/src/app/staff/admin/server/print-template.component.html
+++ b/Open-ILS/src/eg2/src/app/staff/admin/server/print-template.component.html
@@ -59,7 +59,12 @@
       </div>
       <div class="row mt-2">
         <div class="col-lg-6">
-         <h4 i18n>Template</h4>
+          <h4 i18n>
+            Template for "{{template.label()}}"
+            <span class="pl-2 text-warning" *ngIf="template.active() == 'f'">
+              (Inactive)
+            </span>
+          </h4>
          <textarea rows="30" class="form-control"
            spellcheck="false"
            [ngModel]="template.template()"
@@ -72,7 +77,7 @@
           </div>
           <h4 class="mt-3" i18n>Compiled Content</h4>
           <div class="border border-dark w-100">
-            <pre>{{compiledContent}}</pre>
+            <pre class="p-1">{{compiledContent}}</pre>
           </div>
         </div>
       </div>
diff --git a/Open-ILS/src/eg2/src/app/staff/admin/server/print-template.component.ts b/Open-ILS/src/eg2/src/app/staff/admin/server/print-template.component.ts
index 13b89964a2..f5de489626 100644
--- a/Open-ILS/src/eg2/src/app/staff/admin/server/print-template.component.ts
+++ b/Open-ILS/src/eg2/src/app/staff/admin/server/print-template.component.ts
@@ -26,7 +26,7 @@ const SAMPLE_JSON_DATA: any = {
 };
 
 const SAMPLE_TEMPLATE_DATA: any = {
-    'address-label': {
+    patron_address: {
         patron: SAMPLE_JSON_DATA.patron,
         address: SAMPLE_JSON_DATA.address
     }
diff --git a/Open-ILS/src/perlmods/lib/OpenILS/WWW/PrintTemplate.pm b/Open-ILS/src/perlmods/lib/OpenILS/WWW/PrintTemplate.pm
index f8209d451b..79307cac5d 100644
--- a/Open-ILS/src/perlmods/lib/OpenILS/WWW/PrintTemplate.pm
+++ b/Open-ILS/src/perlmods/lib/OpenILS/WWW/PrintTemplate.pm
@@ -128,7 +128,7 @@ sub find_template {
 
     if ($template_id) {
         # Requesting by ID, generally used for testing, 
-        # always pulls the latest value.
+        # always pulls the latest value and ignores the active flag
         return $e->retrieve_config_print_template($template_id);
     }
 
@@ -143,7 +143,8 @@ sub find_template {
         my $template = $e->search_config_print_template({
             name => $name, 
             locale => $locale, 
-            owner => $org->id
+            owner => $org->id,
+            active => 't'
         })->[0];
 
         if ($template) {
diff --git a/Open-ILS/src/sql/Pg/upgrade/XXXX.schema.server-print-templates.sql b/Open-ILS/src/sql/Pg/upgrade/XXXX.schema.server-print-templates.sql
index a520ef9acf..aaa3f9762c 100644
--- a/Open-ILS/src/sql/Pg/upgrade/XXXX.schema.server-print-templates.sql
+++ b/Open-ILS/src/sql/Pg/upgrade/XXXX.schema.server-print-templates.sql
@@ -20,6 +20,7 @@ CREATE TABLE config.print_template (
     name         TEXT NOT NULL, -- programatic name
     label        TEXT NOT NULL, -- i18n
     owner        INT NOT NULL REFERENCES actor.org_unit (id),
+    active       BOOLEAN NOT NULL DEFAULT FALSE,
     locale       TEXT REFERENCES config.i18n_locale(code) 
                  ON UPDATE CASCADE DEFERRABLE INITIALLY DEFERRED,
     content_type TEXT NOT NULL DEFAULT 'text/html',
@@ -30,7 +31,7 @@ CREATE TABLE config.print_template (
 
 INSERT INTO config.print_template (id, name, locale, owner, label, template) 
 VALUES (
-    1, 'address-label', 'en-US',
+    1, 'patron_address', 'en-US',
     (SELECT id FROM actor.org_unit WHERE parent_ou IS NULL),
     oils_i18n_gettext(1, 'Address Label', 'cpt', 'label'),
 $TEMPLATE$
diff --git a/Open-ILS/web/js/ui/default/staff/circ/patron/app.js b/Open-ILS/web/js/ui/default/staff/circ/patron/app.js
index 08374b84d7..a86afae5d6 100644
--- a/Open-ILS/web/js/ui/default/staff/circ/patron/app.js
+++ b/Open-ILS/web/js/ui/default/staff/circ/patron/app.js
@@ -331,8 +331,7 @@ function($scope,  $q , $location , $filter , egCore , egNet , egUser , egAlertDi
     $scope.print_address = function(addr) {
         egCore.print.print({
             context : 'default', 
-            template: 'address-label',
-            server_hosted: true,
+            template: 'patron_address',
             scope : {
                 patron : patronSvc.current,
                 address : addr
-- 
2.11.0