[Koha-bugs] [Bug 20936] Holds History for patrons in OPAC
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Thu Apr 30 16:33:22 CEST 2020
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20936
--- Comment #71 from Tomás Cohen Arazi <tomascohen at gmail.com> ---
(In reply to Josef Moravec from comment #68)
> Comment on attachment 103950 [details] [review]
> Bug 20936: Add patron's hold history menu in OPAC
>
> Review of attachment 103950 [details] [review]:
> -----------------------------------------------------------------
>
> ::: Koha/REST/V1/Patrons/Holds.pm
> @@ +61,5 @@
> > + my $holds = $c->objects->search( $holds_set );
> > + return $c->render( status => 200, openapi => $holds );
> > + }
> > + catch {
> > + $c->unhandlede_exception($_);
>
> typo, should be unhandled_excecption
Done!
> @@ +118,5 @@
> > +[% INCLUDE 'datatables.inc' %]
> > +[% INCLUDE 'js-date-format.inc' %]
> > +<script>
> > + $(document).ready(function(){
> > + [% IF ( GoogleJackets ) %]KOHA.Google.GetCoverFromIsbn();[% END %]
>
> Why Google Jackets and not other services?
That didn't make any sense, I cleaned it.
> ::: opac/opac-holdshistory.pl
> @@ +33,5 @@
> > +use Koha::Patrons;
> > +
> > +use Koha::ItemTypes;
> > +use Koha::Ratings;
> > +
>
> You don't need most of these imports
Cleaned the .pl on behalf of Agustin.
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list