Bug 39254

Summary: PatronSelfRegistrationBorrowerMandatoryField. Show which fields were not set.
Product: Koha Reporter: Olli-Antti Kivilahti <olli-antti.kivilahti>
Component: OPACAssignee: Owen Leonard <oleonard>
Status: NEW --- QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low    
Version: Main   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: ---
Patch complexity: --- Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
Circulation function:

Description Olli-Antti Kivilahti 2025-03-05 15:08:39 UTC
When configuring mandatory fields for Patron OPAC Self Registration, it is possible to configure fields, which are not actually visible. Regardless a notification about

    "Your personal details ->
You have not filled out all required fields. Please fill in all missing fields and resubmit."

Is shown, even if all visible fields are filled. It would be helpful to show the names of the fields that are missing/not-defined, so the system preferences can be more easily updated.

It is a very simple code modification. Something like:

opac-tmpl/bootstrap/en/modules/opac-memberentry.tt =>

  [% IF empty_mandatory_fields %]
    <div class="alert alert-warning">You have not filled out all required fields. Please fill in all missing fields and resubmit. [% empty_mandatory_fields.join(", ") %] </div>
  [% END %]

It would be helpful to show the real column descriptions however.
Comment 1 Owen Leonard 2025-03-06 21:02:42 UTC
(In reply to Olli-Antti Kivilahti from comment #0)
> When configuring mandatory fields for Patron OPAC Self Registration, it is
> possible to configure fields, which are not actually visible.

Can you give an example? I don't think I understand what you're saying.
Comment 2 Olli-Antti Kivilahti 2025-03-07 14:12:50 UTC
(In reply to Owen Leonard from comment #1)
> (In reply to Olli-Antti Kivilahti from comment #0)
> > When configuring mandatory fields for Patron OPAC Self Registration, it is
> > possible to configure fields, which are not actually visible.
> 
> Can you give an example? I don't think I understand what you're saying.

PatronSelfRegistrationBorrowerMandatoryField => define userid

Userid for some reason is not visible.

Koha doesn't allow to submit the form, because userid is not defined.


PatronSelfRegistrationBorrowerUnwantedField => define address
PatronSelfRegistrationBorrowerMandatoryField => define address

Form cannot be submitted.