The sample notice RLIST is used for a notification of the patron when a new issue of a serial arrives (independent from the routing list feature). You can subscribe to the notice from the OPAC - subscription tab - more details. The notice currently allows to include fields from branches, biblio, biblioitems and borrowers. It would be very good to also include subscriptions, serials and items so you can print information about which field has arrived into the notice. Related code is here: http://git.koha-community.org/gitweb/?p=koha.git;a=blob;f=C4/Letters.pm;hb=c16e19b3fdff71e20db1df036931d96dfabcce4d#l411
Created attachment 39487 [details] [review] Bug 13972: Include fields from subscription and serial table in serial notification email Currently it's not possible to include information about which issue has arrived in the serial notification notice the patron can subscribe to from the OPAC. The patch makes the fields from the subscription and serial table available to the notice template. In order to be able to print information about the correct issue, the GetAlert has been modified to expext the serialid as externalid when the module is issue. git grep SendAlerts (only call with 'issue' is in Serial.pm) To test: - Add a subscription, select a patron notification template - Search for the record in the OPAC - Go to the subscription tab - More details - Subscribe to the notification - Edit the notice template you selected for the subscription - add fields from subscription - add fields from serial (serial.serialseq has the issue information) - Receive an issue for the subscription - Check that you have received the notification and that all information has been printed correctly NOTE: notice is sent directly, not through the message_queue
I know I am missing tests for the changed routine SendAlerts. I will probably need some help there.
Created attachment 39489 [details] [review] Bug 13972: Follow-up - Add unit tests for changed parts of SendAlerts To test: - prove t/db_dependent/Letters.t
I found some existing tests for serial claims - tried to adapt them for the other use case 'issue'. Ready for testing now :)
Created attachment 39490 [details] [review] Bug 13972: Follow-up - Add unit tests for changed parts of SendAlerts To test: - prove t/db_dependent/Letters.t
Created attachment 40021 [details] [review] [Signed-off] Bug 13972: Include fields from subscription and serial table in serial notification email Currently it's not possible to include information about which issue has arrived in the serial notification notice the patron can subscribe to from the OPAC. The patch makes the fields from the subscription and serial table available to the notice template. In order to be able to print information about the correct issue, the GetAlert has been modified to expext the serialid as externalid when the module is issue. git grep SendAlerts (only call with 'issue' is in Serial.pm) To test: - Add a subscription, select a patron notification template - Search for the record in the OPAC - Go to the subscription tab - More details - Subscribe to the notification - Edit the notice template you selected for the subscription - add fields from subscription - add fields from serial (serial.serialseq has the issue information) - Receive an issue for the subscription - Check that you have received the notification and that all information has been printed correctly NOTE: notice is sent directly, not through the message_queue Followed test plan, works as expected. Signed-off-by: Marc Véron <veron@veron.ch>
Created attachment 40022 [details] [review] [Signed-off] Bug 13972: Follow-up - Add unit tests for changed parts of SendAlerts To test: - prove t/db_dependent/Letters.t Signed-off-by: Marc Véron <veron@veron.ch>
Created attachment 40273 [details] [review] Bug 13972: Include fields from subscription and serial table in serial notification email Currently it's not possible to include information about which issue has arrived in the serial notification notice the patron can subscribe to from the OPAC. The patch makes the fields from the subscription and serial table available to the notice template. In order to be able to print information about the correct issue, the GetAlert has been modified to expext the serialid as externalid when the module is issue. git grep SendAlerts (only call with 'issue' is in Serial.pm) To test: - Add a subscription, select a patron notification template - Search for the record in the OPAC - Go to the subscription tab - More details - Subscribe to the notification - Edit the notice template you selected for the subscription - add fields from subscription - add fields from serial (serial.serialseq has the issue information) - Receive an issue for the subscription - Check that you have received the notification and that all information has been printed correctly NOTE: notice is sent directly, not through the message_queue Followed test plan, works as expected. Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Jonathan Druart <jonathan.druart@koha-community.org>
Created attachment 40274 [details] [review] Bug 13972: Follow-up - Add unit tests for changed parts of SendAlerts To test: - prove t/db_dependent/Letters.t Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Jonathan Druart <jonathan.druart@koha-community.org>
Patches pushed to master. Thanks Katrina!
*** Bug 12437 has been marked as a duplicate of this bug. ***