columns can not be empty now

Eugene Tupikov · Eugene Tupikov · commit 6f5a24b2e93b · 2021-09-04T19:09:39.000+03:00
diff --git a/src/MultipleInput.php b/src/MultipleInput.php
@@ -222,15 +222,17 @@ class MultipleInput extends InputWidget
      */
     public function init()
     {
+        if (count($this->columns) === 0) {
+            throw new InvalidConfigException('You must specify at least one column');
+        }
+
         if ($this->form !== null && !$this->form instanceof ActiveForm) {
             throw new InvalidConfigException('Property "form" must be an instance of yii\widgets\ActiveForm');
         }
 
         if ($this->showGeneralError && $this->field === null) {
             $this->showGeneralError = false;
         }
-
-        $this->guessColumns();
         $this->initData();
 
         parent::init();
@@ -269,25 +271,6 @@ protected function initData()
         }
     }
 
-    /**
-     * This function tries to guess the columns to show from the given data
-     * if [[columns]] are not explicitly specified.
-     */
-    protected function guessColumns()
-    {
-        if (empty($this->columns)) {
-            $column = [
-                'name' => $this->hasModel() ? $this->attribute : $this->name,
-                'type' => MultipleInputColumn::TYPE_TEXT_INPUT
-            ];
-
-            if ($this->enableGuessTitle && $this->hasModel()) {
-                $column['title'] = $this->model->getAttributeLabel($this->attribute);
-            }
-            $this->columns[] = $column;
-        }
-    }
-
     /**
      * Run widget.
      */
diff --git a/src/TabularInput.php b/src/TabularInput.php
@@ -211,6 +211,10 @@ class TabularInput extends Widget
      */
     public function init()
     {
+        if (count($this->columns) === 0) {
+            throw new InvalidConfigException('You must specify at least one column');
+        }
+
         if (empty($this->models) && !$this->modelClass) {
             throw new InvalidConfigException('You must at least specify "models" or "modelClass"');
         }

Original file line number	Diff line number	Diff line change
`@@ -211,6 +211,10 @@ class TabularInput extends Widget`
`211`	`211`	`*/`
`212`	`212`	`public function init()`
`213`	`213`	`{`
	`214`	`+ if (count($this->columns) === 0) {`
	`215`	`+ throw new InvalidConfigException('You must specify at least one column');`
	`216`	`+ }`
	`217`	`+`
`214`	`218`	`if (empty($this->models) && !$this->modelClass) {`
`215`	`219`	`throw new InvalidConfigException('You must at least specify "models" or "modelClass"');`
`216`	`220`	`}`