From 54fb517170746eab9634025b3f8c8e19f6337c79 Mon Sep 17 00:00:00 2001
From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Date: Tue, 6 Aug 2019 08:56:44 -0500
Subject: [PATCH] Bug 13420: Fallback to the previous behavior if published
 date is not used

---
 catalogue/detail.pl                                         |  3 ++-
 koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt | 10 ++++++++--
 2 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/catalogue/detail.pl b/catalogue/detail.pl
index b4d9bae646..37d2a0c746 100755
--- a/catalogue/detail.pl
+++ b/catalogue/detail.pl
@@ -275,7 +275,7 @@ foreach my $item (@items) {
     $item->{'ccode'} = $collections->{$ccode} if ( defined( $ccode ) && defined($collections) && exists( $collections->{$ccode} ) );
     my $copynumber = $item->{'copynumber'};
     $item->{'copynumber'} = $copynumbers->{$copynumber} if ( defined($copynumber) && defined($copynumbers) && exists( $copynumbers->{$copynumber} ) );
-    foreach (qw(ccode enumchron copynumber stocknumber itemnotes itemnotes_nonpublic uri)) {
+    foreach (qw(ccode enumchron copynumber stocknumber itemnotes itemnotes_nonpublic uri publisheddate)) { # Warning when removing GetItemsInfo - publisheddate (at least) is not part of the items table
         $itemfields{$_} = 1 if ( $item->{$_} );
     }
 
@@ -383,6 +383,7 @@ $template->param(
     itemdata_uri        => $itemfields{uri},
     itemdata_copynumber => $itemfields{copynumber},
     itemdata_stocknumber => $itemfields{stocknumber},
+    itemdata_publisheddate => $itemfields{publisheddate},
     volinfo                => $itemfields{enumchron},
         itemdata_itemnotes  => $itemfields{itemnotes},
         itemdata_nonpublicnotes => $itemfields{itemnotes_nonpublic},
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
index 3f33322018..bfb4b5d080 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
@@ -198,7 +198,13 @@
                 <th>Home library</th>
                 [% IF ( itemdata_ccode ) %]<th>Collection</th>[% END %]
                 <th>Call number</th>
-                [% IF volinfo %]<th class="title-string">Serial enumeration / chronology</th>[% END %]
+                [% IF volinfo %]
+                    [% IF itemdate_publisheddate #If there is at least one published date, use it for sorting%]
+                        <th class="title-string">Serial enumeration / chronology</th>
+                    [% ELSE %]
+                        <th>Serial enumeration / chronology</th>
+                    [% END %]
+                [% END %]
                 <th>Status</th>
                 <th class="title-string">Last seen</th>
                 <th class="title-string">Date accessioned</th>
@@ -241,7 +247,7 @@
                     <td class="itemcallnumber">[% IF ( item.itemcallnumber ) %] [% item.itemcallnumber | html %][% END %]</td>
                     [% IF ( volinfo ) %]
                         <td class="enumchron">
-                            <span title="[% item.publisheddate | html %]">
+                            [% IF itemdate_publisheddate %]<span title="[% item.publisheddate | html %]">[% END %]
                             [% IF ( itemdata_enumchron ) %]
                                 [% IF item.enumchron && item.serialseq %]
                                     <span class="enum">[% item.enumchron | html %]</span>
-- 
2.11.0