Problem/Motivation

We still need to work a little bit on normalization, see related issues.

errors are like : 

Drupal\Core\Render\Component\Exception\InvalidComponentException: [items[1].title] Object value found, but a string is required in Drupal\Core\Theme\Component\ComponentValidator->validateProps() (line 203 of core/lib/Drupal/Core/Theme/Component/ComponentValidator.php).

Issue fork ui_patterns-3492211

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

just_like_good_vibes created an issue. See original summary.

grimreaper’s picture

grimreaper
Primary language French
Location France 🇫🇷
commented

original report in #3491238-18: [2.0.0-beta6] normalization of slots triggers some unwanted filtering

pdureau’s picture

pdureau commented
Title: LinkPropType normalization...suite » [2.0.0-beta6] LinkPropType normalization...suite
just_like_good_vibes’s picture

just_like_good_vibes
Location PARIS
commented
Title: [2.0.0-beta6] LinkPropType normalization...suite » [2.0.0-beta6] Normalization...suite

just_like_good_vibes opened merge request !289

just_like_good_vibes’s picture

just_like_good_vibes
Location PARIS
commented
Assigned: just_like_good_vibes » Unassigned
Status: Active » Needs review

new additions with feedback from multiple persons :) thank you all

just_like_good_vibes credited g4mbini.

just_like_good_vibes’s picture

just_like_good_vibes
Location PARIS
commented
just_like_good_vibes’s picture

just_like_good_vibes
Location PARIS
commented

And finally, i have corrected and give more strength to our normalization processes.

My advice, is that we could be more systematic to this approach, and factorize code a lot to do that process of "cleaning" what is injected into the components.
but, well, let's continue for now with that version, which seems to cope well with all our the submitted problematic usecases (i hope i did not forget any).

just_like_good_vibes’s picture

just_like_good_vibes
Location PARIS
commented
Status: Needs review » Fixed
steveoriol’s picture

steveoriol
Location Grenoble 🇫🇷
commented

It works :-)
Thanks @just_like_good_vibes

just_like_good_vibes’s picture

just_like_good_vibes
Location PARIS
commented

thanks for the feedback :)

grimreaper’s picture

grimreaper
Primary language French
Location France 🇫🇷
commented

Hi,

Thanks, all my test cases are ok now \o/

just_like_good_vibes’s picture

just_like_good_vibes
Location PARIS
commented

great!

pdureau’s picture

pdureau commented
Status: Fixed » Closed (fixed)