rework block visibility settings in edit dialog, fixes #169

controllers/admin.php

@@ -54,6 +54,7 @@ class AdminController extends StudipController
         CSRFProtection::verifyUnsafeRequest();
 
         $block_id = Request::int('block_id');
+        $group_id = Request::option('group_id');
 
         if ($block_id) {
             $block = VipsBlock::find($block_id);
@@ -65,8 +66,8 @@ class AdminController extends StudipController
         vips_require_status('tutor', $block->course_id);
 
         $block->name = Request::get('block_name');
-        $block->group_id = Request::option('group_id') ?: null;
-        $block->visible = Request::int('block_visible', 0);
+        $block->group_id = $group_id ?: null;
+        $block->visible = $group_id !== '';
 
         if (!Request::int('block_grouped')) {
             $block->weight = null;

views/admin/edit_block.php

@@ -11,32 +11,28 @@
     </label>
 
     <label>
-        <input type="checkbox" name="block_grouped" value="1" <?= $block->weight !== null ? 'checked' : '' ?>>
-        <?= _vips('Aufgabenblätter in der Bewertung gruppieren') ?>
-        <?= tooltipIcon(_vips('In der Ergebnisübersicht wird nur der Block anstelle der enthaltenen Aufgabenblätter aufgeführt.')) ?>
-    </label>
-
-    <label>
-        <input type="checkbox" name="block_visible" value="1" <?= $block->visible ? 'checked' : '' ?> data-activates="#group_id">
-        <?= _vips('Aufgabenblätter für Teilnehmer sichtbar') ?>
-        <?= tooltipIcon(_vips('Wenn Aufgabenblätter nur in anderen Tools (z.B. Courseware) verwendet werden, können sie in Vips unsichtbar gemacht werden.')) ?>
-    </label>
-
-    <? if ($groups): ?>
-        <label>
-            <?= _vips('Sichtbar nur für Teilnehmer der Gruppe') ?>
-            <select name="group_id" id="group_id" <?= !$block->visible ? 'disabled' : '' ?>>
+        <?= _vips('Sichtbarkeit') ?>
+        <?= tooltipIcon(_vips('Blöcke und zugeordnete Aufgabenblätter können nur für bestimmte Gruppen sichtbar oder auch komplett unsichtbar gemacht werden.')) ?>
+        <select name="group_id">
             <option value="0">
                 <?= _vips('Alle Teilnehmer (keine Beschränkung)') ?>
             </option>
+            <option value="" <?= !$block->visible ? 'selected' : '' ?>>
+                <?= _vips('Für Teilnehmer unsichtbar') ?>
+            </option>
             <? foreach ($groups as $group): ?>
                 <option value="<?= $group->id ?>" <?= $block->group_id === $group->id ? 'selected' : '' ?>>
-                        <?= htmlReady($group->name) ?>
+                    <?= sprintf(_vips('Gruppe "%s"'), htmlReady($group->name)) ?>
                 </option>
             <? endforeach ?>
         </select>
     </label>
-    <? endif ?>
+
+    <label>
+        <input type="checkbox" name="block_grouped" value="1" <?= $block->weight !== null ? 'checked' : '' ?>>
+        <?= _vips('Aufgabenblätter in der Bewertung gruppieren') ?>
+        <?= tooltipIcon(_vips('In der Ergebnisübersicht wird nur der Block anstelle der enthaltenen Aufgabenblätter aufgeführt.')) ?>
+    </label>
 
     <footer data-dialog-button>
         <?= Studip\Button::createAccept(_vips('Speichern'), 'store_block') ?>