fixes #1815

Closes #1815

Merge request studip/studip!1207

app/controllers/consultation/admin.php

@@ -168,6 +168,10 @@ class Consultation_AdminController extends ConsultationController
                 throw new InvalidArgumentException(_('Die definierte Zeit bis zur Pause ist kleiner als die Dauer eines Termins.'));
             }
 
+            if ($this->range instanceof Institute && !Request::getArray('responsibilities')) {
+                throw new InvalidArgumentException(_('Es muss mindestens eine durchführende Person, Statusgruppe oder Einrichtung ausgewählt werden.'));
+            }
+
             $slot_count = ConsultationBlock::countSlots(
                 $start,
                 $end,

app/views/consultation/admin/block-responsibilities.php

 <?php
-$block = $block ?? false;
+/**
+ * @var ConsultationBlock|null $block
+ */
+
+$block = $block ?? null;
 $selected = function ($type, $id) use ($block) {
     if (!$block ) {
         return '';

app/views/consultation/admin/create.php

 <?php
+/**
+ * @var Consultation_AdminController $controller
+ * @var Trails_Flash $flash
+ * @var string|null $room
+ * @var array $responsible
+ * @var Range $range
+ */
+
 $days_of_the_week = [
     _('Montag')     => 1,
     _('Dienstag')   => 2,
@@ -146,6 +154,20 @@ $intervals = [
     <fieldset>
         <legend><?= _('Durchführende Personen, Gruppen oder Einrichtungen') ?></legend>
 
+    <? if ($range instanceof Institute): ?>
+        <p>
+            <?= _('Bei Einrichtungen muss mindestens eine durchführende Person, Gruppe oder Einrichtung zugewiesen '
+                . 'werden.') ?>
+        </p>
+        <p>
+            <?= _('Bitte beachten Sie, dass bei Zuweisungen von Statusgruppen alle Personen der Gruppe mit dem Status '
+                . '"tutor" und "dozent" als durchführende Personen zugewiesen werden und über alle Buchungen '
+                . 'informiert werden.') ?>
+            <?= _('Gleiches gilt für eine zugewiesene Einrichtung. Bitte achten Sie darauf, dass Sie Ihre hier '
+                . ' getroffene Auswahl in Absprache tätigen.') ?>
+        </p>
+    <? endif; ?>
+
         <?= $this->render_partial('consultation/admin/block-responsibilities.php', compact('responsible')) ?>
     </fieldset>
 <? endif; ?>

lib/models/ConsultationBlock.php

@@ -101,9 +101,7 @@ class ConsultationBlock extends SimpleORMap implements PrivacyObject
                 $persons[] = $block->range;
             } elseif ($block->range instanceof Course) {
                 $persons = ConsultationResponsibility::getCourseResponsibilities($block->range);
-            } elseif ($block->range instanceof Institute) {
-                $persons = ConsultationResponsibility::getInstituteResponsibilites($block->range);
-            } else {
+            } elseif (!($block->range instanceof Institute)) {
                 throw new Exception('Unknown range type');
             }