jenkins-bot has submitted this change and it was merged.

Change subject: Use Maint addDescription method
......................................................................


Use Maint addDescription method

Change-Id: I7ac6283122812337ca3c3b9cce741c247b060d57
---
M client/maintenance/populateEntityUsage.php
M client/maintenance/populateInterwiki.php
M client/maintenance/updateSubscriptions.php
M lib/maintenance/populateSitesTable.php
M repo/maintenance/changePropertyDataType.php
M repo/maintenance/createBlacklistedItems.php
M repo/maintenance/dispatchChanges.php
M repo/maintenance/dumpEntities.php
M repo/maintenance/importProperties.php
M repo/maintenance/populateChangesSubscription.php
M repo/maintenance/pruneChanges.php
M repo/maintenance/rebuildEntityPerPage.php
M repo/maintenance/rebuildItemsPerSite.php
M repo/maintenance/rebuildPropertyInfo.php
M repo/maintenance/rebuildTermsSearchKey.php
15 files changed, 20 insertions(+), 19 deletions(-)

Approvals:
  Aude: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/client/maintenance/populateEntityUsage.php 
b/client/maintenance/populateEntityUsage.php
index c929373..b9ee1bb 100644
--- a/client/maintenance/populateEntityUsage.php
+++ b/client/maintenance/populateEntityUsage.php
@@ -25,7 +25,7 @@
 class PopulateEntityUsage extends LoggedUpdateMaintenance {
 
        public function __construct() {
-               $this->mDescription = 'Populate the wbc_entity_usage table 
based on entries in page_props.';
+               $this->addDescription( 'Populate the wbc_entity_usage table 
based on entries in page_props.' );
 
                $this->addOption( 'start-page', "The page ID to start from.", 
false, true );
 
diff --git a/client/maintenance/populateInterwiki.php 
b/client/maintenance/populateInterwiki.php
index 6bc7b83..7bf0bcb 100644
--- a/client/maintenance/populateInterwiki.php
+++ b/client/maintenance/populateInterwiki.php
@@ -34,7 +34,7 @@
        private $cache;
 
        public function __construct() {
-               $this->mDescription = <<<TEXT
+               $this->addDescription( <<<TEXT
 This script will populate the interwiki table, pulling in interwiki links that 
are used on Wikipedia
 or another MediaWiki wiki.
 
@@ -44,7 +44,8 @@
 --source parameter is the url for the source wiki api, such as 
"https://en.wikipedia.org/w/api.php";
 from which the script fetches the interwiki data and uses here to populate 
this wiki's interwiki
 database table.
-TEXT;
+TEXT
+               );
 
                $this->addOption( 'source', 'Source wiki for interwiki table, 
such as '
                        . 'https://en.wikipedia.org/w/api.php', false, true );
diff --git a/client/maintenance/updateSubscriptions.php 
b/client/maintenance/updateSubscriptions.php
index e1af860..0443a3e 100644
--- a/client/maintenance/updateSubscriptions.php
+++ b/client/maintenance/updateSubscriptions.php
@@ -26,8 +26,8 @@
 class UpdateSubscriptions extends Maintenance {
 
        public function __construct() {
-               $this->mDescription = 'Updates the repo\'s 
wb_changes_subscription table based on entries'
-                       . ' in wbc_entity_usage.';
+               $this->addDescription( 'Updates the repo\'s 
wb_changes_subscription table based on entries'
+                       . ' in wbc_entity_usage.' );
 
                $this->addOption( 'start-item', "The entity ID to start from.", 
false, true );
                $this->addOption(
diff --git a/lib/maintenance/populateSitesTable.php 
b/lib/maintenance/populateSitesTable.php
index f2bd213..74457a8 100644
--- a/lib/maintenance/populateSitesTable.php
+++ b/lib/maintenance/populateSitesTable.php
@@ -39,7 +39,7 @@
        public function __construct() {
                parent::__construct();
 
-               $this->mDescription = 'Populate the sites table from another 
wiki that runs the SiteMatrix extension';
+               $this->addDescription( 'Populate the sites table from another 
wiki that runs the SiteMatrix extension' );
 
                $this->addOption( 'strip-protocols', "Strip http/https from 
URLs to make them protocol relative." );
                $this->addOption( 'force-protocol', "Force a specific protocol 
for all URLs (like http/https).", false, true );
diff --git a/repo/maintenance/changePropertyDataType.php 
b/repo/maintenance/changePropertyDataType.php
index 3a62804..eb62bb7 100644
--- a/repo/maintenance/changePropertyDataType.php
+++ b/repo/maintenance/changePropertyDataType.php
@@ -22,13 +22,13 @@
 
        public function __construct() {
                parent::__construct();
-               $this->mDescription = "Change the data type of a given 
Property.\n"
+               $this->addDescription( "Change the data type of a given 
Property.\n"
                                . "Please note: You probably don't want to use 
this, as this will likely"
                                . " break consumers of your data. Statements 
with the changed property will appear"
                                . " to have changed without there being an edit 
to the entity containing the statement."
                                . " Also derived information based on the old 
type will disappear and derived information"
                                . " based on the new type will appear. In 
almost all cases you"
-                               . " should rather add a new property and make 
user of that.";
+                               . " should rather add a new property and make 
user of that." );
 
                $this->addOption( 'property-id', 'Id of the property to 
change.', true, true );
                $this->addOption(
diff --git a/repo/maintenance/createBlacklistedItems.php 
b/repo/maintenance/createBlacklistedItems.php
index cea5039..4dc4853 100644
--- a/repo/maintenance/createBlacklistedItems.php
+++ b/repo/maintenance/createBlacklistedItems.php
@@ -22,7 +22,7 @@
 class CreateBlacklistedItems extends Maintenance {
 
        public function __construct() {
-               $this->mDescription = 'Created blacklisted items';
+               $this->addDescription( 'Created blacklisted items' );
 
                parent::__construct();
        }
diff --git a/repo/maintenance/dispatchChanges.php 
b/repo/maintenance/dispatchChanges.php
index 6afae15..002e6b0 100644
--- a/repo/maintenance/dispatchChanges.php
+++ b/repo/maintenance/dispatchChanges.php
@@ -37,9 +37,9 @@
        public function __construct() {
                parent::__construct();
 
-               $this->mDescription =
+               $this->addDescription(
                        'Maintenance script that polls for Wikibase changes in 
the shared wb_changes table
-                       and dispatches them to any client wikis using their job 
queue.';
+                       and dispatches them to any client wikis using their job 
queue.' );
 
                $this->addOption( 'verbose', "Report activity." );
                $this->addOption( 'idle-delay', "Seconds to sleep when idle. 
Default: 10", false, true );
diff --git a/repo/maintenance/dumpEntities.php 
b/repo/maintenance/dumpEntities.php
index 9d013d3..1ca5e20 100644
--- a/repo/maintenance/dumpEntities.php
+++ b/repo/maintenance/dumpEntities.php
@@ -43,7 +43,7 @@
        public function __construct() {
                parent::__construct();
 
-               $this->mDescription = 'Generate a JSON dump from entities in 
the repository.';
+               $this->addDescription( 'Generate a JSON dump from entities in 
the repository.' );
 
                $this->addOption( 'list-file', "A file containing one entity ID 
per line.", false, true );
                $this->addOption( 'entity-type', "Only dump this kind of 
entity, e.g. `item` or `property`.", false, true );
diff --git a/repo/maintenance/importProperties.php 
b/repo/maintenance/importProperties.php
index 11c1d9c..ac7c688 100644
--- a/repo/maintenance/importProperties.php
+++ b/repo/maintenance/importProperties.php
@@ -63,7 +63,7 @@
        private $store;
 
        public function __construct() {
-               $this->mDescription = "Import properties in Wikidata.";
+               $this->addDescription( "Import properties in Wikidata." );
 
                $this->addOption( 'skip', "Skip number of entries in the import 
file" );
                $this->addOption( 'only', "Only import the specific entry from 
the import file" );
diff --git a/repo/maintenance/populateChangesSubscription.php 
b/repo/maintenance/populateChangesSubscription.php
index 95dea2d..72a72d8 100644
--- a/repo/maintenance/populateChangesSubscription.php
+++ b/repo/maintenance/populateChangesSubscription.php
@@ -23,7 +23,7 @@
 class PopulateChangesSubscription extends LoggedUpdateMaintenance {
 
        public function __construct() {
-               $this->mDescription = 'Populate the wb_changes_subscription 
table based on entries in wb_items_per_site.';
+               $this->addDescription( 'Populate the wb_changes_subscription 
table based on entries in wb_items_per_site.' );
 
                $this->addOption( 'start-item', "The item ID to start from.", 
false, true );
                $this->addOption( 'verbose', 'Report more detailed script 
progress.' );
diff --git a/repo/maintenance/pruneChanges.php 
b/repo/maintenance/pruneChanges.php
index 0f70c6a..72f8301 100644
--- a/repo/maintenance/pruneChanges.php
+++ b/repo/maintenance/pruneChanges.php
@@ -18,7 +18,7 @@
 
        public function __construct() {
                parent::__construct();
-               $this->mDescription = "Prune the Wikibase changes table to a 
maximum number of entries";
+               $this->addDescription( "Prune the Wikibase changes table to a 
maximum number of entries" );
 
                $this->addOption( 'number-of-days', 'Keep changes at least N 
days (deprecated).', false, true, 'n' );
                $this->addOption( 'keep-days', 'Keep changes at least N days.', 
false, true, 'd' );
diff --git a/repo/maintenance/rebuildEntityPerPage.php 
b/repo/maintenance/rebuildEntityPerPage.php
index f287fb5..de977a4 100644
--- a/repo/maintenance/rebuildEntityPerPage.php
+++ b/repo/maintenance/rebuildEntityPerPage.php
@@ -25,7 +25,7 @@
        public function __construct() {
                parent::__construct();
 
-               $this->mDescription = 'Rebuild the entites_per_page table';
+               $this->addDescription( 'Rebuild the entites_per_page table' );
 
                $this->addOption( 'rebuild-all', "Rebuild the entire table (per 
default, only missing entries are rebuild)" );
                $this->addOption( 'batch-size', "Number of rows to update per 
batch (100 by default)", false, true );
diff --git a/repo/maintenance/rebuildItemsPerSite.php 
b/repo/maintenance/rebuildItemsPerSite.php
index b8e4c0b..9dbc148 100644
--- a/repo/maintenance/rebuildItemsPerSite.php
+++ b/repo/maintenance/rebuildItemsPerSite.php
@@ -26,7 +26,7 @@
        public function __construct() {
                parent::__construct();
 
-               $this->mDescription = 'Rebuild the items_per_site table';
+               $this->addDescription( 'Rebuild the items_per_site table' );
 
                $this->addOption( 'batch-size', "Number of rows to update per 
batch (100 by default)", false, true );
        }
diff --git a/repo/maintenance/rebuildPropertyInfo.php 
b/repo/maintenance/rebuildPropertyInfo.php
index 09469e6..877444e 100644
--- a/repo/maintenance/rebuildPropertyInfo.php
+++ b/repo/maintenance/rebuildPropertyInfo.php
@@ -23,7 +23,7 @@
        public function __construct() {
                parent::__construct();
 
-               $this->mDescription = 'Rebuild the property info table.';
+               $this->addDescription( 'Rebuild the property info table.' );
 
                $this->addOption( 'rebuild-all', "Update property info for all 
properties (per default, only missing entries are created)" );
                $this->addOption( 'start-row', "The ID of the first row to 
update (useful for continuing aborted runs)", false, true );
diff --git a/repo/maintenance/rebuildTermsSearchKey.php 
b/repo/maintenance/rebuildTermsSearchKey.php
index 4816b1b..225862c 100644
--- a/repo/maintenance/rebuildTermsSearchKey.php
+++ b/repo/maintenance/rebuildTermsSearchKey.php
@@ -23,7 +23,7 @@
        public function __construct() {
                parent::__construct();
 
-               $this->mDescription = 'Rebuild the search key of the 
TermSQLCache';
+               $this->addDescription( 'Rebuild the search key of the 
TermSQLCache' );
 
                $this->addOption( 'only-missing', "Update only missing keys 
(per default, all keys are updated)" );
                $this->addOption( 'start-row', "The ID of the first row to 
update (useful for continuing aborted runs)", false, true );

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7ac6283122812337ca3c3b9cce741c247b060d57
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Addshore <addshorew...@gmail.com>
Gerrit-Reviewer: Aude <aude.w...@gmail.com>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to