Skip to content
This repository has been archived by the owner on Sep 16, 2021. It is now read-only.

ContentListener should support multiple content keys #265

Open
benglass opened this issue Oct 1, 2015 · 0 comments
Open

ContentListener should support multiple content keys #265

benglass opened this issue Oct 1, 2015 · 0 comments
Milestone

Comments

@benglass
Copy link
Member

benglass commented Oct 1, 2015

Right now the content listener only supports checking 1 content key in the request. The default is for it to check for the cmf main content key where the CMF stores its main PHPCR content document.

If you want to have the content listener automatically load metadata for other objects in the request, you would have register multiple copies of the listener with different content keys for every argument name in any of your actions that represents an SEO object unless you want to always call your action argument $seoContent which is not a desirable restriction.

For example, what if you have a BlogPost object that implements SeoAware? You would have to register a separate content listener which looks at for a $blogPost key in your action. This would not scale very well.

If it supported checking multiple keys then we could allow the user to configure it to listen on 'cmfMainContent' as well as 'blogPost' and 'job' etc. This seems like it might be a nice enhancement actually to allow configuring multiple contentKeys for the content listener.

Original discussion from: #146

@ElectricMaxxx ElectricMaxxx added this to the 2.0 milestone Feb 13, 2016
@ElectricMaxxx ElectricMaxxx modified the milestones: 2.1, 2.0 Nov 28, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants