Merge "Linker: Remove some else statements, and unnecessary temporary variables"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Mon, 11 Feb 2019 00:43:31 +0000 (00:43 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Mon, 11 Feb 2019 00:43:31 +0000 (00:43 +0000)
1  2 
includes/Linker.php

@@@ -1888,13 -1896,54 +1896,13 @@@ class Linker 
                                $html = $context->msg( 'rollbacklinkcount' )->numParams( $editCount )->parse();
                        }
  
-                       return self::link( $title, $html, $attrs, $query, $options );
-               } else {
-                       $html = $context->msg( 'rollbacklink' )->escaped();
                        return self::link( $title, $html, $attrs, $query, $options );
                }
+               $html = $context->msg( 'rollbacklink' )->escaped();
+               return self::link( $title, $html, $attrs, $query, $options );
        }
  
 -      /**
 -       * @deprecated since 1.28, use TemplatesOnThisPageFormatter directly
 -       *
 -       * Returns HTML for the "templates used on this page" list.
 -       *
 -       * Make an HTML list of templates, and then add a "More..." link at
 -       * the bottom. If $more is null, do not add a "More..." link. If $more
 -       * is a Title, make a link to that title and use it. If $more is a string,
 -       * directly paste it in as the link (escaping needs to be done manually).
 -       * Finally, if $more is a Message, call toString().
 -       *
 -       * @since 1.16.3. $more added in 1.21
 -       * @param Title[] $templates Array of templates
 -       * @param bool $preview Whether this is for a preview
 -       * @param bool $section Whether this is for a section edit
 -       * @param Title|Message|string|null $more An escaped link for "More..." of the templates
 -       * @return string HTML output
 -       */
 -      public static function formatTemplates( $templates, $preview = false,
 -              $section = false, $more = null
 -      ) {
 -              wfDeprecated( __METHOD__, '1.28' );
 -
 -              $type = false;
 -              if ( $preview ) {
 -                      $type = 'preview';
 -              } elseif ( $section ) {
 -                      $type = 'section';
 -              }
 -
 -              if ( $more instanceof Message ) {
 -                      $more = $more->toString();
 -              }
 -
 -              $formatter = new TemplatesOnThisPageFormatter(
 -                      RequestContext::getMain(),
 -                      MediaWikiServices::getInstance()->getLinkRenderer()
 -              );
 -              return $formatter->format( $templates, $type, $more );
 -      }
 -
        /**
         * Returns HTML for the "hidden categories on this page" list.
         *