| Index: trunk/phase3/includes/api/ApiDelete.php |
| — | — | @@ -161,7 +161,8 @@ |
| 162 | 162 | |
| 163 | 163 | if( !FileDeleteForm::haveDeletableFile($file, $oldfile, $oldimage) ) |
| 164 | 164 | return array(array('nofile')); |
| 165 | | - |
| | 165 | + if (is_null($reason)) # Log and RC don't like null reasons |
| | 166 | + $reason = ''; |
| 166 | 167 | $status = FileDeleteForm::doDelete( $title, $file, $oldimage, $reason, $suppress ); |
| 167 | 168 | |
| 168 | 169 | if( !$status->isGood() ) |
| Index: trunk/phase3/RELEASE-NOTES |
| — | — | @@ -505,6 +505,7 @@ |
| 506 | 506 | * (bug 16541) Added block expiry timestamp to list=logevents output |
| 507 | 507 | * (bug 16613) action=protect doesn't tell when &cascade was set but cascading |
| 508 | 508 | protection wasn't allowed |
| | 509 | +* (bug 16626) action=delete now correctly handles empty "reason" param |
| 509 | 510 | |
| 510 | 511 | === Languages updated in 1.14 === |
| 511 | 512 | |