Needs work
Project:
Similar Entries
Version:
7.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
26 Feb 2013 at 10:31 UTC
Updated:
7 May 2017 at 10:37 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
cuman commentedа-я - letters of the Russian (Cyrillic) alphabet
Comment #2
Marko B commentedI have similar problem, croatian letters like "čžđšđ" doesn't work, probably the same reason. When I have node with only english alphabet it works, for non-english it doesnt find similar titles.
Comment #3
Marko B commentedI solved it with this for title. Body guess this code above from cuman can be used.
$title = preg_replace('/[^a-z0-9 _-čćšđžČĆŽŠĐ]+/iu', '', $node->title);Comment #4
Anonymous (not verified) commentedWe seem to have worked on the same thing. See #1958136: Fails to calculate score - perhaps an i18n issue. Please be aware there is another bug in this place: title and body should be taken into account separately. Your current solution seems to fail in case the body is empty.
Comment #5
kedramonHi guys,
have same problems with Cyrillic alphabet, here is my patch,
it is also fix missing body field bug,
and I hope it fix all language.
Comment #6
dillix commentedComment #7
kedramonLas update of this module is December 2, 2014, can we expect some update?
Comment #8
jenlamptonIt looks like the status wasn't set to "Needs Review" so that is likely why this patch hasn't been reviewed yet for inclusion. Please update the status when a patch is uploaded, if possible.
This patch will need to be updated to remove the fix for the other issue. That issue should have it's own patch in it's queue, and applying them both to the project will cause problems. In general keeping things separate will keep progress moving along faster as a problem with one patch won't prevent the other from being merged.
Comment #9
kedramonAfter 2 year ago!
Here is the patch with some codestyle fixes.
Comment #10
pashikk commentedFor what version is this patch? I got 7.x-2.x-dev of 2016-Dec-24 and the patch fails to appy.
Comment #11
dillix commentedThis patch is for beta5. It doesn't work with beta6 & latest dev.