Page MenuHomePhabricator

Point the Phabricator resource map back to webroot/, not webroot/rsrc/
ClosedPublic

Authored by epriestley on Jan 1 2014, 3:19 AM.
Tags
None
Referenced Files
F18759155: D7875.id17817.diff
Mon, Oct 6, 2:10 AM
F18644280: D7875.id17814.diff
Fri, Sep 19, 4:42 AM
F18584060: D7875.id.diff
Sep 11 2025, 11:51 AM
F18576602: D7875.diff
Sep 10 2025, 4:10 PM
F18557075: D7875.id.diff
Sep 8 2025, 7:50 PM
F18513076: D7875.diff
Sep 5 2025, 9:08 AM
F18100805: D7875.id17817.diff
Aug 9 2025, 12:06 PM
F18092898: D7875.id.diff
Aug 7 2025, 1:53 PM
Subscribers

Details

Summary

Ref T4222. Earlier, I adjusted the root from webroot/ to webroot/rsrc/. However, this means that all the /rsrc/x/y/z.jpg fragments in CSS are no longer recognized as resource names.

Since we have like 9,000 things in CSS that do url(/rsrc/xyz.jpg) and I don't want to fix/test them all, so just make them work as-is. There's no real reason either setting is better than the other.

(Both URLs also work fine, but the parsed one will be better once we have real CDN support.)

Test Plan

Verified CSS gets managed resource URIs transformed into it.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

src/infrastructure/celerity/CelerityResourceTransformer.php
115–135

This is just allowing /xyz.jpg or xyz.jpg to work.