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

Change subject: Avoid master queries in GET in onProtectionForm()
......................................................................


Avoid master queries in GET in onProtectionForm()

Bug: T92357
Change-Id: I47153fbc03b37ab424628a247b1a48427e696b1f
---
M frontend/FlaggedRevsUI.hooks.php
1 file changed, 2 insertions(+), 1 deletion(-)

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



diff --git a/frontend/FlaggedRevsUI.hooks.php b/frontend/FlaggedRevsUI.hooks.php
index 3ca0cbb..c474ad4 100644
--- a/frontend/FlaggedRevsUI.hooks.php
+++ b/frontend/FlaggedRevsUI.hooks.php
@@ -650,7 +650,8 @@
                        array() : array( 'disabled' => 'disabled' );
 
                # Get the current config/expiry
-               $config = FRPageConfig::getStabilitySettings( 
$article->getTitle(), FR_MASTER );
+               $mode = $wgRequest->wasPosted() ? FR_MASTER : 0;
+               $config = FRPageConfig::getStabilitySettings( 
$article->getTitle(), $mode );
                $oldExpirySelect = ( $config['expiry'] == 'infinity' ) ? 
'infinite' : 'existing';
 
                # Load requested restriction level, default to current level...

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I47153fbc03b37ab424628a247b1a48427e696b1f
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/FlaggedRevs
Gerrit-Branch: master
Gerrit-Owner: Aaron Schulz <asch...@wikimedia.org>
Gerrit-Reviewer: Addshore <addshorew...@gmail.com>
Gerrit-Reviewer: Catrope <roan.katt...@gmail.com>
Gerrit-Reviewer: Jackmcbarn <jackmcb...@gmail.com>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to