From 56be7177f442f370abeb72f47f7d78796d25bbf6 Mon Sep 17 00:00:00 2001 From: 8ctopus Date: Tue, 10 Dec 2024 12:25:06 +0400 Subject: [PATCH] Cleanup code --- lib/IgraalOSL/StatsTable/StatsTable.php | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/lib/IgraalOSL/StatsTable/StatsTable.php b/lib/IgraalOSL/StatsTable/StatsTable.php index 3b4f65c..5f85d15 100644 --- a/lib/IgraalOSL/StatsTable/StatsTable.php +++ b/lib/IgraalOSL/StatsTable/StatsTable.php @@ -131,7 +131,7 @@ public function removeColumns(array $columns) */ protected function removeColumnsInLine(array &$line, array $columnsMap) { - foreach ($line as $k => $v) { + foreach (array_keys($line) as $k) { if (array_key_exists($k, $columnsMap)) { unset($line[$k]); } @@ -181,13 +181,12 @@ public function sortMultipleColumn($columns) /** * Sort stats table by multiple column with a custom compare function - * @param $columns $columns Associative array : KEY=> column name (string), VALUE=> Custom function (function) + * @param array $columns Associative array : KEY=> column name (string), VALUE=> Custom function (function) * @return $this */ public function uSortMultipleColumn($columns) { - $dataFormats = $this->dataFormats; - $sort = function ($a, $b) use ($columns, $dataFormats) { + $sort = function ($a, $b) use ($columns) { foreach ($columns as $colName => $fn) { $tmp = $fn($a[$colName], $b[$colName]); if ($tmp !== 0) {