From 6a91a29fb1444f6f7d8e1f88d6174a2219a7a51a Mon Sep 17 00:00:00 2001
From: Josef Moravec <josef.moravec@gmail.com>
Date: Thu, 4 Oct 2018 07:44:53 +0000
Subject: [PATCH] Bug 20941: (QA follow-up) Fix return value of get_type, make
 templates more consistant

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
---
 Koha/Illrequest.pm                                           | 3 ++-
 koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt  | 6 +++---
 koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-illrequests.tt | 4 ++--
 t/db_dependent/Illrequests.t                                 | 4 ++--
 4 files changed, 9 insertions(+), 8 deletions(-)

diff --git a/Koha/Illrequest.pm b/Koha/Illrequest.pm
index 0fab257942..076ff3ddd3 100644
--- a/Koha/Illrequest.pm
+++ b/Koha/Illrequest.pm
@@ -735,7 +735,8 @@ Return a string representing the material type of this request or undef
 sub get_type {
     my ($self) = @_;
     my $attr = $self->illrequestattributes->find({ type => 'type'});
-    return $attr ? $attr->value : undef;
+    return if !$attr;
+    return $attr->value;
 };
 
 #### Illrequests Imports
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt
index 1157b0daf0..2e199a486f 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/ill/ill-requests.tt
@@ -438,7 +438,7 @@
                                 </li>
                                 <li class="medium">
                                     <label class="medium">Request type:</label>
-                                    [% type ? type : 'N/A' | html %]
+                                    [% type || 'N/A' | html %]
                                 </li>
                                 <li class="cost">
                                     <label class="cost">Cost:</label>
@@ -538,8 +538,8 @@
                                 </div>
                                 <div class="medium">
                                     <span class="label medium">Request type:</span>
-                                    [% type = request.get_type  %]
-                                    [% type ? type : 'N/A' | html %]
+                                    % type = request.get_type %]
+                                    [% type || 'N/A' | html %]
                                 </div>
                                 <div class="cost">
                                     <span class="label cost">Cost:</span>
diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-illrequests.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-illrequests.tt
index 6345a6e6fc..c5ff708c10 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-illrequests.tt
+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-illrequests.tt
@@ -129,7 +129,7 @@
                                     <td>[% request.metadata.Author || 'N/A' | html %]</td>
                                     <td>[% request.metadata.Title || 'N/A' | html %]</td>
                                     <td>[% request.backend | html %]</td>
-                                    <td>[% type ? type : '<span>N/A</span>' | html %]</td>
+                                    <td>[% type || 'N/A' | html %]</td>
                                     <td>[% request.capabilities.$status.name | html %]</td>
                                     <td>[% request.placed | html %]</td>
                                     <td>[% request.updated | html %]</td>
@@ -169,7 +169,7 @@
                                     </li>
                                     <li>
                                         <label for="medium">Request type:</label>
-                                        [% type ? type : '<span>N/A</span>' | html %]
+                                        [% type || 'N/A' | html %]
                                     </li>
                                     <li>
                                         <label for="placed">Request placed:</label>
diff --git a/t/db_dependent/Illrequests.t b/t/db_dependent/Illrequests.t
index 97df6d85f3..523dedb230 100644
--- a/t/db_dependent/Illrequests.t
+++ b/t/db_dependent/Illrequests.t
@@ -83,8 +83,8 @@ subtest 'Basic object tests' => sub {
     is($illrq_obj->backend, $illrq->{backend},
        "Backend getter works.");
 
-    is($illrq_obj->get_type, '<span>N/A</span>',
-        'get_type() returns placeholder if no type is set');
+    is($illrq_obj->get_type, undef,
+        'get_type() returns undef if no type is set');
     $builder->build({
         source => 'Illrequestattribute',
         value  => {
-- 
2.11.0