From 08548cbf39d8df4c601f6839284e1588c3b76131 Mon Sep 17 00:00:00 2001 From: Marius Hoch Date: Sun, 3 Jan 2016 20:31:05 +0100 Subject: [PATCH] Slightly simplify QueryPage::outputResults By removing/ deprecating unused things. Change-Id: Ib9f7439eeeb261a60903eb23f13c65038ebe4945 --- includes/specialpage/QueryPage.php | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/includes/specialpage/QueryPage.php b/includes/specialpage/QueryPage.php index 2d25710614..27e645afcb 100644 --- a/includes/specialpage/QueryPage.php +++ b/includes/specialpage/QueryPage.php @@ -275,11 +275,14 @@ abstract class QueryPage extends SpecialPage { } /** - * Some special pages (for example SpecialListusers) might not return the + * Some special pages (for example SpecialListusers used to) might not return the * current object formatted, but return the previous one instead. * Setting this to return true will ensure formatResult() is called * one more time to make sure that the very last result is formatted * as well. + * + * @deprecated since 1.27 + * * @return bool */ function tryLastResult() { @@ -660,12 +663,9 @@ abstract class QueryPage extends SpecialPage { // @codingStandardsIgnoreEnd $line = $this->formatResult( $skin, $row ); if ( $line ) { - $attr = ( isset( $row->usepatrol ) && $row->usepatrol && $row->patrolled == 0 ) - ? ' class="not-patrolled"' - : ''; $html[] = $this->listoutput ? $line - : "{$line}\n"; + : "
  • {$line}
  • \n"; } } @@ -674,12 +674,9 @@ abstract class QueryPage extends SpecialPage { $row = null; $line = $this->formatResult( $skin, $row ); if ( $line ) { - $attr = ( isset( $row->usepatrol ) && $row->usepatrol && $row->patrolled == 0 ) - ? ' class="not-patrolled"' - : ''; $html[] = $this->listoutput ? $line - : "{$line}\n"; + : "
  • {$line}
  • \n"; } } -- 2.20.1