Hide similar courses when there are none

GEWIS · Oct 5, 2023 · 3621f6c · 3621f6c
1 parent ac40bad
commit 3621f6c
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 20 deletions.
diff --git a/module/Education/src/Controller/AdminController.php b/module/Education/src/Controller/AdminController.php
@@ -71,10 +71,9 @@ public function addCourseAction(): Response|ViewModel
                 $data = $form->getData(FormInterface::VALUES_AS_ARRAY);
                 $course = $this->courseService->saveCourse($data);
 
-                if ($this->courseService->saveCourse($course)) {
-                    $this->flashMessenger()->addSuccessMessage(
-                        $this->translator->translate('Successfully added course!'),
-                    );
+                $this->flashMessenger()->addSuccessMessage(
+                    $this->translator->translate('Successfully added course!'),
+                );
 
                 return $this->redirect()->toRoute('admin_education/course/edit', ['course' => $course->getCode()]);
             }

diff --git a/module/Education/view/education/education/course.phtml b/module/Education/view/education/education/course.phtml
@@ -124,22 +124,24 @@ foreach ($documents as $document) {
                 <?php endif; ?>
             </div>
         </div>
-        <hr>
-        <div class="row">
-            <div class="col-md-8">
-                <h2><?= $this->translate('Similar courses') ?></h2>
-                <ul>
-                    <?php foreach ($course->getSimilarCoursesTo() as $similarCourse): ?>
-                        <li>
-                            <a href="<?= $this->url('education/course', [
-                                'code' => $similarCourse->getCode(),
-                            ]) ?>">
-                                <?= $this->escapeHtml($similarCourse->getCode()) ?> - <?= $this->escapeHtml($similarCourse->getName()) ?>
-                            </a>
-                        </li>
-                    <?php endforeach; ?>
-                </ul>
+        <?php if (count($course->getSimilarCoursesTo()) > 0): ?>
+            <hr>
+            <div class="row">
+                <div class="col-md-8">
+                    <h2><?= $this->translate('Similar courses') ?></h2>
+                    <ul>
+                        <?php foreach ($course->getSimilarCoursesTo() as $similarCourse): ?>
+                            <li>
+                                <a href="<?= $this->url('education/course', [
+                                    'code' => $similarCourse->getCode(),
+                                ]) ?>">
+                                    <?= $this->escapeHtml($similarCourse->getCode()) ?> - <?= $this->escapeHtml($similarCourse->getName()) ?>
+                                </a>
+                            </li>
+                        <?php endforeach; ?>
+                    </ul>
+                </div>
             </div>
-        </div>
+        <?php endif; ?>
     </div>
 </section>