Problem/Motivation

TypeError: Drupal\image_effects\Plugin\ImageEffect\AspectSwitcherImageEffect::getChildImageStyleToExecute(): Argument #1 ($width) must be of type int, string given, called in /var/www/html/web/modules/contrib/image_effects/src/Plugin/ImageEffect/AspectSwitcherImageEffect.php on line 172 in Drupal\image_effects\Plugin\ImageEffect\AspectSwitcherImageEffect->getChildImageStyleToExecute() (line 196 of modules/contrib/image_effects/src/Plugin/ImageEffect/AspectSwitcherImageEffect.php).

WSOD.
testing in drupal 11 (php 8.3)
seems that type needs to be enforced more.

#3357133 --> ?? possible, does this related to similar type problem. I'm not sure at the moment if this branch is up2date.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork image_effects-3470760

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

skaught created an issue. See original summary.

skaught’s picture

skaught
Primary language English
commented
Issue summary: View changes
skaught’s picture

skaught
Primary language English
commented
Related issues: +#3357133: DivisionByZeroError on AspectSwitcherImageEffect
skaught’s picture

skaught
Primary language English
commented
Version: 4.0.x-dev » 4.0.0-beta1

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

mondrake opened merge request !57

mondrake’s picture

mondrake
Location 🇮🇹
commented
Status: Active » Needs review

I think the type of the values in the dimensions array should be better enforced upstream by core. Let's see if I can file an issue there.

  • mondrake committed abf9272c on 4.0.x 
    Issue #3470760 by mondrake, skaught: AspectSwitcherImageEffect. {width}...
mondrake’s picture

mondrake
Location 🇮🇹
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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