| Index: trunk/phase3/RELEASE-NOTES |
| — | — | @@ -66,6 +66,7 @@ |
| 67 | 67 | * Make rev_deleted log entries more intelligible. |
| 68 | 68 | * Logs are grouped under date headings similar to enhanced changes list |
| 69 | 69 | * (6943) Added PAGESINCATEGORY: magic word |
| | 70 | +* (13624) Fix regression with manual thumb= parameter on images |
| 70 | 71 | |
| 71 | 72 | |
| 72 | 73 | === Bug fixes in 1.13 === |
| Index: trunk/phase3/includes/Parser.php |
| — | — | @@ -4465,7 +4465,16 @@ |
| 4466 | 4466 | $validated = $handler->validateParam( $paramName, $value ); |
| 4467 | 4467 | } else { |
| 4468 | 4468 | # Validate internal parameters |
| 4469 | | - $validated = ( $value === false || is_numeric( trim( $value ) ) ); |
| | 4469 | + switch( $paramName ) { |
| | 4470 | + case "manualthumb": |
| | 4471 | + /// @fixme - possibly check validity here? |
| | 4472 | + /// downstream behavior seems odd with missing manual thumbs. |
| | 4473 | + $validated = true; |
| | 4474 | + break; |
| | 4475 | + default: |
| | 4476 | + // Most other things appear to be empty or numeric... |
| | 4477 | + $validated = ( $value === false || is_numeric( trim( $value ) ) ); |
| | 4478 | + } |
| 4470 | 4479 | } |
| 4471 | 4480 | |
| 4472 | 4481 | if ( $validated ) { |