- Issue created by @pdureau
- 🇫🇷France pdureau Paris
Assigned to @just_like_good_vibes to check if the summary is OK
- 🇫🇷France just_like_good_vibes PARIS
pdureau → credited just_like_good_vibes → .
- 🇫🇷France pdureau Paris
According to @just_like_good_vibes it may also improve performance.
- 🇫🇷France mogtofu33
I am doing work of refactoring here #3545303, see my MR.
I created a service for the default provider instead of the unmanageable trait.
We need to remove any DI or extensive logic from `defaultConfiguration()`, It's not meant to have extensive or complex stuff. That's why I removed it.
So the inverted is what I was thinking about too, and I set the sensitive default values only once on the creation of a profile.
My MR is only missing the pass on the one created from config import, but the inverted logic could simplify this part.At the end this task should be done after #3545303 if not inside instead of separately.
- 🇫🇷France pdureau Paris
So the inverted is what I was thinking about too, and I set the sensitive default values only once on the creation of a profile.
That's great you are both agreeing on this.
- 🇫🇷France mogtofu33
So I am mostly doing it in https://www.drupal.org/project/display_builder/issues/3545303 📌 Island lighter base class Active , is it ok @just_like_good_vibes?