Legoktm has uploaded a new change for review.
https://gerrit.wikimedia.org/r/250844
Change subject: Fix @return phpdoc
......................................................................
Fix @return phpdoc
Change-Id: I18a6a2b153459099d9c065757f5854614ce529c2
---
M includes/QuickSurveys.hooks.php
M includes/Survey.php
2 files changed, 2 insertions(+), 3 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/QuickSurveys
refs/changes/44/250844/1
diff --git a/includes/QuickSurveys.hooks.php b/includes/QuickSurveys.hooks.php
index b565e9c..7cc2f81 100644
--- a/includes/QuickSurveys.hooks.php
+++ b/includes/QuickSurveys.hooks.php
@@ -126,7 +126,7 @@
/**
* Helper method for getting enabled quick surveys
*
- * @return array Enabled survey configuration array
+ * @return Survey[] Enabled survey configuration array
*/
private static function getEnabledSurveys() {
$config = ConfigFactory::getDefaultInstance()->makeConfig(
'quicksurveys' );
diff --git a/includes/Survey.php b/includes/Survey.php
index 06c413a..c8f39fe 100644
--- a/includes/Survey.php
+++ b/includes/Survey.php
@@ -56,12 +56,11 @@
/**
* Gets the list of i18n message keys that the survey uses
*
- * @return [string]
+ * @return string
*/
public function getMessages() {
return array(
$this->question,
-
// FIXME: Should description be optional?
$this->description,
);
--
To view, visit https://gerrit.wikimedia.org/r/250844
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I18a6a2b153459099d9c065757f5854614ce529c2
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/QuickSurveys
Gerrit-Branch: master
Gerrit-Owner: Legoktm <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
