diff --git a/components/ILIAS/OrgUnit/classes/Positions/UserAssignment/class.ilOrgUnitUserAssignmentDBRepository.php b/components/ILIAS/OrgUnit/classes/Positions/UserAssignment/class.ilOrgUnitUserAssignmentDBRepository.php index 1e345c9f30d5..e016d4d44459 100755 --- a/components/ILIAS/OrgUnit/classes/Positions/UserAssignment/class.ilOrgUnitUserAssignmentDBRepository.php +++ b/components/ILIAS/OrgUnit/classes/Positions/UserAssignment/class.ilOrgUnitUserAssignmentDBRepository.php @@ -1,4 +1,5 @@ getAllChildren($orgu_id); + $recursive_orgu_ids = array_merge($recursive_orgu_ids, $tree->getAllChildren($orgu_id)); } - return $recursive_orgu_ids; } @@ -425,13 +426,13 @@ protected function getUserDataByOrgUnitsAndPosition( ; $res = $this->db->query($query); - if($count_only) { + if ($count_only) { return $this->db->numRows($res); } $users = []; while ($rec = $this->db->fetchAssoc($res)) { - $rec['active'] = (bool)$rec['active']; + $rec['active'] = (bool) $rec['active']; $users[] = $rec; } return $users; @@ -459,7 +460,7 @@ public function getRows( $lp_visible = $additional_parameters['lp_visible_ref_ids']; foreach ($this->getUserDataByOrgUnitsAndPosition($orgu_ids, $position_id, false, $range, $order) as $record) { - $row_id = implode('_', [(string)$position_id, (string)$record['usr_id']]); + $row_id = implode('_', [(string) $position_id, (string) $record['usr_id']]); yield $row_builder->buildDataRow($row_id, $record) ->withDisabledAction( 'show_learning_progress', diff --git a/components/ILIAS/OrgUnit/classes/Positions/class.ilOrgUnitPositionDBRepository.php b/components/ILIAS/OrgUnit/classes/Positions/class.ilOrgUnitPositionDBRepository.php index 578832d7f4e9..3d3edd4f275d 100755 --- a/components/ILIAS/OrgUnit/classes/Positions/class.ilOrgUnitPositionDBRepository.php +++ b/components/ILIAS/OrgUnit/classes/Positions/class.ilOrgUnitPositionDBRepository.php @@ -1,4 +1,5 @@ getAllPositions($range, $order) as $pos) { - $row_id = (string)$pos->getId(); + $row_id = (string) $pos->getId(); $record = [ 'title' => $pos->getTitle(), - 'description' => $pos->getDescription() . 'dd', + 'description' => $pos->getDescription(), 'authorities' => implode("
", $this->getAuthorityDescription($pos->getAuthorities())), 'is_core_position' => $pos->isCorePosition(), ];