From edf926dfe23595c6049484cc2f15e4c919827968 Mon Sep 17 00:00:00 2001 From: Sam Reed Date: Wed, 4 Aug 2010 21:21:35 +0000 Subject: [PATCH] Followup r70479 If string is required, but it's value === '', that's missing! --- includes/api/ApiBase.php | 4 ++++ 1 file changed, 4 insertions(+) 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 -- 2.20.1