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