Address symfony#17897 (review)

Author: Mathieu

reference/constraints.rst

@@ -78,6 +78,7 @@ Validation Constraints Reference
    constraints/Valid
    constraints/Traverse
    constraints/CssColor
+   constraints/NoSuspiciousCharacters
 
 The Validator is designed to validate objects against *constraints*.
 In real life, a constraint could be: "The cake must not be burned". In

reference/constraints/NoSuspiciousCharacters.rst

@@ -16,7 +16,7 @@ NoSuspiciousCharacters
 "symfony.com" and "ѕymfony.com" look similar, but the latter actually starts with a
 `cyrillic small letter dze`_. It could make a user think they'll navigate to Symfony's
 website, whereas it would be somewhere else.
-This is a kind of `spoofing attack`_ (called "IDN homograph attack"): it tries to
+This is a kind of `spoofing attack`_ (called "IDN homograph attack"). It tries to
 identify something as something else to exploit the resulting confusion.
 This is why it is recommended to check user-submitted, public-facing identifiers for
 suspicious characters in order to prevent such attacks.
@@ -104,8 +104,8 @@ Options
 This option is a bitmask of the checks you want to perform on the string:
 
 * ``NoSuspiciousCharacters::CHECK_INVISIBLE`` checks for the presence of invisible characters such as zero-width spaces, or character sequences that are likely not to display, such as multiple occurrences of the same non-spacing mark.
-* ``NoSuspiciousCharacters::CHECK_MIXED_NUMBERS`` (since ICU 58) checks for numbers from different numbering systems.
-* ``NoSuspiciousCharacters::CHECK_HIDDEN_OVERLAY`` (since ICU 62) checks for combining characters hidden in their preceding one.
+* ``NoSuspiciousCharacters::CHECK_MIXED_NUMBERS`` (usable with ICU 58 or higher) checks for numbers from different numbering systems.
+* ``NoSuspiciousCharacters::CHECK_HIDDEN_OVERLAY`` (usable with ICU 62 or higher) checks for combining characters hidden in their preceding one.
 
 You can also configure additional requirements using :ref:`locales <locales>` and
 :ref:`restrictionLevel <restrictionlevel>`.
@@ -126,20 +126,20 @@ Passing an empty array, or configuring :ref:`restrictionLevel <restrictionlevel>
 ``restrictionLevel``
 ~~~~~~~~~~~~~~~~~~~~
 
-**type**: ``integer`` **default**: ``NoSuspiciousCharacters::RESTRICTION_LEVEL_MODERATE`` on ICU >= 58, else ``NoSuspiciousCharacters::RESTRICTION_LEVEL_SINGLE_SCRIPT``
+**type**: ``integer`` **default**: ``NoSuspiciousCharacters::RESTRICTION_LEVEL_MODERATE`` on ICU >= 58, otherwise ``NoSuspiciousCharacters::RESTRICTION_LEVEL_SINGLE_SCRIPT``
 
 Configures the set of acceptable characters for the validated string through a
 specified "level":
 
 * ``NoSuspiciousCharacters::RESTRICTION_LEVEL_MINIMAL`` requires the string's characters to match :ref:`the configured locales <locales>`'.
 * ``NoSuspiciousCharacters::RESTRICTION_LEVEL_MODERATE`` also requires the string to be `covered`_ by Latin and any one other `Recommended`_ or `Limited Use`_ script, except Cyrillic, Greek, and Cherokee.
-* ``NoSuspiciousCharacters::RESTRICTION_LEVEL_HIGH`` (since ICU 58) also requires the string to be `covered`_ by any of the following sets of scripts:
+* ``NoSuspiciousCharacters::RESTRICTION_LEVEL_HIGH`` (usable with ICU 58 or higher) also requires the string to be `covered`_ by any of the following sets of scripts:
 
   * Latin + Han + Bopomofo (or equivalently: Latn + Hanb)
   * Latin + Han + Hiragana + Katakana (or equivalently: Latn + Jpan)
   * Latin + Han + Hangul (or equivalently: Latn + Kore)
 * ``NoSuspiciousCharacters::RESTRICTION_LEVEL_SINGLE_SCRIPT`` also requires the string to be `single-script`_.
-* ``NoSuspiciousCharacters::RESTRICTION_LEVEL_ASCII`` (since ICU 58) also requires the string's characters to be in the ASCII range.
+* ``NoSuspiciousCharacters::RESTRICTION_LEVEL_ASCII`` (usable with ICU 58 or higher) also requires the string's characters to be in the ASCII range.
 
 You can accept all characters by setting this option to
 ``NoSuspiciousCharacters::RESTRICTION_LEVEL_NONE``.