Jingga/oms-HumanResourceManagement
1
0
Fork 0
mirror of https://github.com/Karaka-Management/oms-HumanResourceManagement.git synced 2026-01-11 12:28:39 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

correct datamapper self/external usage (invert)

Browse Source
This commit is contained in:
Dennis Eichhorn 2020-10-21 22:50:27 +02:00
parent e153df4ee4
commit b076c12280
2 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
Models/EmployeeHistoryMapper.php
View File

@ -54,19 +54,19 @@ final class EmployeeHistoryMapper extends DataMapperAbstract
protected static array $belongsTo = [
'unit' => [
'mapper' => UnitMapper::class,
'self' => 'hr_staff_history_unit',
'external' => 'hr_staff_history_unit',
],
'department' => [
'mapper' => DepartmentMapper::class,
'self' => 'hr_staff_history_department',
'external' => 'hr_staff_history_department',
],
'position' => [
'mapper' => PositionMapper::class,
'self' => 'hr_staff_history_position',
'external' => 'hr_staff_history_position',
],
'employee' => [
'mapper' => EmployeeMapper::class,
'self' => 'hr_staff_history_staff',
'external' => 'hr_staff_history_staff',
],
];

6
Models/EmployeeMapper.php
View File

@ -51,7 +51,7 @@ final class EmployeeMapper extends DataMapperAbstract
protected static array $belongsTo = [
'profile' => [
'mapper' => ProfileMapper::class,
'self' => 'hr_staff_profile',
'external' => 'hr_staff_profile',
],
];
@ -65,8 +65,8 @@ final class EmployeeMapper extends DataMapperAbstract
'companyHistory' => [
'mapper' => EmployeeHistoryMapper::class,
'table' => 'hr_staff_history', // @todo: is this requried? This is stored in the mapper already. In other places I'm not using this, either use it everywhere or nowhere. Using the mapper is slower but protects us from table name changes!
'external' => 'hr_staff_history_staff',
'self' => null,
'self' => 'hr_staff_history_staff',
'external' => null,
],
];