From 57ef1d307bec09c61b813ca6bafe261e8fe4df22 Mon Sep 17 00:00:00 2001
From: Joas Schilling <coding@schilljs.com>
Date: Wed, 12 Sep 2018 10:24:02 +0200
Subject: [PATCH] Fix the link and anchor for the update notifications

Signed-off-by: Joas Schilling <coding@schilljs.com>
---
 apps/updatenotification/lib/Notification/Notifier.php | 2 +-
 settings/templates/settings/admin/overview.php        | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/apps/updatenotification/lib/Notification/Notifier.php b/apps/updatenotification/lib/Notification/Notifier.php
index c88937f0df..4e3a30f225 100644
--- a/apps/updatenotification/lib/Notification/Notifier.php
+++ b/apps/updatenotification/lib/Notification/Notifier.php
@@ -108,7 +108,7 @@ class Notifier implements INotifier {
 			$notification->setParsedSubject($l->t('Update to %1$s is available.', [$parameters['version']]));
 
 			if ($this->isAdmin()) {
-				$notification->setLink($this->url->linkToRouteAbsolute('settings.AdminSettings.index') . '#updater');
+				$notification->setLink($this->url->linkToRouteAbsolute('settings.AdminSettings.index', ['section' => 'overview']) . '#version');
 			}
 		} else {
 			$appInfo = $this->getAppInfo($notification->getObjectType());
diff --git a/settings/templates/settings/admin/overview.php b/settings/templates/settings/admin/overview.php
index cf725d3101..aad6ae80a7 100644
--- a/settings/templates/settings/admin/overview.php
+++ b/settings/templates/settings/admin/overview.php
@@ -59,7 +59,7 @@
 
 </div>
 
-<div class="section">
+<div id="version" class="section">
 	<!-- should be the last part, so Updater can follow if enabled (it has no heading therefore). -->
 	<h2><?php p($l->t('Version'));?></h2>
 	<p><strong><a href="<?php print_unescaped($theme->getBaseUrl()); ?>" rel="noreferrer noopener" target="_blank"><?php p($theme->getTitle()); ?></a> <?php p(OC_Util::getHumanVersion()) ?></strong></p>