Bugzilla – Attachment 122752 Details for
Bug 13188
Make it possible to configure mandatory patron data differently between OPAC registration and modification
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 13188: Add PatronSelfModificationMandatoryField system preference
Bug-13188-Add-PatronSelfModificationMandatoryField.patch (text/plain), 5.19 KB, created by
Biblibre Sandboxes
on 2021-07-09 14:54:49 UTC
(
hide
)
Description:
Bug 13188: Add PatronSelfModificationMandatoryField system preference
Filename:
MIME Type:
Creator:
Biblibre Sandboxes
Created:
2021-07-09 14:54:49 UTC
Size:
5.19 KB
patch
obsolete
>From 01129fec0f420c5b2f9476d7161b8f99c5da89db Mon Sep 17 00:00:00 2001 >From: Lucas Gass <lucas@bywatersolutions.com> >Date: Tue, 19 Jan 2021 22:39:57 +0000 >Subject: [PATCH] Bug 13188: Add PatronSelfModificationMandatoryField system > preference > >TEST PLAN: >1. Apply patch, updatedatabase, restart_all >2. In the PatronSelfRegistrationBorrowerMandatoryField add the field password. >3. Go to the self registration page on make sure password is required. >4. Now go to the OPAC modification page, make some changes and attempt to submit. It should work. >5. Go to PatronSelfModificationMandatoryField and select some fields. >6. The columns you selected should now be required. >7. On the PatronSelfModificationMandatoryField page, make sure the select all and clear all buttons work. > >Signed-off-by: Kelly McElligott <kelly@bywatersolutions.com> >--- > ...8_add_PatronSelfModificationMandatoryField_syspref.perl | 7 +++++++ > installer/data/mysql/mandatory/sysprefs.sql | 1 + > .../prog/en/modules/admin/preferences/opac.pref | 6 ++++++ > opac/opac-memberentry.pl | 3 ++- > 4 files changed, 16 insertions(+), 1 deletion(-) > create mode 100644 installer/data/mysql/atomicupdate/bug_13188_add_PatronSelfModificationMandatoryField_syspref.perl > >diff --git a/installer/data/mysql/atomicupdate/bug_13188_add_PatronSelfModificationMandatoryField_syspref.perl b/installer/data/mysql/atomicupdate/bug_13188_add_PatronSelfModificationMandatoryField_syspref.perl >new file mode 100644 >index 0000000000..9462d3b3ea >--- /dev/null >+++ b/installer/data/mysql/atomicupdate/bug_13188_add_PatronSelfModificationMandatoryField_syspref.perl >@@ -0,0 +1,7 @@ >+$DBversion = 'XXX'; # will be replaced by the RM >+if( CheckVersion( $DBversion ) ) { >+ $dbh->do(q{ >+ INSERT IGNORE INTO systempreferences (`variable`, `value`, `options`, `explanation`, `type`) VALUES ('PatronSelfModificationMandatoryField','', NULL,'Define the required fields when a patron is editing their information via the OPAC.','multiple'); >+ }); >+ NewVersion($DBversion, 13188, "Allow configuration of required fields when a patron is editing their information via the OPAC"); >+} >diff --git a/installer/data/mysql/mandatory/sysprefs.sql b/installer/data/mysql/mandatory/sysprefs.sql >index 3430bc26e7..abf6f9bda5 100644 >--- a/installer/data/mysql/mandatory/sysprefs.sql >+++ b/installer/data/mysql/mandatory/sysprefs.sql >@@ -512,6 +512,7 @@ INSERT INTO systempreferences ( `variable`, `value`, `options`, `explanation`, ` > ('PatronDuplicateMatchingAddFields','surname|firstname|dateofbirth', NULL,'A list of fields separated by "|" to deduplicate patrons when created','Free'), > ('patronimages','0',NULL,'Enable patron images for the staff interface','YesNo'), > ('PatronRemovalDelay','',NULL,'Delay for removing anonymized patrons', 'Integer'), >+('PatronSelfModificationMandatoryField','',NULL,'Name the fields you don\'t want to be required when a patron is editing their information via the OPAC.','free'), > ('PatronSelfModificationBorrowerUnwantedField','',NULL,'Name the fields you don\'t want to display when a patron is editing their information via the OPAC.','free'), > ('PatronSelfRegistration','0',NULL,'If enabled, patrons will be able to register themselves via the OPAC.','YesNo'), > ('PatronSelfRegistrationAdditionalInstructions','','','A free text field to display additional instructions to newly self registered patrons.','free'), >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/opac.pref b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/opac.pref >index 535f65bd87..11c2989c92 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/opac.pref >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/opac.pref >@@ -827,6 +827,12 @@ OPAC: > class: integer > - "days after account creation." > - "<br><strong>NOTE:</strong> This system preference requires the <code>misc/cronjobs/cleanup_database.pl</code> cronjob. Ask your system administrator to schedule it." >+ - >+ - "The following <a href='http://schema.koha-community.org/__VERSION__/tables/borrowers.html' target='blank'>database columns</a> must be filled in on the patron modification screen:" >+ - pref: PatronSelfModificationMandatoryField >+ type: modalselect >+ source: borrowers >+ exclusions: password|cardnumber|dateexpiry|categorycode > - > - "The following <a href='http://schema.koha-community.org/__VERSION__/tables/borrowers.html' target='blank'>database columns</a> must be filled in on the patron entry screen:" > - pref: PatronSelfRegistrationBorrowerMandatoryField >diff --git a/opac/opac-memberentry.pl b/opac/opac-memberentry.pl >index c9a4790efd..1e92c6c6e9 100755 >--- a/opac/opac-memberentry.pl >+++ b/opac/opac-memberentry.pl >@@ -386,7 +386,8 @@ sub GetMandatoryFields { > > my %mandatory_fields; > >- my $BorrowerMandatoryField = >+ my $BorrowerMandatoryField = $action eq 'edit' || $action eq 'update' ? >+ C4::Context->preference("PatronSelfModificationMandatoryField") : > C4::Context->preference("PatronSelfRegistrationBorrowerMandatoryField"); > > my @fields = split( /\|/, $BorrowerMandatoryField ); >-- >2.20.1
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 13188
:
115371
|
115372
|
116085
|
122752
|
123335
|
125272
|
125273
|
125274
|
128409
|
128413
|
128414
|
128415
|
128416
|
128417
|
128418