ContentHandler: simplify tag tests
authorFramawiki <framawiki@tools.wmflabs.org>
Tue, 12 Dec 2017 18:21:02 +0000 (19:21 +0100)
committerLegoktm <legoktm@member.fsf.org>
Fri, 15 Dec 2017 04:28:28 +0000 (04:28 +0000)
Follows up 62713be990.

Change-Id: Id1f7e6ae470b461e26ef8b7346f715873ccea4c9

includes/content/ContentHandler.php

index 50cc9b5..edfc81c 100644 (file)
@@ -799,13 +799,13 @@ abstract class ContentHandler {
                }
 
                // New page created
-               if ( $flags & EDIT_NEW && $newContent && $newContent->getSize() > 0 ) {
-                       return 'newpage';
-               }
-
-               // New blank page
-               if ( $flags & EDIT_NEW && $newContent && $newContent->getSize() === 0 ) {
-                       return 'newblank';
+               if ( $flags & EDIT_NEW && $newContent ) {
+                       if ( $newContent->getSize() === 0 ) {
+                               // New blank page
+                               return 'newblank';
+                       } else {
+                               return 'newpage';
+                       }
                }
 
                // Removing more than 90% of the page