From f75830dd0c6e9e0df2cc0e7544b6bb0d125be472 Mon Sep 17 00:00:00 2001
From: "andrii.zinkevych" <a.zinkevych@atwix.com>
Date: Tue, 2 Nov 2021 13:07:54 +0100
Subject: [PATCH 1/2] Add Sniff for check autovivification on array

---
 .../Sniffs/PHP/ArrayAutovivificationSniff.php | 66 +++++++++++++++++++
 1 file changed, 66 insertions(+)
 create mode 100644 Magento2/Sniffs/PHP/ArrayAutovivificationSniff.php

diff --git a/Magento2/Sniffs/PHP/ArrayAutovivificationSniff.php b/Magento2/Sniffs/PHP/ArrayAutovivificationSniff.php
new file mode 100644
index 00000000..f814bdba
--- /dev/null
+++ b/Magento2/Sniffs/PHP/ArrayAutovivificationSniff.php
@@ -0,0 +1,66 @@
+<?php
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+declare(strict_types=1);
+
+namespace Magento2\Sniffs\PHP;
+
+use PHP_CodeSniffer\Files\File;
+use PHP_CodeSniffer\Sniffs\Sniff;
+
+/**
+ * Sniff to validate array autovivification.
+ */
+class ArrayAutovivificationSniff implements Sniff
+{
+    /**
+     * String representation of error.
+     *
+     * @var string
+     */
+    private $warningMessage = 'Deprecated: Automatic conversion of false to array is deprecated.';
+
+    /**
+     * Warning violation code.
+     *
+     * @var string
+     */
+    private $warningCode = 'Autovivification';
+
+    /**
+     * @inheritdoc
+     */
+    public function register(): array
+    {
+        return [
+            T_VARIABLE
+        ];
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function process(File $phpcsFile, $stackPtr): void
+    {
+        $positionSquareBracket = $phpcsFile->findNext(T_OPEN_SQUARE_BRACKET, $stackPtr, $stackPtr + 2);
+
+        if ($positionSquareBracket) {
+            $tokens = $phpcsFile->getTokens();
+            $propertyTokenKey = array_keys(array_column($tokens, 'content'), $tokens[$stackPtr]['content']);
+
+            arsort($propertyTokenKey);
+
+            foreach ($propertyTokenKey as $tokenKey) {
+                if ($tokenKey < $stackPtr && $tokens[$tokenKey + 2]['content'] === '=') {
+                    if ($tokens[$tokenKey + 4]['content'] != 'false') {
+                        return;
+                    }
+
+                    $phpcsFile->addWarning($this->warningMessage, $positionSquareBracket, $this->warningCode);
+                }
+            }
+        }
+    }
+}

From 437e157b5073b48e41ab46235de37299419e684b Mon Sep 17 00:00:00 2001
From: "andrii.zinkevych" <a.zinkevych@atwix.com>
Date: Tue, 2 Nov 2021 20:36:30 +0100
Subject: [PATCH 2/2] Created Unit Test for array autovivification sniff

---
 .../Sniffs/PHP/ArrayAutovivificationSniff.php |  7 ++-
 .../PHP/ArrayAutovivificationUnitTest.inc     | 60 +++++++++++++++++++
 .../PHP/ArrayAutovivificationUnitTest.php     | 35 +++++++++++
 Magento2/ruleset.xml                          |  5 ++
 4 files changed, 105 insertions(+), 2 deletions(-)
 create mode 100644 Magento2/Tests/PHP/ArrayAutovivificationUnitTest.inc
 create mode 100644 Magento2/Tests/PHP/ArrayAutovivificationUnitTest.php

diff --git a/Magento2/Sniffs/PHP/ArrayAutovivificationSniff.php b/Magento2/Sniffs/PHP/ArrayAutovivificationSniff.php
index f814bdba..f5893a3d 100644
--- a/Magento2/Sniffs/PHP/ArrayAutovivificationSniff.php
+++ b/Magento2/Sniffs/PHP/ArrayAutovivificationSniff.php
@@ -48,12 +48,15 @@ public function process(File $phpcsFile, $stackPtr): void
 
         if ($positionSquareBracket) {
             $tokens = $phpcsFile->getTokens();
-            $propertyTokenKey = array_keys(array_column($tokens, 'content'), $tokens[$stackPtr]['content']);
+            $positionFunction = $phpcsFile->findPrevious(T_FUNCTION, $positionSquareBracket) ?: 0;
+            $sliceLength = $stackPtr - $positionFunction;
+            $sliceToken = array_slice(array_column($tokens, 'content'), $positionFunction, $sliceLength, true);
+            $propertyTokenKey = array_keys($sliceToken, $tokens[$stackPtr]['content']);
 
             arsort($propertyTokenKey);
 
             foreach ($propertyTokenKey as $tokenKey) {
-                if ($tokenKey < $stackPtr && $tokens[$tokenKey + 2]['content'] === '=') {
+                if ($tokens[$tokenKey + 2]['content'] === '=') {
                     if ($tokens[$tokenKey + 4]['content'] != 'false') {
                         return;
                     }
diff --git a/Magento2/Tests/PHP/ArrayAutovivificationUnitTest.inc b/Magento2/Tests/PHP/ArrayAutovivificationUnitTest.inc
new file mode 100644
index 00000000..82b92c00
--- /dev/null
+++ b/Magento2/Tests/PHP/ArrayAutovivificationUnitTest.inc
@@ -0,0 +1,60 @@
+<?php
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+declare(strict_types=1);
+
+namespace Magento2\Tests\PHP;
+
+/**
+ * Class to test array avtovivification.
+ */
+class Avtovivification
+{
+    /**
+     * @return array
+     */
+    public function testNullAvtovivification()
+    {
+        $productIds = null;
+
+        $productIds[] = 'test_array_value';
+
+        return $productIds;
+    }
+
+    /**
+     * @return array
+     */
+    public function testArrayAvtovivification()
+    {
+        $productIds = [];
+
+        $productIds[] = 'test_array_value';
+
+        return $productIds;
+    }
+
+    /**
+     * @return array
+     */
+    public function testFalseAvtovivification()
+    {
+        $productIds = false;
+
+        $productIds[] = 'test_array_value';
+
+        return $productIds;
+    }
+
+    /**
+     * @return array
+     */
+    public function testUndefineAvtovivification()
+    {
+        $productIds[] = 'test_array_value';
+
+        return $productIds;
+    }
+}
diff --git a/Magento2/Tests/PHP/ArrayAutovivificationUnitTest.php b/Magento2/Tests/PHP/ArrayAutovivificationUnitTest.php
new file mode 100644
index 00000000..faaecde5
--- /dev/null
+++ b/Magento2/Tests/PHP/ArrayAutovivificationUnitTest.php
@@ -0,0 +1,35 @@
+<?php
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+declare(strict_types=1);
+
+namespace Magento2\Tests\PHP;
+
+use PHP_CodeSniffer\Tests\Standards\AbstractSniffUnitTest;
+
+class ArrayAutovivificationUnitTest extends AbstractSniffUnitTest
+{
+    /**
+     * @inheritdoc
+     */
+    public function getErrorList(): array
+    {
+        return [];
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function getWarningList($testFile = ''): array
+    {
+        if ($testFile === 'ArrayAutovivificationUnitTest.inc') {
+            return [
+                46 => 1
+            ];
+        }
+
+        return [];
+    }
+}
diff --git a/Magento2/ruleset.xml b/Magento2/ruleset.xml
index 442d9867..5ea68ff3 100644
--- a/Magento2/ruleset.xml
+++ b/Magento2/ruleset.xml
@@ -400,6 +400,11 @@
         <type>warning</type>
         <exclude-pattern>*\.xml$</exclude-pattern>
     </rule>
+    <rule ref="Magento2.PHP.ArrayAutovivification">
+        <severity>7</severity>
+        <type>warning</type>
+        <exclude-pattern>*\.xml$</exclude-pattern>
+    </rule>
     <rule ref="Magento2.Performance.ForeachArrayMerge">
         <severity>7</severity>
         <type>warning</type>