[MediaWiki-commits] [Gerrit] Doc: Reformat @params declaration - change (mediawiki/core)

2014-07-24 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review.

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

Change subject: Doc: Reformat @params declaration
..

Doc: Reformat @params declaration

I have not found documentation about the @params command, so this seems
not valid.
I have moved the text to the corresponding @param

Change-Id: I443866d602b60570278289e6e24a77d37314cede
---
M includes/jobqueue/JobQueueFederated.php
M includes/jobqueue/JobQueueRedis.php
M includes/jobqueue/aggregator/JobQueueAggregatorMemc.php
M includes/jobqueue/aggregator/JobQueueAggregatorRedis.php
4 files changed, 4 insertions(+), 8 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/82/148982/1

diff --git a/includes/jobqueue/JobQueueFederated.php 
b/includes/jobqueue/JobQueueFederated.php
index d6f9560..01d7ec4 100644
--- a/includes/jobqueue/JobQueueFederated.php
+++ b/includes/jobqueue/JobQueueFederated.php
@@ -64,7 +64,7 @@
const CACHE_TTL_LONG = 300; // integer; seconds to cache info that is 
kept up to date
 
/**
-* @params include:
+* @param array $params Possible keys:
 *  - sectionsByWiki  : A map of wiki IDs to section names.
 *  Wikis will default to using the section 
default.
 *  - partitionsBySection : Map of section names to maps of (partition 
name = weight).
@@ -80,7 +80,6 @@
 *  different partition queues. This improves 
availability
 *  during failure, at the cost of added 
latency and somewhat
 *  less reliable job de-duplication mechanisms.
-* @param array $params
 * @throws MWException
 */
protected function __construct( array $params ) {
diff --git a/includes/jobqueue/JobQueueRedis.php 
b/includes/jobqueue/JobQueueRedis.php
index 522bae1..3519eac 100644
--- a/includes/jobqueue/JobQueueRedis.php
+++ b/includes/jobqueue/JobQueueRedis.php
@@ -73,7 +73,7 @@
protected $key;
 
/**
-* @params include:
+* @param array $params Possible keys:
 *   - redisConfig : An array of parameters to 
RedisConnectionPool::__construct().
 *   Note that the serializer option is ignored as 
none is always used.
 *   - redisServer : A hostname/port combination or the absolute path 
of a UNIX socket.
@@ -83,7 +83,6 @@
 *   - daemonized  : Set to true if the redisJobRunnerService runs in 
the background.
 *   This will disable job recycling/undelaying from 
the MediaWiki side
 *   to avoid redundance and out-of-sync configuration.
-* @param array $params
 */
public function __construct( array $params ) {
parent::__construct( $params );
diff --git a/includes/jobqueue/aggregator/JobQueueAggregatorMemc.php 
b/includes/jobqueue/aggregator/JobQueueAggregatorMemc.php
index d733a42..ae266ef 100644
--- a/includes/jobqueue/aggregator/JobQueueAggregatorMemc.php
+++ b/includes/jobqueue/aggregator/JobQueueAggregatorMemc.php
@@ -34,11 +34,10 @@
protected $cacheTTL; // integer; seconds
 
/**
-* @params include:
+* @param array $params Possible keys:
 *   - objectCache : Name of an object cache registered in 
$wgObjectCaches.
 *   This defaults to the one specified by 
$wgMainCacheType.
 *   - cacheTTL: Seconds to cache the aggregate data before 
regenerating.
-* @param array $params
 */
protected function __construct( array $params ) {
parent::__construct( $params );
diff --git a/includes/jobqueue/aggregator/JobQueueAggregatorRedis.php 
b/includes/jobqueue/aggregator/JobQueueAggregatorRedis.php
index e8e8e30..db9e764 100644
--- a/includes/jobqueue/aggregator/JobQueueAggregatorRedis.php
+++ b/includes/jobqueue/aggregator/JobQueueAggregatorRedis.php
@@ -36,14 +36,13 @@
protected $servers;
 
/**
-* @params include:
+* @param array $params Possible keys:
 *   - redisConfig  : An array of parameters to 
RedisConnectionPool::__construct().
 *   - redisServers : Array of server entries, the first being the 
primary and the
 *others being fallback servers. Each entry is 
either a hostname/port
 *combination or the absolute path of a UNIX socket.
 *If a hostname is specified but no port, the 
standard port number
 *6379 will be used. Required.
-* @param array $params
 */
protected function __construct( array $params ) {
parent::__construct( $params );

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


[MediaWiki-commits] [Gerrit] Doc: Reformat @params declaration - change (mediawiki/core)

2014-07-24 Thread jenkins-bot (Code Review)
jenkins-bot has submitted this change and it was merged.

Change subject: Doc: Reformat @params declaration
..


Doc: Reformat @params declaration

I have not found documentation about the @params command, so this seems
not valid.
I have moved the text to the corresponding @param

Change-Id: I443866d602b60570278289e6e24a77d37314cede
---
M includes/jobqueue/JobQueueFederated.php
M includes/jobqueue/JobQueueRedis.php
M includes/jobqueue/aggregator/JobQueueAggregatorMemc.php
M includes/jobqueue/aggregator/JobQueueAggregatorRedis.php
4 files changed, 4 insertions(+), 8 deletions(-)

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



diff --git a/includes/jobqueue/JobQueueFederated.php 
b/includes/jobqueue/JobQueueFederated.php
index d6f9560..01d7ec4 100644
--- a/includes/jobqueue/JobQueueFederated.php
+++ b/includes/jobqueue/JobQueueFederated.php
@@ -64,7 +64,7 @@
const CACHE_TTL_LONG = 300; // integer; seconds to cache info that is 
kept up to date
 
/**
-* @params include:
+* @param array $params Possible keys:
 *  - sectionsByWiki  : A map of wiki IDs to section names.
 *  Wikis will default to using the section 
default.
 *  - partitionsBySection : Map of section names to maps of (partition 
name = weight).
@@ -80,7 +80,6 @@
 *  different partition queues. This improves 
availability
 *  during failure, at the cost of added 
latency and somewhat
 *  less reliable job de-duplication mechanisms.
-* @param array $params
 * @throws MWException
 */
protected function __construct( array $params ) {
diff --git a/includes/jobqueue/JobQueueRedis.php 
b/includes/jobqueue/JobQueueRedis.php
index 522bae1..3519eac 100644
--- a/includes/jobqueue/JobQueueRedis.php
+++ b/includes/jobqueue/JobQueueRedis.php
@@ -73,7 +73,7 @@
protected $key;
 
/**
-* @params include:
+* @param array $params Possible keys:
 *   - redisConfig : An array of parameters to 
RedisConnectionPool::__construct().
 *   Note that the serializer option is ignored as 
none is always used.
 *   - redisServer : A hostname/port combination or the absolute path 
of a UNIX socket.
@@ -83,7 +83,6 @@
 *   - daemonized  : Set to true if the redisJobRunnerService runs in 
the background.
 *   This will disable job recycling/undelaying from 
the MediaWiki side
 *   to avoid redundance and out-of-sync configuration.
-* @param array $params
 */
public function __construct( array $params ) {
parent::__construct( $params );
diff --git a/includes/jobqueue/aggregator/JobQueueAggregatorMemc.php 
b/includes/jobqueue/aggregator/JobQueueAggregatorMemc.php
index d733a42..ae266ef 100644
--- a/includes/jobqueue/aggregator/JobQueueAggregatorMemc.php
+++ b/includes/jobqueue/aggregator/JobQueueAggregatorMemc.php
@@ -34,11 +34,10 @@
protected $cacheTTL; // integer; seconds
 
/**
-* @params include:
+* @param array $params Possible keys:
 *   - objectCache : Name of an object cache registered in 
$wgObjectCaches.
 *   This defaults to the one specified by 
$wgMainCacheType.
 *   - cacheTTL: Seconds to cache the aggregate data before 
regenerating.
-* @param array $params
 */
protected function __construct( array $params ) {
parent::__construct( $params );
diff --git a/includes/jobqueue/aggregator/JobQueueAggregatorRedis.php 
b/includes/jobqueue/aggregator/JobQueueAggregatorRedis.php
index e8e8e30..db9e764 100644
--- a/includes/jobqueue/aggregator/JobQueueAggregatorRedis.php
+++ b/includes/jobqueue/aggregator/JobQueueAggregatorRedis.php
@@ -36,14 +36,13 @@
protected $servers;
 
/**
-* @params include:
+* @param array $params Possible keys:
 *   - redisConfig  : An array of parameters to 
RedisConnectionPool::__construct().
 *   - redisServers : Array of server entries, the first being the 
primary and the
 *others being fallback servers. Each entry is 
either a hostname/port
 *combination or the absolute path of a UNIX socket.
 *If a hostname is specified but no port, the 
standard port number
 *6379 will be used. Required.
-* @param array $params
 */
protected function __construct( array $params ) {
parent::__construct( $params );

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

Gerrit-MessageType: merged
Gerrit-Change-Id: