Bug 14238

Summary: C4::Matcher is mangling data to match on
Product: Koha Reporter: Kyle M Hall <kyle>
Component: CatalogingAssignee: Galen Charlton <gmcharlt>
Status: RESOLVED DUPLICATE QA Contact: Testopia <testopia>
Severity: major    
Priority: P5 - low CC: dcook, gmcharlt, m.de.rooy, sfayle, tmisilo, tomascohen
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15541
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 14238 - Unit Test to demonstrate issue
Bug 14238 - Unit Test to demonstrate issue

Description Kyle M Hall 2015-05-20 16:19:42 UTC
When setting up an OCLC match point ( see bug 6499 ) the match always fails because C4::Matcher::_normalize converts "(OCoLC)20329436" to "OCOLC20329436" for reasons unknown. The _normalize subroutine has zero documentation as to what it is supposed to do or not do.
Comment 1 Kyle M Hall 2015-05-20 16:23:14 UTC Comment hidden (obsolete)
Comment 2 David Cook 2016-02-05 04:44:41 UTC
Hey Kyle, I've had the same issue when doing matching. 

Since it has no documentation, I've left it in Koha, but I've created a "none" or "raw" normalizer which skips C4::Matcher::_normalize, as it was butchering URLs...

You can see the signed off patch at: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15541
Comment 3 Owen Leonard 2016-06-21 16:12:50 UTC
Is this report specific to 3.18.x? If it is valid in master, please mark it as
such.
Comment 4 Kyle M Hall 2016-06-27 14:33:19 UTC
Created attachment 52897 [details] [review]
Bug 14238 - Unit Test to demonstrate issue
Comment 5 David Cook 2016-06-28 04:52:58 UTC
Oh... I should add a unit test for the URL mangling as well...
Comment 6 David Cook 2016-08-01 23:40:02 UTC
Kyle, what do you need to move forward on this one?
Comment 7 Tomás Cohen Arazi 2016-09-22 17:54:57 UTC
Bug 17318 introduces new normalization routines (including the misterious option 'None') which could be considered a solution to this bug, and probably also 15541.

Kyle, Galen, David: would you agree on closing this one as duplicate of bug 17318?
Comment 8 David Cook 2016-09-26 05:11:47 UTC
(In reply to Tomás Cohen Arazi from comment #7)
> Bug 17318 introduces new normalization routines (including the misterious
> option 'None') which could be considered a solution to this bug, and
> probably also 15541.
> 
> Kyle, Galen, David: would you agree on closing this one as duplicate of bug
> 17318?

Yep, so long as there's no normalization happening during the matching, I'm happy :).
Comment 9 Kyle M Hall 2016-09-26 17:17:30 UTC
(In reply to David Cook from comment #8)
> (In reply to Tomás Cohen Arazi from comment #7)
> > Bug 17318 introduces new normalization routines (including the misterious
> > option 'None') which could be considered a solution to this bug, and
> > probably also 15541.
> > 
> > Kyle, Galen, David: would you agree on closing this one as duplicate of bug
> > 17318?
> 
> Yep, so long as there's no normalization happening during the matching, I'm
> happy :).

Also agreed!
Comment 10 Tomás Cohen Arazi 2016-09-26 17:32:23 UTC

*** This bug has been marked as a duplicate of bug 17318 ***