🇧🇪Belgium @victonator

I've created a patch to fix this.

victonator → created an issue.

I had the same error on a fresh install.
I added the button to a text format without configuring the options below. Configuring the options below got rid of the error and added the button to the text format :).

Hi @mauro_
In AjaxWrapperController.php:183 a new request is created and later on gets pushed in the requestStack.
FormBuilder.php requires a session from the currentRequest which is now the new request without a session from ajax_wrapper.
Adding the session from the mainRequest to the currentRequest (or something like that) is not something I should have to do when building my form in my opinion :) .
This should be handled in the module in some way, like in the attached patch.

Not sure if correct way to fix, but this patch works for me

victonator → created an issue.

Victonator → created an issue.

I deleted the patch from comment #20 from a project and the feature it's attached to is working as intended.
In my case this is not an issue anymore.

Here's a patch

Victonator → created an issue.

#2 works!
We upped our PHP memory limit because of this.
The patch also made our backend loading times shorter.

I had this problem as well and yoast_seo_update_8203 should've taken care of that and created a settings file.
Do you have a yoast_seo.settings.yml file with the contents below?

score_rules:
  0: 'Not available'
  8: Good
  1: Bad
  5: Okay

Hi @arisen,
Sorry for all the back and forth, this is my first time contributing like this :)
I've pushed it to the right branch and made a new MR, I hope this is correct.

Hi @arisen, sorry for my lack of steps, I added some new steps and I was able to reproduce this on a new project.

Victonator → created an issue.

As described in #157 I came across the same issue where we had to press the 'Save' button twice.
Patch #157 fixes this and works great for me.

Nevermind my previous comment #6, Patch #4 works like intended

Patch #4 works, but when a lot of checkboxes are present, a scrollable element is needed (Like before) else the filter will be really long.

When commenting code in the views module in file core/modules/views/src/Plugin/views/filter/FilterPluginBase.php (Line 957-962) It works as expected.

      if ((!empty($this->options['expose']['use_operator']) && !empty($this->options['expose']['operator_id'])) || count(Element::children($form[$value]))) {
        $wrapper = $value . '_wrapper';
        $this->buildValueWrapper($form, $wrapper);
        $form[$wrapper][$value] = $form[$value];
        unset($form[$value]);
      }

Victonator → created an issue.

This MR (#6) works on a fresh Drupal 10 site

Can we get a tagged release with Drupal 10 support?

+1
Merge request #21 works!

I tested the Merge request in comment #12 and on a new Drupal 10 project it works great!

On a fresh Drupal 10 project, patch #2 works great!

No issue's on a fresh Drupal 10 project, works perfectly

Patch #2 works for Drupal 10

Victonator → created an issue.

Patch #2 works for Drupal 10