Page MenuHomePhabricator

Technical: Drop getContentTransformations and setContentTransformations in MobileContext
Closed, ResolvedPublic

Description

These two methods add unnecessary indirection to allow the Special:MobileOptions page to disable transformations that do not even run per the $isSpecialPage check inside the hook here:
https://gerrit.wikimedia.org/g/mediawiki/extensions/MobileFrontend/+/decbf33c9d73b09111134ef7ad46c075959f9a7b/includes/ExtMobileFrontend.php#96

sign off

Event Timeline

Jdlrobson updated the task description. (Show Details)

Change 628577 had a related patch set uploaded (by Ammarpad; owner: Ammarpad):
[mediawiki/extensions/MobileFrontend@master] Drop ContentTransformations getter and setter

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

Change 628577 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Drop ContentTransformations getter and setter

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

Ammarpad updated the task description. (Show Details)
phuedx closed this task as Resolved.Sep 21 2020, 11:04 AM

👍