[MAGNOLIA-6219] ClasspathOrigin: filter resources out; expose mgnl-resources Created: 25/May/15  Updated: 15/Sep/15  Resolved: 27/May/15

Status: Closed
Project: Magnolia
Component/s: resource-loader
Affects Version/s: 5.4
Fix Version/s: 5.4

Type: Improvement Priority: Neutral
Reporter: Magnolia International Assignee: Magnolia International
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Relates
relates to MAGNOLIA-6128 Introduce uniform resource loading API Closed
causality
is causing MGNLUI-3466 Move defaultLoginForm out of legacy-c... Closed
dependency
depends upon MAGNOLIA-6220 Resources: clarify pathPattern usage ... Closed
is depended upon by MGNLRES-144 Implement new origin-based ResourcesS... Closed
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Date of First Response:

 Description   

The current ClasspathOrigin is polluted by a lot of resources, since we load everything from the /.

Additionally, to provide compatibility with previous versions, we need to expose resources under mgnl-resources as if they were at the root.



 Comments   
Comment by Magnolia International [ 27/May/15 ]

Both MAGNOLIA-6219 and MAGNOLIA-6220, preceded by a handful of small QA/clean commits are ready for review on branch feature/MAGNOLIA-6220-6219-resources-2. I'd suggest going at it with git log feature/MAGNOLIA-6220-6219-resources-2 ^master --date=relative --reverse -p

Comment by Mikaël Geljić [ 29/May/15 ]
  • Exclude META-INF altogether for now
  • Log warning in LegacyCPO (openStream/Reader)
  • A few more directories to exclude, at least for now:
    • mgnl-i18n
    • mgnl-nodetypes
    • mgnl-bootstrap*
    • mgnl-files
Generated at Mon Feb 12 04:12:26 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.