Page MenuHomePhabricator

CodeEditor should use ContentHandler
Closed, ResolvedPublic


The code for this extension predates Daniel K.'s work on ContentHandler in core, and as a result its approach to content-types is archaic and ad-hoc. Rather than check the file extension and then give up, it should instead rely on the interface exposed by ContentHandler and ContentModel classes.

Version: unspecified
Severity: normal



Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 1:09 AM
bzimport added a project: CodeEditor.
bzimport set Reference to bz42593.
bzimport added a subscriber: Unknown Object (MLST).
ori created this task.Dec 1 2012, 5:45 AM

Change 104721 had a related patch set uploaded by TheDJ:
ContentModel: Use CM to choose CE language

Slightly related: CM for Scribunto

Change 104721 merged by jenkins-bot:
Use page ContentModel to choose editor language

Patch was merged a while ago - is there more work left to do here (if yes: please reset the bug report status to NEW or ASSIGNED), or can you close this ticket as RESOLVED FIXED?