mirror/php-e107
1
0
Fork 0
mirror of https://github.com/e107inc/e107.git synced 2025-08-08 23:56:58 +02:00
Code Issues Releases Wiki Activity

Code Climate fix: e_tree_model::multiFieldCmp()

Browse Source 
Reduced cognitive complexity
This commit is contained in:
Deltik
2018-02-23 16:39:52 -06:00
parent 5036dfec1e
commit 80a209cbae
1 changed files with 3 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
e107_handlers/model_class.php
View File

@@ -3480,17 +3480,9 @@ class e_tree_model extends e_front_model
*/
protected static function multiFieldCmp($row1, $row2, $sort_field, $sort_order = 1)
{
// Multiple sort fields
if (is_array($sort_field))
{
$field = array_shift($sort_field);
}
// One sort field
else
{
$field = $sort_field;
$sort_field = [];
}
if (!is_array($sort_field))
$sort_field = [$sort_field];
$field = array_shift($sort_field);
$cmp = strnatcmp((string) $row1[$field], (string) $row2[$field]);
if ($sort_order === -1 || $sort_order === 1) $cmp *= $sort_order;