|
Description
Owen Leonard
2019-01-15 14:51:59 UTC
Created attachment 83972 [details] [review] Bug 22134: Add account expiration information to patron details This patch modifies the patron detail template in the staff client so that when a patron's registration is expired a message shows at the top of the page along with the same "Renew" and "Edit" links which appear on the checkout page. Links are also shown next the the expiration date in the body of the page. This patch also corrects the capitalization of the "Edit Details" link on circulation.tt. To test, apply the patch and locate a patron record which is expired. - On the patron detail page (members/moremember.pl) you should see a message at the top of the page. - In the body of the page, next to the expiration date information, you should see similar message and links. - On the patron's checkout page the "Edit details" link in the expiration message should have correct capitalization. Created attachment 83998 [details] [review] Bug 22134: Add account expiration information to patron details This patch modifies the patron detail template in the staff client so that when a patron's registration is expired a message shows at the top of the page along with the same "Renew" and "Edit" links which appear on the checkout page. Links are also shown next the the expiration date in the body of the page. This patch also corrects the capitalization of the "Edit Details" link on circulation.tt. To test, apply the patch and locate a patron record which is expired. - On the patron detail page (members/moremember.pl) you should see a message at the top of the page. - In the body of the page, next to the expiration date information, you should see similar message and links. - On the patron's checkout page the "Edit details" link in the expiration message should have correct capitalization. Signed-off-by: Devinim <kohadevinim@devinim.com.tr> Hi Owen, I think it shoudl be possible to use is_expired and is_going_to_expire from the template without having to pass template variable from the .pl file. I found some examples with is_child: [% IF patron.is_child %] Can you take a look please? Thanks Cait -- You're right that patron.is_expired works right in the template, but patron.is_going_to_expire doesn't seem to work as it should. It always evaluates as true. I'll see if I can figure out why. Created attachment 84147 [details] [review] Bug 22134: (follow-up) Simplify check for expired patron This patch move the check for "is_expired" to the template and removes unnecessary code from the script. The interface should show no difference. Created attachment 84162 [details] [review] Bug 22134: (follow-up) Simplify check for expired patron This patch move the check for "is_expired" to the template and removes unnecessary code from the script. The interface should show no difference. Signed-off-by: Jack Kelliher <jke0429@stacmail.net> Created attachment 84191 [details] [review] Bug 22134: (follow-up) Simplify check for expired patron This patch move the check for "is_expired" to the template and removes unnecessary code from the script. The page now correctly handles messages for patrons who are about to expire, as defined in the NotifyBorrowerDeparture preference. To test, apply the patch and locate a patron record which is expired. - On the patron detail page (members/moremember.pl) you should see a message at the top of the page. - In the body of the page, next to the expiration date information, you should see similar message and links. Select a patron who will expire within the period defined in NotifyBorrowerDeparture. - On the patron detail page you should see a message at the top of the page. - In the body of the page, next to the expiration date information, you should see similar message and links. Created attachment 84214 [details] [review] Bug 22134: Add account expiration information to patron details This patch modifies the patron detail template in the staff client so that when a patron's registration is expired a message shows at the top of the page along with the same "Renew" and "Edit" links which appear on the checkout page. Links are also shown next the the expiration date in the body of the page. This patch also corrects the capitalization of the "Edit Details" link on circulation.tt. To test, apply the patch and locate a patron record which is expired. - On the patron detail page (members/moremember.pl) you should see a message at the top of the page. - In the body of the page, next to the expiration date information, you should see similar message and links. - On the patron's checkout page the "Edit details" link in the expiration message should have correct capitalization. Signed-off-by: Devinim <kohadevinim@devinim.com.tr> Signed-off-by: Maryse Simard <maryse.simard@inlibro.com> Created attachment 84215 [details] [review] Bug 22134: (follow-up) Simplify check for expired patron This patch move the check for "is_expired" to the template and removes unnecessary code from the script. The page now correctly handles messages for patrons who are about to expire, as defined in the NotifyBorrowerDeparture preference. To test, apply the patch and locate a patron record which is expired. - On the patron detail page (members/moremember.pl) you should see a message at the top of the page. - In the body of the page, next to the expiration date information, you should see similar message and links. Select a patron who will expire within the period defined in NotifyBorrowerDeparture. - On the patron detail page you should see a message at the top of the page. - In the body of the page, next to the expiration date information, you should see similar message and links. Signed-off-by: Maryse Simard <maryse.simard@inlibro.com> Created attachment 84238 [details] [review] k Created attachment 84240 [details] [review] Bug 22134: Add account expiration information to patron details This patch modifies the patron detail template in the staff client so that when a patron's registration is expired a message shows at the top of the page along with the same "Renew" and "Edit" links which appear on the checkout page. Links are also shown next the the expiration date in the body of the page. This patch also corrects the capitalization of the "Edit Details" link on circulation.tt. To test, apply the patch and locate a patron record which is expired. - On the patron detail page (members/moremember.pl) you should see a message at the top of the page. - In the body of the page, next to the expiration date information, you should see similar message and links. - On the patron's checkout page the "Edit details" link in the expiration message should have correct capitalization. Signed-off-by: Devinim <kohadevinim@devinim.com.tr> Signed-off-by: Maryse Simard <maryse.simard@inlibro.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Created attachment 84241 [details] [review] Bug 22134: (follow-up) Simplify check for expired patron This patch move the check for "is_expired" to the template and removes unnecessary code from the script. The page now correctly handles messages for patrons who are about to expire, as defined in the NotifyBorrowerDeparture preference. To test, apply the patch and locate a patron record which is expired. - On the patron detail page (members/moremember.pl) you should see a message at the top of the page. - In the body of the page, next to the expiration date information, you should see similar message and links. Select a patron who will expire within the period defined in NotifyBorrowerDeparture. - On the patron detail page you should see a message at the top of the page. - In the body of the page, next to the expiration date information, you should see similar message and links. Signed-off-by: Maryse Simard <maryse.simard@inlibro.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Awesome work all! Pushed to master for 19.05 Enhancement, will not be backported. Looks like this actually created a regression. Once I confirm the bug and fix I'll open up another bug report... |