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