Modify

Opened 2 months ago

Last modified 12 days ago

#13932 assigned enhancement

Move some specific tests to country/regional rules

Reported by: naoliv Owned by: Don-vip
Priority: normal Milestone: 17.01
Component: Core validator Version:
Keywords: Cc:

Attachments (0)

Change History (11)

comment:1 Changed 2 months ago by Klumbumbus

Yes. The "street name contains ss" info can then be raised to warning or even error level. Currently it produces lots of false positives in switzerland.

comment:2 Changed 2 months ago by Klumbumbus

  • Milestone set to 16.11

comment:3 follow-up: Changed 2 months ago by Don-vip

The checks are language-specific, so we can indeed improve the boundaries file to add official languages, based on information from ISO3166 + ISO639, so we could have something like:

*[language("en")] ... {
 ...
}

and the check would be applied only in countries or subdivisions where English is an official language.

comment:4 Changed 2 months ago by Don-vip

  • Owner changed from team to Don-vip
  • Status changed from new to assigned

comment:5 in reply to: ↑ 3 ; follow-up: Changed 2 months ago by Klumbumbus

Replying to Don-vip:

the check would be applied only in countries or subdivisions where English is an official language.

That sounds interresting. However regarding the *[name =~ /(?i).*Strasse.*/] this one is country specific. Germany, Switzerland, Austria and Liechtenstein use the german language but only in Switzerland and Liechtenstein they write the word street with ss instead of ß (strasse instead of straße).

comment:6 Changed 2 months ago by Klumbumbus

The rules at Rules should also be adapted.

comment:7 in reply to: ↑ 5 Changed 7 weeks ago by Klumbumbus

Replying to Klumbumbus:

regarding the *[name =~ /(?i).*Strasse.*/] this one is country specific.

Here is my modification of the code:

*[name =~ /(?i).*Strasse.*/][inside("DE,AT")] {
  throwError: tr("street name contains ss");
}
*[name =~ /(?i).*Straße.*/][inside("LI,CH")] {
  throwError: tr("street name contains ß");
}

comment:8 Changed 6 weeks ago by Don-vip

  • Milestone changed from 16.11 to 16.12

Milestone renamed

comment:9 Changed 3 weeks ago by Don-vip

  • Milestone changed from 16.12 to 17.01

i18n impact => next milestone

comment:10 Changed 3 weeks ago by Klumbumbus

The test can be enhanced to addr:street

*[addr:street =~ /(?i).*Strasse.*/][inside("DE,AT")],
*[name =~ /(?i).*Strasse.*/][inside("DE,AT")] {
  throwError: tr("street name contains ss");
}
*[addr:street =~ /(?i).*Straße.*/][inside("LI,CH")],
*[name =~ /(?i).*Straße.*/][inside("LI,CH")] {
  throwError: tr("street name contains ß");
}
Last edited 3 weeks ago by Klumbumbus (previous) (diff)

comment:11 Changed 12 days ago by Klumbumbus

In 11439/josm:

  • see #13932 - make Straße/Strasse validator test country specific and raise to error level
  • feed ignore list

Add Comment

Modify Ticket

Change Properties
Set your email in Preferences
Action
as assigned The owner will remain Don-vip.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from Don-vip to the specified user. Next status will be 'new'.
Next status will be 'needinfo'.The owner will change to naoliv
as duplicate The resolution will be set to duplicate. Next status will be 'closed'.The specified ticket will be cross-referenced with this ticket
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.