Bug 40105: Patrons cannot add notes when creating an ILL
Testing plan:
- Enable the ILLModule system preference
- Sign into the OPAC and make an ILL request at /cgi-bin/koha/opac-illrequests.pl
- Note there's no option to specify notes
- Apply the patch and restart services
- Make another ILL request
- Use the new notes field
- View the request and note that the notes value was saved
Sponsored-By: California College of the Arts
Signed-off-by: Brendan Lawlor <blawlor@clamsnet.org>
Signed-off-by: Pedro Amorim <pedro.amorim@openfifth.co.uk>
Signed-off-by: Lisette Scheer <lisette@bywatersolutions.com>
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
(cherry picked from commit 5940fb39d5)
Signed-off-by: Jacob O'Mara <Jacob.omara@openfifth.co.uk>
This commit is contained in:
parent
dc84e37030
commit
3cde85ff6f
|
|
@ -948,6 +948,7 @@ sub add_request {
|
|||
$request->branchcode( $params->{other}->{branchcode} );
|
||||
$request->status( $unauthenticated_request ? 'UNAUTH' : 'NEW' );
|
||||
$request->backend( $params->{other}->{backend} );
|
||||
$request->notesopac( $params->{other}->{notesopac} ) if exists $params->{other}->{notesopac};
|
||||
$request->placed( dt_from_string() );
|
||||
$request->updated( dt_from_string() );
|
||||
$request->batch_id(
|
||||
|
|
|
|||
|
|
@ -103,6 +103,7 @@
|
|||
[% IF type %]
|
||||
[% INCLUDE "ill/backends/Standard/shared/forms/${type}.inc" %]
|
||||
[% END %]
|
||||
[% INCLUDE "ill/backends/Standard/shared/notes.inc" %]
|
||||
[% INCLUDE "ill/backends/Standard/shared/custom_fields.inc" %]
|
||||
[% IF whole.value.other.type && unauthenticated_ill && !logged_in_user %]
|
||||
<fieldset class="rows" id="illrequest_captcha">
|
||||
|
|
|
|||
|
|
@ -0,0 +1,5 @@
|
|||
<fieldset class="rows" id="ill-standard-notes">
|
||||
<legend>Notes</legend>
|
||||
<label class="sr-only" for="notesopac">Notes:</label>
|
||||
<textarea name="notesopac" id="notesopac" class="form-control input-fluid">[% whole.value.other.notesopac | html %]</textarea>
|
||||
</fieldset>
|
||||
|
|
@ -428,7 +428,7 @@ subtest 'patron_list() tests' => sub {
|
|||
|
||||
subtest 'add() tests' => sub {
|
||||
|
||||
plan tests => 2;
|
||||
plan tests => 3;
|
||||
|
||||
$schema->storage->txn_begin;
|
||||
|
||||
|
|
@ -537,5 +537,9 @@ subtest 'add() tests' => sub {
|
|||
## Authorized user test
|
||||
$t->post_ok( "//$userid:$password@/api/v1/ill/requests" => json => $body )->status_is(201);
|
||||
|
||||
# Confirm there is a notesopac value
|
||||
my $req = Koha::ILL::Requests->search()->next;
|
||||
ok( $req->notesopac, "notesopac value exists" ) if $req;
|
||||
|
||||
$schema->storage->txn_rollback;
|
||||
};
|
||||
|
|
|
|||
Loading…
Reference in New Issue