- Koha-bugs - lists.koha-community.org

[Bug 10306] Koha to MARC mappings (Part 1): Allow multiple mappings per kohafield (for say 260/RDA 264)
by bugzilla-daemon＠bugs.koha-community.org 23 Oct '17

23 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10306 --- Comment #66 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- Created attachment 68382 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68382&action=ed… Bug 10306: (QA follow-up) More feedback for admins in koha2marclinks As requested by Tomas, this patch does: [1] Add a js alert when you did not type field,subfield [2] Print a yellow alert div when the field,subfield is not found. [3] Bonus: Make it little bit more secure by demanding a POST. (Leaving CSRF etc. for another report.) Signed-off-by: Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 10306] Koha to MARC mappings (Part 1): Allow multiple mappings per kohafield (for say 260/RDA 264)
by bugzilla-daemon＠bugs.koha-community.org 23 Oct '17

23 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10306 --- Comment #65 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- (In reply to Tomás Cohen Arazi from comment #64) > Marcel, I've looked at this patchset, along with bug 19096. Everything looks > correct so far. I'm only concerned about the UI changes, the 'comma' > separator for subfields, and no feedback on wrong input. About the UI changes of koha2marclinks.pl: You mentioned "It seems to be no option to edit a mapping. Only an 'Add' button.". We have a few possibilities: 1) There are no buttons for fields like biblionumber, itemnumber 2) There is only one Add button, when the field is not yet used in a mapping. 3) There are two buttons: Add and Remove. You can remove the current mapping or add a new one. Why is there no Edit button? This is by design. You cant change a mapping but you can remove it. Or add a new one. The comma is used when adding a new field in the js form. Do you have another suggestion? When you type something silly, there is no feedback. This was somewhat lazy indeed, thinking that we have an admin here; if you are not knowing what you are doing, this is a very dangerous form.. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12802] Send notices using several email addresses
by bugzilla-daemon＠bugs.koha-community.org 23 Oct '17

23 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12802 --- Comment #98 from sandboxes(a)biblibre.com <sandboxes(a)biblibre.com> --- Created attachment 68380 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68380&action=ed… Bug 12802: Make tests pass Signed-off-by: Axelle Clarisse <axelle.clarisse(a)univ-amu.fr> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12802] Send notices using several email addresses
by bugzilla-daemon＠bugs.koha-community.org 23 Oct '17

23 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12802 --- Comment #97 from sandboxes(a)biblibre.com <sandboxes(a)biblibre.com> --- Created attachment 68379 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68379&action=ed… Bug 12802: Update all circulation alerts, not only the first one Since SendCirculationAlert can generate multiple entries in message_queue, it should update them all if pending messages already exist Signed-off-by: Axelle Clarisse <axelle.clarisse(a)univ-amu.fr> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12802] Send notices using several email addresses
by bugzilla-daemon＠bugs.koha-community.org 23 Oct '17

23 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12802 --- Comment #96 from sandboxes(a)biblibre.com <sandboxes(a)biblibre.com> --- Created attachment 68378 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68378&action=ed… Bug 12802: Fix tests in Reserves.t Signed-off-by: Axelle Clarisse <axelle.clarisse(a)univ-amu.fr> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12802] Send notices using several email addresses
by bugzilla-daemon＠bugs.koha-community.org 23 Oct '17

23 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12802 --- Comment #95 from sandboxes(a)biblibre.com <sandboxes(a)biblibre.com> --- Created attachment 68377 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68377&action=ed… Bug 12802: Fix tests in Letters.t Signed-off-by: Axelle Clarisse <axelle.clarisse(a)univ-amu.fr> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12802] Send notices using several email addresses
by bugzilla-daemon＠bugs.koha-community.org 23 Oct '17

23 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12802 --- Comment #94 from sandboxes(a)biblibre.com <sandboxes(a)biblibre.com> --- Created attachment 68376 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68376&action=ed… Bug 12802: Sent notices using several email addresses Currently it is not possible to select several email addresses to notify a patron. The only place where the mechanism exists is in the overdue_notices.pl script. This patch reuses the AutoEmailPrimaryAddress syspref and changes its type to "multiple". Like that it is now possible to select several email addresses. Note that there is no sense to select OFF and another value for this pref. So if the 'OFF' (first valid) value exist, it takes the priority. It will add the ability to choose the email addresses to use to notify patrons. The option is "email", "emailpro" and "B_email". If "OFF" is selected, the first valid address will be returned (same behavior as before this patch). Note for the QA step: I found a possible regression, but IMO it's not a big deal: Before this patch if a letter did not contain a "to_address", the C4::Letters::_send_message_by_email subroutine retrieve the email from the given borrowernumber. This is now done in the EnqueueMessage subroutine. What it means: If a borrower didn't have an email address when the notice was sent to the queue, the email address won't be check again when the notice will really sent to the patron. This change permits to sent a letter to the queue (EnqueueLetter) without any information (from_address, to_address), only the borrowernumber is mandatory to retrieve them. The _send_message_by_email subroutine should not contain any useful code, only sent the letter we ask it to sent. What this patch does: The GetNoticeEmailAddress subroutine has been renamed to GetNoticeEmailAddresses (reflect the plural). It only gets the patron' emails defined in the AutoEmailPrimaryAddress pref. If no 'to_address' parameter is given to EnqueueMessage, the emails will be retrieved at this moment. In C4::Message: An old form was found. The AutoEmailPrimaryAddress was not check. The smsalertnumber was sent for the to_address param, which is wrong. C4::Reserves: AddReserve and _koha_notify_reserve: The from address is built in the QueueLetter. It is useless to do it here. overdue_notices.pl: The script could be cleaned a little bit if we remove the --email parameter. Indeed it is redundant with this new enhancement. I can propose another patch with a die statement and a message: "you should use the pref AutoEmailPrimaryAddress" if the --email is provided. opac/opac-shareshelf.pl and opac/opac-memberentry.pl: just remove the from and to address. They will be filled on sending to the queue (because of the borrowernumber). Test plan: 1/ Apply this patch without updating the pref AutoEmailPrimaryAddress (or fill it with a single value if it is not done yet). 2/ Try the different way to sent notices to a patron (check the following letter code: HOLD, CHECKIN, CHECKOUT, PREDUE, RENEW, DUE). 3/ Verify the email is correctly sent to the message_queue. 4/ Fill the pref with email and emailpro (for instance) 5/ Verify that 2 notices are sent to the message_queue: 1 with the email and 1 with the emailpro. 6/ You can also verify that only 1 notice is generated if the emailpro is empty. Important note for testers: Pending messages must be removed each time you change the value of AutoEmailPrimaryAddress. When there are already existing pending messages for a borrower, Koha tries to update them rather than enqueuing new ones, but it will not update the recipient's email address, nor add/remove messages depending on the new value of AutoEmailPrimaryAddress; not sure if this is a bug or not. Signed-off-by: Brendan Gallagher <brendan(a)bywatersolutions.com> Signed-off-by: Axelle Clarisse <axelle.clarisse(a)univ-amu.fr> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12802] Send notices using several email addresses
by bugzilla-daemon＠bugs.koha-community.org 23 Oct '17

23 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12802 --- Comment #93 from sandboxes(a)biblibre.com <sandboxes(a)biblibre.com> --- Created attachment 68375 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=68375&action=ed… Bug 12802: DB changes - Set type multiple for AutoEmailPrimaryAddress Signed-off-by: Brendan Gallagher <brendan(a)bywatersolutions.com> Signed-off-by: Axelle Clarisse <axelle.clarisse(a)univ-amu.fr> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12802] Send notices using several email addresses
by bugzilla-daemon＠bugs.koha-community.org 23 Oct '17

23 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12802 sandboxes(a)biblibre.com <sandboxes(a)biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #68013|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12802] Send notices using several email addresses
by bugzilla-daemon＠bugs.koha-community.org 23 Oct '17

23 Oct '17

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12802 sandboxes(a)biblibre.com <sandboxes(a)biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #68012|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

1 0