- Issue created by @pdureau
- Merge request !260#3483496 - Made autoselect for a single option in the select. Simplified the... β (Merged) created by SmovS
- πΊπ¦Ukraine SmovS Lutsk
Hi guys!
I updated the code to simplify the wording in the "Referenced entities" source.I made autoselect for a single option in selectors. Unfortunately, I stuck with a bug when we changed the Source from "Referenced entities" to "Data from a field" and back.
This error already exists in the current beta4 as well. I debugged the code a bit and noticed weird behavior with$this->settings
When we choose another Source,
$derivableContexts
contains actual data but$this->settings
includes the old one chosen. I tried to find a solution but failed.I added screenshots with the error and debug backtrace.
Any suggestions on how to resolve that? - π«π·France just_like_good_vibes PARIS
Hello, just found the time to look a little bit.
thank you very much for your detailed info.
we will create a separate issue for that."Referenced entities" and "Data from a field" use the same parent source class, and are storing "derivable_context" data at the same place.
When we switch from one source to another, the old data is kept and the selected derivable_context of the old source is injected into the default value of the derivable_context for the new source.we can fix that, independently from the current issue :)
i just created π [2.0.0-beta5] Sources bad on derivable contexts should validate the default option Active , i will push a fix soon. - πΊπ¦Ukraine SmovS Lutsk
Hi guys!
I made autoselect for a single option in selectors and updated the code to simplify the wording in the "Referenced entities" source.
Please review the MR !260
- π«π·France just_like_good_vibes PARIS
Hello,
i just rebased your MR and resolved the merge/rebase conflict.
i will do the review soon.
thank you :) -
just_like_good_vibes β
committed 71517119 on 2.0.x authored by
smovs β
Issue #3483496 by smovs, just_like_good_vibes, pdureau: Preselect option...
-
just_like_good_vibes β
committed 71517119 on 2.0.x authored by
smovs β
- π«π·France just_like_good_vibes PARIS
hello,
i had to change some little things in you MR.
and i merged it :)