| Index: trunk/phase3/includes/EditPage.php |
| — | — | @@ -723,8 +723,11 @@ |
| 724 | 724 | return true; |
| 725 | 725 | } |
| 726 | 726 | |
| 727 | | - # Handle the user preference to force summaries here |
| 728 | | - if( $this->section != 'new' && !$this->allowBlankSummary && $wgUser->getOption( 'forceeditsummary' ) ) { |
| | 727 | + $oldtext = $this->mArticle->getContent(); |
| | 728 | + |
| | 729 | + # Handle the user preference to force summaries here, but not for null edits |
| | 730 | + if( $this->section != 'new' && !$this->allowBlankSummary && $wgUser->getOption( 'forceeditsummary') |
| | 731 | + && 0 != strcmp($oldtext, $text) ) { |
| 729 | 732 | if( md5( $this->summary ) == $this->autoSumm ) { |
| 730 | 733 | $this->missingSummary = true; |
| 731 | 734 | wfProfileOut( $fname ); |
| Index: trunk/phase3/RELEASE-NOTES |
| — | — | @@ -149,8 +149,9 @@ |
| 150 | 150 | * (bug 7819) Move automatic redirect edit summary after pre-save transform |
| 151 | 151 | to work properly with subst: fun |
| 152 | 152 | * (bug 7826) Fix typos in two English messages. |
| | 153 | +* (bug 5365) Stop users being prompted to enter an edit summary for null edits, |
| | 154 | + if they have selected that option in preferences. |
| 153 | 155 | |
| 154 | | - |
| 155 | 156 | == Languages updated == |
| 156 | 157 | |
| 157 | 158 | * Bishnupriya Manipuri (bpy) |