Summary: | Update dateexpiry on categorycode change | ||
---|---|---|---|
Product: | Koha | Reporter: | hebah |
Component: | Patrons | Assignee: | Sam Lau <samalau> |
Status: | Patch doesn't apply --- | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | acampbell, bibliothek, gmcharlt, jaki.frisina, kebliss, kyle.m.hall, lucas, sally.healey, samalau, smhodson123extra, tomascohen |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Bug 32581: Automatically update patron expiration date when changing their category
Bug 32581: Automatically update patron expiration date when changing their category Bug 32581: Add confirmation modal Bug 32581: Automatically update patron expiration date when changing their category Bug 32581: Add confirmation modal |
Description
hebah
2023-01-06 21:18:42 UTC
Yes, without doing a renew on newly preregistered cards, it iss keeping the preregistration expiration date of 1 year, though a regular adult patron expires in 5 years. A preregistered 6 month non resident card is also keeping the 1 year. Changing a previous patron from an adult (5 years) to a non resident 6 month card would still leave them at the old expiration date. We would like to see changing a borrower type and saving it, add any applicable fees and update the expiration date, just by saving it. We would love to see this change! Currently, our staff and part time substitutes often forget this step, which results in confusion for our staff and patrons. It would be nice if the expiration date automatically changed when the patron category is changed. I haven't seen any movement on this. Is it possible to look into a fix? Created attachment 168476 [details] [review] Bug 32581: Automatically update patron expiration date when changing their category To test: 1) In Administration-> Patron categories, set the enrollment period for the 'School' category to 12 months. 2) Find a patron that is from another category, edit their info and change their category from patron to school. 3) Note that the expiry date automatically changes to 12 months from today. 4) Save your changes and ensure on the patron's detail page that their expiration date was successfully changed. Created attachment 168763 [details] [review] Bug 32581: Automatically update patron expiration date when changing their category To test: 1) In Administration-> Patron categories, set the enrollment period for the 'School' category to 12 months. 2) Find a patron that is from another category, edit their info and change their category from patron to school. 3) Note that the expiry date automatically changes to 12 months from today. 4) Save your changes and ensure on the patron's detail page that their expiration date was successfully changed. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> This could pose a problem for libraries that charge library fees on renewal. A category change might happen for multiple reasons, we can not assume that for every library the renewal of the expiration date is correct. I think it needs to be optional, maybe as a question to confirm/not confirm or a system preference (In reply to Katrin Fischer from comment #6) > This could pose a problem for libraries that charge library fees on renewal. > A category change might happen for multiple reasons, we can not assume that > for every library the renewal of the expiration date is correct. > I think it needs to be optional, maybe as a question to confirm/not confirm > or a system preference Sure, that makes sense. So essentially the same thing that happens for the messaging preferences when you change your category? I think that might work, but not completely sure what it looks like right now :) Created attachment 168838 [details] [review] Bug 32581: Add confirmation modal This patch makes changing the expiration date optional, by providing a modal when switching a patron category. Confirming the modal will change the expiration date to the default and cancelling will keep the same expiration date. To test: 1) Same plan as before except now there is a modal when changing the patron category. - Attempt confirming the modal: expiration date should change to default. - Attempt canceling the modal: expiration date should stay the same Created attachment 168895 [details] [review] Bug 32581: Automatically update patron expiration date when changing their category To test: 1) In Administration-> Patron categories, set the enrollment period for the 'School' category to 12 months. 2) Find a patron that is from another category, edit their info and change their category from patron to school. 3) Note that the expiry date automatically changes to 12 months from today. 4) Save your changes and ensure on the patron's detail page that their expiration date was successfully changed. Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl> Signed-off-by: Jan Kissig <jkissig@th-wildau.de> Created attachment 168896 [details] [review] Bug 32581: Add confirmation modal This patch makes changing the expiration date optional, by providing a modal when switching a patron category. Confirming the modal will change the expiration date to the default and cancelling will keep the same expiration date. To test: 1) Same plan as before except now there is a modal when changing the patron category. - Attempt confirming the modal: expiration date should change to default. - Attempt canceling the modal: expiration date should stay the same Signed-off-by: Jan Kissig <jkissig@th-wildau.de> The patch works as described although I think there should be another bug to harmonize the 2 confirm dialogs. The first dialog which asks to accept the change of message preferences is a js confirmation dialog and fires only once. It is followed by the new dialog which is in html and is fired every time the category id is changed. (In reply to Jan Kissig from comment #12) > The patch works as described although I think there should be another bug to > harmonize the 2 confirm dialogs. > The first dialog which asks to accept the change of message preferences is a > js confirmation dialog and fires only once. > It is followed by the new dialog which is in html and is fired every time > the category id is changed. Actually the first dialog also fires every time category id is changed (In reply to Jan Kissig from comment #13) > (In reply to Jan Kissig from comment #12) > > The patch works as described although I think there should be another bug to > > harmonize the 2 confirm dialogs. See Bug 31354 Doesn't apply. Please rebase. I feel like this could also be syspref-controlled. |