From: Sam Reed <reedy@users.mediawiki.org>
Date: Wed, 4 Aug 2010 21:21:35 +0000 (+0000)
Subject: Followup r70479
X-Git-Tag: 1.31.0-rc.0~35717
X-Git-Url: http://git.cyclocoop.org/%40spipnet%40?a=commitdiff_plain;h=edf926dfe23595c6049484cc2f15e4c919827968;p=lhc%2Fweb%2Fwiklou.git

Followup r70479

If string is required, but it's value === '', that's missing!
---

diff --git a/includes/api/ApiBase.php b/includes/api/ApiBase.php
index 44acddadbf..8c79da581b 100644
--- a/includes/api/ApiBase.php
+++ b/includes/api/ApiBase.php
@@ -662,6 +662,10 @@ abstract class ApiBase {
 					case 'NULL': // nothing to do
 						break;
 					case 'string': // nothing to do
+						if ( $value === '' ) {
+							$this->dieUsageMsg( array( 'missingparam', $paramName ) );
+						}
+					
 						break;
 					case 'integer': // Force everything using intval() and optionally validate limits