In the management of patron updates, it seems logical that the requests should show the timestamp of when the request was placed. Yes, you can look this up in a report, but this should be a basic feature of managing requests. Ideally, it should show on the same line as the patron name. Probably something like "Requested: 2016-09-05"
Created attachment 59297 [details] [review] Bug 17334- Added a timestamp to the members-update.tt To show the user when the pending modifications were requested. Test plan: 1.Log into OPAC and click on Lists->Your Lists->your personal details 2. Change/add a value in that form 3. Select the 'Submit update request' 4. Log into the Koha staff interface, then go to the url: cgi-bin/koha/members/members-update.pl 5. You will see near the top of each box containing a pending request the following are displayed: surname, firstname, patron category, branch, and cardnumber but no timestamp. 6. Apply this patch 7. Repeat steps 1,2, 3, 4 8. You will now see timestamp in the format "Requested: YYYY-MM-DD HH:MM:SS" next to each patrons details near the top of each pending request box.
Created attachment 59379 [details] [review] [SIGNED-OFF] Bug 17334- Added a timestamp to the members-update.tt To show the user when the pending modifications were requested. Test plan: 1.Log into OPAC and click on Lists->Your Lists->your personal details 2. Change/add a value in that form 3. Select the 'Submit update request' 4. Log into the Koha staff interface, then go to the url: cgi-bin/koha/members/members-update.pl 5. You will see near the top of each box containing a pending request the following are displayed: surname, firstname, patron category, branch, and cardnumber but no timestamp. 6. Apply this patch 7. Repeat steps 1,2, 3, 4 8. You will now see timestamp in the format "Requested: YYYY-MM-DD HH:MM:SS" next to each patrons details near the top of each pending request box. NOTE: TT variable is excessive. I'll let QA tweak. Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
(in reply to comment 2 by M.Tompsett) I have altered the formatting of the date so that it does not show time, i.e. it is displayed in the format: DD/MM/YY I assume this is what you were referring to when saying that TT variable was excessive? But before I attach my new commit to this bug, I would like to ask if you are happy with the time being displayed being the UTC rather than the localtime? The issue is that the timestamp of the member modification request is stored as UTC in the database
Comment on attachment 59379 [details] [review] [SIGNED-OFF] Bug 17334- Added a timestamp to the members-update.tt To show the user when the pending modifications were requested. Review of attachment 59379 [details] [review]: ----------------------------------------------------------------- This is what I was talking about. ::: koha-tmpl/intranet-tmpl/prog/en/modules/members/members-update.tt @@ +79,4 @@ > <div id="pending_updates"> > [% FOREACH pm IN PendingModifications %] > [% SET borrowernumber = pm.borrowernumber %] > + [% SET timestamp = pm.timestamp %] This is overkill, because... @@ +79,5 @@ > <div id="pending_updates"> > [% FOREACH pm IN PendingModifications %] > [% SET borrowernumber = pm.borrowernumber %] > + [% SET timestamp = pm.timestamp %] > + <h3><a href="#panel_modify_[% pm.borrowernumber %]">[% borrowers.$borrowernumber.surname %][% IF ( borrowers.$borrowernumber.firstname ) %], [% borrowers.$borrowernumber.firstname %][% END %] - [% Categories.GetName( borrowers.$borrowernumber.categorycode ) %] ([% Branches.GetName( borrowers.$borrowernumber.branchcode ) %]) ([% borrowers.$borrowernumber.cardnumber %]) Requested: [% timestamp %]</a></h3> You only use it here.
(In reply to Alex Buckley from comment #3) > But before I attach my new commit to this bug... No need. Let QA do their job. If they are fine with it. Let it go. If not, they'll tweak it and push it through.
Created attachment 59574 [details] [review] Bug 17334: Format the date according to the system preference - Use KohaDates TT plugin to format the date correctly - Do not use a temporary variable - Replace: patron's info Requested: Datetime with patron's info - Requested on Datetime
Created attachment 59575 [details] [review] Bug 17334- Added a timestamp to the members-update.tt To show the user when the pending modifications were requested. Test plan: 1.Log into OPAC and click on Lists->Your Lists->your personal details 2. Change/add a value in that form 3. Select the 'Submit update request' 4. Log into the Koha staff interface, then go to the url: cgi-bin/koha/members/members-update.pl 5. You will see near the top of each box containing a pending request the following are displayed: surname, firstname, patron category, branch, and cardnumber but no timestamp. 6. Apply this patch 7. Repeat steps 1,2, 3, 4 8. You will now see timestamp in the format "Requested: YYYY-MM-DD HH:MM:SS" next to each patrons details near the top of each pending request box. Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Created attachment 59576 [details] [review] Bug 17334: Format the date according to the system preference - Use KohaDates TT plugin to format the date correctly - Do not use a temporary variable - Replace: patron's info Requested: Datetime with patron's info - Requested on Datetime
I'd like to get a signoff on the second patch. Is the display better?
Created attachment 59589 [details] [review] Bug 17334 - Second update following QA feedback This commit contains the following changes: 1. Implemented the use of KohaDates TT plugin to format the date correctly 2. Removed the use of the temporary timestamp variable 3. Changed the date text to: patron's info - Requested on Datetime
(in reply to Jonathan Druart) Sorry I realised you have made all of the neccisary changes and so to remove my second patch ( Bug 17334 - Second update following QA feedback) can I just set it to obsolete?
I am still learning bugzilla and didn't realise your patch contained the necessary changes. so sorry about that
Created attachment 59658 [details] [review] [SIGNED-OFF] Bug 17334: Format the date according to the system preference - Use KohaDates TT plugin to format the date correctly - Do not use a temporary variable - Replace: patron's info Requested: Datetime with patron's info - Requested on Datetime Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 59677 [details] [review] Bug 17334- Added a timestamp to the members-update.tt To show the user when the pending modifications were requested. Test plan: 1.Log into OPAC and click on Lists->Your Lists->your personal details 2. Change/add a value in that form 3. Select the 'Submit update request' 4. Log into the Koha staff interface, then go to the url: cgi-bin/koha/members/members-update.pl 5. You will see near the top of each box containing a pending request the following are displayed: surname, firstname, patron category, branch, and cardnumber but no timestamp. 6. Apply this patch 7. Repeat steps 1,2, 3, 4 8. You will now see timestamp in the format "Requested: YYYY-MM-DD HH:MM:SS" next to each patrons details near the top of each pending request box. Signed-off-by: Mark Tompsett <mtompset@hotmail.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Created attachment 59678 [details] [review] Bug 17334: Format the date according to the system preference - Use KohaDates TT plugin to format the date correctly - Do not use a temporary variable - Replace: patron's info Requested: Datetime with patron's info - Requested on Datetime Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Pushed to master for 17.05, thanks Alex!
This won't get ported back to 16.11.x as it is an enhancement.
Could this be used when a Patron makes a purchase suggestion also? A library has mentioned that they would love to see the time in addition to the date on purchase suggestions made through the OPAC? Thanks!
(In reply to Kelly McElligott from comment #18) > Could this be used when a Patron makes a purchase suggestion also? A > library has mentioned that they would love to see the time in addition to > the date on purchase suggestions made through the OPAC? > > Thanks! That is a separate matter. A new bug should be submitted for that feature. :)