From 3a4eadac828cb09c56e0b877c893c8b7e0da7f80 Mon Sep 17 00:00:00 2001
From: Andreas <6977712+AndreasK79@users.noreply.github.com>
Date: Fri, 11 Aug 2023 20:51:02 +0200
Subject: [PATCH] QSL Carousel - only show prev/next if more than one image

---
 application/controllers/Logbookadvanced.php   |  2 +-
 .../views/logbookadvanced/qslcarousel.php     | 42 +++++++++++++++++++
 application/views/qslcard/qslcarousel.php     | 22 ++++++----
 3 files changed, 56 insertions(+), 10 deletions(-)
 create mode 100644 application/views/logbookadvanced/qslcarousel.php

diff --git a/application/controllers/Logbookadvanced.php b/application/controllers/Logbookadvanced.php
index af66990b1..f25fe68f3 100644
--- a/application/controllers/Logbookadvanced.php
+++ b/application/controllers/Logbookadvanced.php
@@ -231,6 +231,6 @@ class Logbookadvanced extends CI_Controller {
 		$cleanids = $this->security->xss_clean($this->input->post('ids'));
         $this->load->model('logbookadvanced_model');
         $data['qslimages'] = $this->logbookadvanced_model->getQslsForQsoIds($cleanids);
-        $this->load->view('qslcard/qslcarousel', $data);
+        $this->load->view('logbookadvanced/qslcarousel', $data);
 	}
 }
diff --git a/application/views/logbookadvanced/qslcarousel.php b/application/views/logbookadvanced/qslcarousel.php
new file mode 100644
index 000000000..3f26ed004
--- /dev/null
+++ b/application/views/logbookadvanced/qslcarousel.php
@@ -0,0 +1,42 @@
+<div id="carouselExampleIndicators" class="carousel slide" data-ride="carousel">
+<?php if (count($qslimages) > 1) { ?>
+<ol class="carousel-indicators">
+    <?php
+    $i = 0;
+    foreach ($qslimages as $image) {
+        echo '<li data-target="#carouselExampleIndicators" data-slide-to="' . $i . '"';
+        if ($i == 0) {
+            echo 'class="active"';
+        }
+        $i++;
+        echo '></li>';
+    }
+    ?>
+</ol>
+<?php } ?>
+<div class="carousel-inner">
+
+    <?php
+    $i = 1;
+    foreach ($qslimages as $image) {
+        echo '<div class="text-center carousel-item carouselimageid_' . $image->id;
+        if ($i == 1) {
+            echo ' active';
+        }
+        echo '">';
+        echo '<img class="img-fluid w-qsl" src="' . base_url() . '/assets/qslcard/' . $image->filename .'" alt="QSL picture #'. $i++.'">';
+        echo '</div>';
+    }
+    ?>
+</div>
+<?php if (count($qslimages) > 1) { ?>
+	<a class="carousel-control-prev" href="#carouselExampleIndicators" role="button" data-slide="prev">
+		<span class="carousel-control-prev-icon" aria-hidden="true"></span>
+		<span class="sr-only">Previous</span>
+	</a>
+	<a class="carousel-control-next" href="#carouselExampleIndicators" role="button" data-slide="next">
+		<span class="carousel-control-next-icon" aria-hidden="true"></span>
+		<span class="sr-only">Next</span>
+	</a>
+<?php } ?>
+</div>
diff --git a/application/views/qslcard/qslcarousel.php b/application/views/qslcard/qslcarousel.php
index 78aad7123..3f26ed004 100644
--- a/application/views/qslcard/qslcarousel.php
+++ b/application/views/qslcard/qslcarousel.php
@@ -1,4 +1,5 @@
 <div id="carouselExampleIndicators" class="carousel slide" data-ride="carousel">
+<?php if (count($qslimages) > 1) { ?>
 <ol class="carousel-indicators">
     <?php
     $i = 0;
@@ -12,6 +13,7 @@
     }
     ?>
 </ol>
+<?php } ?>
 <div class="carousel-inner">
 
     <?php
@@ -27,12 +29,14 @@
     }
     ?>
 </div>
-<a class="carousel-control-prev" href="#carouselExampleIndicators" role="button" data-slide="prev">
-    <span class="carousel-control-prev-icon" aria-hidden="true"></span>
-    <span class="sr-only">Previous</span>
-</a>
-<a class="carousel-control-next" href="#carouselExampleIndicators" role="button" data-slide="next">
-    <span class="carousel-control-next-icon" aria-hidden="true"></span>
-    <span class="sr-only">Next</span>
-</a>
-</div>
\ No newline at end of file
+<?php if (count($qslimages) > 1) { ?>
+	<a class="carousel-control-prev" href="#carouselExampleIndicators" role="button" data-slide="prev">
+		<span class="carousel-control-prev-icon" aria-hidden="true"></span>
+		<span class="sr-only">Previous</span>
+	</a>
+	<a class="carousel-control-next" href="#carouselExampleIndicators" role="button" data-slide="next">
+		<span class="carousel-control-next-icon" aria-hidden="true"></span>
+		<span class="sr-only">Next</span>
+	</a>
+<?php } ?>
+</div>