- Issue created by @mglaman
- Merge request !891ApiLayoutController::get should not default model to stdClass β (Merged) created by mglaman
- πΊπΈUnited States mglaman WI, USA
π Once previewed in XB an entity with no changes will still show up in "Review x changes" Active introduced this method and possibly missed in the refactor
- π¦πΊAustralia larowlan π¦πΊπ.au GMT+10
This is needed to ensure an empty model is json encoded as
{}
and not[]
I think the calling code should cast it to an array. - π¦πΊAustralia larowlan π¦πΊπ.au GMT+10
We do this on line 366 but missed line 344, I'll add a test and a fix.
- πΊπΈUnited States mglaman WI, USA
which is unique in that there are enabled global regions but they're all empty.
Seems like it'd be a common place for anyone starting fresh with XB and just turned on their global regions. Great catch
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
Seems like it'd be a common place for anyone starting fresh with XB and just turned on their global regions. Great catch
Only when 0
Block
config entities existed! SeePageRegion::createFromBlockLayout()
+experience_builder_form_system_theme_settings_submit()
- π§πͺBelgium wim leers Ghent π§πͺπͺπΊ
Nothing to do here: one-line change, expanded test coverage by @larowlan β π’
-
wim leers β
committed 974fa24c on 0.x authored by
mglaman β
Issue #3518834 by mglaman, larowlan: ApiLayoutController::get() should...
-
wim leers β
committed 974fa24c on 0.x authored by
mglaman β
- πΊπΈUnited States mglaman WI, USA
Only when 0 Block config entities existed!
π par for the course for me to hit that oddball edge case.