Remove rel="archives" from HTML output
authorKevin Israel <pleasestand@live.com>
Sun, 1 Mar 2015 11:05:25 +0000 (06:05 -0500)
committerKrinkle <krinklemail@gmail.com>
Thu, 5 Mar 2015 22:57:58 +0000 (22:57 +0000)
Follows-up fab055d6a1.
Introduced in 4299d27956 (r45988)

That link type was removed from HTML5 in 2011. It is not recognized
by the W3C Markup Validator.

https://html5.org/r/5925
https://www.w3.org/Bugs/Public/show_bug.cgi?id=11486
https://html.spec.whatwg.org/multipage/semantics.html#linkTypes

Change-Id: I61ddd7ee3df41bfb62423d0498217d31e7952902

RELEASE-NOTES-1.25
includes/page/Article.php
includes/skins/SkinTemplate.php

index 642fef8..efaaafa 100644 (file)
@@ -166,6 +166,8 @@ production.
   (instead of returning incorrect results) even when the page ID is known.
 * (T74070) Duplicate search for archived files on file upload now omits the extension.
   This requires the fa_sha1 field being populated.
+* Removed rel="archives" from the "View history" link, as it did not pass
+  HTML validation.
 
 === Action API changes in 1.25 ===
 * (T67403) XML tag highlighting is now only performed for formats
index 817b4b6..cc87a10 100644 (file)
@@ -1634,7 +1634,7 @@ class Article implements Page {
                                $context->msg( 'historywarning' )->numParams( $revisions )->parse() .
                                $context->msg( 'word-separator' )->escaped() . Linker::linkKnown( $title,
                                        $context->msg( 'history' )->escaped(),
-                                       array( 'rel' => 'archives' ),
+                                       array(),
                                        array( 'action' => 'history' ) ) .
                                '</strong>'
                        );
index 98a1b03..b0390e9 100644 (file)
@@ -948,7 +948,6 @@ class SkinTemplate extends Skin {
                                                'text' => wfMessageFallback( "$skname-view-history", 'history_short' )
                                                        ->setContext( $this->getContext() )->text(),
                                                'href' => $title->getLocalURL( 'action=history' ),
-                                               'rel' => 'archives',
                                        );
 
                                        if ( $title->quickUserCan( 'delete', $user ) ) {