diff --git a/application/controllers/jobs/ZeiterfassungInfoJob.php b/application/controllers/jobs/ZeiterfassungInfoJob.php index 82254d40f..e99e1306a 100644 --- a/application/controllers/jobs/ZeiterfassungInfoJob.php +++ b/application/controllers/jobs/ZeiterfassungInfoJob.php @@ -54,8 +54,6 @@ class ZeiterfassungInfoJob extends JOB_Controller public function sendMail() { $allMitarbeiter = $this->_getEmplyeeUids(); - //var_dump($allMitarbeiter); - //$allMitarbeiter = ['ma0068','ma0080']; $vorgesetzte_to_approve_vacation = $this->_getVorgesetztetoApproveVacationList(); $vorgesetzte_to_approve_timesheets = $this->_getVorgesetztetoApproveTimesheetList(); @@ -63,7 +61,7 @@ class ZeiterfassungInfoJob extends JOB_Controller $mitarbeiter_to_send_timesheet_lastmonth = $this->_getEmployeeTimesheetList(); $mitarbeiter_to_record_times_lastweek = $this->_getEmployeeLastWeeksTimeList(); - //$mitarbeiter_without_zeitmodell = $this->_filterMitarbeiter(); + $mitarbeiter_without_zeitmodell = $this->_filterMitarbeiter(); $cnt_sup_to_approve_vacation = 0; $cnt_sup_to_approve_timesheets = 0; @@ -113,16 +111,15 @@ class ZeiterfassungInfoJob extends JOB_Controller { $ma->EmpWeek = false; } - //fehlendes Zeitmodell?? - // if(array_key_exists($uid, $mitarbeiter_without_zeitmodell)) - // { - // $ma->EmpZeitMod = true; - // $cnt_ma_without_zeitmodell++; - // } - // else - // { - // $ma->EmpZeitMod = false; - // } + if(array_key_exists($uid, $mitarbeiter_without_zeitmodell)) + { + $ma->EmpZeitMod = true; + $cnt_ma_without_zeitmodell++; + } + else + { + $ma->EmpZeitMod = false; + } if($ma->SupVac || $ma->SupMonth || $ma->EmpMonth || $ma->EmpWeek || $ma->EmpZeitMod) { @@ -229,7 +226,6 @@ class ZeiterfassungInfoJob extends JOB_Controller $vorgesetzte [] = getData($this->MitarbeiterModel->getVorgesetzte($mitarbeiter_uid)); } - //var_dump($vorgesetzte); Unittest URLAUB foreach ($vorgesetzte as $v) { if(!(is_null($v))) @@ -316,7 +312,6 @@ class ZeiterfassungInfoJob extends JOB_Controller } - //var_dump($names); //unittest MONATSLISTE abschicken return $names; } @@ -344,7 +339,6 @@ class ZeiterfassungInfoJob extends JOB_Controller $uids[$uid] = $uid; } } - //var_dump($uids); //unittest ZEITEN last week return $uids; } @@ -370,31 +364,28 @@ class ZeiterfassungInfoJob extends JOB_Controller return $mitarbeiterUIDs; } + private function _filterMitarbeiter() { - $mResult = $this->TimesheetModel->getAllMissingZeitmodelle(); - $mitarbeiterList = getData($mResult); - $mitarbeiterWithoutZeitmodell = array(); + $mitarbeiter = $this->MitarbeiterModel->getPersonal(true,null,true)->retval; + $mResult = $this->TimesheetModel->getAllMissingZeitmodelle(); + $mResult = $mResult[1]; + $mitarbeiterWithoutZeitmodell = array(); + $uids = array(); + foreach ($mResult as $ma) + { + array_push($uids, strtolower($ma[0])); + } - foreach ($mitarbeiterList as $mitarbeiter) - { - $uid = $mitarbeiter->uid; - if($this->MitarbeiterModel->isMitarbeiter($uid)) - { - $mitarbeiterWithoutZeitmodell [$uid] = $uid; - - } - - - } - - // foreach ($mitarbeiter as $ma) - // { - // print_r - // } - return $mitarbeiterWithoutZeitmodell; + foreach ($mitarbeiter as $ma) + { + $uid = $ma->uid; + if(!in_array($uid,$uids)) + { + $mitarbeiterWithoutZeitmodell [$uid] = $uid; + } + } + return $mitarbeiterWithoutZeitmodell; } - - } diff --git a/application/models/ressource/Mitarbeiter_model.php b/application/models/ressource/Mitarbeiter_model.php index 7a5ad525c..fc8378988 100644 --- a/application/models/ressource/Mitarbeiter_model.php +++ b/application/models/ressource/Mitarbeiter_model.php @@ -206,11 +206,6 @@ class Mitarbeiter_model extends DB_Model public function getEmployeesZeitaufzeichnungspflichtig() { $qry = "SELECT DISTINCT mitarbeiter_uid FROM bis.tbl_bisverwendung WHERE beginn <= now() and (ende >= now() OR ende is NULL) AND zeitaufzeichnungspflichtig is TRUE - --AND mitarbeiter_uid in ('sabo','wala','ma0080') --Urlaubsfreigabe - --AND mitarbeiter_uid in ('markl','ma0080') -- Monatlistenzu bestätigen - --AND mitarbeiter_uid in ('hrisenko','kiciak','zenzi','holnthon','ma0068') -- Monatlisten abzuschicken - --AND mitarbeiter_uid in ('ma0068','felleitn','wala','ma0080','ma002') --ZEITEN - AND mitarbeiter_uid in ('ma0068') --kein Zeitmodell ORDER BY mitarbeiter_uid"; return $this->execQuery($qry); } diff --git a/application/models/ressource/Timesheet_model.php b/application/models/ressource/Timesheet_model.php index e685dcab6..419b7f7fb 100644 --- a/application/models/ressource/Timesheet_model.php +++ b/application/models/ressource/Timesheet_model.php @@ -53,7 +53,6 @@ class Timesheet_model extends DB_Model if(curl_errno($ch)) { - //print_r($ch); return 'Curl error: ' . curl_error($ch); curl_close($ch); } @@ -64,12 +63,10 @@ class Timesheet_model extends DB_Model if(isset($data->STATUS) && $data->STATUS=='OK') { - // print_r($data[2]); return $data->RESULT; } else - //print_r($data); - return $data; + return $data; } } }