| Index: trunk/phase3/includes/api/ApiBase.php |
| — | — | @@ -664,7 +664,7 @@ |
| 665 | 665 | case 'NULL': // nothing to do |
| 666 | 666 | break; |
| 667 | 667 | case 'string': |
| 668 | | - if ( $value === '' ) { |
| | 668 | + if ( $required && $value === '' ) { |
| 669 | 669 | $this->dieUsageMsg( array( 'missingparam', $paramName ) ); |
| 670 | 670 | } |
| 671 | 671 | |
| Index: trunk/phase3/RELEASE-NOTES |
| — | — | @@ -350,7 +350,8 @@ |
| 351 | 351 | * (bug 24330) Add &redirect parameter to ?action=edit |
| 352 | 352 | * (bug 24722) For list=allusers&auprop=blockinfo, only show blockedby and blockreason if |
| 353 | 353 | the user is actually blocked. |
| 354 | | -* Add format=dump and format=dumpfm, outputs results in PHP's var_dump() format. |
| | 354 | +* Add format=dump and format=dumpfm, outputs results in PHP's var_dump() format |
| | 355 | +* For required string parameters, if '' is provided, this is now classed as missing |
| 355 | 356 | |
| 356 | 357 | === Languages updated in 1.17 === |
| 357 | 358 | |