[BUGFIX] Remove trailing or leading comma from CSV list of columns

Resolves: #83514 Releases: master, 8-0, 7-0 Change-Id: If154c64f884376f356238909f671caf27204aff4 Reviewed-on: https://review.typo3.org/55302 Reviewed-by: Jo Hasenau <info@cybercraft.de> Tested-by: Jo Hasenau <info@cybercraft.de>
TYPO3-extensions · Jan 10, 2018 · 0a28d8d · 0a28d8d
1 parent 7a7abec
commit 0a28d8d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/Classes/DataHandler/AfterDatabaseOperations.php b/Classes/DataHandler/AfterDatabaseOperations.php
@@ -380,7 +380,7 @@ public function getAvailableColumns($layout = '', $table = '', $id = 0)
                 }
             }
             // Implode into a CSV string as BackendLayoutView->getColPosListItemsParsed returns an array
-            $tcaColumns = '-2,-1,' . implode(',', $temp);
+            $tcaColumns = rtrim('-2,-1,' . implode(',', $temp), ',');
         }
         return $tcaColumns;
     }

diff --git a/Classes/Plugin/Gridelements.php b/Classes/Plugin/Gridelements.php
@@ -100,7 +100,7 @@ public function main($content = '', $conf = [])
         $layoutSetup->init($this->cObj->data['pid'], $conf);
 
         $availableColumns = $layoutSetup->getLayoutColumns($layout);
-        $csvColumns = str_replace('-2,-1,', '', $availableColumns['CSV']);
+        $csvColumns = ltrim(str_replace('-2,-1', '', $availableColumns['CSV']), ',');
         $this->getChildren($element, $pid, $csvColumns);
 
         // and we have to determine the frontend setup related to the backend layout record which is assigned to this container