fix input name in case of one coumn and enabled sorting

Author: Eugene Tupikov

CHANGELOG.md

@@ -4,6 +4,7 @@ Yii2 multiple input change log
 2.23.0 (in development)
 =======================
 - Ability to add custom tabindex via options array
+- #335 fix input name in case of one coumn and enabled sorting
 
 2.22.0
 =======================

src/MultipleInputColumn.php

@@ -70,7 +70,11 @@ public function getElementName($index, $withPrefix = true)
      */
     private function isRendererHasOneColumn()
     {
-        return count($this->renderer->columns) === 1; 
+        $columns = \array_filter($this->renderer->columns, function(self $column) {
+            return $column->type !== self::TYPE_DRAGCOLUMN;
+        });
+        
+        return count($columns) === 1;
     }
 
     /**
@@ -85,10 +89,10 @@ protected function getInputNamePrefix()
             if (empty($this->renderer->columns) || ($this->isRendererHasOneColumn() && $this->hasModelAttribute($this->name))) {
                 return $model->formName();
             }
-            
+
             return Html::getInputName($this->context->model, $this->context->attribute);
         }
-        
+
         return $this->context->name;
     }
 
@@ -120,7 +124,7 @@ public function getFirstError($index)
         if ($index === null) {
             return null;
         }
-        
+
         if ($this->isRendererHasOneColumn()) {
             $attribute = $this->name . '[' . $index . ']';
         } else {
@@ -161,12 +165,12 @@ protected function renderWidget($type, $name, $value, $options)
             $options['attribute'] = $attribute;
 
             // Remember current name and mark the widget as embedded to prevent
-            // generation of wrong prefix in case when column is associated with AR relation
+            // generation of wrong prefix in case the column is associated with AR relation
             // @see https://github.com/unclead/yii2-multiple-input/issues/92
             $options['name'] = $name;
             $options['isEmbedded'] = true;
         }
 
         return parent::renderWidget($type, $name, $value, $options);
     }
-}
+}

src/renderers/DivRenderer.php

@@ -16,7 +16,8 @@
 use yii\helpers\UnsetArrayValue;
 
 /**
- * Class DivRenderer is a list renderer who use divs
+ * Class DivRenderer is a list renderer which uses divs
+ * 
  * @package unclead\multipleinput\renderers
  */
 class DivRenderer extends BaseRenderer
@@ -185,9 +186,10 @@ public function renderCellContent($column, $index, $columnIndex = null)
          * via the $options array
          */
         $options = ['id' => $id];
-        if (substr($id, -4) === 'drag') {
+        if ($column->type === BaseColumn::TYPE_DRAGCOLUMN) {
             $options = ArrayHelper::merge($options, ['class' => $this->iconMap['drag-handle']]);
         }
+
         $input = $column->renderInput($name, $options, [
             'id' => $id,
             'name' => $name,

src/renderers/ListRenderer.php

@@ -201,7 +201,7 @@ public function renderCellContent($column, $index, $columnIndex = null)
          * via the $options array
          */
         $options = ['id' => $id];
-        if (substr($id, -4) === 'drag') {
+        if ($column->type === BaseColumn::TYPE_DRAGCOLUMN) {
             $options = ArrayHelper::merge($options, ['class' => $this->iconMap['drag-handle']]);
         }
 

src/renderers/TableRenderer.php

@@ -278,10 +278,11 @@ public function renderCellContent($column, $index, $columnIndex = null)
          * via the $options array
          */
         $options = ['id' => $id];
-        if (substr($id, -4) === 'drag') {
+        if ($column->type === BaseColumn::TYPE_DRAGCOLUMN) {
             $options = ArrayHelper::merge($options, ['class' => $this->iconMap['drag-handle']]);
         }
 
+
         $input = $column->renderInput($name, $options, [
             'id' => $id,
             'name' => $name,