https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=37434

--- Comment #17 from Baptiste Wojtkowski (bwoj) 
<[email protected]> ---
(In reply to Marcel de Rooy from comment #13)
> Do you need these changes:
> +                                    [% ELSIF shelf.sortfield ==
> "publicationyear" %]
> +                                        <option value="publicationyear"
> selected="selected">Copyrightdate</option>
> 
> and also twice
> +        $sortfield = 'title' unless grep { $_ eq $sortfield } qw( title
> author copyrightdate publicationyear itemcallnumber dateadded );
> 
> Is the test on copyrightdate and UNIMARC not sufficient ?

In fact, they are not used the same way : "copyrightdate" is de facto the name
used in database to sort by date of publication, in MARC21, "copyrightdate" and
in UNIMARC 'publicationyear'. This is why we need to tell the template "I need
to sort by publicationyear"
The code added in the template is just the corresponding entry for the
sortfield publicationyear, as there is one for each possible sortfield.

Or you mean you want to do the test directly on the template ? I don't feel
like it would make it more clean, do you ?

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to