'#multiple__no_items_message' added to every field

This regression is caused by Drupal core's CKEditor wrapping the string in P tags. Since we want to fully support CKEditor via Drupal 6.2.x, we need to support the P tags.

Works well. The patch applies and the update correctly removed all the extra properties.

When I run this update I get Error: Class "WebformYaml" not found in webform_post_update_ckeditor01() (line 113 of /app/web/modules/contrib/webform/webform.post_update.php). Does this need to use Drupal\webform\Utility\WebformYaml; or maybe I'm missing something since the patch applied for others?

Do you have the same problem if you clear the cache before running the updates?

Yup, same issue even after cache clear (I clear caches before updb out of habit xD)

What version of webform are you using? I'm using 6.2.0-beta5, and my webform/webform.post_update.php file does not have a line 113, nor does it have a webform_post_update_ckeditor01() function.

It just has this, which doesn't have a reference to WebformYaml:

/**
 * Move from custom CKEditor to hidden 'webform_default' text format.
 */
function webform_post_update_ckeditor() {
  $config = \Drupal::configFactory()->getEditable('webform.settings');
  if (empty($config->get('html_editor.element_format'))) {
    $config->set('html_editor.element_format', WebformHtmlEditor::DEFAULT_FILTER_FORMAT);
  }
  if (empty($config->get('html_editor.mail_format'))) {
    $config->set('html_editor.mail_format', WebformHtmlEditor::DEFAULT_FILTER_FORMAT);
  }
  $config->save();

  _webform_update_html_editor();
}

beta5 is from January, this fix is in the latest 6.2 dev version

I don't have dev installed. I just have the patch via Composer. So it's possible your issue is actually related to another update, namely the webform_post_update_ckeditor01() function you mentioned. You should open a separate new issue.

The update function references this issue ID so I assumed it was introduced here:

/**
 * Issue #3351348: '#multiple__no_items_message' added to every field.
 */
function webform_post_update_ckeditor01() {

Ah okay. I'm using patch #2. The actual commit is slightly different.

I switched to using the commit patch. I had no issues running the DB update.

There isn't an equivalent of a manual registry rebuild. According to this post, you may need to restart php-fpm, then try running the updates.

Made a patch for #6 see 🐛 Error: Class "WebformYaml" not found in webform_post_update_ckeditor01() Fixed

Automatically closed - issue fixed for 2 weeks with no activity.