more performance using union
This commit is contained in:
parent
eccbf32f68
commit
75945fdbbd
|
@ -76,8 +76,17 @@ function NeededAngeltypes_by_ShiftsFilter(ShiftsFilter $shiftsFilter, $user) {
|
||||||
JOIN `AngelTypes` ON `AngelTypes`.`id`= `NeededAngelTypes`.`angel_type_id`
|
JOIN `AngelTypes` ON `AngelTypes`.`id`= `NeededAngelTypes`.`angel_type_id`
|
||||||
WHERE `Shifts`.`RID` IN (" . implode(',', $shiftsFilter->getRooms()) . ")
|
WHERE `Shifts`.`RID` IN (" . implode(',', $shiftsFilter->getRooms()) . ")
|
||||||
AND `start` BETWEEN " . $shiftsFilter->getStartTime() . " AND " . $shiftsFilter->getEndTime() . "
|
AND `start` BETWEEN " . $shiftsFilter->getStartTime() . " AND " . $shiftsFilter->getEndTime() . "
|
||||||
ORDER BY `Shifts`.`start`";
|
AND `Shifts`.`PSID` IS NULL
|
||||||
// FIXME: Use needed angeltypes on rooms!
|
|
||||||
|
UNION
|
||||||
|
|
||||||
|
SELECT `NeededAngelTypes`.*, `AngelTypes`.`id`, `AngelTypes`.`name`, `AngelTypes`.`restricted`, `AngelTypes`.`no_self_signup`
|
||||||
|
FROM `Shifts`
|
||||||
|
JOIN `NeededAngelTypes` ON `NeededAngelTypes`.`room_id`=`Shifts`.`RID`
|
||||||
|
JOIN `AngelTypes` ON `AngelTypes`.`id`= `NeededAngelTypes`.`angel_type_id`
|
||||||
|
WHERE `Shifts`.`RID` IN (" . implode(',', $shiftsFilter->getRooms()) . ")
|
||||||
|
AND `start` BETWEEN " . $shiftsFilter->getStartTime() . " AND " . $shiftsFilter->getEndTime() . "
|
||||||
|
AND NOT `Shifts`.`PSID` IS NULL";
|
||||||
$result = sql_select($SQL);
|
$result = sql_select($SQL);
|
||||||
if ($result === false) {
|
if ($result === false) {
|
||||||
engelsystem_error("Unable to load needed angeltypes by filter.");
|
engelsystem_error("Unable to load needed angeltypes by filter.");
|
||||||
|
@ -92,22 +101,20 @@ function NeededAngeltype_by_Shift_and_Angeltype($shift, $angeltype) {
|
||||||
JOIN `AngelTypes` ON `AngelTypes`.`id`= `NeededAngelTypes`.`angel_type_id`
|
JOIN `AngelTypes` ON `AngelTypes`.`id`= `NeededAngelTypes`.`angel_type_id`
|
||||||
WHERE `Shifts`.`SID`=" . sql_escape($shift['SID']) . "
|
WHERE `Shifts`.`SID`=" . sql_escape($shift['SID']) . "
|
||||||
AND `AngelTypes`.`id`=" . sql_escape($angeltype['id']) . "
|
AND `AngelTypes`.`id`=" . sql_escape($angeltype['id']) . "
|
||||||
ORDER BY `Shifts`.`start`");
|
AND `Shifts`.`PSID` IS NULL
|
||||||
if ($result === false) {
|
|
||||||
engelsystem_error("Unable to load needed angeltypes by filter.");
|
UNION
|
||||||
}
|
|
||||||
if (count($result) == 0) {
|
SELECT `NeededAngelTypes`.*, `AngelTypes`.`id`, `AngelTypes`.`name`, `AngelTypes`.`restricted`, `AngelTypes`.`no_self_signup`
|
||||||
$result = sql_select("SELECT `NeededAngelTypes`.*, `AngelTypes`.`id`, `AngelTypes`.`name`, `AngelTypes`.`restricted`, `AngelTypes`.`no_self_signup`
|
|
||||||
FROM `Shifts`
|
FROM `Shifts`
|
||||||
JOIN `NeededAngelTypes` ON `NeededAngelTypes`.`room_id`=`Shifts`.`RID`
|
JOIN `NeededAngelTypes` ON `NeededAngelTypes`.`room_id`=`Shifts`.`RID`
|
||||||
JOIN `AngelTypes` ON `AngelTypes`.`id`= `NeededAngelTypes`.`angel_type_id`
|
JOIN `AngelTypes` ON `AngelTypes`.`id`= `NeededAngelTypes`.`angel_type_id`
|
||||||
WHERE `Shifts`.`SID`=" . sql_escape($shift['SID']) . "
|
WHERE `Shifts`.`SID`=" . sql_escape($shift['SID']) . "
|
||||||
AND `AngelTypes`.`id`=" . sql_escape($angeltype['id']) . "
|
AND `AngelTypes`.`id`=" . sql_escape($angeltype['id']) . "
|
||||||
ORDER BY `Shifts`.`start`");
|
AND NOT `Shifts`.`PSID` IS NULL");
|
||||||
if ($result === false) {
|
if ($result === false) {
|
||||||
engelsystem_error("Unable to load needed angeltypes by filter.");
|
engelsystem_error("Unable to load needed angeltypes by filter.");
|
||||||
}
|
}
|
||||||
}
|
|
||||||
if(count($result) == 0) {
|
if(count($result) == 0) {
|
||||||
return null;
|
return null;
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue