Page MenuHomePhabricator
Create Task
Maniphest T123006

Do not directly access $_SESSION superglobal in DonationInterface
Closed, ResolvedPublic2 Estimated Story Points
Actions

Description

Make an accessor class and plug it in. Suddenly, this is urgent. Core change a73c5b7395a07d490f7052fd3b2491ebd656b190 adds a SessionManager class and now none of our tests pass. We need to patch DI similar to how CentralAuth was patched in 0f38ef2177433db7a27e38516c9588eebee07da3, so it works with or without SessionManager.

Related Objects

Event Timeline

• cwdent created this task.Jan 6 2016, 10:41 PM
• cwdent raised the priority of this task from to Needs Triage.
• cwdent updated the task description. (Show Details)
• cwdent added projects: Fundraising-Backlog, MediaWiki-extensions-DonationInterface.
• cwdent subscribed.
Restricted Application added subscribers: StudiesWorld, Aklapper. · View Herald TranscriptJan 6 2016, 10:41 PM
Ejegg updated the task description. (Show Details)Jan 13 2016, 7:41 PM
Ejegg added projects: Unplanned-Sprint-Work, Fundraising Sprint Asbestos Removal 2016.
Ejegg set Security to None.
Ejegg edited a custom field.
gerritbot subscribed.Jan 13 2016, 10:43 PM

Change 264017 had a related patch set uploaded (by Ejegg):
Use SessionManager instead of deprecated fn

https://gerrit.wikimedia.org/r/264017

gerritbot added a project: Patch-For-Review.Jan 13 2016, 10:43 PM
• DStrine moved this task from Triage to Current Sprint & Completed in Q3 1516 on the Fundraising-Backlog board.Jan 19 2016, 5:50 PM
Ejegg claimed this task.Jan 19 2016, 9:29 PM
Ejegg moved this task from Backlog to Review on the Fundraising Sprint Asbestos Removal 2016 board.
gerritbot added a comment.Jan 20 2016, 8:11 PM

Change 265367 had a related patch set uploaded (by Ejegg):
Update session handling for 1.27

https://gerrit.wikimedia.org/r/265367

gerritbot added a comment.Jan 20 2016, 9:01 PM

Change 265381 had a related patch set uploaded (by Ejegg):
$_SESSION cleanup: Amazon adapter

https://gerrit.wikimedia.org/r/265381

gerritbot added a comment.Jan 20 2016, 9:21 PM

Change 265384 had a related patch set uploaded (by Ejegg):
$_SESSION cleanup: session_velocity filter

https://gerrit.wikimedia.org/r/265384

gerritbot added a comment.Jan 20 2016, 9:21 PM

Change 265385 had a related patch set uploaded (by Ejegg):
$_SESSION cleanup: GatewayPage

https://gerrit.wikimedia.org/r/265385

gerritbot added a comment.Jan 20 2016, 9:24 PM

Change 265389 had a related patch set uploaded (by Ejegg):
$_SESSION cleanup: Worldpay adapter

https://gerrit.wikimedia.org/r/265389

gerritbot added a comment.Jan 20 2016, 9:26 PM

Change 265390 had a related patch set uploaded (by Ejegg):
$_SESSION cleanup: instance methods, edit tokens

https://gerrit.wikimedia.org/r/265390

gerritbot added a comment.Jan 20 2016, 9:31 PM

Change 265392 had a related patch set uploaded (by Ejegg):
$_SESSION cleanup: GlobalCollect

https://gerrit.wikimedia.org/r/265392

gerritbot added a comment.Jan 20 2016, 9:45 PM

Change 265396 had a related patch set uploaded (by Ejegg):
$_SESSION cleanup: miscellaneous tests

https://gerrit.wikimedia.org/r/265396

• DStrine added a project: Fundraising Sprint Bloodletting 2016.Jan 20 2016, 11:51 PM
Ejegg moved this task from Backlog to Review on the Fundraising Sprint Bloodletting 2016 board.Jan 21 2016, 12:02 AM
gerritbot added a comment.Jan 21 2016, 6:38 PM

Change 265367 merged by jenkins-bot:
Update session handling for 1.27

https://gerrit.wikimedia.org/r/265367

ReleaseTaggerBot added a project: MW-1.27-release (WMF-deploy-2016-02-02_(1.27.0-wmf.12)).Jan 21 2016, 8:01 PM
gerritbot added a comment.Jan 21 2016, 9:50 PM

Change 264017 abandoned by Ejegg:
WIP update session handling for 1.27

Reason:
Split into smaller commits

https://gerrit.wikimedia.org/r/264017

gerritbot added a comment.Jan 21 2016, 11:34 PM

Change 265381 merged by jenkins-bot:
$_SESSION cleanup: Amazon adapter

https://gerrit.wikimedia.org/r/265381

gerritbot added a comment.Jan 21 2016, 11:59 PM

Change 265384 merged by jenkins-bot:
$_SESSION cleanup: session_velocity filter

https://gerrit.wikimedia.org/r/265384

gerritbot added a comment.Jan 21 2016, 11:59 PM

Change 265385 merged by jenkins-bot:
$_SESSION cleanup: GatewayPage

https://gerrit.wikimedia.org/r/265385

gerritbot added a comment.Jan 22 2016, 12:00 AM

Change 265389 merged by jenkins-bot:
$_SESSION cleanup: Worldpay adapter

https://gerrit.wikimedia.org/r/265389

gerritbot added a comment.Jan 22 2016, 12:00 AM

Change 265390 merged by jenkins-bot:
$_SESSION cleanup: instance methods, edit tokens

https://gerrit.wikimedia.org/r/265390

gerritbot added a comment.Jan 22 2016, 12:00 AM

Change 265392 merged by jenkins-bot:
$_SESSION cleanup: GlobalCollect

https://gerrit.wikimedia.org/r/265392

gerritbot added a comment.Jan 22 2016, 12:00 AM

Change 265396 merged by jenkins-bot:
$_SESSION cleanup: miscellaneous tests

https://gerrit.wikimedia.org/r/265396

Ejegg moved this task from Review to Pending Deployment on the Fundraising Sprint Bloodletting 2016 board.Jan 22 2016, 7:13 PM
• DStrine removed a project: Unplanned-Sprint-Work.Jan 22 2016, 7:18 PM
Ejegg closed this task as Resolved.Feb 3 2016, 10:47 PM
Ejegg moved this task from Pending Deployment to Done on the Fundraising Sprint Bloodletting 2016 board.
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits