Page MenuHomePhabricator

Update the celerity resource controller to be module-aware
AbandonedPublic

Authored by hach-que on Dec 11 2013, 3:05 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 21, 4:15 AM
Unknown Object (File)
Sun, Nov 3, 7:06 PM
Unknown Object (File)
Oct 21 2024, 10:52 AM
Unknown Object (File)
Oct 16 2024, 12:32 PM
Unknown Object (File)
Oct 13 2024, 5:09 PM
Unknown Object (File)
Sep 4 2024, 6:54 PM
Unknown Object (File)
Aug 12 2024, 2:41 PM
Unknown Object (File)
Aug 5 2024, 4:02 AM

Details

Summary

Depends on D7753. This updates the celerity resource controller so that it will serve up resources located in external libphutil modules.

Test Plan

Ensured that Phabricator resources still rendered correctly (I did break it at one point so I know they're working again). Also went to a URL in another module's resource map and Phabricator served it up correctly.

Diff Detail

Branch
celerity-external-lookup
Lint
Lint Passed
Unit
Tests Passed

Event Timeline

src/infrastructure/celerity/CelerityResourceController.php
63–67

I don't know what performance impacts this change might have...

Throwing this out since @epriestley is implementing a better version.