gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bug 30868: (bug 29684 follow-up) Fix borrowernumber var in template 

borrower_data does not contain the borrowernumber, it is sent
separately.

Test plan:
1. Edit a patron record (for example: Henry Acevedo).
2. Note that the URL is http://127.0.0.1:8081/cgi-bin/koha/members/memberentry.pl?op=modify&destination=circ&borrowernumber=19
3. Change date of birth to yesterday.
4. Save.
5. Error message displayed above the Save and Cancel buttons:
   The following fields are wrong. Please fix them.
   - Patron's age is incorrect for their category. Ages allowed are 18-999.
6. Note that the URL is now http://127.0.0.1:8081/cgi-bin/koha/members/memberentry.pl
7. Fix date of birth (for example: change year to 2000).
8. Select Save.
=> Without this patch you get: "Patron not found. Return to search."
=> With this patch applied the patron's data is saved

Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
Jonathan Druart 2022-05-31 08:23:30 +02:00 committed by Tomas Cohen Arazi
parent f0062c67d6
commit 9f1cfa0bd3
No known key found for this signature in database
GPG Key ID: 0A272EA1B2F3C15F
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt
View File

@ -51,7 +51,7 @@ legend:hover {
</li>
[% IF (borrower_data.firstname || borrower_data.surname ) %]
<li>
<a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% borrower_data.borrowernumber | uri %]">
<a href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% borrowernumber | uri %]">
[% INCLUDE 'patron-title.inc' %]
</a>
</li>
@ -229,7 +229,7 @@ legend:hover {
[% IF opadd %]
<a class="btn btn-default" href="/cgi-bin/koha/members/member.pl" class="toggler save_entryform">
[% ELSE %]
<a class="btn btn-default" href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% borrower_data.borrowernumber | html %]">
<a class="btn btn-default" href="/cgi-bin/koha/members/moremember.pl?borrowernumber=[% borrowernumber | html %]">
[% END %]
<i class="fa fa-times"></i> Cancel
</a>
@ -246,7 +246,7 @@ legend:hover {
<input type="hidden" name="updtype" value="[% updtype | html %]" />
<input type="hidden" name="destination" value="[% destination | html %]" />
<input type="hidden" name="check_member" value="[% check_member | html %]" />
<input type="hidden" name="borrowernumber" value="[% borrower_data.borrowernumber | html UNLESS opduplicate %]" />
<input type="hidden" name="borrowernumber" value="[% borrowernumber | html UNLESS opduplicate %]" />
<input type="hidden" name="nodouble" value="[% nodouble | html UNLESS opduplicate %]" />
<input type="hidden" name="csrf_token" value="[% csrf_token | html %]" />
[% IF ( step ) %]