itiva has reported to us that quotes in the title of a record cause the call to not be made to the patron. The fix is to remove quotes from the title, as quotes are not spoken anyway ( That is, "Queens" and "Queen's" are pronounced the same ).
Created attachment 115909 [details] [review] Bug 27562: itiva notices break if record title contains quotes itiva has reported to us that quotes in the title of a record cause the call to not be made to the patron. The fix is to remove quotes from the title, as quotes are not spoken anyway ( That is, "Queens" and "Queen's" are pronounced the same ). Test Plan: 1) Set up itiva to send phone notes 2) Find a record with quotes in the title 3) Trigger an itiva notice ( checkout, checkin, place hold, etc ) using the itiva outbound cronjob 4) View the CSV, note the title has the quotes in it 5) Apply this patch 6) Repeat steps 2-3 7) View the CSV, note the title contains no quotes!
Created attachment 120351 [details] [review] Bug 27562: itiva notices break if record title contains quotes itiva has reported to us that quotes in the title of a record cause the call to not be made to the patron. The fix is to remove quotes from the title, as quotes are not spoken anyway ( That is, "Queens" and "Queen's" are pronounced the same ). Test Plan: 1) Set up itiva to send phone notes 2) Find a record with quotes in the title 3) Trigger an itiva notice ( checkout, checkin, place hold, etc ) using the itiva outbound cronjob 4) View the CSV, note the title has the quotes in it 5) Apply this patch 6) Repeat steps 2-3 7) View the CSV, note the title contains no quotes! Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
works :) git author needs a fix: * Author 'Koha user' seems invalid
Created attachment 120688 [details] [review] Bug 27562: itiva notices break if record title contains quotes itiva has reported to us that quotes in the title of a record cause the call to not be made to the patron. The fix is to remove quotes from the title, as quotes are not spoken anyway ( That is, "Queens" and "Queen's" are pronounced the same ). Test Plan: 1) Set up itiva to send phone notes 2) Find a record with quotes in the title 3) Trigger an itiva notice ( checkout, checkin, place hold, etc ) using the itiva outbound cronjob 4) View the CSV, note the title has the quotes in it 5) Apply this patch 6) Repeat steps 2-3 7) View the CSV, note the title contains no quotes! Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
(In reply to Victor Grousset/tuxayo from comment #3) > git author needs a fix: I have taken the liberty of correcting it for Kyle so that it can go into the QA queue.
Thanks Owen :) QA: Looks good to me. I'll try to find someone to signoff. Test note: here are instruction to setup i-tiva https://wiki.koha-community.org/wiki/External_Integration#i-tiva
Created attachment 120802 [details] [review] Bug 27562: itiva notices break if record title contains quotes itiva has reported to us that quotes in the title of a record cause the call to not be made to the patron. The fix is to remove quotes from the title, as quotes are not spoken anyway ( That is, "Queens" and "Queen's" are pronounced the same ). Test Plan: 1) Set up itiva to send phone notes 2) Find a record with quotes in the title 3) Trigger an itiva notice ( checkout, checkin, place hold, etc ) using the itiva outbound cronjob 4) View the CSV, note the title has the quotes in it 5) Apply this patch 6) Repeat steps 2-3 7) View the CSV, note the title contains no quotes! Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
Thanks Owen :)
Pushed to master for 21.05, thanks to everybody involved!
Pushed to 20.11.x for 20.11.06
Pushed to 20.05.x for 20.05.12
Not backported to oldoldstable (19.11.x). Feel free to ask if it's needed.