[magnolia-dev] [JIRA] (MGNLSITE-72) Add required configurationSourceType while adding theme by SiteModule

2016-10-17 Thread on behalf of Michael Mühlebach 
Title: Message Title
 
 
 
 
 
 
 
 
 
 
  
 
 Michael Mühlebach updated an issue 
 
 
 
 
 
 
 
 
 
 

 
 
 
 
 
 
 
 Magnolia Site Module /  MGNLSITE-72 
 
 
 
  Add required configurationSourceType while adding theme by SiteModule  
 
 
 
 
 
 
 
 
 

Change By:
 
 Michael Mühlebach 
 
 
 

Story Points:
 
 1 
 
 
 

Account:
 
 null (null) 
 
 
 
 
 
 
 
 
 
 
 
 

 
 Add Comment 
 
 
 
 
 
 
 
 
 
 

 
 
 
 
 
 
 
 
 
 

 This message was sent by Atlassian JIRA (v6.4.13#64028-sha1:b7939e9) 
 
 
 
 
  
 
 
 
 
 
 
 
 
   




For list details, see: http://www.magnolia-cms.com/community/mailing-lists.html
Alternatively, use our forums: http://forum.magnolia-cms.com/
To unsubscribe, E-mail to: 

[magnolia-dev] [JIRA] (MGNLSITE-72) Add required configurationSourceType while adding theme by SiteModule

2016-10-11 Thread JIRA (on behalf of Philip Mundt) 
Title: Message Title
 
 
 
 
 
 
 
 
 
 
  
 
 Philip Mundt updated an issue 
 
 
 
 
 
 
 
 
 
 

 
 
 
 
 
 
 
 Magnolia Site Module /  MGNLSITE-72 
 
 
 
  Add required configurationSourceType while adding theme by SiteModule  
 
 
 
 
 
 
 
 
 

Change By:
 
 Philip Mundt 
 
 
 
 
 
 
 
 
 
 When using the groupBy functionality of the definitions-app and selecting by {{definition source}} the legacy mechanism of the ThemeRegistry causes NPE because the underlying field is not set in the deprecated method. h2 h4 . Step to reproduce:* Open Definition app* Choose grouped by definition source[^grouped-by-definition-source-does-not-work.png] h2 h4 . Problem: All the null pointer exceptions come from nodes under /modules/site/config/themes/ because there are no configurationSourceType for theme h2 h4 . Error log:{code}2016-10-11 18:23:02,079 ERROR info.magnolia.event.SimpleEventBus: Exception caught when dispatching info.magnolia.definitions.app.overview.toolbar.events.GroupDefinitionsEvent with info.magnolia.definitions.app.overview.tree.DefinitionsTreePresenter$$Lambda$30/1286817053 eventHandler.java.lang.NullPointerException at info.magnolia.config.registry.DefinitionQueryImpl$QueryPredicate.exactMatch(DefinitionQueryImpl.java:99) at info.magnolia.config.registry.DefinitionQueryImpl$QueryPredicate.apply(DefinitionQueryImpl.java:90) at info.magnolia.config.registry.DefinitionQueryImpl$QueryPredicate.apply(DefinitionQueryImpl.java:77) at com.google.common.collect.Iterators$7.computeNext(Iterators.java:652) at com.google.common.collect.AbstractIterator.tryToComputeNext(AbstractIterator.java:143) at com.google.common.collect.AbstractIterator.hasNext(AbstractIterator.java:138) at com.google.common.collect.TransformedIterator.hasNext(TransformedIterator.java:43) at com.google.common.collect.Iterators$5.hasNext(Iterators.java:547) at com.google.common.collect.Iterators.addAll(Iterators.java:361) at com.google.common.collect.Iterables.addAll(Iterables.java:354) at com.google.common.collect.Lists.newLinkedList(Lists.java:259) at info.magnolia.config.registry.RegistryFacadeImpl$AggregatingQuery.findMultiple(RegistryFacadeImpl.java:113) at info.magnolia.definitions.app.overview.tree.DefinitionsTreePresenter.getProviderMap(DefinitionsTreePresenter.java:176) at info.magnolia.definitions.app.overview.tree.DefinitionsTreePresenter.lambda$createContainer$13(DefinitionsTreePresenter.java:114) at info.magnolia.definitions.app.overview.tree.DefinitionsTreePresenter$$Lambda$30/1286817053.onGroupDefinitionsChange(Unknown Source) at info.magnolia.definitions.app.overview.toolbar.events.GroupDefinitionsEvent.dispatch(GroupDefinitionsEvent.java:52) at info.magnolia.definitions.app.overview.toolbar.events.GroupDefinitionsEvent.dispatch(GroupDefinitionsEvent.java:42) at info.magnolia.event.SimpleEventBus.fireEvent(SimpleEventBus.java:78) at info.magnolia.definitions.app.overview.toolbar.ToolbarPresenter.updateGroupBy(ToolbarPresenter.java:119) at info.magnolia.definitions.app.overview.toolbar.ToolbarViewImpl.lambda$new$cc77f013$1(ToolbarViewImpl.java:110) at info.magnolia.definitions.app.overview.toolbar.ToolbarViewImpl$$Lambda$33/1016375528.valueChange(Unknown Source) at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) at 

[magnolia-dev] [JIRA] (MGNLSITE-72) Add required configurationSourceType while adding theme by SiteModule

2016-10-11 Thread JIRA (on behalf of Philip Mundt) 
Title: Message Title
 
 
 
 
 
 
 
 
 
 
  
 
 Philip Mundt updated an issue 
 
 
 
 
 
 
 
 
 
 

 
 
 
 
 
 
 
 Magnolia Site Module /  MGNLSITE-72 
 
 
 
  Add required configurationSourceType while adding theme by SiteModule  
 
 
 
 
 
 
 
 
 

Change By:
 
 Philip Mundt 
 
 
 
 
 
 
 
 
 
 When using the groupBy functionality of the definitions-app and selecting by {{ configurationSourceType definition source }} the legacy mechanism of the ThemeRegistry causes NPE because the underlying field is not set in the deprecated method.h2. Step to reproduce:* Open Definition app* Choose grouped by definition source[^grouped-by-definition-source-does-not-work.png]h2. Problem: All the null pointer exceptions come from nodes under /modules/site/config/themes/ because there are no configurationSourceType for themeh2. Error log:{code}2016-10-11 18:23:02,079 ERROR info.magnolia.event.SimpleEventBus: Exception caught when dispatching info.magnolia.definitions.app.overview.toolbar.events.GroupDefinitionsEvent with info.magnolia.definitions.app.overview.tree.DefinitionsTreePresenter$$Lambda$30/1286817053 eventHandler.java.lang.NullPointerException at info.magnolia.config.registry.DefinitionQueryImpl$QueryPredicate.exactMatch(DefinitionQueryImpl.java:99) at info.magnolia.config.registry.DefinitionQueryImpl$QueryPredicate.apply(DefinitionQueryImpl.java:90) at info.magnolia.config.registry.DefinitionQueryImpl$QueryPredicate.apply(DefinitionQueryImpl.java:77) at com.google.common.collect.Iterators$7.computeNext(Iterators.java:652) at com.google.common.collect.AbstractIterator.tryToComputeNext(AbstractIterator.java:143) at com.google.common.collect.AbstractIterator.hasNext(AbstractIterator.java:138) at com.google.common.collect.TransformedIterator.hasNext(TransformedIterator.java:43) at com.google.common.collect.Iterators$5.hasNext(Iterators.java:547) at com.google.common.collect.Iterators.addAll(Iterators.java:361) at com.google.common.collect.Iterables.addAll(Iterables.java:354) at com.google.common.collect.Lists.newLinkedList(Lists.java:259) at info.magnolia.config.registry.RegistryFacadeImpl$AggregatingQuery.findMultiple(RegistryFacadeImpl.java:113) at info.magnolia.definitions.app.overview.tree.DefinitionsTreePresenter.getProviderMap(DefinitionsTreePresenter.java:176) at info.magnolia.definitions.app.overview.tree.DefinitionsTreePresenter.lambda$createContainer$13(DefinitionsTreePresenter.java:114) at info.magnolia.definitions.app.overview.tree.DefinitionsTreePresenter$$Lambda$30/1286817053.onGroupDefinitionsChange(Unknown Source) at info.magnolia.definitions.app.overview.toolbar.events.GroupDefinitionsEvent.dispatch(GroupDefinitionsEvent.java:52) at info.magnolia.definitions.app.overview.toolbar.events.GroupDefinitionsEvent.dispatch(GroupDefinitionsEvent.java:42) at info.magnolia.event.SimpleEventBus.fireEvent(SimpleEventBus.java:78) at info.magnolia.definitions.app.overview.toolbar.ToolbarPresenter.updateGroupBy(ToolbarPresenter.java:119) at info.magnolia.definitions.app.overview.toolbar.ToolbarViewImpl.lambda$new$cc77f013$1(ToolbarViewImpl.java:110) at info.magnolia.definitions.app.overview.toolbar.ToolbarViewImpl$$Lambda$33/1016375528.valueChange(Unknown Source) at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) at 

[magnolia-dev] [JIRA] (MGNLSITE-72) Add required configurationSourceType while adding theme by SiteModule

2016-10-11 Thread JIRA (on behalf of Philip Mundt) 
Title: Message Title
 
 
 
 
 
 
 
 
 
 
  
 
 Philip Mundt updated an issue 
 
 
 
 
 
 
 
 
 
 

 
 
 
 
 
 
 
 Magnolia Site Module /  MGNLSITE-72 
 
 
 
  Add required configurationSourceType while adding theme by SiteModule  
 
 
 
 
 
 
 
 
 

Change By:
 
 Philip Mundt 
 
 
 
 
 
 
 
 
 
 When using the groupBy functionality of the definitions-app and selecting by {{configurationSourceType}} the legacy mechanism of the ThemeRegistry causes NPE because the underlying field is not set in the deprecated method. h2. Step to reproduce:* Open Definition app* Choose grouped by definition source[^grouped-by-definition-source-does-not-work.png]h2. Problem: All the null pointer exceptions come from nodes under /modules/site/config/themes/ because there are no configurationSourceType for themeh2. Error log:{code}2016-10-11 18:23:02,079 ERROR info.magnolia.event.SimpleEventBus: Exception caught when dispatching info.magnolia.definitions.app.overview.toolbar.events.GroupDefinitionsEvent with info.magnolia.definitions.app.overview.tree.DefinitionsTreePresenter$$Lambda$30/1286817053 eventHandler.java.lang.NullPointerException at info.magnolia.config.registry.DefinitionQueryImpl$QueryPredicate.exactMatch(DefinitionQueryImpl.java:99) at info.magnolia.config.registry.DefinitionQueryImpl$QueryPredicate.apply(DefinitionQueryImpl.java:90) at info.magnolia.config.registry.DefinitionQueryImpl$QueryPredicate.apply(DefinitionQueryImpl.java:77) at com.google.common.collect.Iterators$7.computeNext(Iterators.java:652) at com.google.common.collect.AbstractIterator.tryToComputeNext(AbstractIterator.java:143) at com.google.common.collect.AbstractIterator.hasNext(AbstractIterator.java:138) at com.google.common.collect.TransformedIterator.hasNext(TransformedIterator.java:43) at com.google.common.collect.Iterators$5.hasNext(Iterators.java:547) at com.google.common.collect.Iterators.addAll(Iterators.java:361) at com.google.common.collect.Iterables.addAll(Iterables.java:354) at com.google.common.collect.Lists.newLinkedList(Lists.java:259) at info.magnolia.config.registry.RegistryFacadeImpl$AggregatingQuery.findMultiple(RegistryFacadeImpl.java:113) at info.magnolia.definitions.app.overview.tree.DefinitionsTreePresenter.getProviderMap(DefinitionsTreePresenter.java:176) at info.magnolia.definitions.app.overview.tree.DefinitionsTreePresenter.lambda$createContainer$13(DefinitionsTreePresenter.java:114) at info.magnolia.definitions.app.overview.tree.DefinitionsTreePresenter$$Lambda$30/1286817053.onGroupDefinitionsChange(Unknown Source) at info.magnolia.definitions.app.overview.toolbar.events.GroupDefinitionsEvent.dispatch(GroupDefinitionsEvent.java:52) at info.magnolia.definitions.app.overview.toolbar.events.GroupDefinitionsEvent.dispatch(GroupDefinitionsEvent.java:42) at info.magnolia.event.SimpleEventBus.fireEvent(SimpleEventBus.java:78) at info.magnolia.definitions.app.overview.toolbar.ToolbarPresenter.updateGroupBy(ToolbarPresenter.java:119) at info.magnolia.definitions.app.overview.toolbar.ToolbarViewImpl.lambda$new$cc77f013$1(ToolbarViewImpl.java:110) at info.magnolia.definitions.app.overview.toolbar.ToolbarViewImpl$$Lambda$33/1016375528.valueChange(Unknown Source) at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) at 

[magnolia-dev] [JIRA] (MGNLSITE-72) Add required configurationSourceType while adding theme by SiteModule

2016-10-11 Thread JIRA (on behalf of Ilgun Ilgun) 
Title: Message Title
 
 
 
 
 
 
 
 
 
 
  
 
 Ilgun Ilgun updated an issue 
 
 
 
 
 
 
 
 
 
 

 
 
 
 
 
 
 
 Magnolia Site Module /  MGNLSITE-72 
 
 
 
  Add required configurationSourceType while adding theme by SiteModule  
 
 
 
 
 
 
 
 
 

Change By:
 
 Ilgun Ilgun 
 
 
 

Summary:
 
 Group Add required configurationSourceType while adding theme  by  definition source does not work  SiteModule 
 
 
 
 
 
 
 
 
 
 
 
 

 
 Add Comment 
 
 
 
 
 
 
 
 
 
 

 
 
 
 
 
 
 
 
 
 

 This message was sent by Atlassian JIRA (v6.4.13#64028-sha1:b7939e9) 
 
 
 
 
  
 
 
 
 
 
 
 
 
   




For list details, see: http://www.magnolia-cms.com/community/mailing-lists.html
Alternatively, use our forums: http://forum.magnolia-cms.com/
To unsubscribe, E-mail to: