Browse Source

Merge pull request #11183 from nextcloud/bugfix/10765/fix-link-for-update-notifications

Fix the link and anchor for the update notifications
pull/11222/head
blizzz 7 years ago
committed by GitHub
parent
commit
2e72efef3b
No known key found for this signature in database GPG Key ID: 4AEE18F83AFDEB23
  1. 2
      apps/updatenotification/lib/Notification/Notifier.php
  2. 2
      settings/templates/settings/admin/overview.php

2
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());

2
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>

Loading…
Cancel
Save