I'm not so sure if a whitelist instead of ranges are the way to go, but as an easy fix for now, I'd like to add the remaining latin accented characters to the valid list of chars.

The first patch is simply a reordering of the current list. The interdiff and extended patch will follow in #1.

CommentFileSizeAuthor
#2 validate_all_latin-2499165-1.patch2.75 KBjcnventura
#2 interdiff.txt1.51 KBjcnventura
link-sort-chars.patch2.52 KBjcnventura

Comments

Status: Needs review » Needs work

The last submitted patch, link-sort-chars.patch, failed testing.

jcnventura’s picture

jcnventura
Primary language English
commented
Status: Needs work » Needs review
StatusFileSize
new interdiff.txt1.51 KB
new validate_all_latin-2499165-1.patch2.75 KB
renatog’s picture

renatog commented
Assigned: Unassigned » renatog
Status: Needs review » Reviewed & tested by the community
Issue tags: +ciandt-contrib

There really is a comment to add letters

Thanks

  • RenatoG committed a30f0b1 on 7.x-1.x authored by jcnventura 
    Issue #2499165 by jcnventura, RenatoG: Validate all latin accented chars
renatog’s picture

renatog commented
Status: Reviewed & tested by the community » Fixed

Fixed.

Comitted in dev branch.

Good Work.

Regards.

jcnventura’s picture

jcnventura
Primary language English
commented

Oi Renato!!

Looking forward to Link 7.x-1.5!

Obrigado.

João Ventura

renatog’s picture

renatog commented

Hi João, how are you?

You're right.

I created this issue to plan the new release:

#2894892: Create new Release: 7.x-1.5

Let's create the new release as soon as possible.

When this release is released I'll let you know.

Thank you very much for your contributions João.

Bom Trabalho e Bom Final de Semana.

Regards.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

jrochate’s picture

jrochate commented

Hi.

From what I've seen this only protects domains.

How can this be applied to full path, like this: http://www.example.com/Sónia ?

thanks.