Details
-
Task
-
Resolution: Fixed
-
Neutral
-
None
-
None
-
-
Empty show more show less
-
Empty show more show less
-
Kromeriz 69
-
3
Description
- Request http://localhost:8080/magnoliaPublic/demo-project~mgnlArea=branding,metaNavigation~ sets selector to mgnlArea=branding,metaNavigation and also sets context attribute mgnlArea=branding,metaNavigation.
- A subsequent forward or include without a selector (http://localhost:8080/magnoliaPublic/demo-project/stage/component) does:
if (StringUtils.isNotEmpty(selector)) { selectors = this.selector.split(Path.SELECTOR_DELIMITER); //selectors are not assigned, it keeps mgnlArea=branding,metaNavigation } for (String sel : selectors) { ... MgnlContext.setAttribute(splitSelector[0], splitSelector[1]); //mgnlArea=branding,metaNavigation is set incorrectly ... }info.magnolia.cms.core.AggregationState#setSelector should not set the previously set selectors if the current selector is empty and could possible also clear the attributes set by the previous selector.
Checklists
Acceptance criteria
Attachments
Issue Links
- is depended upon by
-
MSITEMESH-49 Remove workarounds for MAGNOLIA-6821 and MGNLADVCACHE-79
-
- Closed
-