Page MenuHomePhabricator

Rework Special:Upload JS to use api and serverside upload checks
Open, LowPublic

Description

It would be best if we retooled upload.js into a single ajax/api call.

one that checks for:
-name exists
-name allowed (file extensions checks)
-name normalization (think $wgCapitalizedNamespaces)

Better to do these checks serverside, then to reproduce them (often incompletely) in Javascript on the clientside.


Version: unspecified
Severity: enhancement

Details

Reference
bz24217

Event Timeline

bzimport raised the priority of this task from to Low.Nov 21 2014, 11:10 PM
bzimport set Reference to bz24217.
bzimport added a subscriber: Unknown Object (MLST).
TheDJ created this task.Jul 1 2010, 8:00 PM

Is this still valid?

Jdforrester-WMF moved this task from Untriaged to Backlog on the Multimedia board.Sep 4 2015, 5:58 PM
Restricted Application added subscribers: Steinsplitter, Matanya, Aklapper. · View Herald TranscriptSep 4 2015, 5:58 PM

@matmarex I know you guys changed quite some things in the upload script pipelines, not sure if this was part of it/came up.

We didn't touch it, but Special:Upload does some AJAX checks these days.

gpaumier removed a subscriber: gpaumier.Jul 18 2018, 6:01 PM