MAGETWO-56063: After upgrading from 2.0.7 to 2.1, editing a category gives a 500 error

- fixed logger injection

Author: buskamuza

app/code/Magento/Eav/Model/ResourceModel/ReadHandler.php

@@ -14,6 +14,7 @@
 use Magento\Framework\Model\Entity\ScopeInterface;
 use Magento\Framework\EntityManager\Operation\AttributeInterface;
 use Magento\Eav\Model\Entity\AttributeCache;
+use Psr\Log\LoggerInterface;
 
 /**
  * @SuppressWarnings(PHPMD.CouplingBetweenObjects)
@@ -51,7 +52,7 @@ class ReadHandler implements AttributeInterface
     protected $scopeResolver;
 
     /**
-     * @var \Psr\Log\LoggerInterface
+     * @var LoggerInterface
      */
     private $logger;
 
@@ -64,21 +65,24 @@ class ReadHandler implements AttributeInterface
      * @param AppResource $appResource
      * @param ScopeResolver $scopeResolver
      * @param AttributeCache $attributeCache
+     * @param LoggerInterface $logger
      */
     public function __construct(
         AttributeRepository $attributeRepository,
         MetadataPool $metadataPool,
         SearchCriteriaBuilder $searchCriteriaBuilder,
         AppResource $appResource,
         ScopeResolver $scopeResolver,
-        AttributeCache $attributeCache
+        AttributeCache $attributeCache,
+        LoggerInterface $logger
     ) {
         $this->attributeRepository = $attributeRepository;
         $this->metadataPool = $metadataPool;
         $this->searchCriteriaBuilder = $searchCriteriaBuilder;
         $this->appResource = $appResource;
         $this->scopeResolver = $scopeResolver;
         $this->attributeCache = $attributeCache;
+        $this->logger = $logger;
     }
 
     /**
@@ -172,7 +176,7 @@ public function execute($entityType, $entityData, $arguments = [])
                 if (isset($attributesMap[$attributeValue['attribute_id']])) {
                     $entityData[$attributesMap[$attributeValue['attribute_id']]] = $attributeValue['value'];
                 } else {
-                    $this->getLogger()->warning(
+                    $this->logger->warning(
                         "Attempt to load value of nonexistent EAV attribute '{$attributeValue['attribute_id']}' 
                         for entity type '$entityType'."
                     );
@@ -181,18 +185,4 @@ public function execute($entityType, $entityData, $arguments = [])
         }
         return $entityData;
     }
-
-    /**
-     * Get logger
-     *
-     * @return \Psr\Log\LoggerInterface
-     * @deprecated
-     */
-    private function getLogger()
-    {
-        if ($this->logger == null) {
-            $this->logger = ObjectManager::getInstance()->get(\Psr\Log\LoggerInterface::class);
-        }
-        return $this->logger;
-    }
 }