diff --git a/core/modules/file/file.module b/core/modules/file/file.module index 7c862ed005d51f9bdfac4817b5776da93d13bf75..4f251e03471b1077c057427605f932cfdcec67f8 100644 --- a/core/modules/file/file.module +++ b/core/modules/file/file.module @@ -289,6 +289,8 @@ function file_validate_image_resolution(FileInterface $file, $maximum_dimensions if ($image->scale($width, $height)) { $scaling = TRUE; $image->save(); + // Update the file size now that the image has been resized. + $file->setSize($image->getFileSize()); if (!empty($width) && !empty($height)) { $message = t('The image was resized to fit within the maximum allowed dimensions of %dimensions pixels. The new dimensions of the resized image are %new_widthx%new_height pixels.', [ diff --git a/core/modules/file/tests/src/Kernel/LegacyValidatorTest.php b/core/modules/file/tests/src/Kernel/LegacyValidatorTest.php index 357dcb357eb4e40de02ffdf3aa9799e2f48f353d..20c27fe2d8b08a4df6d47bb6121fedd140c884ec 100644 --- a/core/modules/file/tests/src/Kernel/LegacyValidatorTest.php +++ b/core/modules/file/tests/src/Kernel/LegacyValidatorTest.php @@ -37,6 +37,7 @@ protected function setUp(): void { /** @var \Drupal\Core\File\FileSystemInterface $file_system */ $file_system = \Drupal::service('file_system'); $this->image->setFilename($file_system->basename($this->image->getFileUri())); + $this->image->setSize(@filesize($this->image->getFileUri())); $this->nonImage = File::create(); $this->nonImage->setFileUri('core/assets/vendor/jquery/jquery.min.js'); @@ -191,6 +192,8 @@ public function testFileValidateImageResolution() { // Verify that the image was scaled to the correct width and height. $this->assertLessThanOrEqual(10, $image->getWidth()); $this->assertLessThanOrEqual(5, $image->getHeight()); + // Verify that the file size has been updated after resizing. + $this->assertEquals($this->image->getSize(), $image->getFileSize()); // Once again, now with negative width and height to force an error. copy('core/misc/druplicon.png', 'temporary://druplicon.png');