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

[Bug 11175] Show the parent record's component parts in the detailed views
by bugzilla-daemon＠bugs.koha-community.org 04 Oct '18

04 Oct '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11175 Josef Moravec <josef.moravec(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |21484 --- Comment #47 from Josef Moravec <josef.moravec(a)gmail.com> --- (In reply to Katrin Fischer from comment #44) > (In reply to Josef Moravec from comment #39) > > And please add support for elasticsearch. > > I don't think we should make this a requirement at the moment until > Elasticsearch has left experimental status and this has been added to the > coding guidelines as a requirement. For now, I think, not breaking > Elasticsearch functionality with new code would be enough. Other things can > be dealt with later. Ok, I could live without elastic for now, but I have filled a follow-up bug report - bug 21484 to address this. > > This is something we have been asked about for an awful long time, it would > be sad to see the first steps abandoned. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21484 [Bug 21484] Show the parent record's component parts in the detailed views - Elasticsearch -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 12996] Tool to bulk export patrons images
by bugzilla-daemon＠bugs.koha-community.org 04 Oct '18

04 Oct '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12996 --- Comment #2 from Pablo AB <pablo.bianchi(a)gmail.com> --- The attached script wasn't working on current stable release. I've updated on this two places: - https://wiki.koha-community.org/wiki/Debian_packages_script_library#Bulk_ex… - https://gist.github.com/pabloab/347dc307a9650a5c4309c8fd6296c906 Tested on 17.11 and 18.05. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15554] Organize entries in ccl.properties
by bugzilla-daemon＠bugs.koha-community.org 04 Oct '18

04 Oct '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15554 --- Comment #4 from David Cook <dcook(a)prosentient.com.au> --- Looking at ccl.properties, it looks like it's still disorganized, but at this point in time... I don't know that it really matters. I think Zebra's days are numbered, so there's probably not much point investing time in this bug report. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15529] Process Message Queue may send duplicate emails if process is launched twice
by bugzilla-daemon＠bugs.koha-community.org 04 Oct '18

04 Oct '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15529 M. Tompsett <mtompset(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off --- Comment #27 from M. Tompsett <mtompset(a)hotmail.com> --- I signed off Alex's having tested with all three. Alex has signed off mine having tests with all three. Marking as signed off. -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15529] Process Message Queue may send duplicate emails if process is launched twice
by bugzilla-daemon＠bugs.koha-community.org 04 Oct '18

04 Oct '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15529 M. Tompsett <mtompset(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #79910|0 |1 is obsolete| | --- Comment #26 from M. Tompsett <mtompset(a)hotmail.com> --- Created attachment 79913 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=79913&action=ed… Bug 15529: Followup to send without more libraries This uses the letter_code parameter to limit the current duplicate run of the process_message_queue.pl script to just this one message. We don't care about receiving duplicate warning messages as much as we care about sending patrons duplicate messages. Works as described, passes QA test tool. Signed-off-by: Alex Buckley <alexbuckley(a)catalyst.net.nz> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15529] Process Message Queue may send duplicate emails if process is launched twice
by bugzilla-daemon＠bugs.koha-community.org 04 Oct '18

04 Oct '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15529 M. Tompsett <mtompset(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #79649|0 |1 is obsolete| | --- Comment #25 from M. Tompsett <mtompset(a)hotmail.com> --- Created attachment 79912 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=79912&action=ed… Bug 15529 - Wrapping email sending up in the verbose flag Thus the cron email telling the administrator or support company that a second execution of the process_message_queue.pl cron whilst it is already running is only sent if it is wanted and verbose is set. Sponsored-By: New Zealand Parliamentary Library Signed-off-by: Mark Tompsett <mtompset(a)hotmail.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15529] Process Message Queue may send duplicate emails if process is launched twice
by bugzilla-daemon＠bugs.koha-community.org 04 Oct '18

04 Oct '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15529 M. Tompsett <mtompset(a)hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #78918|0 |1 is obsolete| | --- Comment #24 from M. Tompsett <mtompset(a)hotmail.com> --- Created attachment 79911 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=79911&action=ed… Bug 15529 - Locking process_message_queue.pl cron whilst it is running This amended patch is called by koha-common. koha-common runs this process_message_queue.pl cronjob foreach enabled Koha instance on the server. As this cronjob is run for every Koha instance the retrival of all Koha instance names from debian/scripts/koha-scripts in previous obsolete patches on this bug report and the looping through every name to check if the lock file existed is redundant. Instead this patch retrieves the 'logdir' of the Koha instance being iterated over and checks if the messagequeue.lock file in that directory is locked. If the aforementioned file is not locked then the SendQueuedMessages() in C4/Letters.pm is called, otherwise an email is sent (to the manually inputted email server, and to/from addresses) and duplicate execution of the cronjob is prevented. Test plan: 1. In two terminals open koha-shell and try to run ./process_message_queue.pl and notice you can run the script multiple times simulataneously without a warning/error being displayed 2. Apply patch and input the email server, to/from email addresses you would like the email to be sent to. Also confirm your Koha instance is enabled to send emails. 3. Repeat step 1 and notice that the second of the two scripts to be run will output text saying './process_message_queue.pl is already running. Exiting' In this case the aforementioned text is displayed and the body of the cronjob is not run. 4. The first cronjob script to run, runs without issues 5. At the end of the execution of the first script a 15 second pause is run 6. Notice you have received an email Sponsored-By: New Zealand Parliamentary Library Signed-off-by: Mark Tompsett <mtompset(a)hotmail.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15529] Process Message Queue may send duplicate emails if process is launched twice
by bugzilla-daemon＠bugs.koha-community.org 03 Oct '18

03 Oct '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15529 --- Comment #23 from Alex Buckley <alexbuckley(a)catalyst.net.nz> --- Created attachment 79910 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=79910&action=ed… Bug 15529: Followup to send without more libraries This uses the letter_code parameter to limit the current duplicate run of the process_message_queue.pl script to just this one message. We don't care about receiving duplicate warning messages as much as we care about sending patrons duplicate messages. Works as described, passes QA test tool. Signed-off-by: Alex Buckley <alexbuckley(a)catalyst.net.nz> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15529] Process Message Queue may send duplicate emails if process is launched twice
by bugzilla-daemon＠bugs.koha-community.org 03 Oct '18

03 Oct '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15529 Alex Buckley <alexbuckley(a)catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #79726|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15529] Process Message Queue may send duplicate emails if process is launched twice
by bugzilla-daemon＠bugs.koha-community.org 03 Oct '18

03 Oct '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15529 --- Comment #22 from Alex Buckley <alexbuckley(a)catalyst.net.nz> --- Hi M. Tompsett I have tested and your patch, in combination with my first two patches, works nicely. I will sign off on your patch, can you please do the same on my two patches. Cheers, Alex -- You are receiving this mail because: You are watching all bug changes.

1 0