jenkins-bot has submitted this change and it was merged.
Change subject: Update documentation for TranslateUtils::getIcon
......................................................................
Update documentation for TranslateUtils::getIcon
Also readd the warning removed in Change-Id:
I24fb73e31ab9433bb6046014e5e6937910a8f8e4 with an updated
description.
Change-Id: I654ba943be0effd3d0568e1eca11115d3355f6ae
---
M TranslateUtils.php
1 file changed, 6 insertions(+), 2 deletions(-)
Approvals:
Nikerabbit: Looks good to me, approved
jenkins-bot: Verified
diff --git a/TranslateUtils.php b/TranslateUtils.php
index edef392..ace3a8a 100644
--- a/TranslateUtils.php
+++ b/TranslateUtils.php
@@ -395,8 +395,11 @@
}
/**
- * For the give message group get the icons in vector and raster formats
- * @return array
+ * Get URLs for icons if available.
+ * @param MessageGroup $g
+ * @param int $size Length of the edge of a bounding box to fit the
icon.
+ * @return null|array
+ * @since 2013-04-01
*/
public static function getIcon( MessageGroup $g, $size ) {
global $wgServer;
@@ -410,6 +413,7 @@
$filename = substr( $icon, 7 );
$file = wfFindFile( $filename );
if ( !$file ) {
+ wfWarn( "Unknown message group icon file $icon" );
return null;
}
--
To view, visit https://gerrit.wikimedia.org/r/56139
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I654ba943be0effd3d0568e1eca11115d3355f6ae
Gerrit-PatchSet: 4
Gerrit-Project: mediawiki/extensions/Translate
Gerrit-Branch: master
Gerrit-Owner: Nikerabbit <niklas.laxst...@gmail.com>
Gerrit-Reviewer: Amire80 <amir.ahar...@mail.huji.ac.il>
Gerrit-Reviewer: Nikerabbit <niklas.laxst...@gmail.com>
Gerrit-Reviewer: Siebrand <siebr...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
