Hey John, yes I saw that the patch was accepted. Thanks for that. I overlooked that function was accessible, so that should handle other cases. Thanks again.
I have seen it in both cases, additionally this is just a part of a bigger issue I think with how the form is working.
in setHostEntity(FormStateInterface $form_state), we are first checking if $form_state already has a host entity, if not we fall back to using routeMatch paramaters, which will fail in this situation: $this->getRouteMatch()->getParameters().
I think the more/better solution would be to add a third parameter onto the form method, that takes a \Drupal\Core\Entity\ContentEntityInterface object. Then in the Field Formatter, pass in the host entity when calling the form to be rendered. This would ensure that the host entity is always present.
Attaching patch for now.
cmcintosh → created an issue.
Will need to test and see i have not used the module in a good minute.
Attaching patch for this issue.
cmcintosh → created an issue.
I created several repos for the various glaze modules and themes, and updated it to support Modules/themes.
- https://github.com/WembassyCo/glazed_builder
- https://github.com/WembassyCo/glazed
I moved the helper modules all into the glazed_builder module to make it simpler to install, and then the theme itself uses bootstrap as the base theme. I have used it on around 4-5 sites, including my own https://www.wembassy.com and it is working well with minimal issues.
cmcintosh → created an issue.
I know its been a minute but i have been working on upgrading the component elements for the glaze builder into Drupal 9, most of them are being hosted as forks on associated repos on github. Not sure if there is still a interest in it or not.
hotwebmatter → credited cmcintosh → .