Page MenuHomePhabricator
Differential D7871

Make resolveResources() and packageResources() private on CelerityResourceMap
ClosedPublic
Actions

Authored by epriestley on Jan 1 2014, 12:36 AM.
Tags
None
Referenced Files
F13293620: D7871.id17809.diff
Wed, Jun 5, 10:17 AM
F13290691: D7871.diff
Tue, Jun 4, 7:46 PM
F13288415: D7871.diff
Tue, Jun 4, 10:25 AM
F13281264: D7871.diff
Sun, Jun 2, 10:29 AM
F13267276: D7871.diff
Wed, May 29, 3:06 AM
F13252802: D7871.diff
Sat, May 25, 2:05 AM
F13247753: D7871.id17800.diff
Thu, May 23, 10:16 PM
F13246558: D7871.id17809.diff
Thu, May 23, 11:25 AM
View All Files
Subscribers
aran

Details

Reviewers
hach-que
btrahan
Maniphest Tasks
T4222: Permit Celerity to serve resources in external libphutil modules
Commits
Restricted Diffusion Commit
rP60cb65bfbb2d: Make resolveResources() and packageResources() private on CelerityResourceMap
Summary

Ref T4222. These are the last two "return a big ball of mud" methods. Make the API stronger so I can swap out the implementations.

Test Plan

Reloaded pages.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Revision Contents
Changeset List

PathSize
src/
applications/
phame/
skins/
4 lines
infrastructure/
celerity/
75 lines
2 lines
48 lines
2 lines

Diff 17809

View Options

src/applications/phame/skins/PhameBasicTemplateBlogSkin.php

Loading...
View Options

src/infrastructure/celerity/CelerityResourceMap.php

Loading...
View Options

src/infrastructure/celerity/CelerityResourceTransformer.php

Loading...
View Options

src/infrastructure/celerity/CelerityStaticResourceResponse.php

Loading...
View Options

src/infrastructure/celerity/api.php

Loading...
Phabricator · Built by Phacility