Merge pull request #23 from oliverschroeder/dev

Fixing 2 bugs when exporting backend users (#22)
Tuurlijk · Nov 27, 2017 · 132ec2e · 132ec2e
2 parents 96a1f2d + 5ef8780
commit 132ec2e
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/Classes/Command/ExportCommandController.php b/Classes/Command/ExportCommandController.php
@@ -206,14 +206,17 @@ public function exportTable(
                         continue;
                     }
 
-                    // do not update usergroups by UID when exporting to other systems
-                    // UID maybe diffrent for the same usergroup name
-                    if($table == 'be_users' && $column == 'usergroup' && $value) {
-                        $usergroups = $this->databaseConnection->exec_SELECTgetRows('title', 'be_groups', 'uid IN ('.$value.')');
+                    // Do not update usergroups by UID when exporting to other systems
+                    // UID maybe different for the same usergroup name
+                    if ($table == 'be_users' && $column == 'usergroup' && $value) {
+                        $usergroups = $this->databaseConnection->exec_SELECTgetRows('title', 'be_groups', 'uid IN (' . $value . ')');
+                        // @todo Currently the sorting of usergroups in the original records is ignored when exporting usergroups
+                        $usergroupsTitles = [];
                         foreach ($usergroups as $singleUserGroup) {
                             $usergroupsTitles[] = $singleUserGroup['title'];
                         }
                         $explodedValue = $usergroupsTitles;
+                        $value = $usergroupsTitles[0]; // Overwrite $value for case count() == 1, see below
                     } else {
                         $explodedValue = explode(',', $value);
                     }