[5.4] Fix fonts URL

[dgrammatiko]

Pull Request for Issue #46717 .

Summary of Changes

Adjust the code so that Fonts follow the logic of Modern/legacy templates

Testing Instructions

Follow the instruction of the issue #46717

Actual result BEFORE applying this Pull Request

Broken link is logged into the browser's console, no font preview

Expected result AFTER applying this Pull Request

Works

Link to documentations

Please select:

• Documentation link for docs.joomla.org:

• No documentation changes for docs.joomla.org needed

• Pull Request link for manual.joomla.org:

• No documentation changes for manual.joomla.org needed

[brianteeman]

I have tested this item ✅ successfully on 06fa1b2

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46757.}

[krishnagandhicode]

I have tested this item ✅ successfully on 06fa1b2

For test 1:
Fonts located in /media/templates/site/cassiopeia/fonts
Font preview works correctly, no 404 errors in browser console

For test 2:
Fonts located in /templates/cassiopeia/fonts
404 error occurs (expected for modern templates)

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46757.}

[richard67]

RTC

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/46757.}

[muhme]

TL;DR The 2nd folder now works, but the 1st one no longer does. @dgrammatiko Could you please check? #46717 states The font is previewed in both tests.

❌ Final test before merge with JBT and used Ubuntu TTF font

• Executing mkdir media/templates/site/cassiopeia/fonts administrator/templates/atum/fonts media/templates/administrator/atum/fonts first
• Before PR, seen rendering is working in templates/cassiopeia/fonts dir, but not is the font only in media/templates/site/cassiopeia/fonts dir and 404 Not Found in JS console log
• Same, working for administrator/templates/atum/fonts, but not working if font is only in media/templates/administrator/atum/fonts folder
• Applied PR with Patch Tester
  • ✅ Font from media/templates/site/cassiopeia/fonts dir only is correctly rendered and no 404 Not Found anymore
  • ❌ Font from templates/cassiopeia/fonts dir is not rendered and 404 Not Found
  • ✅ Font from media/templates/administrator/atum/fonts only is correctly rendered and no 404 Not Found anymore
  • ❌ Font from administrator/templates/atum/fonts dir only is not rendered and 404 Not Found

[dgrammatiko]

That’s correct, the modern templates expect all the static assets to be served from the media folder (compatibility with the public folder and better security). In sort your test was correct and the results are the expected ones

[muhme]

As discussed yesterday in the maintainer meeting: Old templates load their assets from the template folder, new templates (inheritable) use the media folder, so the PR is fine and can be merged.

[muhme]

Thank you @dgrammatiko for your contribution. Thank you @brianteeman and @krishnaGandhi11 for testing.