[Bug 9520] New: Update default LOC Z39.50 target
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9520 Bug ID: 9520 Summary: Update default LOC Z39.50 target Classification: Unclassified Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: minor Priority: P5 - low Component: Z39.50 / SRU / OpenSearch Servers Assignee: bgkriegel@gmail.com Reporter: bgkriegel@gmail.com CC: m.de.rooy@rijksmuseum.nl Depends on: 9494 Followup to Bug 9494 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9520 --- Comment #1 from Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> --- Created attachment 14983 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=14983&action=edit Bug 9520 - Update default LOC Z39.50 target This patch updates (if present) target settings of Library of Congress Z39.50 server. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9520 Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9520 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #14983|0 |1 is obsolete| | --- Comment #2 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 15004 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15004&action=edit [SIGNED-OFF] Bug 9520 - Update default LOC Z39.50 target This patch updates (if present) target settings of Library of Congress Z39.50 server. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9520 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |kyle@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9520 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA CC| |jonathan.druart@biblibre.co | |m --- Comment #3 from Jonathan Druart <jonathan.druart@biblibre.com> --- QA comment: The query is not the good one, mysql raises a warning: Data truncated for column 'type' at row 4 type is an enum (cf kohastructure: `type` enum('zed','opensearch') NOT NULL default 'zed') Marked as Failed QA. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9520 --- Comment #4 from Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> --- (In reply to comment #3)
QA comment:
The query is not the good one, mysql raises a warning: Data truncated for column 'type' at row 4 type is an enum (cf kohastructure: `type` enum('zed','opensearch') NOT NULL default 'zed')
Marked as Failed QA.
Sorry but.. did the comment applies to this bug? bgkriegel@sentey:~/kohaclone$ mysql -uUUU -pPPP kohadev --verbose -e "UPDATE z3950servers SET host = 'lx2.loc.gov', port = 210, db = 'LCDB', type = 'USMARC', encoding = 'utf8' WHERE name = 'LIBRARY OF CONGRESS'" -------------- UPDATE z3950servers SET host = 'lx2.loc.gov', port = 210, db = 'LCDB', type = 'USMARC', encoding = 'utf8' WHERE name = 'LIBRARY OF CONGRESS' -------------- No warnings :) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9520 --- Comment #5 from Jonathan Druart <jonathan.druart@biblibre.com> --- (In reply to comment #4)
Sorry but.. did the comment applies to this bug?
Yes! mysql> show create table z3950servers; [...] `type` enum('zed','opensearch') NOT NULL DEFAULT 'zed', [...] mysql> UPDATE z3950servers SET host = 'lx2.loc.gov', port = 210, db = 'LCDB', type = 'USMARC', encoding = 'utf8' WHERE name = 'LIBRARY OF CONGRESS'; Query OK, 0 rows affected, 1 warning (0.06 sec) Rows matched: 1 Changed: 0 Warnings: 1 mysql> show warnings; +---------+------+-------------------------------------------+ | Level | Code | Message | +---------+------+-------------------------------------------+ | Warning | 1265 | Data truncated for column 'type' at row 4 | +---------+------+-------------------------------------------+ 1 row in set (0.00 sec) mysql> select type from z3950servers where name='LIBRARY OF CONGRESS'; +------+ | type | +------+ | | +------+ 1 row in set (0.00 sec) -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9520 --- Comment #6 from Jonathan Druart <jonathan.druart@biblibre.com> --- Bug 9494 does not update the type column. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9520 Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #15004|0 |1 is obsolete| | --- Comment #7 from Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> --- Created attachment 15769 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15769&action=edit Bug 9520 -[Revised] - Update default LOC Z39.50 target This patch updates (if present) target settings of Library of Congress Z39.50 server. Revision: Was trying to update wrong column 'type', now correct column 'syntax'. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9520 --- Comment #8 from Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> --- (In reply to comment #6)
Bug 9494 does not update the type column. It doesn't need to. The correct column to update is 'syntax'. I was fooled by the name. I'm sorry (again).
-- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9520 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Passed QA Patch complexity|--- |Trivial patch QA Contact| |jonathan.druart@biblibre.co | |m --- Comment #9 from Jonathan Druart <jonathan.druart@biblibre.com> --- (In reply to comment #8)
(In reply to comment #6)
Bug 9494 does not update the type column. It doesn't need to. The correct column to update is 'syntax'. I was fooled by the name. I'm sorry (again).
Don't worry, thanks for your quick revised patch! Now it is good. This patch updates the LOC z3950 server for current installations. Marked as Passed QA. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9520 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #15769|0 |1 is obsolete| | --- Comment #10 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 15771 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15771&action=edit Bug 9520 -[Revised] - Update default LOC Z39.50 target This patch updates (if present) target settings of Library of Congress Z39.50 server. Revision: Was trying to update wrong column 'type', now correct column 'syntax'. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Amended patch: add the SetVersion call -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9520 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master --- Comment #11 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- This patch has been pushed to master. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9520 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |Pushed to Stable CC| |chris@bigballofwax.co.nz --- Comment #12 from Chris Cormack <chris@bigballofwax.co.nz> --- Pushed to 3.8.x and 3.10.x will be in 3.8.11 and 3.10.4 -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org