[2.0.0-beta1] Component validator: false positive errors

Also:

• Add trans filter to the whitelist
• Add keys filter to the whitelist
• Add striptags filter to the whitelist
• Move include function from error to warning
• Maybe it is too soon to raise "Return true for mapping and sequence" warning. Twig 3.11 is not release yet and iterable test is still the only choice

Wording changes:

• "Careful with null test." when raised by bar is null >> "Not needed in Drupal because strict_variables=false.
• "Careful with null test." when raised by bar ?? foo >> "Use `|default(foo)` filter instead of null ternary `??`" ?
• "Required props are not recommended." >> "Required props are not recommended. Use default values instead"
• "PHP URL object, or useless if URL string." >> no proposal yet
• "Manipulate objects is not compatible with Design system." >> "PHP object manipulation must be avoided"
• "Filter `default` is not for booleans, your prop is a boolean!" >> "Don't use `default`filter on boolean"

New tests proposals:

• Can we raise a warning when foo|default(false) ? We already check if foo is a boolean, but do we check if the filter value is a boolean? Is it relevant?
• Can we detect when expression is the same in the input and value of default filter: foo|default(foo). Yes, I haev seen that in contrib projects.

Kept in the grey list for beta1:

• clean_class filter
• cycle function
• spaceless filter
• safe_join filter
• nl2br filter

Another false positive found:

[props.properties.links.description] NULL value found, but a string is required

Every components have this error, even if no links prop.

Anyway, description property is not mandatory in props defrinitions.

It seems to be an error from ui_patterns_legacy, for example on ui_suite_dsfr:translate the description of the props 'links' is empty, when transformed by ui_patterns_legacy it become: description: null, and so the Validator raise this schema error.

interesting, thanks. I will create a beta1 issue about this

I couldn't find why so for now I removed to proxy to componentValidator until I can do it properly.

that was the right thing to do

Since the last update, I have this error

An exception has been thrown during the rendering of a template ("Drupal\ui_patterns_legacy\RenderableConverter::getNamespacedId(): Argument #1 ($component_id) must be of type string, null given, called in modules/ui_patterns_legacy/src/RenderableConverter.php on line 81").

On those components, everytim it is because of the pattern() function:

• ui_suite_dsfr:consent_banner
• ui_suite_dsfr:consent_manager because of:
  

    {{ pattern('button_group', {
      attributes: {class: ['fr-consent-manager__buttons', 'fr-btns-group--inline-sm']},
      buttons: [pattern('button', {label: 'Confirm my choices'|t})]
    }, 'right') }}

• ui_suite_bootstrap:dropdown
• ui_suite_uswds:modal
• ui_suite_uswds:tooltip because of:
  

  {{ pattern('button', {
    'attributes': attributes,
    'label': label|default('Show')
  }) }}

But not every component template with pattern() function raise that. Mysterious...

OK, i check ui_patterns_legacy again