[Bug 5737] New: ZOOM connection problem in merge_authority.pl when trying to do a batch merge.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5737 Summary: ZOOM connection problem in merge_authority.pl when trying to do a batch merge. Change sponsored?: --- Product: Koha Version: rel_3_2 Platform: All OS/Version: All Status: NEW Severity: normal Priority: P5 Component: Command-line Utilities AssignedTo: gmcharlt@gmail.com ReportedBy: fcapovilla@live.ca QAContact: koha-bugs@lists.koha-community.org Estimated Hours: 0.0 Created attachment 3122 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=3122 Modified the merge subroutine in AuthoritiesMarc.pm so it won't destroy the connection to the biblioserver. When trying to use the script "./merge_authority.pl -b" to merge all authorities, an error is returned. Here is an example using the verbose mode : $ ./merge_authority.pl -b -v Merging managing 76871 managing 14291 {_conn} undefined: has this Connection been destroy()ed? at /usr/lib/perl5/ZOOM.pm line 341. The problem seems to happen because the ZOOM connection to the biblioserver is destroyed on line 1305 of AuthoritiesMarc.pm. The merge is done correctly for the first authority, but an error is returned when we get to the second one because the connection was destroyed by the first call to &merge. There seems to be no way to check if a connection was destroyed, so I modified the &merge subroutine so it won't destroy the connection, but will return it to its original state after &merge is done with it. Patch attached. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5737 Frédérick Capovilla <fcapovilla@live.ca> changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|P5 |PATCH-Sent Patch Status|--- |Needs Signoff -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5737 Frédérick Capovilla <fcapovilla@live.ca> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|gmcharlt@gmail.com |fcapovilla@live.ca -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5737 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #3122|0 |1 is obsolete| | --- Comment #1 from Chris Cormack <chris@bigballofwax.co.nz> 2011-07-28 22:02:43 UTC --- Created attachment 4771 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=4771 Bug 5737: Modified the merge subroutine in AuthoritiesMarc.pm so it won't destroy the connection to the biblioserver. Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5737 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |chris@bigballofwax.co.nz Patch Status|Needs Signoff |Signed Off -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5737 Ian Walls <ian.walls@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |ian.walls@bywatersolutions. | |com Patch Status|Signed Off |Passed QA --- Comment #2 from Ian Walls <ian.walls@bywatersolutions.com> 2011-08-02 17:39:16 UTC --- Patch just introduces a new variable to store a value which is redefined in the next line. Patch then applies the old value before exiting the subroutine. Clean, easy to read, marking as Passed QA. -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5737 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch Status|Passed QA |Patch Pushed --- Comment #3 from Chris Cormack <chris@bigballofwax.co.nz> 2011-08-03 02:06:18 UTC --- Pushed, please test -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=5737 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|PATCH-Sent (DO NOT USE) |P4 Status|Pushed to Master |RESOLVED Resolution| |FIXED -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug. You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org