MaxSem has uploaded a new change for review.
https://gerrit.wikimedia.org/r/85388
Change subject: Remove b/c cookie code
......................................................................
Remove b/c cookie code
The old cookies should've expired since July 30 by now
Change-Id: Idcea645c7d3e0eafb33947ad64b2785e23735064
---
M includes/MobileContext.php
1 file changed, 1 insertion(+), 13 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MobileFrontend
refs/changes/88/85388/1
diff --git a/includes/MobileContext.php b/includes/MobileContext.php
index 847a513..ee3031e 100644
--- a/includes/MobileContext.php
+++ b/includes/MobileContext.php
@@ -192,20 +192,8 @@
if ( $mobileAction === 'alpha' || $mobileAction ===
'beta' ) {
$this->mobileMode = $mobileAction;
} else {
- // First check old cookie
- // @todo: Remove in September when old cookies
expire
$req = $this->getRequest();
- $alpha = $req->getCookie( 'mf_alpha', '' );
- if ( $alpha == 1 ) {
- $req->response()->setcookie(
'mf_alpha', '', 0, '' );
- $this->setMobileMode( 'alpha' );
- } else {
- $this->mobileMode =
$this->getRequest()->getCookie( 'optin', '' );
- // Old cookie format - handle it but no
point in overwriting the cookie
- if ( $this->mobileMode == '1' ) {
- $this->mobileMode = 'beta';
- }
- }
+ $this->mobileMode = $req->getCookie( 'optin',
'' );
}
}
return $this->mobileMode;
--
To view, visit https://gerrit.wikimedia.org/r/85388
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Idcea645c7d3e0eafb33947ad64b2785e23735064
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: MaxSem <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
