Improve update notification layout and show non-default update server

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

1 files changed, 30 insertions, 19 deletions

diff --git a/apps/updatenotification/templates/admin.php b/apps/updatenotification/templates/admin.php
index bda27abf25e..e09d19848e7 100644
--- a/apps/updatenotification/templates/admin.php
+++ b/apps/updatenotification/templates/admin.php
@@ -1,5 +1,6 @@
 <?php
 	script('updatenotification', 'admin');
+	style('updatenotification', 'admin');
 
 	/** @var array $_ */
 	/** @var bool $isNewVersionAvailable */
@@ -14,22 +15,36 @@
 	$channels = $_['channels'];
 	/** @var string $currentChannel */
 	$currentChannel = $_['currentChannel'];
+	/** @var string $updateServerURL */
+	$updateServerURL = $_['updateServerURL'];
+	/** @var bool $isDefaultUpdateServerURL */
+	$isDefaultUpdateServerURL = $_['isDefaultUpdateServerURL'];
 ?>
 <form id="oca_updatenotification_section" class="followupsection">
-	<?php if($isNewVersionAvailable === true) { ?>
-		<strong><?php p($l->t('A new version is available: %s', [$newVersionString])); ?></strong>
-		<?php if ($_['updaterEnabled']) { ?>
-			<input type="button" id="oca_updatenotification_button" value="<?php p($l->t('Open updater')) ?>">
+	<p>
+		<?php if ($isNewVersionAvailable === true) { ?>
+			<strong><?php p($l->t('A new version is available: %s', [$newVersionString])); ?></strong>
+			<?php if ($_['updaterEnabled']) { ?>
+				<input type="button" id="oca_updatenotification_button" value="<?php p($l->t('Open updater')) ?>">
+			<?php } ?>
+			<?php if (!empty($_['downloadLink'])) { ?>
+				<a href="<?php p($_['downloadLink']); ?>" class="button<?php if ($_['updaterEnabled']) { p(' hidden'); } ?>"><?php p($l->t('Download now')) ?></a>
+			<?php } ?>
+		<?php } elseif (!$isUpdateChecked) { ?>
+			<?php p($l->t('The update check is not yet finished. Please refresh the page.')); ?>
+		<?php } else { ?>
+			<?php p($l->t('Your version is up to date.')); ?>
+			<span class="icon-info svg" title="<?php p($l->t('Checked on %s', [$lastCheckedDate])) ?>"></span>
 		<?php } ?>
-		<?php if (!empty($_['downloadLink'])) { ?>
-			<a href="<?php p($_['downloadLink']); ?>" class="button<?php if ($_['updaterEnabled']) { p(' hidden'); } ?>"><?php p($l->t('Download now')) ?></a>
+
+		<?php if (!$isDefaultUpdateServerURL) { ?>
+			<br />
+			<em>
+				<?php p($l->t("A non-default update server is in use to be checked for updates:")); ?>
+				<code><?php p($updateServerURL); ?></code>
+			</em>
 		<?php } ?>
-	<?php } elseif (!$isUpdateChecked) { ?>
-		<?php p($l->t('The update check is not yet finished. Please refresh the page.')); ?>
-	<?php } else { ?>
-		<?php p($l->t('Your version is up to date.')); ?>
-		<span class="icon-info svg" title="<?php p($l->t('Checked on %s', [$lastCheckedDate])) ?>"></span>
-	<?php } ?>
+	</p>
 
 	<p>
 		<label for="release-channel"><?php p($l->t('Update channel:')) ?></label>
@@ -41,20 +56,16 @@
 				</option>
 			<?php } ?>
 		</select>
-		<span id="channel_save_msg" class="msg"></span>
-	</p>
-	<p>
-		<em><?php p($l->t('You can always update to a newer version / experimental channel. But you can never downgrade to a more stable channel.')); ?></em>
+		<span id="channel_save_msg" class="msg"></span><br />
+		<em><?php p($l->t('You can always update to a newer version / experimental channel. But you can never downgrade to a more stable channel.')); ?></em><br />
 		<em><?php p($l->t('Note that after a new release it can take some time before it shows up here. We roll out new versions spread out over time to our users and sometimes skip a version when issues are found.')); ?></em>
 	</p>
 
 
 	<p id="oca_updatenotification_groups">
-		<br />
 		<?php p($l->t('Notify members of the following groups about available updates:')); ?>
-		<input name="oca_updatenotification_groups_list" type="hidden" id="oca_updatenotification_groups_list" value="<?php p($_['notify_groups']) ?>" style="width: 400px">
+		<input name="oca_updatenotification_groups_list" type="hidden" id="oca_updatenotification_groups_list" value="<?php p($_['notify_groups']) ?>" style="width: 400px"><br />
 		<em class="<?php if (!in_array($currentChannel, ['daily', 'git'])) p('hidden'); ?>">
-			<br />
 			<?php p($l->t('Only notification for app updates are available.')); ?>
 			<?php if ($currentChannel === 'daily') p($l->t('The selected update channel makes dedicated notifications for the server obsolete.')); ?>
 			<?php if ($currentChannel === 'git') p($l->t('The selected update channel does not support updates of the server.')); ?>