Problem/Motivation

PHP Fatal error: Type of Drupal\acquia_dam\Plugin\Field\FieldFormatter\AcquiaDamMediaThumbnailFormatter::$renderer must be Drupal\Core\Render\RendererInterface (as in class Drupal\media\Plugin\Field\FieldFormatter\MediaThumbnailFormatter) in /var/www/docroot/modules/contrib/acquia_dam/src/Plugin/Field/FieldFormatter/AcquiaDamMediaThumbnailFormatter.php on line 21

Fatal error: Type of Drupal\acquia_dam\Plugin\Field\FieldFormatter\AcquiaDamMediaThumbnailFormatter::$renderer must be Drupal\Core\Render\RendererInterface (as in class Drupal\media\Plugin\Field\FieldFormatter\MediaThumbnailFormatter) in /var/www/docroot/modules/contrib/acquia_dam/src/Plugin/Field/FieldFormatter/AcquiaDamMediaThumbnailFormatter.php on line 21

Steps to reproduce

With Drupal 11.4, the code is not compatible.

Proposed resolution

fix the definition.

CommentFileSizeAuthor
#2 3607942-acquia-dam-php-compatibility.patch624 bytesnehajyoti

Issue fork acquia_dam-3607942

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

nehajyoti created an issue. See original summary.

nehajyoti’s picture

nehajyoti commented
StatusFileSize
new 3607942-acquia-dam-php-compatibility.patch624 bytes

thoward216 made their first commit to this issue’s fork.

thoward216 opened merge request !304

thoward216’s picture

thoward216 commented

I just came across this issue also, I've created a branch & MR to move this forward.

thoward216’s picture

thoward216 commented
Issue summary: View changes
Status: Active » Needs review

rajeshreeputra made their first commit to this issue’s fork.

rajeshreeputra’s picture

rajeshreeputra
he/him
Location Pune
commented
Status: Needs review » Fixed

MR 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.

danflanagan8’s picture

danflanagan8
Primary language English
Location St. Louis, US
commented

Can we please get that into a new 1.1.x release or put out a 1.2.x release?

This is at least a major if not critical issue. Without this fix the acquia_dam module brings down a site running on D11.4.x.