Merge pull request #90 from ggoffy/master

fixed bug in MigrateHelper
XoopsModules25x · Aug 18, 2022 · 83456e3 · 83456e3
2 parents d26df1e + 57304c9
commit 83456e3
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/class/Common/MigrateHelper.php b/class/Common/MigrateHelper.php
@@ -212,8 +212,11 @@ private function getColumns (string $line)
         } else {
             return false;
         }
-        $attributes = \trim(\str_replace([$name, '`', ','], '', $line));
 
+        $attributes = \trim(\str_replace([$name, '`'], '', $line));
+        if (',' == \substr($attributes, - 1)) {
+            $attributes = substr($attributes, 0, strlen($attributes) - 1);
+        }
         $columns['name'] = $name;
         // update quotes
         if (\strpos($attributes, "''") > 0) {

diff --git a/docs/changelog.txt b/docs/changelog.txt
@@ -9,6 +9,7 @@
  - fixed bug with download zip (cesagonchu/goffy)
  - adapted update due to new semantic versioning (goffy)
  - updated viewerjs (goffy)
+ - fixed bug in MigrateHelper (goffy)
 
 <h5>1.15 RC2</h5>  [2021/04/19]
 <hr>