Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Improvement of page definition for page engine #797

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Collaborator

flongo commented Nov 5, 2013

Two features are added with this commit:

1 - property "contents" is no longer mandatory in a page composition
2 - The bean definition for "placeholderContents" has been changed in order to reflect the code: values in the map can be either a single content or an array of them. The code was already taking that possibility into account. Tests have been added also for the case in which a template and a contentId are set in a placeholder configuration.

feat #797 Improvement of page definition for page engine
Two features are added with this commit:

1 - property "contents" is no longer mandatory in a page composition
2 - The bean definition for "placeholderContents" has been changed in order to reflect the code: values in the map can be either a single content or an array of them. The code was already taking that possibility into account. Tests have been added also for the case in which a template and a contentId are set in a placeholder configuration.

@ghost ghost assigned divdavem Nov 20, 2013

@flongo flongo closed this in ad746fa Nov 20, 2013

@flongo flongo deleted the flongo:pageEngine_contents branch Jan 24, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment