-
-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
/admin/content causes fatal error with Retrofit #176
Comments
I was getting a similar error in the Appearance tab. Fixed this by adding another case to ArrayAccessFormState. public function offsetSet(mixed $offset, mixed $value): void
{
switch ($offset) {
...
case 'build_info':
$this->setBuildInfo($value);
break; |
Thanks for reporting this @mariosm1 !! we can get that added |
I'm surprised the existing methods didn't work, since the property name is the same internally still https://git.drupalcode.org/project/drupal/-/blob/11.x/core/lib/Drupal/Core/Form/FormState.php?ref_type=heads#L987 |
It looks like FormState::set() adds the build_info to the $form_state->storage array, but FormBuilder::retrieveForm() grabs the build_info directly from $form_state->build_info and it was coming up as an empty array |
Thanks @mariosm1, now that I'm back at a computer I will add that. |
Merged and cut a release, thanks for digging in @mariosm1 |
@mglaman No problem! |
Running Drupal 10.3.10 with retrofit 0.1.14 and seeing pretty much the same error at
Previously I had retrofit 0.1.10 working well with Drupal 10.2.5. I tried downgrading from 0.1.14 to 0.1.10 on Drupal 10.3.10 and that at least cleared the above error |
Came here from your video, and after implementation i get the same error on
As @emasters suggested, downgrading to 0.1.10 got it up and running again |
This was reported via Slack.
The text was updated successfully, but these errors were encountered: