From a6eb3d4378c5eb969eb3bd29abf0f2b4eb53feb9 Mon Sep 17 00:00:00 2001 From: Nik Okuntseff Date: Sun, 3 Mar 2019 15:11:20 +0000 Subject: [PATCH] Prohibited adding overlapping timesheets. --- WEB-INF/lib/ttTimesheetHelper.class.php | 40 +++++++++++++++++++++++++ WEB-INF/templates/footer.tpl | 2 +- timesheet_add.php | 1 + 3 files changed, 42 insertions(+), 1 deletion(-) diff --git a/WEB-INF/lib/ttTimesheetHelper.class.php b/WEB-INF/lib/ttTimesheetHelper.class.php index 1fff1955..f214963b 100644 --- a/WEB-INF/lib/ttTimesheetHelper.class.php +++ b/WEB-INF/lib/ttTimesheetHelper.class.php @@ -374,4 +374,44 @@ class ttTimesheetHelper { return false; } + + // The overlaps function determines if a new timesheet overlaps with + // an already existing timesheet. + static function overlaps($fields) { + global $user; + $mdb2 = getConnection(); + + $user_id = $user->getUser(); + $group_id = $user->getGroup(); + $org_id = $user->org_id; + + if (isset($fields['client_id'])) $client_id = (int) $fields['client_id']; + if (isset($fields['project_id'])) $project_id = (int) $fields['project_id']; + + $start_date = new DateAndTime($user->date_format, $fields['start_date']); + $start = $start_date->toString(DB_DATEFORMAT); + $quoted_start = $mdb2->quote($start); + + $end_date = new DateAndTime($user->date_format, $fields['end_date']); + $end = $end_date->toString(DB_DATEFORMAT); + $quoted_end = $mdb2->quote($end); + + // sql parts. + if ($client_id) $client_part = " and client_id = $client_id"; + if ($project_id) $project_part = " and project_id = $project_id"; + + $sql = "select id from tt_timesheets". + " where status is not null $client_part $project_part". + " and (($quoted_start >= start_date and $quoted_start <= end_date)". + " or ($quoted_end >= start_date and $quoted_end <= end_date))". + " and user_id = $user_id and group_id = $group_id and org_id = $org_id"; + $res = $mdb2->query($sql); + if (!is_a($res, 'PEAR_Error')) { + $val = $res->fetchRow(); + if ($val['id']) { + return true; + } + } + return false; + } } diff --git a/WEB-INF/templates/footer.tpl b/WEB-INF/templates/footer.tpl index 66e0203b..1a9296dc 100644 --- a/WEB-INF/templates/footer.tpl +++ b/WEB-INF/templates/footer.tpl @@ -12,7 +12,7 @@
-
 Anuko Time Tracker 1.18.52.4813 | Copyright © Anuko | +  Anuko Time Tracker 1.18.52.4814 | Copyright © Anuko | {$i18n.footer.credits} | {$i18n.footer.license} | {$i18n.footer.improve} diff --git a/timesheet_add.php b/timesheet_add.php index 09a2d384..7c7633c6 100644 --- a/timesheet_add.php +++ b/timesheet_add.php @@ -87,6 +87,7 @@ if ($request->isPost()) { 'end_date' => $cl_finish, 'comment' => $cl_comment); if ($err->no() && !ttTimesheetHelper::timesheetItemsExist($fields)) $err->add($i18n->get('error.no_records')); + if ($err->no() && ttTimesheetHelper::overlaps($fields)) $err->add($i18n->get('error.overlap')); // Finished validating user input. if ($err->no()) { -- 2.20.1