using cyrillic symbols (купить/адрес) - error
using node/114 (cyrillic alias: купить/адрес) is not converted to node/114, it's still node/114
using get-request in cyrillic (may be and others domains) domain (купить.рф/buy&foo=bar) will converted to купить.рф/buy%3Ffoo%3dbar

CommentFileSizeAuthor
#19 link-cyrillic_symbols-1236012-19.patch2.99 KBsuper_romeo
#18 link-cyrillic_symbols-1236012-17.patch2.98 KBsuper_romeo
#17 link-cyrillic_symbols-1236012-16.patch3.08 KBsuper_romeo
#15 link-cyrillic_symbols-1236012-15.patch3.07 KBNamzhilma Zhambalova
#14 link-cyrillic_symbols-1236012-14.patch3.08 KBsuper_romeo
#11 link-cyrillic_symbols-1236012-11.patch3.04 KBjcnventura
#8 link-cyrillic_symbols-1236012-8.diff3 KBartematem
#6 link_cyrillic_symbols_support-1236012-6.patch2.5 KBAnonymous (not verified)
#1 link_cyrillic_symbols_support.patch2.5 KBantonnavi

Comments

antonnavi’s picture

antonnavi
Primary language Ukrainian
Location Leipzig
commented
Issue summary: View changes
StatusFileSize
new link_cyrillic_symbols_support.patch2.5 KB

Usage cyrillic symbols in URL path I fix with attached patch.
Added: Russian, Ukrainian, Belorussian, Serbian symbols support.

antonnavi’s picture

antonnavi
Primary language Ukrainian
Location Leipzig
commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 1: link_cyrillic_symbols_support.patch, failed testing.

antonnavi’s picture

antonnavi
Primary language Ukrainian
Location Leipzig
commented
Version: 7.x-1.0-alpha3 » 7.x-1.3

Status: Needs work » Needs review

Antonnavi queued 1: link_cyrillic_symbols_support.patch for re-testing.

Anonymous’s picture

Anonymous (not verified) commented
StatusFileSize
new link_cyrillic_symbols_support-1236012-6.patch2.5 KB

Re-rolling patch for tesbot.

Anonymous’s picture

Anonymous (not verified) commented
Version: 7.x-1.3 » 7.x-1.x-dev

Needs to be against dev version.

artematem’s picture

artematem commented
StatusFileSize
new link-cyrillic_symbols-1236012-8.diff3 KB

Added "рф" to LINK_DOMAINS constant.

Status: Needs review » Needs work

The last submitted patch, 8: link-cyrillic_symbols-1236012-8.diff, failed testing.

jcnventura’s picture

jcnventura
Primary language English
commented
Related issues: +#2385565: link_validate_url() LINK_ICHARS needs to be completed
jcnventura’s picture

jcnventura
Primary language English
commented
Status: Needs work » Needs review
StatusFileSize
new link-cyrillic_symbols-1236012-11.patch3.04 KB

@artamem: there are a lot more TLDs in that line..

Status: Needs review » Needs work

The last submitted patch, 11: link-cyrillic_symbols-1236012-11.patch, failed testing.

artematem’s picture

artematem commented

It still very site or country specific patch, and I don't think it will be committed some day. What this module needs is more general approach to handle Cyrillic, Greek, Latin etc. characters.

super_romeo’s picture

super_romeo commented
StatusFileSize
new link-cyrillic_symbols-1236012-14.patch3.08 KB

updating patch

Namzhilma Zhambalova’s picture

Namzhilma Zhambalova commented
Status: Needs work » Needs review
StatusFileSize
new link-cyrillic_symbols-1236012-15.patch3.07 KB

Hello,

I added patch for 7.x-1.5 version.

Status: Needs review » Needs work

The last submitted patch, 15: link-cyrillic_symbols-1236012-15.patch, failed testing. View results

super_romeo’s picture

super_romeo commented
StatusFileSize
new link-cyrillic_symbols-1236012-16.patch3.08 KB

Patch rerolled for dev version.

super_romeo’s picture

super_romeo commented
StatusFileSize
new link-cyrillic_symbols-1236012-17.patch2.98 KB

Fixed.

super_romeo’s picture

super_romeo commented
StatusFileSize
new link-cyrillic_symbols-1236012-19.patch2.99 KB

Patch rerolled for dev version.