From 6cdcbd555d68416552ba1727af575d70f92e798d Mon Sep 17 00:00:00 2001
From: Andrew Isherwood <andrew.isherwood@ptfs-europe.com>
Date: Wed, 3 Oct 2018 12:05:40 +0100
Subject: [PATCH] Bug 20941: (follow-up) Carry out comment #4 advice

- Switch getType -> get_type
- get_type now returns a type or undef
- Ternary now performed in template

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

diff --git a/Koha/Illrequest.pm b/Koha/Illrequest.pm
index 7893568..b64b143 100644
--- a/Koha/Illrequest.pm
+++ b/Koha/Illrequest.pm
@@ -736,18 +736,18 @@ sub getPrefix {
         || "";                  # "the empty prefix"
 }
 
-=head3 getType
+=head3 get_type
 
-    my $type = $abstract->getType();
+    my $type = $abstract->get_type();
 
-Return a string representing the material type of this request
+Return a string representing the material type of this request or undef
 
 =cut
 
-sub getType {
+sub get_type {
     my ($self) = @_;
     my $attr = $self->illrequestattributes->find({ type => 'type'});
-    return $attr ? $attr->value : '<span>N/A</span>';
+    return $attr ? $attr->value : undef;
 };
 
 #### 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 85fbaf8..aaa3ed0 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
@@ -436,6 +436,7 @@
                         <fieldset class="rows">
                             <legend>Request details</legend>
                             <ol>
+                                [% type = request.get_type %]
                                 <li class="borrowernumber">
                                     <label for="borrowernumber">Patron ID:</label>
                                     [% request.borrowernumber | html %]
@@ -461,7 +462,7 @@
                                 </li>
                                 <li class="medium">
                                     <label class="medium">Request type:</label>
-                                    [% request.getType | html %]
+                                    [% type ? type : 'N/A' | html %]
                                 </li>
                                 <li class="cost">
                                     <label class="cost">Cost:</label>
@@ -570,7 +571,8 @@
                                 </div>
                                 <div class="medium">
                                     <span class="label medium">Request type:</span>
-                                    [% request.getType | html %]
+                                    [% type = request.get_type  %]
+                                    [% 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 4912fa7..4f4aa84 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-illrequests.tt
+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-illrequests.tt
@@ -125,6 +125,7 @@
                         <tbody>
                             [% FOREACH request IN requests %]
                                 [% status = request.status | html %]
+                                [% type = request.get_type %]
                                 <tr>
                                     <td>[% request.id | html %]</td>
                                     <td>
@@ -134,7 +135,7 @@
                                         [% IF request.metadata.Title %][% request.metadata.Title | html %][% ELSE %]<span>N/A</span>[% END %]
                                     </td>
                                     <td>[% request.backend | html %]</td>
-                                    <td>[% request.getType | html %]</td>
+                                    <td>[% type ? type : '<span>N/A</span>' | html %]</td>
                                     <td>[% request.capabilities.$status.name | html %]</td>
                                     <td>[% request.placed | html %]</td>
                                     <td>[% request.updated | html %]</td>
@@ -153,6 +154,7 @@
                             <fieldset class="rows">
                                 <legend id="library_legend">Details from library</legend>
                                 <ol>
+                                    [% type = request.get_type %]
                                     <li>
                                         <label for="request_id">Request ID:</label>
                                         [% request.id | html %]
@@ -177,7 +179,7 @@
                                     </li>
                                     <li>
                                         <label for="medium">Request type:</label>
-                                        [% request.getType | html %]
+                                        [% type ? type : '<span>N/A</span>' | html %]
                                     </li>
                                     <li>
                                         <label for="placed">Request placed:</label>
diff --git a/t/db_dependent/Illrequests.t b/t/db_dependent/Illrequests.t
index 511bff4..97df6d8 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->getType, '<span>N/A</span>',
-        'getType() returns placeholder if no type is set');
+    is($illrq_obj->get_type, '<span>N/A</span>',
+        'get_type() returns placeholder if no type is set');
     $builder->build({
         source => 'Illrequestattribute',
         value  => {
@@ -93,8 +93,8 @@ subtest 'Basic object tests' => sub {
             value => 'book'
         }
     });
-    is($illrq_obj->getType, 'book',
-        'getType() returns correct type if set');
+    is($illrq_obj->get_type, 'book',
+        'get_type() returns correct type if set');
 
     isnt($illrq_obj->status, 'COMP',
          "ILL is not currently marked complete.");
-- 
2.1.4