Bugzilla – Attachment 167170 Details for
Bug 30372
Patron self registration: Extended patron attributes are emptied on submit when mandatory field isn't filled in
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 30372: Fix value->attribute
Bug-30372-Fix-value-attribute.patch (text/plain), 1.25 KB, created by
Pedro Amorim
on 2024-05-24 16:11:37 UTC
(
hide
)
Description:
Bug 30372: Fix value->attribute
Filename:
MIME Type:
Creator:
Pedro Amorim
Created:
2024-05-24 16:11:37 UTC
Size:
1.25 KB
patch
obsolete
>From eeadbefc3992ae0386f3605bafc29dc1cc07ac5f Mon Sep 17 00:00:00 2001 >From: Pedro Amorim <pedro.amorim@ptfs-europe.com> >Date: Fri, 24 May 2024 16:10:44 +0000 >Subject: [PATCH] Bug 30372: Fix value->attribute > >Test plan: >- Activate patron self registration without email verification >- Create several patron attributes as visible and editable in the OPAC, make one mandatory >- Register as a new patron from the OPAC >- Fill in all required fields but the extended attribute >- Fill in at least one of the non-required extended attributes >- Submit >- Verify that the contents of the other extended attribute fields are still present. >--- > opac/opac-memberentry.pl | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/opac/opac-memberentry.pl b/opac/opac-memberentry.pl >index 279417431d..17700b8847 100755 >--- a/opac/opac-memberentry.pl >+++ b/opac/opac-memberentry.pl >@@ -671,7 +671,7 @@ sub GeneratePatronAttributesForm { > # or taken from the patron itself > if ( defined $entered_attributes ) { > foreach my $attr (@$entered_attributes) { >- push @{ $attr_values{ $attr->{code} } }, $attr->{value}; >+ push @{ $attr_values{ $attr->{code} } }, $attr->{attribute}; > } > } > elsif ( defined $borrowernumber ) { >-- >2.39.2
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 30372
:
167170
|
167177
|
168112