refactoring

Author: Alexander Nesterov

src/assets/src/js/jquery.multipleInput.js

@@ -131,7 +131,7 @@
 
             $wrapper.on('click.multipleInput', '.js-input-clone', function (e) {
                 e.stopPropagation();
-                addInput($(this), getRowValues(e));
+                addInput($(this), getRowValues($(this)));
             });
 
             var i = 0,
@@ -435,10 +435,9 @@
             }).length;
     };
 
-    var getRowValues = function (event) {
-        var tr = $(event.currentTarget).closest('tr');
+    var getRowValues = function (element) {
         var values = {};
-        tr.find('td').each(function (index, value) {
+        element.closest('tr').find('td').each(function (index, value) {
             $(value).find('input, select, textarea').each(function (k, v) {
                 var ele = $(v),
                     id = getInputId(ele),

src/renderers/ListRenderer.php

@@ -140,7 +140,10 @@ private function renderRowContent($index = null, $item = null)
             $content[] = $this->renderActionColumn($index);
         }
 
-        $this->cloneButton && $content[] = $this->renderCloneColumn();
+        if ($this->cloneButton) {
+            $content[] = $this->renderCloneColumn();
+        }
+
         $content = Html::tag('tr', implode("\n", $content), $this->prepareRowOptions($index, $item));
 
         if ($index !== null) {

src/renderers/TableRenderer.php

@@ -63,7 +63,9 @@ public function renderHeader()
 
             $cells[] = $this->renderButtonHeaderCell($button);
 
-            $this->cloneButton && $cells[] = $this->renderButtonHeaderCell();
+            if ($this->cloneButton) {
+                $cells[] = $this->renderButtonHeaderCell();
+            }
         }
 
         return Html::tag('thead', Html::tag('tr', implode("\n", $cells)));
@@ -198,7 +200,10 @@ private function renderRowContent($index = null, $item = null)
                 $cells[] = $this->renderCellContent($column, $index);
             }
         }
-        $this->cloneButton && $cells[] = $this->renderCloneColumn();
+        if ($this->cloneButton) {
+            $cells[] = $this->renderCloneColumn();
+        }
+        
         if (!$isLastRow) {
             $cells[] = $this->renderActionColumn($index);
         }