- Issue created by @bnjmnm
- πΊπΈUnited States bnjmnm Ann Arbor, MI
bnjmnm β changed the visibility of the branch 3481719-confirm-semi-coupled-form to hidden.
- πΊπΈUnited States bnjmnm Ann Arbor, MI
The Radix version on 0.x
The native version in this MR
- ππΊHungary GΓ‘bor Hojtsy Hungary
Is this at a state where it can be tried with Pathauto and other entity form extending modules? :) Or can you signal when it is? :)
- πΊπΈUnited States bnjmnm Ann Arbor, MI
@gΓ‘bor hojtsy the scope of this issue includes getting the pathauto widget functional. With this MR. The form elements in that widget work as expected and send the correct data for submission. The autosave data is aware of pathauto being enabled/disabled and the value of the custom alias.
However, the publish process is not processing these values correctly. This.is out of scope for this widget-targeted issue. Even if it's out of scope, I figured it can't hurt to offer an additional draft MR 3481719-states-api-with-targeted-pathauto that takes the in-scope MR and adds a few lines to
ApiAutoSaveController
to get pathauto workingif (get_class($entity->path) === 'Drupal\pathauto\PathautoFieldItemList' && $auto_save['data']['entity_form_fields']['path[0][pathauto]'] === FALSE) { $entity->path->__set('pathauto', 0); }
I believe this is necessary because the publish process updates fields, but
path
is being updated as an entity property.
I suspect there are more holistic solutions available so the 3481719-states-api-with-targeted-pathauto branch should probably not be part of this issue's commit, so I created π Autosave publish process does not acknowledge pathauto deactivation Postponed , which can use the solution I already found as a starting point. - πΊπΈUnited States hooroomoo
Looks good to me! but still needs a /ui/src/components/form/inputBehaviors.tsx approval
-
bnjmnm β
committed fb5bb437 on 0.x
Issue #3481719: Get Semi-coupled Checkbox form elements to work with the...
-
bnjmnm β
committed fb5bb437 on 0.x
- π³π±Netherlands balintbrews Amsterdam, NL
bnjmnm β credited balintbrews β .
- πΊπΈUnited States bnjmnm Ann Arbor, MI
Thanks for the reviews @hooroomoo & @balintbrews