Remove the Zend_Json call from the Setup/Migration.php

 - inject the new \Magento\Framework\Serialize\Serializer\Json class,
 - update _jsonDecode method parameter default from Zend_Json constant,
 - update _jsonDecode to call the new Serializer method version

Author: dmanners

lib/internal/Magento/Framework/Module/Setup/Migration.php

@@ -132,6 +132,11 @@ class Migration
      */
     private $setup;
 
+    /**
+     * @var \Magento\Framework\Serialize\Serializer\Json
+     */
+    private $serializer;
+
     /**
      * @param ModuleDataSetupInterface $setup
      * @param Filesystem $filesystem
@@ -144,7 +149,8 @@ public function __construct(
         Filesystem $filesystem,
         MigrationData $migrationData,
         $confPathToMapFile,
-        $compositeModules = []
+        $compositeModules = [],
+        \Magento\Framework\Serialize\Serializer\Json $serializer = null
     ) {
         $this->_directory = $filesystem->getDirectoryRead(DirectoryList::ROOT);
         $this->_pathToMapFile = $confPathToMapFile;
@@ -155,6 +161,8 @@ public function __construct(
         ];
         $this->_compositeModules = $compositeModules;
         $this->setup = $setup;
+        $this->serializer = $serializer?: \Magento\Framework\App\ObjectManager::getInstance()
+            ->get(\Magento\Framework\Serialize\Serializer\Json::class);
     }
 
     /**
@@ -694,10 +702,12 @@ public function getCompositeModules()
      *
      * @param string $encodedValue
      * @param int $objectDecodeType
-     * @return mixed
+     * @return array|bool|float|int|mixed|null|string
+     * @throws \InvalidArgumentException
+     * @SuppressWarnings(PHPMD.UnusedFormalParameter)
      */
-    protected function _jsonDecode($encodedValue, $objectDecodeType = \Zend_Json::TYPE_ARRAY)
+    protected function _jsonDecode($encodedValue, $objectDecodeType = 1)
     {
-        return \Zend_Json::decode($encodedValue, $objectDecodeType);
+        return $this->serializer->unserialize($encodedValue);
     }
 }

lib/internal/Magento/Framework/Module/Test/Unit/Setup/MigrationTest.php

@@ -147,7 +147,9 @@ public function testAppendClassAliasReplace()
             $setupMock,
             $filesystemMock,
             $migrationData,
-            'app/etc/aliases_to_classes_map.json'
+            'app/etc/aliases_to_classes_map.json',
+            [],
+            $this->getSerializerMock()
         );
 
         $setupModel->appendClassAliasReplace(
@@ -203,7 +205,8 @@ public function testDoUpdateClassAliases($replaceRules, $tableData, $expected, $
             $filesystemMock,
             $migrationData,
             'app/etc/aliases_to_classes_map.json',
-            $this->_getModelDependencies($tableRowsCount, $tableData, $aliasesMap)
+            $this->_getModelDependencies($tableRowsCount, $tableData, $aliasesMap),
+            $this->getSerializerMock()
         );
 
         foreach ($replaceRules as $replaceRule) {
@@ -248,4 +251,23 @@ protected function _getFilesystemMock()
         $mock = $this->getMockBuilder(\Magento\Framework\Filesystem::class)->disableOriginalConstructor()->getMock();
         return $mock;
     }
+
+    /**
+     * @return \PHPUnit_Framework_MockObject_MockObject|\Magento\Framework\Serialize\Serializer\Json
+     * @throws \PHPUnit_Framework_Exception
+     */
+    private function getSerializerMock()
+    {
+        $serializerMock = $this->getMockBuilder(\Magento\Framework\Serialize\Serializer\Json::class)
+            ->getMock();
+
+        $serializerMock->expects($this->any())
+            ->method('unserialize')
+            ->willReturnCallback(
+                function ($serializedData) {
+                    return json_decode($serializedData, true);
+                }
+            );
+        return $serializerMock;
+    }
 }