Implement the new Resources app
(MGNLRES-140)
|
|
| Status: | Closed |
| Project: | Magnolia Resources Module |
| Component/s: | app |
| Affects Version/s: | None |
| Fix Version/s: | 2.4 |
| Type: | Sub-task | Priority: | Major |
| Reporter: | Mikaël Geljić | Assignee: | Mikaël Geljić |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | platform-cell | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Template: |
|
| Date of First Response: | |
| Sprint: | 5.4 Sprint 4 |
| Team: |
| Description |
|
Read-only to begin with. |
| Comments |
| Comment by Mikaël Geljić [ 26/May/15 ] |
|
Ready for review on branch feature/resources-app-2 |
| Comment by Mikaël Geljić [ 01/Jun/15 ] |
|
Btw ReaderProperty was most basic/temporary. We should really wrap the ResourcePath and call Origin#openReader() only upon #getValue. |
| Comment by Magnolia International [ 01/Jun/15 ] |
|
Found flaw: every resource was opened eagerly rather than when the content was actually needed (which would lead to resource exhaustion since this was opening streams of every resource without ever closing them) |
| Comment by Magnolia International [ 01/Jun/15 ] |
|
Fix pushed on fix/app-resource-content branch; not sure how to approach testing just yet. |