Page MenuHomePhabricator
Differential D19200

PhabricatorMemeEngine HA HA HA HA
ClosedPublic
Actions

Authored by epriestley on Mar 8 2018, 5:38 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jan 25, 12:38 AM
Unknown Object (File)
Sat, Jan 25, 12:37 AM
Unknown Object (File)
Sat, Jan 25, 12:37 AM
Unknown Object (File)
Tue, Jan 21, 9:50 AM
Unknown Object (File)
Sun, Jan 19, 3:15 AM
Unknown Object (File)
Sun, Jan 12, 5:38 PM
Unknown Object (File)
Sun, Jan 5, 7:38 PM
Unknown Object (File)
Mon, Dec 30, 9:21 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T5258: Memes use older image transform code without modern error handling
T13101: The "{img ...}" and "{meme ...}" remarkup rules violate the new Content-Security-Policy
Commits
rPc7408f27977d: PhabricatorMemeEngine HA HA HA HA
Summary

Depends on D19198. Ref T13101. Ref T5258. Pull compositing logic out of the Controller.

This is moving toward fixing memes in email.

Test Plan

Used new and old memes. Used API memes.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
src/
2 lines
applications/
macro/
conduit/
21 lines
controller/
100 lines
engine/
163 lines
markup/
30 lines

Diff 45987

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/macro/conduit/MacroCreateMemeConduitAPIMethod.php

Loading...
View Options

src/applications/macro/controller/PhabricatorMacroMemeController.php

Loading...
View Options

src/applications/macro/engine/PhabricatorMemeEngine.php

Loading...
View Options

src/applications/macro/markup/PhabricatorMemeRemarkupRule.php

Loading...
Phabricator · Built by Phacility