Bug 35964 - User cannot update their account if a column in PatronSelfModificationMandatoryField is also in PatronSelfRegistrationBorrowerUnwantedField
Summary: User cannot update their account if a column in PatronSelfModificationMandato...
Status: NEW
Alias: None
Product: Koha
Classification: Unclassified
Component: OPAC (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal
Assignee: Owen Leonard
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-01-31 15:55 UTC by Nick Clemens (kidclamp)
Modified: 2024-05-08 20:24 UTC (History)
1 user (show)

See Also:
GIT URL:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Nick Clemens (kidclamp) 2024-01-31 15:55:41 UTC
subroutine ParseCgiForBorrower is used in both self-reg and self-modification,however,it always removes the unwanted fields in PatronSelfRegistrationBorrowerUnwantedField

564     delete $borrower{$_} for split( /\s*\|\s*/, C4::Context->preference('PatronSelfRegistrationBorrowerUnwantedField') || q{} );

To recreate:
1 - Add 'fax' to PatronSelfRegistrationBorrowerUnwantedField
2 - Add 'fax' to PatronSelfModificationMandatoryField
3 - Self register on the OPAC, note there is no 'fax' field
4 - Sign in as the new borrower
5 - Attempt to edit, note 'fax' is required, fill it
6 - Save the form - 'you are missing a required field'
7 - Remove 'fax' from PatronSelfRegistrationBorrowerUnwantedField
8 - Save the form again, now it works