summaryrefslogtreecommitdiff
path: root/includes/model
diff options
context:
space:
mode:
authormsquare <msquare@notrademark.de>2016-11-12 12:50:52 +0100
committermsquare <msquare@notrademark.de>2016-11-12 12:50:52 +0100
commitc10ab2d8d2d2b85925768b93f2847ffd3de72e4a (patch)
treec78185ec8bcb28bb22db16cf52d7686070ee7008 /includes/model
parent5f2871538a44e5f878ee99316322c5fe4ffb4f36 (diff)
reduce complexity of shift signup allowed
Diffstat (limited to 'includes/model')
-rw-r--r--includes/model/Shifts_model.php87
1 files changed, 22 insertions, 65 deletions
diff --git a/includes/model/Shifts_model.php b/includes/model/Shifts_model.php
index f232360e..aab822b7 100644
--- a/includes/model/Shifts_model.php
+++ b/includes/model/Shifts_model.php
@@ -90,6 +90,26 @@ function Shift_collides($shift, $shifts) {
}
/**
+ * Returns true if a shift has no more free slots for angels.
+ *
+ * @param int $shift_id
+ * ID of the shift to check
+ */
+function Shift_occupied($shift_id) {
+ $needed_angeltypes = NeededAngelTypes_by_shift($shift['SID']);
+
+ foreach ($needed_angeltypes as $needed_angeltype) {
+ if ($needed_angeltype['angel_type_id'] == $angeltype['id']) {
+ if ($needed_angeltype['taken'] < $needed_angeltype['count']) {
+ return false;
+ }
+ }
+ }
+
+ return true;
+}
+
+/**
* Check if an angel can sign up for given shift.
*
* @param Shift $shift
@@ -103,8 +123,6 @@ function Shift_signup_allowed($shift, $angeltype, $user_angeltype = null, $user_
$user_shifts = Shifts_by_user($user);
}
- $collides = Shift_collides($shift, $user_shifts);
-
if ($user_angeltype == null) {
$user_angeltype = UserAngelType_by_User_and_AngelType($user, $angeltype);
}
@@ -117,26 +135,11 @@ function Shift_signup_allowed($shift, $angeltype, $user_angeltype = null, $user_
}
}
- $needed_angeltypes = NeededAngelTypes_by_shift($shift['SID']);
- if ($needed_angeltypes === false) {
- engelsystem_error('Unable to load needed angel types.');
- }
-
- // is the shift still running or alternatively is the user shift admin?
- $user_may_join_shift = true;
-
// you canot join if shift is full
- foreach ($needed_angeltypes as $needed_angeltype) {
- if ($needed_angeltype['angel_type_id'] == $angeltype['id']) {
- if ($needed_angeltype['taken'] >= $needed_angeltype['count']) {
- $user_may_join_shift = false;
- }
- break;
- }
- }
+ $user_may_join_shift = ! Shift_occupied($shift['SID']);
// you cannot join if user alread joined a parallel or this shift
- $user_may_join_shift &= ! $collides;
+ $user_may_join_shift &= ! Shift_collides($shift, $user_shifts);
// you cannot join if you already singed up for this shift
$user_may_join_shift &= ! $signed_up;
@@ -257,52 +260,6 @@ function Shifts_by_user($user) {
}
/**
- * TODO: $_REQUEST is not allowed in model!
- * Returns Shift id array
- */
-function Shifts_filtered() {
- global $_REQUEST;
- $filter = "";
-
- // filterRoom (Array of integer) - Array of Room IDs (optional, for list request)
- if (isset($_REQUEST['filterRoom']) && is_array($_REQUEST['filterRoom'])) {
- foreach ($_REQUEST['filterRoom'] as $key => $value) {
- $filter .= ", `RID`='" . sql_escape($value) . "' ";
- }
- }
-
- // filterTask (Array of integer) - Array if Task (optional, for list request)
- if (isset($_REQUEST['filterTask']) && is_array($_REQUEST['filterTask'])) {
- foreach ($_REQUEST['filterTask'] as $key => $value) {
- // TODO $filter .= ", `RID`=" . sql_escape($value) . " ";
- }
- }
-
- // filterOccupancy (integer) - Occupancy state: (optional, for list request)
- // 1 occupied, 2 free, 3 occupied and free
- if (isset($_REQUEST['filterOccupancy']) && is_array($_REQUEST['filterOccupancy'])) {
- foreach ($_REQUEST['filterOccupancy'] as $key => $value) {
- // TODO $filter .= ", `RID`=" . sql_escape($value) . " ";
- }
- }
-
- // format filter
- if ($filter != "") {
- $filter = ' WHERE ' . substr($filter, 1);
- }
-
- // real request
- $shifts_source = sql_select("SELECT `SID` FROM `Shifts`" . $filter);
- if ($shifts_source === false) {
- return false;
- }
- if (count($shifts_source) > 0) {
- return $shifts_source;
- }
- return null;
-}
-
-/**
* Returns Shift by id.
*
* @param $shift_id Shift