Simplified and fixed broken multi name parameter code

Author: HLeithner

src/Mysqli/MysqliStatement.php

@@ -158,7 +158,7 @@ public function prepareParameterKeyMapping($sql)
         $quoteChar  = '';
         $literal    = '';
         $mapping    = [];
-        $replace    = [];
+        $position   = 0;
         $matches    = [];
         $pattern    = '/([:][a-zA-Z0-9_]+)/';
 
@@ -198,15 +198,14 @@ public function prepareParameterKeyMapping($sql)
                     }
 
                     if (isset($mapping[$match[0]])) {
-                        $mapping[$match[0]] = is_array($mapping[$match[0]]) ? $mapping[$match[0]] : [$mapping[$match[0]]];
-                        $mapping[$match[0]][] = \count($mapping);
-                    } else {
-                        $mapping[$match[0]] = \count($mapping);
+                        $mapping[$match[0]][] = [];
                     }
-                    $endOfPlaceholder = $match[1] + strlen($match[0]);
+
+                    $mapping[$match[0]][]   = $position++;
+                    $endOfPlaceholder       = $match[1] + strlen($match[0]);
                     $beginOfNextPlaceholder = $matches[0][$i + 1][1] ?? strlen($substring);
                     $beginOfNextPlaceholder -= $endOfPlaceholder;
-                    $literal .= '?' . substr($substring, $endOfPlaceholder, $beginOfNextPlaceholder);
+                    $literal                .= '?' . substr($substring, $endOfPlaceholder, $beginOfNextPlaceholder);
                 }
             } else {
                 $literal .= $substring;
@@ -378,14 +377,9 @@ public function execute(?array $parameters = null)
                 foreach ($this->bindedValues as $key => &$value) {
                     $paramKey = $this->parameterKeyMapping[$key];
 
-                    if (is_scalar($this->parameterKeyMapping[$key])) {
-                        $params[$paramKey] =& $value;
-                        $types[$paramKey]  = $this->typesKeyMapping[$key];
-                    } else {
-                        foreach ($paramKey as $currentKey) {
-                            $params[$currentKey] =& $value;
-                            $types[$currentKey]  = $this->typesKeyMapping[$key];
-                        }
+                    foreach ($paramKey as $currentKey) {
+                        $params[$currentKey] =& $value;
+                        $types[$currentKey]  = $this->typesKeyMapping[$key];
                     }
                 }
             } else {

src/Sqlsrv/SqlsrvStatement.php

@@ -163,7 +163,7 @@ public function prepareParameterKeyMapping($sql)
         $quoteChar  = '';
         $literal    = '';
         $mapping    = [];
-        $replace    = [];
+        $position   = 0;
         $matches    = [];
         $pattern    = '/([:][a-zA-Z0-9_]+)/';
 
@@ -203,16 +203,14 @@ public function prepareParameterKeyMapping($sql)
                     }
 
                     if (isset($mapping[$match[0]])) {
-                        $mapping[$match[0]] = is_array($mapping[$match[0]]) ? $mapping[$match[0]] : [$mapping[$match[0]]];
-                        $mapping[$match[0]][] = \count($mapping);
-                    } else {
-                        $mapping[$match[0]] = \count($mapping);
+                        $mapping[$match[0]][] = [];
                     }
 
+                    $mapping[$match[0]][]   = $position++;
                     $endOfPlaceholder       = $match[1] + strlen($match[0]);
                     $beginOfNextPlaceholder = $matches[0][$i + 1][1] ?? strlen($substring);
                     $beginOfNextPlaceholder -= $endOfPlaceholder;
-                    $literal .= '?' . substr($substring, $endOfPlaceholder, $beginOfNextPlaceholder);
+                    $literal                .= '?' . substr($substring, $endOfPlaceholder, $beginOfNextPlaceholder);
                 }
             } else {
                 $literal .= $substring;
@@ -492,12 +490,8 @@ private function prepare()
             if (isset($this->parameterKeyMapping[$key])) {
                 $paramKey = $this->parameterKeyMapping[$key];
 
-                if (is_scalar($this->parameterKeyMapping[$key])) {
-                    $params[$paramKey] = $variable;
-                } else {
-                    foreach ($paramKey as $currentKey) {
-                        $params[$currentKey] = $variable;
-                    }
+                foreach ($paramKey as $currentKey) {
+                    $params[$currentKey] = $variable;
                 }
             } else {
                 $params[] = $variable;