MC-38613: Support by Magento CatalogGraphQl

Author: duhon

app/code/Magento/AwsS3/Driver/AwsS3.php

@@ -344,7 +344,11 @@ public function isFile($path): bool
         $path = $this->normalizeRelativePath($path);
         $path = rtrim($path, '/');
 
-        return $this->adapter->has($path) && $this->adapter->getMetadata($path)['type'] === self::TYPE_FILE;
+        if ($this->adapter->has($path) && ($meta = $this->adapter->getMetadata($path))) {
+            return ($meta['type'] ?? null) === self::TYPE_FILE;
+        }
+
+        return false;
     }
 
     /**

dev/tests/integration/testsuite/Magento/Catalog/Model/Product/Gallery/ProcessorTest.php

@@ -46,9 +46,10 @@ public static function setUpBeforeClass(): void
         $mediaDirectory->create($config->getBaseTmpMediaPath());
         $mediaDirectory->create($config->getBaseMediaPath());
 
-        copy($fixtureDir . "/magento_image.jpg", self::$_mediaTmpDir . "/magento_image.jpg");
-        copy($fixtureDir . "/magento_image.jpg", self::$_mediaDir . "/magento_image.jpg");
-        copy($fixtureDir . "/magento_small_image.jpg", self::$_mediaTmpDir . "/magento_small_image.jpg");
+        $mediaDirectory->getDriver()->filePutContents(self::$_mediaTmpDir . "/magento_image.jpg", file_get_contents($fixtureDir . "/magento_image.jpg"));
+        $mediaDirectory->getDriver()->filePutContents(self::$_mediaDir . "/magento_image.jpg", file_get_contents($fixtureDir . "/magento_image.jpg"));
+        $mediaDirectory->getDriver()->filePutContents(self::$_mediaTmpDir . "/magento_small_image.jpg", file_get_contents($fixtureDir . "/magento_small_image.jpg"));
+
     }
 
     public static function tearDownAfterClass(): void

dev/tests/integration/testsuite/Magento/Catalog/Model/ProductTest.php

@@ -246,7 +246,7 @@ protected function _copyFileToBaseTmpMediaPath($sourceFile)
 
         $mediaDirectory->create($config->getBaseTmpMediaPath());
         $targetFile = $config->getTmpMediaPath(basename($sourceFile));
-        copy($sourceFile, $mediaDirectory->getAbsolutePath($targetFile));
+        $mediaDirectory->getDriver()->filePutContents($mediaDirectory->getAbsolutePath($targetFile), file_get_contents($sourceFile));
 
         return $targetFile;
     }

dev/tests/integration/testsuite/Magento/Catalog/_files/catalog_category_image.php

@@ -10,13 +10,14 @@
 $objectManager = \Magento\TestFramework\Helper\Bootstrap::getObjectManager();
 
 /** @var $mediaDirectory \Magento\Framework\Filesystem\Directory\WriteInterface */
-$mediaDirectory = $objectManager->get(\Magento\Framework\Filesystem::class)
-    ->getDirectoryWrite(DirectoryList::MEDIA);
+$mediaDirectory = $objectManager->get(\Magento\Framework\Filesystem::class)->getDirectoryWrite(DirectoryList::MEDIA);
 $fileName = 'magento_small_image.jpg';
 $fileNameLong = 'magento_long_image_name_magento_long_image_name_magento_long_image_name.jpg';
 $filePath = 'catalog/category/' . $fileName;
 $filePathLong = 'catalog/category/' . $fileNameLong;
 $mediaDirectory->create('catalog/category');
-
-copy(__DIR__ . DIRECTORY_SEPARATOR . $fileName, $mediaDirectory->getAbsolutePath($filePath));
-copy(__DIR__ . DIRECTORY_SEPARATOR . $fileNameLong, $mediaDirectory->getAbsolutePath($filePathLong));
+$shortImageContent = file_get_contents(__DIR__ . DIRECTORY_SEPARATOR . $fileName);
+$longImageContent = file_get_contents(__DIR__ . DIRECTORY_SEPARATOR . $fileNameLong);
+$mediaDirectory->getDriver()->filePutContents($mediaDirectory->getAbsolutePath($filePath), $shortImageContent);
+$mediaDirectory->getDriver()->filePutContents($mediaDirectory->getAbsolutePath($filePathLong), $longImageContent);
+unset($shortImageContent, $longImageContent);

dev/tests/integration/testsuite/Magento/Catalog/_files/catalog_tmp_category_image.php

@@ -15,5 +15,4 @@
 $fileName = 'magento_small_image.jpg';
 $tmpFilePath = 'catalog/tmp/category/' . $fileName;
 $mediaDirectory->create('catalog/tmp/category');
-
-copy(__DIR__ . DIRECTORY_SEPARATOR . $fileName, $mediaDirectory->getAbsolutePath($tmpFilePath));
+$mediaDirectory->getDriver()->filePutContents($mediaDirectory->getAbsolutePath($tmpFilePath), file_get_contents(__DIR__ . DIRECTORY_SEPARATOR . $fileName));

dev/tests/integration/testsuite/Magento/Catalog/_files/product_image.php

@@ -24,12 +24,11 @@
 $images = ['magento_image.jpg', 'magento_small_image.jpg', 'magento_thumbnail.jpg'];
 
 foreach ($images as $image) {
-    $targetTmpFilePath = $mediaDirectory->getAbsolutePath() . DIRECTORY_SEPARATOR . $targetTmpDirPath
-        . DIRECTORY_SEPARATOR . $image;
+    $targetTmpFilePath = $mediaDirectory->getAbsolutePath() . $targetTmpDirPath . $image;
 
     $sourceFilePath = __DIR__ . DIRECTORY_SEPARATOR . $image;
+    $mediaDirectory->getDriver()->filePutContents($targetTmpFilePath, file_get_contents($sourceFilePath));
 
-    copy($sourceFilePath, $targetTmpFilePath);
     // Copying the image to target dir is not necessary because during product save, it will be moved there from tmp dir
     $database->saveFile($targetTmpFilePath);
 }

dev/tests/integration/testsuite/Magento/Catalog/_files/product_simple_with_image.php

@@ -38,7 +38,7 @@
 $mediaDirectory->create($targetTmpDirPath);
 
 $dist = $mediaDirectory->getAbsolutePath($mediaConfig->getBaseMediaPath() .  DIRECTORY_SEPARATOR . 'magento_image.jpg');
-copy(__DIR__ . '/magento_image.jpg', $dist);
+$mediaDirectory->getDriver()->filePutContents($dist, file_get_contents(__DIR__ . '/magento_image.jpg'));
 
 /** @var \Magento\Catalog\Api\ProductRepositoryInterface $productRepository */
 $productRepository = $objectManager->create(\Magento\Catalog\Api\ProductRepositoryInterface::class);

dev/tests/integration/testsuite/Magento/Catalog/_files/validate_image_info.php

@@ -12,10 +12,10 @@
 /** @var Magento\Catalog\Model\Product\Media\Config $config */
 $config = $objectManager->get(\Magento\Catalog\Model\Product\Media\Config::class);
 
-/** @var $tmpDirectory \Magento\Framework\Filesystem\Directory\WriteInterface */
-$tmpDirectory = $filesystem->getDirectoryWrite(\Magento\Framework\App\Filesystem\DirectoryList::MEDIA);
-$tmpDirectory->create($config->getBaseTmpMediaPath());
+/** @var $mediaDirectory \Magento\Framework\Filesystem\Directory\WriteInterface */
+$mediaDirectory = $filesystem->getDirectoryWrite(\Magento\Framework\App\Filesystem\DirectoryList::MEDIA);
+$mediaDirectory->create($config->getBaseTmpMediaPath());
 
-$targetTmpFilePath = $tmpDirectory->getAbsolutePath($config->getBaseTmpMediaPath() . '/magento_small_image.jpg');
-copy(__DIR__ . '/magento_small_image.jpg', $targetTmpFilePath);
+$targetTmpFilePath = $mediaDirectory->getAbsolutePath($config->getBaseTmpMediaPath() . '/magento_small_image.jpg');
+$mediaDirectory->getDriver()->filePutContents($targetTmpFilePath, file_get_contents(__DIR__ . '/magento_small_image.jpg'));
 // Copying the image to target dir is not necessary because during product save, it will be moved there from tmp dir

dev/tests/integration/testsuite/Magento/Catalog/controllers/_files/products.php

@@ -24,7 +24,7 @@
 
 $baseTmpMediaPath = $config->getBaseTmpMediaPath();
 $mediaDirectory->create($baseTmpMediaPath);
-copy(__DIR__ . '/product_image.png', $mediaDirectory->getAbsolutePath($baseTmpMediaPath . '/product_image.png'));
+$mediaDirectory->getDriver()->filePutContents($mediaDirectory->getAbsolutePath($baseTmpMediaPath . '/product_image.png'), file_get_contents(__DIR__ . '/product_image.png'));
 
 /** @var $productOne \Magento\Catalog\Model\Product */
 $productOne = $objectManager->create(\Magento\Catalog\Model\Product::class);

lib/internal/Magento/Framework/Api/Uploader.php

@@ -39,20 +39,4 @@ public function processFileAttributes($fileAttributes)
             $this->_fileExists = true;
         }
     }
-
-    /**
-     * Move files from TMP folder into destination folder
-     *
-     * @param string $tmpPath
-     * @param string $destPath
-     * @return bool|void
-     */
-    protected function _moveFile($tmpPath, $destPath)
-    {
-        if (is_uploaded_file($tmpPath)) {
-            return move_uploaded_file($tmpPath, $destPath);
-        } elseif (is_file($tmpPath)) {
-            return rename($tmpPath, $destPath);
-        }
-    }
 }