🇮🇳India @Jaykumar95

Ahmedabad
Account created on 29 April 2019, over 5 years ago
#

Recent comments

🇮🇳India Jaykumar95 Ahmedabad

Thank you for the patch. It is fixed in new version.

🇮🇳India Jaykumar95 Ahmedabad

I have removed the Polyfill dependency in newer version.

🇮🇳India Jaykumar95 Ahmedabad

Thank you @priyanka277 for pointing out.

I have dig deeper and found that in core\modules\system\src\Plugin\ImageToolkit\GDToolkit.php file we are using getimagesize() function which is creating this issue.
Please find attached patch which fixes the issue using below reference link.

PHP getimagesize() mixes up width and height

Image Orientation has Strange Behavior

🇮🇳India Jaykumar95 Ahmedabad

I can confirm that this is coming in Drupal 10.2.4 and PHP 8.3.

My finding is when we manually change the image orientation from landscape to portrait or portrait to landscape then only this issue is occurs.
Say I have an original image of size 500x1000 and manually changes it to 1000x500 using Rotate right or Rotate left feature of Windows system, Drupal is saving it's width as 500 and height as 1000 as original image size, but when I rotate that same image using the MS Paint the image width and height saved correctly in Drupal.

🇮🇳India Jaykumar95 Ahmedabad

attached patch #5 while follow up on #3 failed in test.

🇮🇳India Jaykumar95 Ahmedabad

As per previous code for gatsby_entity_insert hook we add preview log first and then build log but in #10 patch it's reversed.
which will cause test case failure in \tests\src\Kernel\GatsbyEntityLoggerTest.php as the conditions are changed.

🇮🇳India Jaykumar95 Ahmedabad

When trying to check fastbuild logs explicit accessCheck exception thrown.
attached patch will resolve the issue.

Production build 0.71.5 2024