mirror of
https://github.com/grocy/grocy.git
synced 2026-04-06 21:06:15 +02:00
fix chores service
This commit is contained in:
parent
5b214dbf27
commit
90c00680fa
|
|
@ -29,7 +29,7 @@ class ChoresService extends BaseService
|
||||||
public function GetCurrent()
|
public function GetCurrent()
|
||||||
{
|
{
|
||||||
$sql = 'SELECT * from chores_current';
|
$sql = 'SELECT * from chores_current';
|
||||||
return $this->DatabaseService->ExecuteDbQuery($sql)->fetchAll(\PDO::FETCH_OBJ);
|
return $this->getDatabaseService()->ExecuteDbQuery($sql)->fetchAll(\PDO::FETCH_OBJ);
|
||||||
}
|
}
|
||||||
|
|
||||||
public function GetChoreDetails(int $choreId)
|
public function GetChoreDetails(int $choreId)
|
||||||
|
|
@ -42,12 +42,12 @@ class ChoresService extends BaseService
|
||||||
$usersService = new UsersService();
|
$usersService = new UsersService();
|
||||||
$users = $usersService->GetUsersAsDto();
|
$users = $usersService->GetUsersAsDto();
|
||||||
|
|
||||||
$chore = $this->Database->chores($choreId);
|
$chore = $this->getDatabase()->chores($choreId);
|
||||||
$choreTrackedCount = $this->Database->chores_log()->where('chore_id = :1 AND undone = 0', $choreId)->count();
|
$choreTrackedCount = $this->getDatabase()->chores_log()->where('chore_id = :1 AND undone = 0', $choreId)->count();
|
||||||
$choreLastTrackedTime = $this->Database->chores_log()->where('chore_id = :1 AND undone = 0', $choreId)->max('tracked_time');
|
$choreLastTrackedTime = $this->getDatabase()->chores_log()->where('chore_id = :1 AND undone = 0', $choreId)->max('tracked_time');
|
||||||
$nextExecutionTime = $this->Database->chores_current()->where('chore_id', $choreId)->min('next_estimated_execution_time');
|
$nextExecutionTime = $this->getDatabase()->chores_current()->where('chore_id', $choreId)->min('next_estimated_execution_time');
|
||||||
|
|
||||||
$lastChoreLogRow = $this->Database->chores_log()->where('chore_id = :1 AND tracked_time = :2 AND undone = 0', $choreId, $choreLastTrackedTime)->fetch();
|
$lastChoreLogRow = $this->getDatabase()->chores_log()->where('chore_id = :1 AND tracked_time = :2 AND undone = 0', $choreId, $choreLastTrackedTime)->fetch();
|
||||||
$lastDoneByUser = null;
|
$lastDoneByUser = null;
|
||||||
if ($lastChoreLogRow !== null && !empty($lastChoreLogRow))
|
if ($lastChoreLogRow !== null && !empty($lastChoreLogRow))
|
||||||
{
|
{
|
||||||
|
|
@ -77,25 +77,25 @@ class ChoresService extends BaseService
|
||||||
throw new \Exception('Chore does not exist');
|
throw new \Exception('Chore does not exist');
|
||||||
}
|
}
|
||||||
|
|
||||||
$userRow = $this->Database->users()->where('id = :1', $doneBy)->fetch();
|
$userRow = $this->getDatabase()->users()->where('id = :1', $doneBy)->fetch();
|
||||||
if ($userRow === null)
|
if ($userRow === null)
|
||||||
{
|
{
|
||||||
throw new \Exception('User does not exist');
|
throw new \Exception('User does not exist');
|
||||||
}
|
}
|
||||||
|
|
||||||
$chore = $this->Database->chores($choreId);
|
$chore = $this->getDatabase()->chores($choreId);
|
||||||
if ($chore->track_date_only == 1)
|
if ($chore->track_date_only == 1)
|
||||||
{
|
{
|
||||||
$trackedTime = substr($trackedTime, 0, 10) . ' 00:00:00';
|
$trackedTime = substr($trackedTime, 0, 10) . ' 00:00:00';
|
||||||
}
|
}
|
||||||
|
|
||||||
$logRow = $this->Database->chores_log()->createRow(array(
|
$logRow = $this->getDatabase()->chores_log()->createRow(array(
|
||||||
'chore_id' => $choreId,
|
'chore_id' => $choreId,
|
||||||
'tracked_time' => $trackedTime,
|
'tracked_time' => $trackedTime,
|
||||||
'done_by_user_id' => $doneBy
|
'done_by_user_id' => $doneBy
|
||||||
));
|
));
|
||||||
$logRow->save();
|
$logRow->save();
|
||||||
$lastInsertId = $this->Database->lastInsertId();
|
$lastInsertId = $this->getDatabase()->lastInsertId();
|
||||||
|
|
||||||
$this->CalculateNextExecutionAssignment($choreId);
|
$this->CalculateNextExecutionAssignment($choreId);
|
||||||
|
|
||||||
|
|
@ -109,13 +109,13 @@ class ChoresService extends BaseService
|
||||||
|
|
||||||
private function ChoreExists($choreId)
|
private function ChoreExists($choreId)
|
||||||
{
|
{
|
||||||
$choreRow = $this->Database->chores()->where('id = :1', $choreId)->fetch();
|
$choreRow = $this->getDatabase()->chores()->where('id = :1', $choreId)->fetch();
|
||||||
return $choreRow !== null;
|
return $choreRow !== null;
|
||||||
}
|
}
|
||||||
|
|
||||||
public function UndoChoreExecution($executionId)
|
public function UndoChoreExecution($executionId)
|
||||||
{
|
{
|
||||||
$logRow = $this->Database->chores_log()->where('id = :1 AND undone = 0', $executionId)->fetch();
|
$logRow = $this->getDatabase()->chores_log()->where('id = :1 AND undone = 0', $executionId)->fetch();
|
||||||
if ($logRow == null)
|
if ($logRow == null)
|
||||||
{
|
{
|
||||||
throw new \Exception('Execution does not exist or was already undone');
|
throw new \Exception('Execution does not exist or was already undone');
|
||||||
|
|
@ -135,11 +135,11 @@ class ChoresService extends BaseService
|
||||||
throw new \Exception('Chore does not exist');
|
throw new \Exception('Chore does not exist');
|
||||||
}
|
}
|
||||||
|
|
||||||
$chore = $this->Database->chores($choreId);
|
$chore = $this->getDatabase()->chores($choreId);
|
||||||
$choreLastTrackedTime = $this->Database->chores_log()->where('chore_id = :1 AND undone = 0', $choreId)->max('tracked_time');
|
$choreLastTrackedTime = $this->getDatabase()->chores_log()->where('chore_id = :1 AND undone = 0', $choreId)->max('tracked_time');
|
||||||
$lastChoreLogRow = $this->Database->chores_log()->where('chore_id = :1 AND tracked_time = :2 AND undone = 0', $choreId, $choreLastTrackedTime)->fetch();
|
$lastChoreLogRow = $this->getDatabase()->chores_log()->where('chore_id = :1 AND tracked_time = :2 AND undone = 0', $choreId, $choreLastTrackedTime)->fetch();
|
||||||
$lastDoneByUserId = $lastChoreLogRow->done_by_user_id;
|
$lastDoneByUserId = $lastChoreLogRow->done_by_user_id;
|
||||||
|
|
||||||
$usersService = new UsersService();
|
$usersService = new UsersService();
|
||||||
$users = $usersService->GetUsersAsDto();
|
$users = $usersService->GetUsersAsDto();
|
||||||
$assignedUsers = array();
|
$assignedUsers = array();
|
||||||
|
|
@ -198,7 +198,7 @@ class ChoresService extends BaseService
|
||||||
}
|
}
|
||||||
else if ($chore->assignment_type == self::CHORE_ASSIGNMENT_TYPE_WHO_LEAST_DID_FIRST)
|
else if ($chore->assignment_type == self::CHORE_ASSIGNMENT_TYPE_WHO_LEAST_DID_FIRST)
|
||||||
{
|
{
|
||||||
$row = $this->Database->chores_execution_users_statistics()->where('chore_id = :1', $choreId)->orderBy('execution_count')->limit(1)->fetch();
|
$row = $this->getDatabase()->chores_execution_users_statistics()->where('chore_id = :1', $choreId)->orderBy('execution_count')->limit(1)->fetch();
|
||||||
if ($row != null)
|
if ($row != null)
|
||||||
{
|
{
|
||||||
$nextExecutionUserId = $row->user_id;
|
$nextExecutionUserId = $row->user_id;
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue
Block a user