Some libraries would like to see the patron's email address in the Patron holds queue column.
Created attachment 164841 [details] [review] Bug 36595: Added email address field to the holds queue table To test: 1. Make sure that the system preference ‘HidePatronName’ is set to ‘Show’ 2. Select a patron that has a primary email address set in their contact information. 1. Place a hold for that patron 3. Build the holds queue: perl /kohadevbox/koha/misc/cronjobs/holds/build_holds_queue.pl 4. Check the holds queue and notice that the patron column includes the patron’s email 5. Disable the ‘HidePatronName’ system preference 1. Check the holds queue again and notice that no contact information can be seen for the patron 6. Sign off and have an amazing day :D
Do we want to wrap the email in a mailto: link like we do on the holds awaiting pickup page? I'm not sure how useful it is in practice, but we usually do that with email addresses.
Created attachment 164851 [details] [review] Bug 36595: Updated to add mailto: -- Add patron email to the holds queue table To test: 1. Make sure that the system preference ‘HidePatronName’ is set to ‘Show’ 2. Select a patron that has a primary email address set in their contact information. 1. Place a hold for that patron 3. Build the holds queue: perl /kohadevbox/koha/misc/cronjobs/holds/build_holds_queue.pl 4. Check the holds queue and notice that the patron column includes the patron’s email 5. Click on the hyperlinked email. It will open a new email with the subject of "Hold item: (your biblio title)" 6. Disable the ‘HidePatronName’ system preference 1. Check the holds queue again and notice that no contact information can be seen for the patron 7. Sign off and have an amazing day :D
(In reply to Owen Leonard from comment #2) > Do we want to wrap the email in a mailto: link like we do on the holds > awaiting pickup page? I'm not sure how useful it is in practice, but we > usually do that with email addresses. Thanks for the feedback, Owen! I've updated the patch so that it includes a mailto:
Created attachment 164854 [details] [review] Bug 36595: Updated to add mailto: -- Add patron email to the holds queue table To test: 1. Make sure that the system preference ‘HidePatronName’ is set to ‘Show’ 2. Select a patron that has a primary email address set in their contact information. 1. Place a hold for that patron 3. Build the holds queue: perl /kohadevbox/koha/misc/cronjobs/holds/build_holds_queue.pl 4. Check the holds queue and notice that the patron column includes the patron’s email 5. Click on the hyperlinked email. It will open a new email with the subject of "Hold item: (your biblio title)" 6. Disable the ‘HidePatronName’ system preference 1. Check the holds queue again and notice that no contact information can be seen for the patron 7. Sign off and have an amazing day :D Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 166626 [details] [review] Bug 36595: Updated to add mailto: -- Add patron email to the holds queue table To test: 1. Make sure that the system preference ‘HidePatronName’ is set to ‘Show’ 2. Select a patron that has a primary email address set in their contact information. 1. Place a hold for that patron 3. Build the holds queue: perl /kohadevbox/koha/misc/cronjobs/holds/build_holds_queue.pl 4. Check the holds queue and notice that the patron column includes the patron’s email 5. Click on the hyperlinked email. It will open a new email with the subject of "Hold item: (your biblio title)" 6. Disable the ‘HidePatronName’ system preference 1. Check the holds queue again and notice that no contact information can be seen for the patron 7. Sign off and have an amazing day :D Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Nice little enhancement, no QA issues.. Passing
The "Hold item" here is not translatable: + <p class="patron-email"><a href="mailto:[% itemsloo.patron.email | uri %]?subject=[% "Hold item: " | uri %][% itemsloo.title | uri %]">[% itemsloo.patron.notice_email_address | html %]</a></p> Maybe if you used a variable instead? Would need some testing.
Created attachment 169518 [details] [review] Bug 36595: Updated to make "Hold item:" translatable To test: 1. Make sure that the system preference ‘HidePatronName’ is set to ‘Show’ 2. Select a patron that has a primary email address set in their contact information. 1. Place a hold for that patron 3. Build the holds queue: perl /kohadevbox/koha/misc/cronjobs/holds/build_holds_queue.pl 4. Check the holds queue and notice that the patron column includes the patron’s email 5. Click on the hyperlinked email. It will open a new email with the subject of "Hold item: (your biblio title)" 6. Disable the ‘HidePatronName’ system preference 1. Check the holds queue again and notice that no contact information can be seen for the patron 7. Sign off and have an amazing day :D
Created attachment 169701 [details] [review] Bug 36595: Updated to make "Hold item:" translatable To test: 1. Make sure that the system preference ‘HidePatronName’ is set to ‘Show’ 2. Select a patron that has a primary email address set in their contact information. 1. Place a hold for that patron 3. Build the holds queue: perl /kohadevbox/koha/misc/cronjobs/holds/build_holds_queue.pl 4. Check the holds queue and notice that the patron column includes the patron’s email 5. Click on the hyperlinked email. It will open a new email with the subject of "Hold item: (your biblio title)" 6. Disable the ‘HidePatronName’ system preference 1. Check the holds queue again and notice that no contact information can be seen for the patron 7. Sign off and have an amazing day :D Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
Created attachment 169738 [details] [review] Bug 36595: Added email address field to the holds queue table To test: 1. Make sure that the system preference ‘HidePatronName’ is set to ‘Show’ 2. Select a patron that has a primary email address set in their contact information. 1. Place a hold for that patron 3. Build the holds queue: perl /kohadevbox/koha/misc/cronjobs/holds/build_holds_queue.pl 4. Check the holds queue and notice that the patron column includes the patron’s email 5. Click on the hyperlinked email. It will open a new email with the subject of "Hold item: (your biblio title)" 6. Disable the ‘HidePatronName’ system preference 1. Check the holds queue again and notice that no contact information can be seen for the patron 7. Sign off and have an amazing day :D Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Great work, thanks Laura. Passing QA
I see some small issues here. You don't check if the patron actually has an email set. The link will be broken if the email field is empty I believe. This will also not be translatable: ?subject=Hold item: [% itemsloo.title | uri %]"> I think when I looked at this trick in the past, there was also an issue with UTF8/diacritics. As you include the title, it might be worth testing with some Chinese or umlauts as these could happen to occur there. I don't thinks this should be a blocker here, but it might also be worth filing a separate bug for some template trick to show the email address according to EmailFieldPrimary and EmailFieldPrecedence. +1 for adding the class :)
Sorry, spotted something else: You use itemsloo.patron.email in the link, but itemsloo.patron.notice_email_address in the display text.