Page MenuHomePhabricator

MobileFrontend makes UserLoginForm hook to pass incompatible parameter
Closed, ResolvedPublic


MobileFrontend creates it's own Userlogin and UserCreate templates. These templates inherits QuickTemplate while core templates inherits BaseTemplate. If another extension (which is included _after_ MobileFrontend in LocalSettings.php) requires functions from BaseTemplate in it's UserLoginFrom or UserCreateForm hook handler (which is legitim, because UserCreateTemplate is the expected class), php ends with a fatal error. MobileFrontend should also inherit BaseTemplate instead of QuickTemplate to prevent these problems.

Encountered with installed MobileFrontend and GoogleLogin extension.

Event Timeline

Florian claimed this task.
Florian raised the priority of this task from to Needs Triage.
Florian updated the task description. (Show Details)
Florian added a project: MobileFrontend.
Florian added a subscriber: Florian.
Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change 203622 had a related patch set uploaded (by Florianschmidtwelzow):
Use BaseTemplate instead of QuickTemplate to create UserLoginAndCreateTemplate

Change 203622 merged by jenkins-bot:
Use BaseTemplate instead of QuickTemplate to create UserLoginAndCreateTemplate

Florian added a project: GoogleLogin.
Florian removed a project: Patch-For-Review.
Florian set Security to None.