Shirayuki has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/79331


Change subject: Give grep a chance to find the usages
......................................................................

Give grep a chance to find the usages

Change-Id: I18846326539b814fa7fa93ca54117dac3572e4b0
---
M includes/CategoryViewer.php
M includes/EditPage.php
M includes/Preferences.php
M includes/ProtectionForm.php
M includes/Skin.php
M includes/SkinTemplate.php
M includes/User.php
M includes/UserMailer.php
M includes/actions/InfoAction.php
M includes/installer/CliInstaller.php
M includes/installer/WebInstaller.php
M includes/installer/WebInstallerPage.php
M includes/job/jobs/DoubleRedirectJob.php
M includes/specials/SpecialJavaScriptTest.php
M includes/specials/SpecialProtectedpages.php
M includes/specials/SpecialProtectedtitles.php
M includes/specials/SpecialUserrights.php
M languages/Language.php
18 files changed, 58 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/31/79331/1

diff --git a/includes/CategoryViewer.php b/includes/CategoryViewer.php
index 562ca0c..0c44e96 100644
--- a/includes/CategoryViewer.php
+++ b/includes/CategoryViewer.php
@@ -649,11 +649,6 @@
         * returned?  This function says what. Each type is considered 
independently
         * of the other types.
         *
-        * Note for grepping: uses the messages category-article-count,
-        * category-article-count-limited, category-subcat-count,
-        * category-subcat-count-limited, category-file-count,
-        * category-file-count-limited.
-        *
         * @param int $rescnt The number of items returned by our database 
query.
         * @param int $dbcnt The number of items according to the category 
table.
         * @param string $type 'subcat', 'article', or 'file'
@@ -701,8 +696,12 @@
                        $this->cat->refreshCounts();
                } else {
                        # Case 3: hopeless.  Don't give a total count at all.
+                       # Give grep a chance to find the usages: 
category-subcat-count-limited,
+                       # category-article-count-limited, 
category-file-count-limited
                        return $this->msg( "category-$type-count-limited" 
)->numParams( $rescnt )->parseAsBlock();
                }
+               # Give grep a chance to find the usages: 
category-subcat-count-limited,
+               # category-article-count-limited, category-file-count-limited
                return $this->msg( "category-$type-count" )->numParams( 
$rescnt, $totalcnt )->parseAsBlock();
        }
 }
diff --git a/includes/EditPage.php b/includes/EditPage.php
index 59b7480..f244828 100644
--- a/includes/EditPage.php
+++ b/includes/EditPage.php
@@ -967,6 +967,7 @@
                                                $undoMsg = 'norev';
                                        }
 
+                                       // Give grep a chance to find the 
usages: undo-success, undo-failure, undo-norev
                                        $class = ( $undoMsg == 'success' ? '' : 
'error ' ) . "mw-undo-{$undoMsg}";
                                        $this->editFormPageTop .= 
$wgOut->parse( "<div class=\"{$class}\">" .
                                                wfMessage( 'undo-' . $undoMsg 
)->plain() . '</div>', true, /* interface */true );
diff --git a/includes/Preferences.php b/includes/Preferences.php
index f5f63ea..2bb1e64 100644
--- a/includes/Preferences.php
+++ b/includes/Preferences.php
@@ -978,6 +978,8 @@
 
                foreach ( $watchTypes as $action => $pref ) {
                        if ( $user->isAllowed( $action ) ) {
+                               // Give grep a chance to find the usages:
+                               // tog-watchdefault, tog-watchmoves, 
tog-watchdeletion, tog-watchcreations
                                $defaultPreferences[$pref] = array(
                                        'type' => 'toggle',
                                        'section' => 
'watchlist/advancedwatchlist',
diff --git a/includes/ProtectionForm.php b/includes/ProtectionForm.php
index 4d41d9e..94ebc61 100644
--- a/includes/ProtectionForm.php
+++ b/includes/ProtectionForm.php
@@ -365,6 +365,8 @@
 
                foreach ( $this->mRestrictions as $action => $selected ) {
                        /* Not all languages have V_x <-> N_x relation */
+                       // Give grep a chance to find the usages:
+                       // restriction-edit, restriction-move, 
restriction-create, restriction-upload
                        $msg = wfMessage( 'restriction-' . $action );
                        $out .= "<tr><td>" .
                        Xml::openElement( 'fieldset' ) .
diff --git a/includes/Skin.php b/includes/Skin.php
index de9c9dc..b714886 100644
--- a/includes/Skin.php
+++ b/includes/Skin.php
@@ -82,6 +82,8 @@
        static function getSkinNameMessages() {
                $messages = array();
                foreach ( self::getSkinNames() as $skinKey => $skinName ) {
+                       // Give grep a chance to find the usages:
+                       // skinname-cologneblue, skinname-monobook, 
skinname-modern, skinname-vector
                        $messages[] = "skinname-$skinKey";
                }
                return $messages;
diff --git a/includes/SkinTemplate.php b/includes/SkinTemplate.php
index cca1c6c..bce9ca4 100644
--- a/includes/SkinTemplate.php
+++ b/includes/SkinTemplate.php
@@ -291,6 +291,7 @@
                if ( $out->isSyndicated() ) {
                        $feeds = array();
                        foreach ( $out->getSyndicationLinks() as $format => 
$link ) {
+                               // Give grep a chance to find the usages: 
feed-atom, feed-rss
                                $feeds[$format] = array(
                                        'text' => $this->msg( "feed-$format" 
)->text(),
                                        'href' => $link
diff --git a/includes/User.php b/includes/User.php
index 3c0e1f6..72f66f0 100644
--- a/includes/User.php
+++ b/includes/User.php
@@ -3739,6 +3739,8 @@
                } elseif ( $type === true ) {
                        $message = 'confirmemail_body_changed';
                } else {
+                       // Give grep a chance to find the usages:
+                       // confirmemail_body_changed, confirmemail_body_set
                        $message = 'confirmemail_body_' . $type;
                }
 
diff --git a/includes/UserMailer.php b/includes/UserMailer.php
index 0d77159..0c880d6 100644
--- a/includes/UserMailer.php
+++ b/includes/UserMailer.php
@@ -752,9 +752,13 @@
                $postTransformKeys['$PAGESUMMARY'] = $this->summary == '' ? ' - 
' : $this->summary;
 
                # Now build message's subject and body
+               # Give grep a chance to find the usages: 
enotif_subject_deleted, enotif_subject_created,
+               # enotif_subject_moved, enotif_subject_restored, 
enotif_subject_changed
                $this->subject = wfMessage( 'enotif_subject_' . 
$this->pageStatus )->inContentLanguage()
                        ->params( $pageTitle, $keys['$PAGEEDITOR'] )->text();
 
+               # Give grep a chance to find the usages: 
enotif_body_intro_deleted, enotif_body_intro_created,
+               # enotif_body_intro_moved, enotif_body_intro_restored, 
enotif_body_intro_changed
                $keys['$PAGEINTRO'] = wfMessage( 'enotif_body_intro_' . 
$this->pageStatus )
                        ->inContentLanguage()->params( $pageTitle, 
$keys['$PAGEEDITOR'], $pageTitleUrl )
                        ->text();
diff --git a/includes/actions/InfoAction.php b/includes/actions/InfoAction.php
index 012a655..1fe19d5 100644
--- a/includes/actions/InfoAction.php
+++ b/includes/actions/InfoAction.php
@@ -116,6 +116,9 @@
 
                // Render page information
                foreach ( $pageInfo as $header => $infoTable ) {
+                       // Give grep a chance to find the usages:
+                       // pageinfo-header-basic, pageinfo-header-edits, 
pageinfo-header-restrictions,
+                       // pageinfo-header-properties, pageinfo-category-info
                        $content .= $this->makeHeader( $this->msg( 
"pageinfo-${header}" )->escaped() ) . "\n";
                        $table = "\n";
                        foreach ( $infoTable as $infoRow ) {
@@ -278,6 +281,7 @@
 
                // Use robot policy logic
                $policy = $this->page->getRobotPolicy( 'view', $pOutput );
+               // Give grep a chance to find the usages: pageinfo-robot-index, 
pageinfo-robot-noindex
                $pageInfo['header-basic'][] = array(
                        $this->msg( 'pageinfo-robot-policy' ), $this->msg( 
"pageinfo-robot-${policy['index']}" )
                );
@@ -393,6 +397,8 @@
                                $message = $this->msg( 'protect-default' 
)->escaped();
                        } else {
                                // Administrators only
+                               // Give grep a chance to find the usages:
+                               // protect-level-autoconfirmed, 
protect-level-sysop
                                $message = $this->msg( 
"protect-level-$protectionLevel" );
                                if ( $message->isDisabled() ) {
                                        // Require "$1" permission
@@ -402,6 +408,8 @@
                                }
                        }
 
+                       // Give grep a chance to find the usages:
+                       // restriction-edit, restriction-move, 
restriction-create, restriction-upload
                        $pageInfo['header-restrictions'][] = array(
                                $this->msg( "restriction-$restrictionType" ), 
$message
                        );
diff --git a/includes/installer/CliInstaller.php 
b/includes/installer/CliInstaller.php
index e3bcb6f..d4e19f6 100644
--- a/includes/installer/CliInstaller.php
+++ b/includes/installer/CliInstaller.php
@@ -137,6 +137,9 @@
        }
 
        public function startStage( $step ) {
+               // Give grep a chance to find the usages: 
config-install-database, config-install-tables,
+               // config-install-interwiki, config-install-stats, 
config-install-keys, config-install-sysop,
+               // config-install-mainpage
                $this->showMessage( "config-install-$step" );
        }
 
diff --git a/includes/installer/WebInstaller.php 
b/includes/installer/WebInstaller.php
index 10c7b96..da84316 100644
--- a/includes/installer/WebInstaller.php
+++ b/includes/installer/WebInstaller.php
@@ -540,6 +540,11 @@
                $s .= "</ul><br/><ul>\n";
                $s .= $this->getPageListItem( 'Restart', true, $currentPageName 
);
                $s .= "</ul></div>\n"; // end list pane
+               // Give grep a chance to find the usages:
+               // config-page-language, config-page-welcome, 
config-page-dbconnect, config-page-upgrade,
+               // config-page-dbsettings, config-page-name, 
config-page-options, config-page-install,
+               // config-page-complete, config-page-restart, 
config-page-readme, config-page-releasenotes,
+               // config-page-copying, config-page-upgradedoc, 
config-page-existingwiki
                $s .= Html::element( 'h2', array(),
                        wfMessage( 'config-page-' . strtolower( 
$currentPageName ) )->text() );
 
@@ -557,6 +562,11 @@
         */
        private function getPageListItem( $pageName, $enabled, $currentPageName 
) {
                $s = "<li class=\"config-page-list-item\">";
+               // Give grep a chance to find the usages:
+               // config-page-language, config-page-welcome, 
config-page-dbconnect, config-page-upgrade,
+               // config-page-dbsettings, config-page-name, 
config-page-options, config-page-install,
+               // config-page-complete, config-page-restart, 
config-page-readme, config-page-releasenotes,
+               // config-page-copying, config-page-upgradedoc, 
config-page-existingwiki
                $name = wfMessage( 'config-page-' . strtolower( $pageName ) 
)->text();
 
                if ( $enabled ) {
diff --git a/includes/installer/WebInstallerPage.php 
b/includes/installer/WebInstallerPage.php
index 87c4a8f..a017894 100644
--- a/includes/installer/WebInstallerPage.php
+++ b/includes/installer/WebInstallerPage.php
@@ -1219,6 +1219,9 @@
        }
 
        public function startStage( $step ) {
+               // Give grep a chance to find the usages: 
config-install-database, config-install-tables,
+               // config-install-interwiki, config-install-stats, 
config-install-keys, config-install-sysop,
+               // config-install-mainpage
                $this->addHTML( "<li>" . wfMessage( "config-install-$step" 
)->escaped() . wfMessage( 'ellipsis' )->escaped() );
                if ( $step == 'extension-tables' ) {
                        $this->startLiveBox();
diff --git a/includes/job/jobs/DoubleRedirectJob.php 
b/includes/job/jobs/DoubleRedirectJob.php
index 0be03b3..918d9b3 100644
--- a/includes/job/jobs/DoubleRedirectJob.php
+++ b/includes/job/jobs/DoubleRedirectJob.php
@@ -142,6 +142,8 @@
                $oldUser = $wgUser;
                $wgUser = $user;
                $article = WikiPage::factory( $this->title );
+               // Give grep a chance to find the usages:
+               // double-redirect-fixed-move, double-redirect-fixed-maintenance
                $reason = wfMessage( 'double-redirect-fixed-' . $this->reason,
                        $this->redirTitle->getPrefixedText(), 
$newTitle->getPrefixedText()
                )->inContentLanguage()->text();
diff --git a/includes/specials/SpecialJavaScriptTest.php 
b/includes/specials/SpecialJavaScriptTest.php
index f79fdd7..5f4c1d2 100644
--- a/includes/specials/SpecialJavaScriptTest.php
+++ b/includes/specials/SpecialJavaScriptTest.php
@@ -62,6 +62,7 @@
                        $out->addHtml( $summary );
                } elseif ( isset( self::$frameworks[$framework] ) ) {
                        // Matched! Display proper title and initialize the 
framework
+                       // Give grep a chance to find the usages: 
javascripttest-qunit-name
                        $out->setPageTitle( $this->msg(
                                'javascripttest-title',
                                $this->msg( "javascripttest-$framework-name" 
)->plain()
@@ -92,6 +93,7 @@
        private function getFrameworkListHtml() {
                $list = '<ul>';
                foreach ( self::$frameworks as $framework => $initFn ) {
+                       // Give grep a chance to find the usages: 
javascripttest-qunit-name
                        $list .= Html::rawElement(
                                'li',
                                array(),
diff --git a/includes/specials/SpecialProtectedpages.php 
b/includes/specials/SpecialProtectedpages.php
index 78931a0..90c0782 100644
--- a/includes/specials/SpecialProtectedpages.php
+++ b/includes/specials/SpecialProtectedpages.php
@@ -123,6 +123,8 @@
 
                $description_items = array();
 
+               // Give grep a chance to find the usages:
+               // restriction-level-sysop, restriction-level-autoconfirmed
                $protType = $this->msg( 'restriction-level-' . $row->pr_level 
)->escaped();
 
                $description_items[] = $protType;
@@ -308,6 +310,8 @@
 
                // First pass to load the log names
                foreach ( Title::getFilteredRestrictionTypes( true ) as $type ) 
{
+                       // Give grep a chance to find the usages:
+                       // restriction-edit, restriction-move, 
restriction-create, restriction-upload
                        $text = $this->msg( "restriction-$type" )->text();
                        $m[$text] = $type;
                }
diff --git a/includes/specials/SpecialProtectedtitles.php 
b/includes/specials/SpecialProtectedtitles.php
index 21bdf02..540d0d0 100644
--- a/includes/specials/SpecialProtectedtitles.php
+++ b/includes/specials/SpecialProtectedtitles.php
@@ -101,6 +101,8 @@
 
                $link = Linker::link( $title );
                $description_items = array();
+               // Give grep a chance to find the usages:
+               // restriction-level-sysop, restriction-level-autoconfirmed
                $protType = $this->msg( 'restriction-level-' . 
$row->pt_create_perm )->escaped();
                $description_items[] = $protType;
                $lang = $this->getLanguage();
@@ -183,6 +185,8 @@
                // First pass to load the log names
                foreach ( $wgRestrictionLevels as $type ) {
                        if ( $type != '' && $type != '*' ) {
+                               // Give grep a chance to find the usages:
+                               // restriction-level-sysop, 
restriction-level-autoconfirmed
                                $text = $this->msg( "restriction-level-$type" 
)->text();
                                $m[$text] = $type;
                        }
diff --git a/includes/specials/SpecialUserrights.php 
b/includes/specials/SpecialUserrights.php
index 77f2063..26923f7 100644
--- a/includes/specials/SpecialUserrights.php
+++ b/includes/specials/SpecialUserrights.php
@@ -598,6 +598,7 @@
                        if ( $column === array() ) {
                                continue;
                        }
+                       // Give grep a chance to find the usages: 
userrights-changeable-col, userrights-unchangeable-col
                        $ret .= Xml::element( 'th', null, $this->msg( 
'userrights-' . $name . '-col', count( $column ) )->text() );
                }
                $ret .= "</tr>\n<tr>\n";
diff --git a/languages/Language.php b/languages/Language.php
index 262d86f..2523814 100644
--- a/languages/Language.php
+++ b/languages/Language.php
@@ -2089,6 +2089,9 @@
                $segments = array();
 
                foreach ( $intervals as $intervalName => $intervalValue ) {
+                       // Give grep a chance to find the usages:
+                       // duration-centuries, duration-decades, 
duration-years, duration-days,
+                       // duration-hours, duration-minutes, duration-seconds
                        $message = wfMessage( 'duration-' . $intervalName 
)->numParams( $intervalValue );
                        $segments[] = $message->inLanguage( $this )->escaped();
                }

-- 
To view, visit https://gerrit.wikimedia.org/r/79331
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I18846326539b814fa7fa93ca54117dac3572e4b0
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Shirayuki <shirayuk...@gmail.com>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to