Add new Custom Rector Rule to Pass true to 3rd parameter of in_array when no value provided

Author: samsonasik

app/Config/Mimes.php

@@ -507,14 +507,14 @@ public static function guessExtensionFromType(string $type, string $proposedExte
 
 		$proposedExtension = trim(strtolower($proposedExtension));
 
-		if ($proposedExtension !== '' && array_key_exists($proposedExtension, static::$mimes) && in_array($type, is_string(static::$mimes[$proposedExtension]) ? [static::$mimes[$proposedExtension]] : static::$mimes[$proposedExtension]))
+		if ($proposedExtension !== '' && array_key_exists($proposedExtension, static::$mimes) && in_array($type, is_string(static::$mimes[$proposedExtension]) ? [static::$mimes[$proposedExtension]] : static::$mimes[$proposedExtension], true))
 		{
 			return $proposedExtension;
 		}
 
 		foreach (static::$mimes as $ext => $types)
 		{
-			if ((is_string($types) && $types === $type) || (is_array($types) && in_array($type, $types)))
+			if ((is_string($types) && $types === $type) || (is_array($types) && in_array($type, $types, true)))
 			{
 				return $ext;
 			}

rector.php

@@ -4,6 +4,7 @@
 use Rector\CodeQuality\Rector\Return_\SimplifyUselessVariableRector;
 use Rector\SOLID\Rector\If_\RemoveAlwaysElseRector;
 use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;
+use Utils\Rector\PassTrueToThirdParameterInArrayRector;
 use Utils\Rector\UnderscoreToCamelCaseVariableNameRector;
 
 return static function (ContainerConfigurator $containerConfigurator): void {
@@ -40,4 +41,5 @@
 	$services->set(UnderscoreToCamelCaseVariableNameRector::class);
 	$services->set(SimplifyUselessVariableRector::class);
 	$services->set(RemoveAlwaysElseRector::class);
+	$services->set(PassTrueToThirdParameterInArrayRector::class);
 };

system/Common.php

@@ -662,7 +662,7 @@ function helper($filenames)
 			}
 
 			// Check if this helper has already been loaded
-			if (in_array($filename, $loaded))
+			if (in_array($filename, $loaded, true))
 			{
 				continue;
 			}

system/Database/Sqlsrv/Forge.php

@@ -301,12 +301,12 @@ protected function _processForeignKeys(string $table): string
 						. ' FOREIGN KEY (' . $this->db->escapeIdentifiers($field) . ') '
 						. ' REFERENCES ' . $this->db->escapeIdentifiers($this->db->getPrefix() . $fkey['table']) . ' (' . $this->db->escapeIdentifiers($fkey['field']) . ')';
 
-				if ($fkey['onDelete'] !== false && in_array($fkey['onDelete'], $allowActions))
+				if ($fkey['onDelete'] !== false && in_array($fkey['onDelete'], $allowActions, true))
 				{
 					$sql .= ' ON DELETE ' . $fkey['onDelete'];
 				}
 
-				if ($fkey['onUpdate'] !== false && in_array($fkey['onUpdate'], $allowActions))
+				if ($fkey['onUpdate'] !== false && in_array($fkey['onUpdate'], $allowActions, true))
 				{
 					$sql .= ' ON UPDATE ' . $fkey['onUpdate'];
 				}

system/Router/RouteCollection.php

@@ -1514,7 +1514,7 @@ private function checkSubdomains($subdomains): bool
 
 		// Routes can be limited to any sub-domain. In that case, though,
 		// it does require a sub-domain to be present.
-		if (! empty($this->currentSubdomain) && in_array('*', $subdomains))
+		if (! empty($this->currentSubdomain) && in_array('*', $subdomains, true))
 		{
 			return true;
 		}

system/Validation/Validation.php

@@ -249,7 +249,7 @@ protected function processRules(string $field, string $label = null, $value, $ru
 			$rules = array_diff($rules, ['if_exist']);
 		}
 
-		if (in_array('permit_empty', $rules))
+		if (in_array('permit_empty', $rules, true))
 		{
 			if (! in_array('required', $rules, true) && (is_array($value) ? empty($value) : (trim($value) === '')))
 			{

utils/Rector/PassTrueToThirdParameterInArrayRector.php

@@ -0,0 +1,70 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Utils\Rector;
+
+use PhpParser\Node;
+use PhpParser\Node\Arg;
+use PhpParser\Node\Expr\ConstFetch;
+use PhpParser\Node\Expr\FuncCall;
+use PhpParser\Node\Name;
+use Rector\Core\Rector\AbstractRector;
+use Rector\Core\RectorDefinition\CodeSample;
+use Rector\Core\RectorDefinition\RectorDefinition;
+
+/**
+ * Pass true to 3rd parameter of in_array when no value provided
+ */
+final class PassTrueToThirdParameterInArrayRector extends AbstractRector
+{
+	public function getDefinition(): RectorDefinition
+	{
+		return new RectorDefinition('Pass true to 3rd parameter of in_array if no value provided', [
+			new CodeSample(
+				<<<'CODE_SAMPLE'
+in_array('a', $array);
+CODE_SAMPLE
+,
+				<<<'CODE_SAMPLE'
+in_array('a', $array, true);
+CODE_SAMPLE
+			),
+		]);
+	}
+
+	/**
+	 * @return string[]
+	 */
+	public function getNodeTypes(): array
+	{
+		return [FuncCall::class];
+	}
+
+	/**
+	 * @param FuncCall $node
+	 */
+	public function refactor(Node $node): ?Node
+	{
+		$name = $node->name;
+		if (! method_exists($name, 'toString'))
+		{
+			return null;
+		}
+
+		if ($name->toString() !== 'in_array')
+		{
+			return null;
+		}
+
+		if (isset($node->args[2]))
+		{
+			return null;
+		}
+
+		$name          = new Name('true');
+		$node->args[2] = new Arg(new ConstFetch($name));
+
+		return $node;
+	}
+}