jenkins-bot has submitted this change and it was merged.

Change subject: EditPage::newSectionSummary should return a value in all code 
paths
......................................................................


EditPage::newSectionSummary should return a value in all code paths

Followup I72890c06.

Change-Id: Id63c3f3d2c180d193c37091dfe5f1c56e3edb45f
---
M includes/EditPage.php
1 file changed, 1 insertion(+), 2 deletions(-)

Approvals:
  Jackmcbarn: Looks good to me, approved
  Eranroz: Looks good to me, but someone else must approve
  jenkins-bot: Verified



diff --git a/includes/EditPage.php b/includes/EditPage.php
index 6454cfa..ff63fe8 100644
--- a/includes/EditPage.php
+++ b/includes/EditPage.php
@@ -1468,9 +1468,8 @@
                        $cleanSummary = $wgParser->stripSectionName( 
$this->summary );
                        return wfMessage( 'newsectionsummary' )
                                ->rawParams( $cleanSummary 
)->inContentLanguage()->text();
-               } else {
-                       return $this->summary;
                }
+               return $this->summary;
        }
 
        /**

-- 
To view, visit https://gerrit.wikimedia.org/r/151645
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Id63c3f3d2c180d193c37091dfe5f1c56e3edb45f
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Anomie <[email protected]>
Gerrit-Reviewer: Aaron Schulz <[email protected]>
Gerrit-Reviewer: Eranroz <[email protected]>
Gerrit-Reviewer: Jackmcbarn <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to