From 67d07eab15f401cd9477d8c34fcf6d501a7014a5 Mon Sep 17 00:00:00 2001
From: Phil Davis <phil.davis@inf.org>
Date: Sat, 22 Nov 2014 19:36:51 +0545
Subject: AutoConfigBackup current host appears twice in dropdown

When building the lists of available host names on the Restore tab, the current host is listed twice if it already has some backups.
This makes the code a bit smarter to not do that.
IMHO not worth updating the version number, since this is purely cosmetic.
---
 config/autoconfigbackup/autoconfigbackup.php | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

(limited to 'config/autoconfigbackup')

diff --git a/config/autoconfigbackup/autoconfigbackup.php b/config/autoconfigbackup/autoconfigbackup.php
index 755c47aa..52b3eca9 100644
--- a/config/autoconfigbackup/autoconfigbackup.php
+++ b/config/autoconfigbackup/autoconfigbackup.php
@@ -357,10 +357,17 @@ EOF;
 					<center>
 					<b>Hostname:</b>
 					<select id="hostname" name="hostname" onChange="document.location='autoconfigbackup.php?hostname=' + this.value;">
-						<?foreach($hostnames as $hn):?>
-						<option value='<?=$hn?>'><?=$hn?></option>
+						<?
+						$host_not_found = true;
+						foreach($hostnames as $hn):
+						?>
+						<option value='<?=$hn?>' <? if ($hn == $hostname) {echo " selected=\"selected\""; $host_not_found = false;}?>>
+							<?=$hn?>
+						</option>
 						<?endforeach?>
-						<option value='<?=$hostname?>' SELECTED><?=$hostname?></option>
+						<? if ($host_not_found) { ?>
+							<option value='<?=$hostname?>' SELECTED><?=$hostname?></option>
+						<? } ?>
 					</select>
 				</td>
 			</tr>
-- 
cgit v1.2.3