Summary: | Self-registration defaults to the first library in the list | ||
---|---|---|---|
Product: | Koha | Reporter: | Jason Robb <jrobb> |
Component: | OPAC | Assignee: | Owen Leonard <oleonard> |
Status: | NEW --- | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | andrew, dcook |
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
Jason Robb
2025-08-21 21:41:30 UTC
This is an interesting one for sure. I took a look at some of our examples, and it looks like we use Javascript in a number of cases to do similar things. (Same for category code too actually.) This would be a good fix for everyone I think. (In reply to Jason Robb from comment #0) > we had to set branchcode as mandatory in > PatronSelfRegistrationBorrowerMandatoryField to avoid that issue. It sounds like we need to make it mandatory for the field to be mandatory! Otherwise we need a way to decide which library is the default. (In reply to Owen Leonard from comment #2) > (In reply to Jason Robb from comment #0) > > we had to set branchcode as mandatory in > > PatronSelfRegistrationBorrowerMandatoryField to avoid that issue. > > It sounds like we need to make it mandatory for the field to be mandatory! > > Otherwise we need a way to decide which library is the default. I suppose the logic would be if branchcode is not unwanted, then it must be mandatory. If branchcode is unwanted, then a default library would need to be provided. With categorycode, if one isn't provided (ie categorycode is unwanted), then the default category is used. -- Looking at branchcode and categorycode, neither let you show an empty option on the UI. Probably because they're the 2 essential fields for a borrower DB entry. If they did have empty options, then I think we'd need to make them mandatory/required. Anyway, it seems doable... |