fix(HumanResources): sort by employe_id dosen´t work

tine-groupware · Jun 4, 2024 · 8056231 · 8056231
1 parent 3166979
commit 8056231
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/tests/tine20/HumanResources/Controller/DailyWTReportTests.php b/tests/tine20/HumanResources/Controller/DailyWTReportTests.php
@@ -87,6 +87,16 @@ public function testCorrectionFilter()
         $result = $result->{HumanResources_Model_MonthlyWTReport::FLDS_CORRECTIONS}->getFirstRecord();
         $this->assertNotNull($result);
         $this->assertSame($monthlyCorrection->getId(), $result->getId());
+
+        $paging = new Tinebase_Model_Pagination(array(
+            'start' => 0,
+            'limit' => 0,
+            'sort' => 'employee_id',
+            'dir' => 'ASC'
+
+        ));
+
+        $result = HumanResources_Controller_MonthlyWTReport::getInstance()->search(null , $paging)->getFirstRecord();
     }
 
     public function testCalculateAllReports()

diff --git a/tine20/HumanResources/Model/Employee.php b/tine20/HumanResources/Model/Employee.php
@@ -48,6 +48,7 @@ class HumanResources_Model_Employee extends Tinebase_Record_Abstract
         self::DELEGATED_ACL_FIELD => 'division_id',
 
         'titleProperty'     => '{{number}} {{n_fn}}',
+        self::DEFAULT_SORT_INFO => [self::FIELD => 'number'],
         'appName'           => 'HumanResources',
         'modelName'         =>  self::MODEL_NAME_PART,