For example word [[дождь]] is not recognized by module. Fresh Drupal 7.7 install with default module settings.

CommentFileSizeAuthor
#2 international_regex-1385832-2.patch681 bytesmisc

Comments

navik’s picture

Status: Active » Needs review

In freelinking.module

Change
'/(?<!\\\\)\[\[([A-Za-z0-9]{1}.+' . $separator . '?.+)]]/Uu';
to
'/(?<!\\\\)\[\[([A-Za-z0-9]{1}.+' . $separator . '?.+|.+)]]/Uu';

Thanks xandeadx (http://xandeadx.ru)

misc’s picture

StatusFileSize
new681 bytes

Attached patch for the above fix.

rhblake’s picture

I had a similar problem when a Swedish character (åäö) was the first character of a node title. E.g., [[Göran Persson]] would work but not [[Örjan Persson]]. This patch solved it. Thanks!

misc’s picture

Status: Needs review » Reviewed & tested by the community

@rhblake, I mark as reviewed & tested by the community, if you think this is wrong, but it back in need review.

juampynr’s picture

Version: 7.x-3.1 » 7.x-3.x-dev
Status: Reviewed & tested by the community » Active
Issue tags: +needs backport to 6.x
torbasow’s picture

This patch is not quite correct. Hasn’t anyone noticed that /(?<!\\)\[\[([A-Za-z0-9]{1}.+\:?.+|.+)]]/Uu fully equivalent to simplier /(?<!\\)\[\[(.+)]]/Uu?

It should be /(?<!\\)\[\[(([A-Za-z0-9]{1}.+\:)?.+)]]/Uu, I think. The quantifier ? must be applied not to only : but to pluginname: entirely.

juampynr’s picture

Version: 7.x-3.x-dev » 6.x-3.x-dev

Moved to 6.x-3.x branch so it can be ported.

Torbasow, could you post here some examples of why this pattern is wrong?

rsvelko’s picture

the patch from #2 fixed it for me on a drupal 6.25 with freelinking 3.2. for cyrillic.

gisle’s picture

Version: 6.x-3.x-dev » 6.x-3.4
Issue summary: View changes
Status: Active » Closed (fixed)

It looks like this has been fixed somewhere along the way.

I am unable to reproduce this when testing version 6.x-3.4.