@@ -273,7 +273,7 @@ discard block |
||
| 273 | 273 | $vars, |
| 274 | 274 | $minifier |
| 275 | 275 | ); |
| 276 | - $content = (string)$content; |
|
| 276 | + $content = (string) $content; |
|
| 277 | 277 | if ($scriptAttrs !== null) { |
| 278 | 278 | $attrs = array_merge([ |
| 279 | 279 | ], |
@@ -308,7 +308,7 @@ discard block |
||
| 308 | 308 | $vars, |
| 309 | 309 | $minifier |
| 310 | 310 | ); |
| 311 | - $content = (string)$content; |
|
| 311 | + $content = (string) $content; |
|
| 312 | 312 | if ($scriptAttrs !== null) { |
| 313 | 313 | $attrs = array_merge([ |
| 314 | 314 | ], |
@@ -372,7 +372,7 @@ discard block |
||
| 372 | 372 | . '%'; |
| 373 | 373 | Craft::info($message, __METHOD__); |
| 374 | 374 | if (Craft::$app instanceof ConsoleApplication) { |
| 375 | - echo $message . PHP_EOL; |
|
| 375 | + echo $message.PHP_EOL; |
|
| 376 | 376 | } |
| 377 | 377 | // Create any image variants |
| 378 | 378 | $this->createImageVariants( |
@@ -410,14 +410,14 @@ discard block |
||
| 410 | 410 | */ |
| 411 | 411 | public function saveTransformToTempFile(AssetTransformIndex $index, Image $image): string |
| 412 | 412 | { |
| 413 | - $tempFilename = uniqid(pathinfo($index->filename, PATHINFO_FILENAME), true) . '.' . $index->detectedFormat; |
|
| 414 | - $tempPath = Craft::$app->getPath()->getTempPath() . DIRECTORY_SEPARATOR . $tempFilename; |
|
| 413 | + $tempFilename = uniqid(pathinfo($index->filename, PATHINFO_FILENAME), true).'.'.$index->detectedFormat; |
|
| 414 | + $tempPath = Craft::$app->getPath()->getTempPath().DIRECTORY_SEPARATOR.$tempFilename; |
|
| 415 | 415 | try { |
| 416 | 416 | $image->saveAs($tempPath); |
| 417 | 417 | } catch (ImageException $e) { |
| 418 | - Craft::error('Transformed image save failed: ' . $e->getMessage(), __METHOD__); |
|
| 418 | + Craft::error('Transformed image save failed: '.$e->getMessage(), __METHOD__); |
|
| 419 | 419 | } |
| 420 | - Craft::info('Transformed image saved to: ' . $tempPath, __METHOD__); |
|
| 420 | + Craft::info('Transformed image saved to: '.$tempPath, __METHOD__); |
|
| 421 | 421 | |
| 422 | 422 | return $tempPath; |
| 423 | 423 | } |
@@ -529,7 +529,7 @@ discard block |
||
| 529 | 529 | . '%'; |
| 530 | 530 | Craft::info($message, __METHOD__); |
| 531 | 531 | if (Craft::$app instanceof ConsoleApplication) { |
| 532 | - echo $message . PHP_EOL; |
|
| 532 | + echo $message.PHP_EOL; |
|
| 533 | 533 | } |
| 534 | 534 | // Copy the image variant into place |
| 535 | 535 | $this->copyImageVariantToVolume( |
@@ -628,9 +628,9 @@ discard block |
||
| 628 | 628 | // Handle auto-sharpening scaled down images |
| 629 | 629 | if ($settings->autoSharpenScaledImages) { |
| 630 | 630 | // See if the image has been scaled >= 50% |
| 631 | - $widthScale = (int)(($image->getWidth() / $asset->getWidth()) * 100); |
|
| 632 | - $heightScale = (int)(($image->getHeight() / $asset->getHeight()) * 100); |
|
| 633 | - if (($widthScale >= (int)$settings->sharpenScaledImagePercentage) || ($heightScale >= (int)$settings->sharpenScaledImagePercentage)) { |
|
| 631 | + $widthScale = (int) (($image->getWidth() / $asset->getWidth()) * 100); |
|
| 632 | + $heightScale = (int) (($image->getHeight() / $asset->getHeight()) * 100); |
|
| 633 | + if (($widthScale >= (int) $settings->sharpenScaledImagePercentage) || ($heightScale >= (int) $settings->sharpenScaledImagePercentage)) { |
|
| 634 | 634 | $imagineImage->effects() |
| 635 | 635 | ->sharpen(); |
| 636 | 636 | Craft::debug( |
@@ -685,7 +685,7 @@ discard block |
||
| 685 | 685 | . escapeshellarg($tempPath); |
| 686 | 686 | // Execute the command |
| 687 | 687 | $shellOutput = $this->executeShellCommand($cmd); |
| 688 | - Craft::info($cmd . "\n" . $shellOutput, __METHOD__); |
|
| 688 | + Craft::info($cmd."\n".$shellOutput, __METHOD__); |
|
| 689 | 689 | } else { |
| 690 | 690 | Craft::error( |
| 691 | 691 | $thisProcessor['commandPath'] |
@@ -737,7 +737,7 @@ discard block |
||
| 737 | 737 | // Make sure the command exists |
| 738 | 738 | if (is_file($variantCreatorCommand['commandPath'])) { |
| 739 | 739 | // Get the output file for this image variant |
| 740 | - $outputPath .= '.' . $variantCreatorCommand['imageVariantExtension']; |
|
| 740 | + $outputPath .= '.'.$variantCreatorCommand['imageVariantExtension']; |
|
| 741 | 741 | // Set any options for the command |
| 742 | 742 | $commandOptions = ''; |
| 743 | 743 | if (!empty($variantCreatorCommand['commandOptions'])) { |
@@ -772,7 +772,7 @@ discard block |
||
| 772 | 772 | . escapeshellarg($tempPath); |
| 773 | 773 | // Execute the command |
| 774 | 774 | $shellOutput = $this->executeShellCommand($cmd); |
| 775 | - Craft::info($cmd . "\n" . $shellOutput, __METHOD__); |
|
| 775 | + Craft::info($cmd."\n".$shellOutput, __METHOD__); |
|
| 776 | 776 | } else { |
| 777 | 777 | Craft::error( |
| 778 | 778 | $variantCreatorCommand['commandPath'] |
@@ -810,23 +810,23 @@ discard block |
||
| 810 | 810 | } catch (InvalidConfigException $e) { |
| 811 | 811 | $volume = null; |
| 812 | 812 | Craft::error( |
| 813 | - 'Asset volume error: ' . $e->getMessage(), |
|
| 813 | + 'Asset volume error: '.$e->getMessage(), |
|
| 814 | 814 | __METHOD__ |
| 815 | 815 | ); |
| 816 | 816 | } |
| 817 | 817 | try { |
| 818 | - $variantPath = $asset->getFolder()->path . $assetTransforms->getTransformSubpath( |
|
| 818 | + $variantPath = $asset->getFolder()->path.$assetTransforms->getTransformSubpath( |
|
| 819 | 819 | $asset, |
| 820 | 820 | $transformIndex |
| 821 | 821 | ); |
| 822 | 822 | } catch (InvalidConfigException $e) { |
| 823 | 823 | $variantPath = ''; |
| 824 | 824 | Craft::error( |
| 825 | - 'Asset folder does not exist: ' . $e->getMessage(), |
|
| 825 | + 'Asset folder does not exist: '.$e->getMessage(), |
|
| 826 | 826 | __METHOD__ |
| 827 | 827 | ); |
| 828 | 828 | } |
| 829 | - $variantPath .= '.' . $variantCreatorCommand['imageVariantExtension']; |
|
| 829 | + $variantPath .= '.'.$variantCreatorCommand['imageVariantExtension']; |
|
| 830 | 830 | // Delete the variant file in case it is stale |
| 831 | 831 | try { |
| 832 | 832 | $volume->deleteFile($variantPath); |
@@ -834,7 +834,7 @@ discard block |
||
| 834 | 834 | // We're fine with that. |
| 835 | 835 | } |
| 836 | 836 | Craft::info( |
| 837 | - 'Deleted variant: ' . $variantPath, |
|
| 837 | + 'Deleted variant: '.$variantPath, |
|
| 838 | 838 | __METHOD__ |
| 839 | 839 | ); |
| 840 | 840 | } |
@@ -864,21 +864,21 @@ discard block |
||
| 864 | 864 | } catch (InvalidConfigException $e) { |
| 865 | 865 | $volume = null; |
| 866 | 866 | Craft::error( |
| 867 | - 'Asset volume error: ' . $e->getMessage(), |
|
| 867 | + 'Asset volume error: '.$e->getMessage(), |
|
| 868 | 868 | __METHOD__ |
| 869 | 869 | ); |
| 870 | 870 | } |
| 871 | 871 | $assetTransforms = Craft::$app->getAssetTransforms(); |
| 872 | 872 | try { |
| 873 | - $transformPath = $asset->getFolder()->path . $assetTransforms->getTransformSubpath($asset, $index); |
|
| 873 | + $transformPath = $asset->getFolder()->path.$assetTransforms->getTransformSubpath($asset, $index); |
|
| 874 | 874 | } catch (InvalidConfigException $e) { |
| 875 | 875 | $transformPath = ''; |
| 876 | 876 | Craft::error( |
| 877 | - 'Error getting asset folder: ' . $e->getMessage(), |
|
| 877 | + 'Error getting asset folder: '.$e->getMessage(), |
|
| 878 | 878 | __METHOD__ |
| 879 | 879 | ); |
| 880 | 880 | } |
| 881 | - $variantPath = $transformPath . '.' . $variantCreatorCommand['imageVariantExtension']; |
|
| 881 | + $variantPath = $transformPath.'.'.$variantCreatorCommand['imageVariantExtension']; |
|
| 882 | 882 | |
| 883 | 883 | // Delete the variant file in case it is stale |
| 884 | 884 | try { |
@@ -888,7 +888,7 @@ discard block |
||
| 888 | 888 | } |
| 889 | 889 | |
| 890 | 890 | Craft::info( |
| 891 | - 'Variant output path: ' . $outputPath . ' - Variant path: ' . $variantPath, |
|
| 891 | + 'Variant output path: '.$outputPath.' - Variant path: '.$variantPath, |
|
| 892 | 892 | __METHOD__ |
| 893 | 893 | ); |
| 894 | 894 | |
@@ -926,9 +926,9 @@ discard block |
||
| 926 | 926 | protected function swapPathExtension(string $path, string $extension): string |
| 927 | 927 | { |
| 928 | 928 | $pathParts = pathinfo($path); |
| 929 | - $newPath = $pathParts['filename'] . '.' . $extension; |
|
| 929 | + $newPath = $pathParts['filename'].'.'.$extension; |
|
| 930 | 930 | if (!empty($pathParts['dirname']) && $pathParts['dirname'] !== '.') { |
| 931 | - $newPath = $pathParts['dirname'] . DIRECTORY_SEPARATOR . $newPath; |
|
| 931 | + $newPath = $pathParts['dirname'].DIRECTORY_SEPARATOR.$newPath; |
|
| 932 | 932 | $newPath = preg_replace('#/+#', '/', $newPath); |
| 933 | 933 | } |
| 934 | 934 | |
@@ -326,14 +326,14 @@ discard block |
||
| 326 | 326 | $sizesWrapperId = Craft::$app->getView()->namespaceInputId('sizes-wrapper'); |
| 327 | 327 | $view = Craft::$app->getView(); |
| 328 | 328 | $view->registerJs( |
| 329 | - 'document.addEventListener("vite-script-loaded", function (e) {' . |
|
| 330 | - 'if (e.detail.path === "../src/web/assets/src/js/OptimizedImagesField.js") {' . |
|
| 331 | - 'new Craft.OptimizedImagesInput(' . |
|
| 332 | - '"' . $namespacedId . '", ' . |
|
| 333 | - '"' . $namespacePrefix . '",' . |
|
| 334 | - '"' . $sizesWrapperId . '"' . |
|
| 335 | - ');' . |
|
| 336 | - '}' . |
|
| 329 | + 'document.addEventListener("vite-script-loaded", function (e) {'. |
|
| 330 | + 'if (e.detail.path === "../src/web/assets/src/js/OptimizedImagesField.js") {'. |
|
| 331 | + 'new Craft.OptimizedImagesInput('. |
|
| 332 | + '"'.$namespacedId.'", '. |
|
| 333 | + '"'.$namespacePrefix.'",'. |
|
| 334 | + '"'.$sizesWrapperId.'"'. |
|
| 335 | + ');'. |
|
| 336 | + '}'. |
|
| 337 | 337 | '});' |
| 338 | 338 | ); |
| 339 | 339 | |
@@ -407,12 +407,12 @@ discard block |
||
| 407 | 407 | $jsonVars = Json::encode($jsonVars); |
| 408 | 408 | $view = Craft::$app->getView(); |
| 409 | 409 | $view->registerJs( |
| 410 | - 'document.addEventListener("vite-script-loaded", function (e) {' . |
|
| 411 | - 'if (e.detail.path === "../src/web/assets/src/js/OptimizedImagesField.js") {' . |
|
| 412 | - "$('#{$nameSpaceId}-field').ImageOptimizeOptimizedImages(" . |
|
| 413 | - $jsonVars . |
|
| 414 | - ");" . |
|
| 415 | - '}' . |
|
| 410 | + 'document.addEventListener("vite-script-loaded", function (e) {'. |
|
| 411 | + 'if (e.detail.path === "../src/web/assets/src/js/OptimizedImagesField.js") {'. |
|
| 412 | + "$('#{$nameSpaceId}-field').ImageOptimizeOptimizedImages(". |
|
| 413 | + $jsonVars. |
|
| 414 | + ");". |
|
| 415 | + '}'. |
|
| 416 | 416 | '});' |
| 417 | 417 | ); |
| 418 | 418 | |
@@ -258,7 +258,7 @@ discard block |
||
| 258 | 258 | Event::on( |
| 259 | 259 | CraftVariable::class, |
| 260 | 260 | CraftVariable::EVENT_INIT, |
| 261 | - function (Event $event) { |
|
| 261 | + function(Event $event) { |
|
| 262 | 262 | /** @var CraftVariable $variable */ |
| 263 | 263 | $variable = $event->sender; |
| 264 | 264 | $variable->set('imageOptimize', [ |
@@ -272,7 +272,7 @@ discard block |
||
| 272 | 272 | Event::on( |
| 273 | 273 | Fields::class, |
| 274 | 274 | Fields::EVENT_REGISTER_FIELD_TYPES, |
| 275 | - function (RegisterComponentTypesEvent $event) { |
|
| 275 | + function(RegisterComponentTypesEvent $event) { |
|
| 276 | 276 | Craft::debug( |
| 277 | 277 | 'Fields::EVENT_REGISTER_FIELD_TYPES', |
| 278 | 278 | __METHOD__ |
@@ -286,7 +286,7 @@ discard block |
||
| 286 | 286 | Event::on( |
| 287 | 287 | Utilities::class, |
| 288 | 288 | Utilities::EVENT_REGISTER_UTILITY_TYPES, |
| 289 | - function (RegisterComponentTypesEvent $event) { |
|
| 289 | + function(RegisterComponentTypesEvent $event) { |
|
| 290 | 290 | $event->types[] = ImageOptimizeUtility::class; |
| 291 | 291 | } |
| 292 | 292 | ); |
@@ -322,7 +322,7 @@ discard block |
||
| 322 | 322 | Event::on( |
| 323 | 323 | Assets::class, |
| 324 | 324 | Assets::EVENT_GET_ASSET_URL, |
| 325 | - function (GetAssetUrlEvent $event) { |
|
| 325 | + function(GetAssetUrlEvent $event) { |
|
| 326 | 326 | Craft::debug( |
| 327 | 327 | 'Assets::EVENT_GET_ASSET_URL', |
| 328 | 328 | __METHOD__ |
@@ -338,7 +338,7 @@ discard block |
||
| 338 | 338 | Event::on( |
| 339 | 339 | Assets::class, |
| 340 | 340 | Assets::EVENT_GET_ASSET_THUMB_URL, |
| 341 | - function (GetAssetThumbUrlEvent $event) { |
|
| 341 | + function(GetAssetThumbUrlEvent $event) { |
|
| 342 | 342 | Craft::debug( |
| 343 | 343 | 'Assets::EVENT_GET_ASSET_THUMB_URL', |
| 344 | 344 | __METHOD__ |
@@ -354,7 +354,7 @@ discard block |
||
| 354 | 354 | Event::on( |
| 355 | 355 | AssetTransforms::class, |
| 356 | 356 | AssetTransforms::EVENT_GENERATE_TRANSFORM, |
| 357 | - function (GenerateTransformEvent $event) { |
|
| 357 | + function(GenerateTransformEvent $event) { |
|
| 358 | 358 | Craft::debug( |
| 359 | 359 | 'AssetTransforms::EVENT_GENERATE_TRANSFORM', |
| 360 | 360 | __METHOD__ |
@@ -370,7 +370,7 @@ discard block |
||
| 370 | 370 | Event::on( |
| 371 | 371 | AssetTransforms::class, |
| 372 | 372 | AssetTransforms::EVENT_AFTER_DELETE_TRANSFORMS, |
| 373 | - function (AssetTransformImageEvent $event) { |
|
| 373 | + function(AssetTransformImageEvent $event) { |
|
| 374 | 374 | Craft::debug( |
| 375 | 375 | 'AssetTransforms::EVENT_AFTER_DELETE_TRANSFORMS', |
| 376 | 376 | __METHOD__ |
@@ -386,7 +386,7 @@ discard block |
||
| 386 | 386 | Event::on( |
| 387 | 387 | Assets::class, |
| 388 | 388 | Assets::EVENT_BEFORE_REPLACE_ASSET, |
| 389 | - function (ReplaceAssetEvent $event) { |
|
| 389 | + function(ReplaceAssetEvent $event) { |
|
| 390 | 390 | Craft::debug( |
| 391 | 391 | 'Assets::EVENT_BEFORE_REPLACE_ASSET', |
| 392 | 392 | __METHOD__ |
@@ -405,7 +405,7 @@ discard block |
||
| 405 | 405 | Event::on( |
| 406 | 406 | Assets::class, |
| 407 | 407 | Assets::EVENT_AFTER_REPLACE_ASSET, |
| 408 | - function (ReplaceAssetEvent $event) { |
|
| 408 | + function(ReplaceAssetEvent $event) { |
|
| 409 | 409 | Craft::debug( |
| 410 | 410 | 'Assets::EVENT_AFTER_REPLACE_ASSET', |
| 411 | 411 | __METHOD__ |
@@ -428,7 +428,7 @@ discard block |
||
| 428 | 428 | Event::on( |
| 429 | 429 | Assets::class, |
| 430 | 430 | Elements::EVENT_BEFORE_SAVE_ELEMENT, |
| 431 | - function (ElementEvent $event) { |
|
| 431 | + function(ElementEvent $event) { |
|
| 432 | 432 | Craft::debug( |
| 433 | 433 | 'Elements::EVENT_BEFORE_SAVE_ELEMENT', |
| 434 | 434 | __METHOD__ |
@@ -449,7 +449,7 @@ discard block |
||
| 449 | 449 | Event::on( |
| 450 | 450 | Asset::class, |
| 451 | 451 | Elements::EVENT_BEFORE_DELETE_ELEMENT, |
| 452 | - function (ElementEvent $event) { |
|
| 452 | + function(ElementEvent $event) { |
|
| 453 | 453 | Craft::debug( |
| 454 | 454 | 'Elements::EVENT_BEFORE_DELETE_ELEMENT', |
| 455 | 455 | __METHOD__ |
@@ -474,7 +474,7 @@ discard block |
||
| 474 | 474 | Event::on( |
| 475 | 475 | Fields::class, |
| 476 | 476 | Fields::EVENT_AFTER_SAVE_FIELD, |
| 477 | - function (FieldEvent $event) { |
|
| 477 | + function(FieldEvent $event) { |
|
| 478 | 478 | Craft::debug( |
| 479 | 479 | 'Fields::EVENT_AFTER_SAVE_FIELD', |
| 480 | 480 | __METHOD__ |
@@ -491,7 +491,7 @@ discard block |
||
| 491 | 491 | Event::on( |
| 492 | 492 | Plugins::class, |
| 493 | 493 | Plugins::EVENT_AFTER_SAVE_PLUGIN_SETTINGS, |
| 494 | - function (PluginEvent $event) { |
|
| 494 | + function(PluginEvent $event) { |
|
| 495 | 495 | if ($event->plugin === $this) { |
| 496 | 496 | Craft::debug( |
| 497 | 497 | 'Plugins::EVENT_AFTER_SAVE_PLUGIN_SETTINGS', |
@@ -510,7 +510,7 @@ discard block |
||
| 510 | 510 | Event::on( |
| 511 | 511 | Volumes::class, |
| 512 | 512 | Volumes::EVENT_AFTER_SAVE_VOLUME, |
| 513 | - function (VolumeEvent $event) { |
|
| 513 | + function(VolumeEvent $event) { |
|
| 514 | 514 | Craft::debug( |
| 515 | 515 | 'Volumes::EVENT_AFTER_SAVE_VOLUME', |
| 516 | 516 | __METHOD__ |
@@ -531,7 +531,7 @@ discard block |
||
| 531 | 531 | Event::on( |
| 532 | 532 | Plugins::class, |
| 533 | 533 | Plugins::EVENT_AFTER_INSTALL_PLUGIN, |
| 534 | - function (PluginEvent $event) { |
|
| 534 | + function(PluginEvent $event) { |
|
| 535 | 535 | if ($event->plugin === $this) { |
| 536 | 536 | $request = Craft::$app->getRequest(); |
| 537 | 537 | if ($request->isCpRequest) { |
@@ -565,7 +565,7 @@ discard block |
||
| 565 | 565 | Event::on( |
| 566 | 566 | UrlManager::class, |
| 567 | 567 | UrlManager::EVENT_REGISTER_SITE_URL_RULES, |
| 568 | - function (RegisterUrlRulesEvent $event) { |
|
| 568 | + function(RegisterUrlRulesEvent $event) { |
|
| 569 | 569 | Craft::debug( |
| 570 | 570 | 'UrlManager::EVENT_REGISTER_SITE_URL_RULES', |
| 571 | 571 | __METHOD__ |
@@ -588,12 +588,12 @@ discard block |
||
| 588 | 588 | Event::on( |
| 589 | 589 | Plugins::class, |
| 590 | 590 | Plugins::EVENT_AFTER_LOAD_PLUGINS, |
| 591 | - function () { |
|
| 591 | + function() { |
|
| 592 | 592 | // Install these only after all other plugins have loaded |
| 593 | 593 | Event::on( |
| 594 | 594 | View::class, |
| 595 | 595 | View::EVENT_REGISTER_CP_TEMPLATE_ROOTS, |
| 596 | - function (RegisterTemplateRootsEvent $e) { |
|
| 596 | + function(RegisterTemplateRootsEvent $e) { |
|
| 597 | 597 | // Register the root directodies |
| 598 | 598 | $allImageTransformTypes = ImageOptimize::$plugin->optimize->getAllImageTransformTypes(); |
| 599 | 599 | /** @var ImageTransformInterface $imageTransformType */ |