Problem/Motivation

A Drupal build I inherited has been using a patch from #2913598: Accessibility enhancements for html output of iframe tag (title, id, name, inner A link) for years. Specifically https://www.drupal.org/files/issues/2019-09-19/video_embed_field-title_a...

With the release of 8.x-2.6, the patch can longer be applied. I removed the patch from Composer and the build successed, but now the install profile fails with...

Error: Class "Drupal\Core\Render\Element\RenderElementBase" not found in /var/www/html/web/modules/contrib/video_embed_field/src/Element/VideoEmbedIFrame.php on line 13 #0 /var/www/html/vendor/composer/ClassLoader.php(576): include()

I tried installing just this module on D9 with https://simplytest.me/, but that no longer works because of the Composer 2.9 security-audit requirements.

I haven't dug into this too far yet, but based on other issues related to RenderElementBase my guess is that 8.x-2.6 wasn't actually tested on D9. I know the solution is to upgrade to a supported version of Drupal and we are working on that, but if this is no longer going to support D9 please update the info.yml to reflect that.

For now, I'm just going to continue using the patch with the previous release.

Issue fork video_embed_field-3562351

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

kreynen created an issue. See original summary.

i-trokhanenko made their first commit to this issue’s fork.

i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
commented

I have the same error using Video Embed Field 8.x-2.6 and Drupal core 9.5.11

i-trokhanenko opened merge request !100

i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
commented
Status: Active » Needs review
mably’s picture

mably commented

Looks like 2.6 is not compatible with D9 anymore. Merging.

  • mably committed 01ec6a16 on 8.x-2.x authored by i-trokhanenko 
    Issue #3562351 by kreynen, i-trokhanenko, mably: Does 8.x-2.6 really...
mably’s picture

mably commented
Status: Needs review » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

mably’s picture

mably commented

Looks like it even requires Drupal 10.3+. Will push an update.

mably opened merge request !101

mably’s picture

mably commented
Related issues: +#3483115: Fix phpstan warnings

Looks like the change was introduced in issue #3483115: Fix phpstan warnings.

mably’s picture

mably commented

Looks like we could easily rollback the change. Will give it a try.

I do not have any Drupal 9 instance easily available, so I'll need help from the community to validate that everything is working fine again on Drupal 9 after I push the rollback.

The problem is that Gitlab CI tests do not cover EOL versions as far as I know, so there don't seem to be any easy way to verify that updates do not break compatibility with Drupal 9.

Feel free to chime in if you have any solution to this problem to share.

mably opened merge request !102

mably’s picture

mably commented
Status: Fixed » Needs review
mably’s picture

mably commented

Can someone confirm that MR 102 restores compatibility with Drupal 9.2+ please?

Ping @kreynen @i-trokhanenko

beretor’s picture

beretor commented

Hello,
I have this Issue:
Error: Class 'Drupal\Core\Render\Element\RenderElementBase' not found in include() (line 13 of modules/contrib/video_embed_field/src/Element/VideoEmbedIFrame.php).

drupal/core 9.4.4
drupal/video_embed_field 2.6.0

Context: We updated drupal/video_embed_field from 2.4 to 2.6 to get the YouTube error 153 fix.

I last tried at: 16.12.2025 11am.

I tried Merge request!102 on my local Drupal, and it worked like a charm.

No more errors.
Thank you.

If you require further information, please feel free to ask.

mably’s picture

mably commented

Hi @beretor, thanks for your feedback.

Can you please switch this issue to RTBC so I can merge it? Thanks.

beretor’s picture

beretor commented
Status: Needs review » Reviewed & tested by the community

Tested with

drupal/core 9.4.4
drupal/video_embed_field 2.6.0

beretor’s picture

beretor commented

@mably done.

beretor’s picture

beretor commented

@mably ok, done

mably’s picture

mably commented
Status: Reviewed & tested by the community » Fixed

It's merged.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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