[MGNLDAM-252] Assets app should be bootstrapped into app launcher layout Created: 09/Jun/13  Updated: 09/Jul/13  Resolved: 05/Jul/13

Status: Closed
Project: Magnolia DAM Module
Component/s: None
Affects Version/s: 1.0
Fix Version/s: 1.0.1

Type: Bug Priority: Neutral
Reporter: Tobias Mattsson Assignee: Eric Hechinger
Resolution: Fixed Votes: 0
Labels: architecture
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
relation
is related to CNTCTSAPP-25 Contacts app should be bootstrapped i... Closed
is related to MGNLUI-1604 Pages, Security, Sample and Messages ... 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)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled

 Description   

It is currently bootstrapped in by ui-framework.



 Comments   
Comment by Tobias Mattsson [ 27/Jun/13 ]

The version handler needs to make sure the assets app appears after the pages app, maybe even directly after

Comment by Tobias Mattsson [ 04/Jul/13 ]

Magnolia fails to install if dam is installed before pages, this is happening in the EE bundle. We need a fix similar to what was done in MGNLUI-1604 and CNTCTSAPP-25.

2013-07-04 16:10:25,100 ERROR info.magnolia.module.InstallContextImpl           : > Could not install or update dam module. Task 'Order Assets app after Pages' failed. (PathNotFoundException: modules/ui-framework/config/appLauncherLayout/groups/edit/apps/assets)
info.magnolia.module.delta.TaskExecutionException: Could not execute task: Path not found: modules/ui-framework/config/appLauncherLayout/groups/edit/apps/assets
	at info.magnolia.module.delta.AbstractRepositoryTask.execute(AbstractRepositoryTask.java:62)
	at info.magnolia.module.ModuleManagerImpl.applyDeltas(ModuleManagerImpl.java:512)
	at info.magnolia.module.ModuleManagerImpl.installOrUpdateModule(ModuleManagerImpl.java:494)
	at info.magnolia.module.ModuleManagerImpl$1.doExec(ModuleManagerImpl.java:277)
	at info.magnolia.context.MgnlContext$VoidOp.exec(MgnlContext.java:411)
	at info.magnolia.context.MgnlContext$VoidOp.exec(MgnlContext.java:408)
	at info.magnolia.context.MgnlContext.doInSystemContext(MgnlContext.java:382)
	at info.magnolia.module.ModuleManagerImpl.performInstallOrUpdate(ModuleManagerImpl.java:271)
	at info.magnolia.module.ui.ModuleManagerWebUI$2.run(ModuleManagerWebUI.java:127)
	at java.lang.Thread.run(Thread.java:680)
Caused by: javax.jcr.PathNotFoundException: modules/ui-framework/config/appLauncherLayout/groups/edit/apps/assets
	at org.apache.jackrabbit.core.NodeImpl$8.perform(NodeImpl.java:2151)
	at org.apache.jackrabbit.core.NodeImpl$8.perform(NodeImpl.java:2145)
	at org.apache.jackrabbit.core.session.SessionState.perform(SessionState.java:216)
	at org.apache.jackrabbit.core.ItemImpl.perform(ItemImpl.java:91)
	at org.apache.jackrabbit.core.NodeImpl.getNode(NodeImpl.java:2145)
	at info.magnolia.jcr.wrapper.DelegateNodeWrapper.getNode(DelegateNodeWrapper.java:197)
	at info.magnolia.jcr.decoration.ContentDecoratorNodeWrapper.getNode(ContentDecoratorNodeWrapper.java:120)
	at info.magnolia.jcr.wrapper.DelegateNodeWrapper.getNode(DelegateNodeWrapper.java:197)
	at info.magnolia.jcr.decoration.ContentDecoratorNodeWrapper.getNode(ContentDecoratorNodeWrapper.java:120)
	at info.magnolia.cms.core.DefaultContent.<init>(DefaultContent.java:113)
	at info.magnolia.cms.core.DefaultHierarchyManager.wrapAsContent(DefaultHierarchyManager.java:230)
	at info.magnolia.cms.core.DefaultHierarchyManager.getContent(DefaultHierarchyManager.java:222)
	at info.magnolia.module.delta.OrderNodeAfterTask.doExecute(OrderNodeAfterTask.java:66)
	at info.magnolia.module.delta.AbstractRepositoryTask.execute(AbstractRepositoryTask.java:60)
	... 9 more
Generated at Mon Feb 12 04:58:00 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.