linuxfrorg
fix multiple sharp signs in URIs
rename HttpUrlValidator as UriValidator
Suivi: https://linuxfr.org/suivi/impossible-de-mettre-un-lien-vers-un-salon-matrix-dans-les-liens-d-une-depeche
Beware of the invisible merge conflict with #355: the change won't cause a conflict but it won't work due to renaming the validator.
Beware of the invisible merge conflict with #355: the change won't cause a conflict but it won't work due to renaming the validator.
Indeed, I've just merged #355 in the current PR so @Oumph will just need to merge this one as the conflict is resolved.
I've just added commits to add an after_validation method too, to revert the workaround with the sharp sign before saving data inside the database. So user input will match with the data saved and visually the user will not be confused with the %23 encoding.
@Trim given that #355 was trivial and that this one doesn't show any sign of being merged, wouldn't it be easier to at least fix the existing bug by merging #355 first? Or would you merge this one?