MAGETWO-52737: Deleting category's image on custom store view - deletes category's images for all store views

Author: slopukhov

app/code/Magento/Catalog/Controller/Adminhtml/Category/Save.php

@@ -85,8 +85,11 @@ protected function _filterCategoryPostData(array $rawData)
             if (is_array($data['image']) && !empty($data['image']['delete'])) {
                 $data['image'] = null;
             } else {
-                $data['image_new_data'] = $data['image'];
-                unset($data['image']);
+                if (isset($data['image'][0]['name']) && isset($data['image'][0]['tmp_name'])) {
+                    $data['image'] = $data['image'][0]['name'];
+                } else {
+                    unset($data['image']);
+                }
             }
         }
         return $data;

app/code/Magento/Catalog/Model/Category/Attribute/Backend/Image.php

@@ -94,14 +94,11 @@ private function getImageUploader()
      */
     public function afterSave($object)
     {
-        $value = $object->getData($this->getAttribute()->getName() . '_new_data');
+        $image = $object->getData($this->getAttribute()->getName(), null);
 
-        if (isset($value[0]['name']) && isset($value[0]['tmp_name'])) {
+        if ($image !== null) {
             try {
-                $result = $this->getImageUploader()->moveFileFromTmp($value[0]['name']);
-
-                $object->setData($this->getAttribute()->getName(), $result);
-                $this->getAttribute()->getEntity()->saveAttribute($object, $this->getAttribute()->getName());
+                $this->getImageUploader()->moveFileFromTmp($image);
             } catch (\Exception $e) {
                 $this->_logger->critical($e);
             }

app/code/Magento/Catalog/Model/CategoryRepository.php

@@ -94,8 +94,11 @@ public function save(\Magento\Catalog\Api\Data\CategoryInterface $category)
                 if (is_array($existingData['image']) && !empty($existingData['image']['delete'])) {
                     $existingData['image'] = null;
                 } else {
-                    $existingData['image_new_data'] = $existingData['image'];
-                    unset($existingData['image']);
+                    if (isset($existingData['image'][0]['name']) && isset($existingData['image'][0]['tmp_name'])) {
+                        $existingData['image'] = $existingData['image'][0]['name'];
+                    } else {
+                        unset($existingData['image']);
+                    }
                 }
             }
         } else {

app/code/Magento/Catalog/Test/Unit/Controller/Adminhtml/Category/SaveTest.php

@@ -433,7 +433,7 @@ public function testExecute($categoryId, $storeId, $parentId)
             ->method('getPostValue')
             ->willReturn($postData);
         $addData = $postData;
-        $addData['image_new_data'] = ['delete' => true];
+        $addData['image'] = ['delete' => true];
         $categoryMock->expects($this->once())
             ->method('addData')
             ->with($addData);

app/code/Magento/Catalog/Test/Unit/Model/CategoryRepositoryTest.php

@@ -173,7 +173,6 @@ public function filterExtraFieldsOnUpdateCategoryDataProvider()
                 ['level' => '1', 'path' => '1/2', 'image' => ['categoryImage'], 'name' => 'category'],
                 [
                     'store_id' => 1,
-                    'image_new_data' => ['categoryImage'],
                     'name' => 'category',
                     'entity_id' => null
                 ]