From 3882dc314896c26f13550796922ff9392c923464 Mon Sep 17 00:00:00 2001
From: Jan-Hendrik Willms <tleilax+github@gmail.com>
Date: Wed, 12 Aug 2015 11:24:37 +0200
Subject: [PATCH] simplify data structure and view; add proxy checkboxes

---
 controllers/show.php    |  6 +++---
 views/show/settings.php | 34 ++++++++++++++++++----------------
 2 files changed, 21 insertions(+), 19 deletions(-)

diff --git a/controllers/show.php b/controllers/show.php
index 1f019a3..6527884 100644
--- a/controllers/show.php
+++ b/controllers/show.php
@@ -117,7 +117,7 @@ protected function prepareCourses($ids)
             $_course['visitdate'] = object_get_visit($course->id, 'sem', '');
 
             $user_status = @$member_ships[$course->id]['status'];
-            if(!$user_status && Config::get()->DEPUTIES_ENABLE && isDeputy($GLOBALS['user']->id, $course->id)) {
+            if (!$user_status && Config::get()->DEPUTIES_ENABLE && isDeputy($GLOBALS['user']->id, $course->id)) {
                 $user_status = 'dozent';
                 $is_deputy = true;
             } else {
@@ -158,7 +158,7 @@ protected function getCourses()
                         $_ids[] = $a['Seminar_id'];
                     }
                 });
-                $courses[$sem['name']]['courses'] = $cm;
+                $courses[$sem['name']] = $cm;
             }
         }
 
@@ -177,7 +177,7 @@ protected function getCourses()
                         $_ids[] = $a['Seminar_id'];
                     }
                 });
-                $courses['unbegrenzt laufende']['courses'] = $cm;
+                $courses['unbegrenzt laufende'] = $cm;
             }
         }
 
diff --git a/views/show/settings.php b/views/show/settings.php
index 2ada69a..4074733 100644
--- a/views/show/settings.php
+++ b/views/show/settings.php
@@ -1,25 +1,27 @@
 <form action="<?= $controller->url_for('show/save_settings')?>" method="post">
     <?=CSRFProtection::tokenTag()?>
-    <table class="default">
 
-        <? foreach ($courses as $sem => $c) : ?>
-            <thead>
+    <table class="default">
+        <colgroup>
+            <col width="20px">
+            <col>
+        </colgroup>
+    <? foreach ($courses as $sem => $courses): ?>
+        <tbody id="sem-<?= md5($sem) ?>">
             <tr>
-                <th></th>
+                <th>
+                    <input type="checkbox" data-proxyfor="#sem-<?= md5($sem) ?> td :checkbox">
+                </th>
                 <th><?= htmlReady($sem) ?></th>
             </tr>
-            </thead>
-            <tbody>
-            <? foreach ($c['courses'] as $cm) : ?>
-
-                <tr>
-                    <td><input type="checkbox" name="favorites[]" value="<?= $cm['Seminar_id']?>" <?= in_array($cm['Seminar_id'], $ids) ? 'checked' : ''?> /></td>
-                    <td><?= htmlReady($cm['course_name']) ?></td>
-                </tr>
-
-            <? endforeach ?>
-            </tbody>
-        <? endforeach ?>
+        <? foreach ($courses as $course): ?>
+            <tr>
+                <td><input type="checkbox" name="favorites[]" value="<?= $course['Seminar_id']?>" <?= in_array($course['Seminar_id'], $ids) ? 'checked' : ''?> /></td>
+                <td><?= htmlReady($course['course_name']) ?></td>
+            </tr>
+        <? endforeach; ?>
+        </tbody>
+    <? endforeach; ?>
     </table>
 
     <div data-dialog-button>