search

osm-fr / osmose-backend

Part of osmose that runs the analysis, and send the results to the frontend.
GNU General Public License v3.0
94 stars 116 forks source link

Do not warn on newlines in values #2397

Closed Famlam closed 6 days ago

Famlam commented 6 days ago

See #2302 1) The wiki was updated 2) JOSM has added this to this months milestone

So I think we should stop warning about it too.

This PR stops warning for newlines in values (replacing it by spaces, so "at position X" is not affected). It keeps warning for keys with newlines.

I didn't change the regex but opted for a replace because: 1) the same regex is used for keys (and keys with newlines are wrong) 2) There's no convenient alternative for \p{Control} in regex that matches the same minus \n. I think [\u0000-\u0009\u000b\u001F\u007F-\u009F] is a lot less readable.

Also remove the newline from the example, and fix a few typos