Page MenuHomePhabricator

Multiple calls to nonexistent, super ancient global function wfImageArchiveDir in uploading special pages
Closed, ResolvedPublic

Description

SocialProfile's Special:GiftManagerLogo and Special:SystemGiftManagerLogo as well as SportsTeams' (which doesn't have its own Phab component since it's a rather site-specific extension that has been for years and remains a work-in-progress) Special:SportsManagerLogo and Special:SportsTeamsManagerLogo special pages reference the global function wfImageArchiveDir, which, according to some googling, existed in MediaWiki core's ImageFunctions.php...back in the MediaWiki 1.5 (!) era, which is when the social tools were originally written.

Questions:

  1. What's the present-day replacement for wfImageArchiveDir?
  2. Can the code in practise even hit the case where that function would be called? wfImageArchiveDir is called by these special pages' saveTempUploadedFile method, which in turn is called by their stashSession, which does look like it could be called from somewhere...but I don't understand the concept of "stashed uploads" well enough, but I think that social tools don't really support that?

In any case the uploading special pages in Social-Tools are horrible and could use quite a bit of TLC...

Event Timeline

Vvjjkkii renamed this task from Multiple calls to nonexistent, super ancient global function wfImageArchiveDir in uploading special pages to 1aaaaaaaaa.Jul 1 2018, 1:01 AM
Vvjjkkii triaged this task as High priority.
Vvjjkkii updated the task description. (Show Details)
JJMC89 renamed this task from 1aaaaaaaaa to Multiple calls to nonexistent, super ancient global function wfImageArchiveDir in uploading special pages.Jul 1 2018, 2:49 AM
JJMC89 raised the priority of this task from High to Needs Triage.
JJMC89 updated the task description. (Show Details)
JJMC89 added a subscriber: Aklapper.

Change 729940 had a related patch set uploaded (by Jack Phoenix; author: Jack Phoenix):

[mediawiki/extensions/SportsTeams@master] Don't call a nonexistent global function, wfImageArchiveDir, which hasn't existed since the MW 1.10 days (or so)

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

Change 730224 had a related patch set uploaded (by Jack Phoenix; author: Jack Phoenix):

[mediawiki/extensions/SocialProfile@master] Don't call a nonexistent global function, wfImageArchiveDir, which hasn't existed since the MW 1.10 days (or so)

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

Change 730224 merged by jenkins-bot:

[mediawiki/extensions/SocialProfile@master] Don't call a nonexistent global function, wfImageArchiveDir, which hasn't existed since the MW 1.10 days (or so)

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

Change 729940 merged by Jack Phoenix:

[mediawiki/extensions/SportsTeams@master] Don't call a nonexistent global function, wfImageArchiveDir, which hasn't existed since the MW 1.10 days (or so)

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

ashley claimed this task.
ashley removed a project: Patch-For-Review.