search

Adobe-Consulting-Services / acs-aem-commons

http://adobe-consulting-services.github.io/acs-aem-commons/
Apache License 2.0
453 stars 600 forks source link

3.7.0 breaks system notifications. #946

Closed cru-Luis-Rodriguez closed 7 years ago

cru-Luis-Rodriguez commented 7 years ago

Hello,

I upgraded ACS commons from 3.4.0 to latest 3.7.0 on an AEM 6.2 instance. Now the notification feature does not work and throws an exception. I downgraded to 3.6.2 and the feature works on that version. Are there plans to fix this in 3.7.x next release? 

Error during include of component '/apps/acs-commons/components/utilities/system-notifications'

Error Message:

java.lang.ExceptionInInitializerError
Processing Info:

Page    =   /etc/acs-commons/notifications/test 
Resource Path   =   /etc/acs-commons/notifications/test/jcr:content 
Cell    =   system-notifications    
Cell Search Path    =   system-notifications    
Component Path  =   /apps/acs-commons/components/utilities/system-notifications 
Sling Request Progress:

Full exception

davidjgonzalez commented 7 years ago

@cru-Luis-Rodriguez now there is!

cru-Luis-Rodriguez commented 7 years ago

@davidjgonzalez

Thank you!

-Luis

davidjgonzalez commented 7 years ago

@cru-Luis-Rodriguez if you have a chance could you try this snapshot out?

acs-aem-commons-content-3.7.1-SNAPSHOT-min.zip acs-aem-commons-content-3.7.1-SNAPSHOT.zip

cru-Luis-Rodriguez commented 7 years ago

@davidjgonzalez That did the trick 👍

Thanks screen shot 2017-03-30 at 3 45 44 pm

justinedelson commented 7 years ago

@davidjgonzalez has this been fixed in the feature/6.2-compatibility branch?

davidjgonzalez commented 7 years ago

@justinedelson #954 fixes this; travis is checking it out as we speak.