@osopolar i did some (x)debugging and found the reason. We applied a patch which solved a validation error in JSON:API if an image name has a space in the filename. (see uri-reference in vendor/justinrainbow/json-schema/src/JsonSchema/Constraints/FormatConstraint.php
)This patch urlencoded the $uri which caused the StreamWrapperManager::getScheme($uri)
to return false
as mentioned. I removed the patch and it is working as expected. The error which occured when a filename contains a space does not appear anymoree either.
So you were right, it was another issue in my case.
Yesterday i updated the module on an installation where we are also using S3 Module β :
drupal/consumer_image_styles (4.0.8 => 4.0.10)
After the upgrade i get
Error: Call to a member function getType() on false in Drupal\consumer_image_styles\ImageStylesProvider->buildDerivativeLink() (line 128 of /var/www/html/htdocs/modules/contrib/consumer_image_styles/src/ImageStylesProvider.php
i tried the patch from MR #27 , but still got the error.
Problem seems that
$this->streamWrapperManager->getViaScheme(StreamWrapperManager::getScheme($uri)
returns false
now.
Had the same issue and MR21 is working fine for me. As 'UA-' properties are deprected now, the format should not be supported anymore. The MR only supports 'G-' format, which is correct in my opinion.
But the approach to handle both Formats which was mentioned in the Issue description seems outdated now.
Would be happy if MR21 gets merged!
makkus183 β made their first commit to this issueβs fork.
Can reproduce the mentioned SyntaxError in version 6.0.2
The Update is also necessary for Drupal 9.x. Also not sure how to handle it.
"Solved" it as mentioned in #4 for now..