diff --git a/controllers/message.php b/controllers/message.php index 5f0f39dc14707950e12b9ac6365aa918b304ab69..936ad7d7601d641f00e84963130289dc2059e0f5 100644 --- a/controllers/message.php +++ b/controllers/message.php @@ -767,7 +767,7 @@ class MessageController extends AuthenticatedController { dgettext('garuda', 'Ihre Nachricht an %u Personen wurde gespeichert.'), count($this->flash['users']))); } else { - PageLayout::postSuccess(sprintf( + PageLayout::postError(sprintf( dgettext('garuda', 'Ihre Nachricht an %u Personen konnte nicht gespeichert werden.'), count($this->flash['users']))); } diff --git a/models/GarudaModel.php b/models/GarudaModel.php index 77b64d8ce26fda641b1a890900bd77ce1895cc0a..06a7dac4b3144805d4804cf2c303d507d7b5e88b 100644 --- a/models/GarudaModel.php +++ b/models/GarudaModel.php @@ -256,7 +256,7 @@ class GarudaModel { JOIN `auth_user_md5` a USING (`user_id`) WHERE s.`fach_id` NOT IN (:ignored) AND a.`visible` != 'never'", - ['ignored' => Config::get()->GARUDA_IGNORE_SUBJECT_IDS] + ['ignored' => Config::get()->GARUDA_IGNORE_SUBJECT_IDS ?: ['']] ); } else { $query = "SELECT DISTINCT `user_id` FROM `user_studiengang`"; @@ -273,15 +273,13 @@ class GarudaModel { $parameters[] = $entry['fach_id']; } $where .= " AND `fach_id` NOT IN (?)"; - $parameters[] = Config::get()->GARUDA_IGNORE_SUBJECT_IDS; + $parameters[] = Config::get()->GARUDA_IGNORE_SUBJECT_IDS ?: ['']; $query .= $where; return DBManager::get()->fetchFirst($query, $parameters); } else { return []; } - return DBManager::get()->fetchFirst($query, $parameters); } - return array(); } public static function getAllEmployees($userId, $config) diff --git a/plugin.manifest b/plugin.manifest index 4e25c7a9cd9794977ebe4cd7676943a6251188ba..31e81755e3a116d4bb1e326469557f81a9a94fe6 100644 --- a/plugin.manifest +++ b/plugin.manifest @@ -1,7 +1,7 @@ pluginname=Garuda pluginclassname=GarudaPlugin origin=intelec -version=2.14 +version=2.14.1 description=Allows mail sending to selectable target audiences. studipMinVersion=5.3 studipMaxVersion=5.9.99