Koha-clms/opac
Jonathan Druart 70e95a7dc6 Bug 21192: Do not pick SelfRegistration hidden fields for SelfModification
There was a mix up between "modification" and "edit" for $action in
opac-memberentry.pl.

If a patron is logged into the OPAC and they attempt to submit changes to their
personal details without having actually changed anything, the page reloads with
a message stating 'No changes were made.' However, the fields that now appear
are those that have been allowed/hidden according to the preferences set for
patron self registration, even if patron self registration is disabled. This can
lead to problems if there are discrepancies between the fields allowed for self
register settings and those allowed for patron self modification settings.

To replicate:

1.) In Administration › System preferences > OPAC, set PatronSelfRegistration to
'don't allow'

2.) Modify the unwanted fields for PatronSelfModificationBorrowerUnwantedField
and PatronSelfRegistrationBorrowerUnwantedField so that at least one field is
different between the two settings and save (for example, specify 'mobile' to be
hidden in the former, but don't include in the list of fields to be hidden in
the latter).

3.) Log into the OPAC and navigate to the 'your personal details' tab. Notice
that the editable fields correspond to those not hidden by
PatronSelfModificationBorrowerUnwantedField

4.) Without making any changes, click on 'Submit update request'.

5.) Note that on this page the fields correspond to those not hidden by
PatronSelfRegistrationBorrowerUnwantedField and are now editable (including any
fields that would otherwise be hidden by the Self Modification settings).

Reported-By: Chris Slone <cslone@camdencountylibrary.org>
Signed-off-by: Devinim <kohadevinim@devinim.com.tr>
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
(cherry picked from commit e0b1f945f9)
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
(cherry picked from commit 63ebdb8159)

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
2019-02-01 17:09:41 +00:00
..
clubs
…
errors Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
external/overdrive
…
rss
…
sci Bug 15492: (QA followup) Make page refresh work correctly 2018-03-26 17:31:19 -03:00
sco Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
svc Bug 22030: Use preference to determine username sent to overdrive 2019-01-11 18:54:41 +00:00
ilsdi.pl Bug 21719: Fix typos 2018-11-16 12:59:57 +00:00
maintenance.pl Bug 17989: Include full path logic in _get_template_file 2017-11-01 13:10:17 -03:00
oai.pl
…
opac-ISBDdetail.pl Bug 19301: Move C4::Reserves::OnShelfHoldsAllowed to get_onshelfholds_policy 2018-02-13 13:36:00 -03:00
opac-MARCdetail.pl Bug 19301: (QA follow-up) Add POD, use statements and correct typo 2018-02-13 13:36:00 -03:00
opac-account-pay-paypal-return.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
opac-account-pay-return.pl Bug 19173: Make OPAC online payments pluggable 2017-09-19 14:15:52 -03:00
opac-account-pay.pl Bug 19173: Make OPAC online payments pluggable 2017-09-19 14:15:52 -03:00
opac-account.pl Bug 12001: Move GetMemberAccountRecords to the Koha namespace 2018-02-23 10:57:30 -03:00
opac-addbybiblionumber.pl
…
opac-alert-subscribe.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
opac-article-request-cancel.pl
…
opac-authorities-home.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
opac-authoritiesdetail.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
opac-basket.pl Bug 19708: Improve code for printing the cart in OPAC 2018-03-26 17:31:16 -03:00
opac-blocked.pl
…
opac-browser.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
opac-changelanguage.pl Bug 18946 [QA Followup] - code cleanup 2017-09-01 13:02:26 -03:00
opac-course-details.pl
…
opac-course-reserves.pl
…
opac-detail.pl Bug 20737: Use https for baker and taylor cover images 2018-05-11 11:36:23 -03:00
opac-discharge.pl Bug 20953: Prevent several discharge requests on OPAC 2018-07-15 20:01:34 +01:00
opac-downloadcart.pl Bug 19040: Refactor GetMarcBiblio parameters 2017-08-25 10:23:42 -03:00
opac-downloadshelf.pl Bug 19040: Refactor GetMarcBiblio parameters 2017-08-25 10:23:42 -03:00
opac-export.pl Bug 20097: Use same regex in marc2dcxml and opac-export 2018-03-19 12:23:19 -03:00
opac-ics.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
opac-idref.pl Bug 19640: fix OPAC IdRef webservice display 2017-12-07 09:37:09 -03:00
opac-illrequests.pl Bug 20548: Remove copyright clearance for staff 2018-10-15 13:39:36 +01:00
opac-image.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
opac-imageviewer.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
opac-issue-note.pl Bug 18915: Fix checkout note email message 2018-01-15 12:14:00 -03:00
opac-main.pl Bug 19892: Replace numbersphr variable with Koha.Preference('OPACNumbersPreferPhrase') in OPAC 2018-04-02 18:08:07 -03:00
opac-memberentry.pl Bug 21192: Do not pick SelfRegistration hidden fields for SelfModification 2019-02-01 17:09:41 +00:00
opac-messaging.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
opac-modrequest-suspend.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
opac-modrequest.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
opac-mymessages.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
opac-news-rss.pl
…
opac-overdrive-search.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
opac-passwd.pl Bug 21719: Fix typos 2018-11-16 12:59:57 +00:00
opac-password-recovery.pl Bug 21771: (follow-up) Revert the shift statement 2018-11-16 12:37:51 +00:00
opac-patron-image.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
opac-privacy.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
opac-ratings-ajax.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
opac-ratings.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
opac-readingrecord.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
opac-registration-verify.pl Bug 19579: Do not confirm the registration if email already used - PatronSelfRegistrationEmailMustBeUnique 2018-05-11 10:52:46 -03:00
opac-renew.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
opac-request-article.pl
…
opac-reserve.pl Bug 21719: Fix typos 2018-11-16 12:59:57 +00:00
opac-restrictedpage.pl
…
opac-retrieve-file.pl
…
opac-review.pl Bug 21986: Do not escape quotation marks when cataloguing 2018-12-22 20:46:39 +00:00
opac-routing-lists.pl Bug 20666: Correct permissions on opac-routing-lists.pl 2018-05-03 13:26:48 -03:00
opac-search-history.pl Bug 12497: Fix OPAC search history reachable by URL even when disabled 2017-12-07 10:09:52 -03:00
opac-search.pl Bug 21719: Fix typos 2018-11-16 12:59:57 +00:00
opac-sendbasket.pl Bug 21874: Fix encoding of cart and list email subjects 2018-12-18 13:47:47 +00:00
opac-sendshelf.pl Bug 21874: Fix encoding of cart and list email subjects 2018-12-18 13:47:47 +00:00
opac-serial-issues.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
opac-shareshelf.pl Bug 20687: (follow-up) Look for invitekey in show_accept and fix error codes 2018-05-07 11:55:13 -03:00
opac-shelves.pl Bug 21629: Use cn_sort instead of itemcallnumber when displaying a list 2018-11-06 13:34:22 +00:00
opac-showmarc.pl Bug 20083: (follow-up) use same logic in opac-showmarc 2018-04-04 15:45:34 -03:00
opac-showreviews.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
opac-suggestions.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
opac-tags.pl Bug 19040: Refactor GetMarcBiblio parameters 2017-08-25 10:23:42 -03:00
opac-tags_subject.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
opac-topissues.pl Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00
opac-user.pl Bug 19991: Remove warning from opac-user 2018-09-05 11:20:48 +01:00
tracklinks.pl Bug 21018: Allow uris defined in the item 2018-08-09 11:08:29 +01:00
unapi Bug 19991: use Modern::Perl in OPAC perl scripts 2018-09-05 11:14:29 +01:00