TabularInput: now new row does't copy values from the most recent row

Author: Eugene Tupikov

CHANGELOG.md

@@ -4,6 +4,8 @@ Yii2 multiple input change log
 1.2.4 in development
 --------------------
 
+- Bug #39: TabularInput: now new row does't copy values from  the most recent row
+
 1.2.3
 -----
 

examples/actions/TabularInputAction.php

@@ -25,7 +25,8 @@ public function run()
     {
         Yii::setAlias('@unclead-examples', realpath(__DIR__ . '/../'));
 
-        $models = [new Item()];
+        $models = $this->getItems();
+
         $request = Yii::$app->getRequest();
         if ($request->isPost && $request->post('ajax') !== null) {
             $data = Yii::$app->request->post('Item', []);
@@ -42,7 +43,30 @@ public function run()
             // your magic
         }
 
-
         return $this->controller->render('@unclead-examples/views/tabular-input.php', ['models' => $models]);
     }
+
+    private function getItems()
+    {
+        $data = [
+            [
+                'id' => 1,
+                'title' => 'Title 1',
+                'description' => 'Description 1'
+            ],
+            [
+                'id' => 2,
+                'title' => 'Title 2',
+                'description' => 'Description 2'
+            ],
+        ];
+
+        $items = [];
+        foreach ($data as $row) {
+            $item = new Item();
+            $item->setAttributes($row);
+            $items[] = $item;
+        }
+        return $items;
+    }
 }

examples/models/Item.php

@@ -10,15 +10,14 @@
 
 use Yii;
 use yii\base\Model;
-// you have to install https://github.com/vova07/yii2-fileapi-widget
-use vova07\fileapi\behaviors\UploadBehavior;
 
 /**
  * Class Item
  * @package unclead\widgets\examples\models
  */
 class Item extends Model
 {
+    public $id;
 
     public $title;
 
@@ -32,24 +31,26 @@ class Item extends Model
     public function behaviors()
     {
         return [
+            // you have to install https://github.com/vova07/yii2-fileapi-widget
+            /*
             'uploadBehavior' => [
-                'class' => UploadBehavior::className(),
+                'class' => \vova07\fileapi\behaviors\UploadBehavior::className(),
                 'attributes' => [
                     'file' => [
                         'path' => Yii::getAlias('@webroot') . '/images/',
                         'tempPath' => Yii::getAlias('@webroot') . '/images/tmp/',
                         'url' => '/images/'
                     ],
                 ]
-            ]
+            ]*/
         ];
     }
 
     public function rules()
     {
         return [
             [['title', 'description'], 'required'],
-            ['file', 'safe']
+            [['id', 'file'], 'safe']
         ];
     }
 

examples/views/multiple-input.php

@@ -21,19 +21,18 @@
     'validateOnBlur'            => false,
 ]);?>
 
-    <h3>Single column</h3>
+<h3>Single column</h3>
 <?php
     echo $form->field($model, 'emails')->widget(MultipleInput::className(), [
-        'limit' => 5,
-        'allowEmptyList' => true,
-        'enableGuessTitle' => true
+        'limit'             => 5,
+        'allowEmptyList'    => true,
+        'enableGuessTitle'  => true
     ])
     ->label(false);
 ?>
 
 <h3>Multiple columns</h3>
 <?= $form->field($model, 'schedule')->widget(MultipleInput::className(), [
-    'id' => 'schedule-wrapper',
     'limit' => 4,
     'allowEmptyList' => true,
     'columns' => [
@@ -132,13 +131,14 @@
 
 <?php
 $js = <<< JS
-        $('#schedule-wrapper').on('afterInit', function(){
+        $('#examplemodel-schedule').on('afterInit', function(){
             console.log('calls on after initialization event');
         }).on('beforeAddRow', function(e) {
             console.log('calls on before add row event');
         }).on('afterAddRow', function(e) {
             console.log('calls on after add row event');
-        }).on('beforeDeleteRow', function(){
+        }).on('beforeDeleteRow', function(e, item){
+            console.log(item);
             console.log('calls on before remove row event');
             return confirm('Are you sure you want to delete row?')
         }).on('afterDeleteRow', function(){

examples/views/tabular-input.php

@@ -31,6 +31,10 @@
         'validateOnBlur'            => false,
     ],
     'columns' => [
+        [
+            'name' => 'id',
+            'type' => \unclead\widgets\TabularColumn::TYPE_HIDDEN_INPUT
+        ],
         [
             'name'  => 'title',
             'title' => 'Title',
@@ -40,16 +44,16 @@
             'name'  => 'description',
             'title' => 'Description',
         ],
-        [
-            'name'  => 'file',
-            'title' => 'File',
-            'type'  => \vova07\fileapi\Widget::className(),
-            'options' => [
-                'settings' => [
-                    'url' => ['site/fileapi-upload']
-                ]
-            ]
-        ],
+//        [
+//            'name'  => 'file',
+//            'title' => 'File',
+//            'type'  => \vova07\fileapi\Widget::className(),
+//            'options' => [
+//                'settings' => [
+//                    'url' => ['site/fileapi-upload']
+//                ]
+//            ]
+//        ],
         [
             'name'  => 'date',
             'type'  => \kartik\date\DatePicker::className(),

src/TabularColumn.php

@@ -55,4 +55,24 @@ protected function ensureModel($model)
     {
         return $model instanceof Model;
     }
+
+    /**
+     * @inheritdoc
+     */
+    public function setModel($model)
+    {
+        $currentModel = $this->getModel();
+
+        // If model is null and current model is not empty it means that widget renders a template
+        // In this case we have to unset all model attributes
+        if ($model === null && $currentModel !== null) {
+            foreach ($currentModel->attributes() as $attribute) {
+                $currentModel->$attribute = null;
+            }
+        } else {
+            parent::setModel($model);
+        }
+    }
+
+
 }

src/components/BaseColumn.php

@@ -171,14 +171,13 @@ public function isHiddenInput()
     protected function prepareValue()
     {
         $data = $this->getModel();
-        if ($this->value !== null) {
-            $value = $this->value;
-            if ($value instanceof \Closure) {
-                $value = call_user_func($value, $data);
-            }
+        if ($this->value instanceof \Closure) {
+            $value = call_user_func($this->value, $data);
         } else {
-            if ($data instanceof ActiveRecord) {
+            if ($data instanceof ActiveRecord ) {
                 $value = $data->getAttribute($this->name);
+            } elseif ($data instanceof Model) {
+                $value = $data->{$this->name};
             } elseif (is_array($data)) {
                 $value = ArrayHelper::getValue($data, $this->name, null);
             } elseif(is_string($data)) {