Parsys: override allowance definition of resourceSuperType

Description

Currently the definitions from parsysConfigs of child and parent will be merged:

If you have an inheritance hierarchy of page components using sling:resourceSuperType you can define a wcmioarsysConfig on each level of this hierarchy. The definitions are merged together according to the inheritance hierarchy.

It would be nice to have the possibility to override the allowance definition of parent for certain path.

Gliffy Diagrams

Activity

Show:

Stefan Seifert December 7, 2017 at 8:33 PM

i've implemented both approaches:

  1. add deniedChildren parameter (which was aldready implemented for osgi configs, it's now available for settting in the resource type definition as well)

  2. add inherited parameter. this is true by default, if you set it to false it cancels the inheritance from the super resource type

Misha Pekarskij October 10, 2017 at 2:53 PM

don't understand this approach cause I'd like to use the same sightly template from the super page component

Stefan Seifert October 10, 2017 at 2:41 PM

and the approach of just using a different name than "content" for the parsys where you do not want to inherit any child component definitions?

Misha Pekarskij October 10, 2017 at 2:37 PM

I'd like to inherit everything from the super page component (all configs and html). The only difference is that the inherited page should allow the reduced list of children components in its content.

Your approach with deniedChildren would surely also solve the problem.

Stefan Seifert October 10, 2017 at 12:35 PM

when you do not want to inherit anything from the super resource type, there might by better alternatives:

  • do not inherit from the super page component

  • choose a different name then "content" for the parsys, this also breaks the inheritance chain

thus i'm not sure if an approach with overrideSuperType:true makes sense.

Fixed

Details

Assignee

Reporter

Components

Fix versions

Priority

Created September 13, 2017 at 3:32 PM
Updated December 20, 2017 at 1:58 PM
Resolved December 7, 2017 at 8:33 PM