[MGNLPUR-161] Prepare PUR to be used in travel demo Created: 11/Nov/15  Updated: 17/Oct/16  Resolved: 02/Dec/15

Status: Closed
Project: Magnolia Public User Registration
Component/s: None
Affects Version/s: None
Fix Version/s: 2.5.1

Type: Task Priority: Neutral
Reporter: Roman Kovařík Assignee: Roman Kovařík
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: 0d
Time Spent: 3h 10m
Original Estimate: Not Specified

Issue Links:
dependency
depends upon MGNLPUR-163 Update bootstrap files for dialogs so... Closed
is depended upon by MGNLDEMO-104 Demonstrate PUR Closed
is depended upon by MGNLPUR-162 Deprecate obsolete templates Closed
Template:
Acceptance criteria:
Empty
Task DoR:
Empty
Date of First Response:
Sprint: Kromeriz 21
Story Points: 3

 Description   
  • fix MailChangePasswordLinkStrategy so is doesn't require defaultBaseUrl to be set.
  • Introduce form model which pre fills fields with user properties.
  • Add new legacy-adminterface free templates.


 Comments   
Comment by Philip Mundt [ 16/Nov/15 ]

Found three some issues:

  • Binary compat. broken due to making constants final
    • Option: deprecate them with a notice that they will be made final?
  • Usage of renderType=site without dependency to site: possibly change to renderType=freemarker
  • Reference of a templateScript from travel-demo without dependency: remove property?
  • info.magnolia.module.publicuserregistration.strategy.Mail:
    • New ctor should have @Inject annotation
    • Old ctor should have @Deprecated annotation
  • info.magnolia.module.publicuserregistration.frontend.Register, info.magnolia.module.publicuserregistration.frontend.ProfileUpdate, info.magnolia.module.publicuserregistration.frontend.EnableByUuid and info.magnolia.module.publicuserregistration.frontend.PasswordReminder were deprecated, but those classes are still in use: they are bootstrapped by default from /mgnl-bootstrap/public-user-registration/pages!
    • As those are legacy info.magnolia.module.admininterface.PageMVCHandler pages and the dependency to magnolia-module-legacy-admininterface has been made optional bootstrapping those page should be made optional too!
  • info.magnolia.module.publicuserregistration.processors.EnableUserByUuidProcessor is a new class. Shouldn't it be (minimally) tested?
Comment by Roman Kovařík [ 18/Nov/15 ]

Found three some issues:

  • Binary compat. broken due to making constants final
    • final removed, deprecation probably doesn't worth the effort + could be confusing
  • Usage of renderType=site without dependency to site: possibly change to renderType=freemarker
  • Reference of a templateScript from travel-demo without dependency: remove property?
    • page template moved to travel-demo
  • info.magnolia.module.publicuserregistration.strategy.Mail:
  • info.magnolia.module.publicuserregistration.frontend.Register, info.magnolia.module.publicuserregistration.frontend.ProfileUpdate, info.magnolia.module.publicuserregistration.frontend.EnableByUuid and info.magnolia.module.publicuserregistration.frontend.PasswordReminder were deprecated, but those classes are still in use: they are bootstrapped by default from /mgnl-bootstrap/public-user-registration/pages!
  • info.magnolia.module.publicuserregistration.processors.EnableUserByUuidProcessor is a new class. Shouldn't it be (minimally) tested?
Comment by Philip Mundt [ 24/Nov/15 ]

UserFormModel has to be made more failsafe. It's causing javax.jcr.PathNotFoundException when properties do not exist:

[INFO] [talledLocalContainer] 2015-11-24 08:34:24,711 ERROR a.module.publicuserregistration.form.UserFormModel: email
[INFO] [talledLocalContainer] javax.jcr.PathNotFoundException: email
[INFO] [talledLocalContainer] 	at org.apache.jackrabbit.core.NodeImpl$11.perform(NodeImpl.java:2265)
[INFO] [talledLocalContainer] 	at org.apache.jackrabbit.core.NodeImpl$11.perform(NodeImpl.java:2252)
[INFO] [talledLocalContainer] 	at org.apache.jackrabbit.core.session.SessionState.perform(SessionState.java:216)
[INFO] [talledLocalContainer] 	at org.apache.jackrabbit.core.ItemImpl.perform(ItemImpl.java:91)
[INFO] [talledLocalContainer] 	at org.apache.jackrabbit.core.NodeImpl.getProperty(NodeImpl.java:2252)
[INFO] [talledLocalContainer] 	at info.magnolia.jcr.wrapper.DelegateNodeWrapper.getProperty(DelegateNodeWrapper.java:242)
[INFO] [talledLocalContainer] 	at info.magnolia.jcr.decoration.ContentDecoratorNodeWrapper.getProperty(ContentDecoratorNodeWrapper.java:194)
[INFO] [talledLocalContainer] 	at info.magnolia.jcr.wrapper.DelegateNodeWrapper.getProperty(DelegateNodeWrapper.java:242)
[INFO] [talledLocalContainer] 	at info.magnolia.jcr.decoration.ContentDecoratorNodeWrapper.getProperty(ContentDecoratorNodeWrapper.java:194)
[INFO] [talledLocalContainer] 	at info.magnolia.jcr.wrapper.DelegateNodeWrapper.getProperty(DelegateNodeWrapper.java:242)
[INFO] [talledLocalContainer] 	at info.magnolia.jcr.decoration.ContentDecoratorNodeWrapper.getProperty(ContentDecoratorNodeWrapper.java:194)
[INFO] [talledLocalContainer] 	at info.magnolia.jcr.wrapper.DelegateNodeWrapper.getProperty(DelegateNodeWrapper.java:242)
[INFO] [talledLocalContainer] 	at info.magnolia.jcr.decoration.ContentDecoratorNodeWrapper.getProperty(ContentDecoratorNodeWrapper.java:194)
[INFO] [talledLocalContainer] 	at info.magnolia.module.publicuserregistration.form.UserFormModel.getFormField(UserFormModel.java:84)
[INFO] [talledLocalContainer] 	at info.magnolia.module.form.templates.components.FormFieldModel.validate(FormFieldModel.java:85)
[INFO] [talledLocalContainer] 	at info.magnolia.module.form.templates.components.FormFieldModel.execute(FormFieldModel.java:75)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.renderer.AbstractRenderer.render(AbstractRenderer.java:135)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.engine.DefaultRenderingEngine.render(DefaultRenderingEngine.java:118)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.engine.DefaultRenderingEngine.render(DefaultRenderingEngine.java:99)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.engine.DefaultRenderingEngine.render(DefaultRenderingEngine.java:94)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.engine.DefaultRenderingEngine$$EnhancerByCGLIB$$89664a12.render(<generated>)
[INFO] [talledLocalContainer] 	at info.magnolia.templating.elements.ComponentElement.begin(ComponentElement.java:193)
[INFO] [talledLocalContainer] 	at info.magnolia.templating.freemarker.AbstractDirective.execute(AbstractDirective.java:93)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visit(Environment.java:333)
[INFO] [talledLocalContainer] 	at freemarker.core.UnifiedCall.accept(UnifiedCall.java:100)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visitByHiddingParent(Environment.java:278)
[INFO] [talledLocalContainer] 	at freemarker.core.IteratorBlock$Context.runLoop(IteratorBlock.java:157)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visitIteratorBlock(Environment.java:501)
[INFO] [talledLocalContainer] 	at freemarker.core.IteratorBlock.accept(IteratorBlock.java:67)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visit(Environment.java:257)
[INFO] [talledLocalContainer] 	at freemarker.core.MixedContent.accept(MixedContent.java:57)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visit(Environment.java:257)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.process(Environment.java:235)
[INFO] [talledLocalContainer] 	at freemarker.template.Template.process(Template.java:262)
[INFO] [talledLocalContainer] 	at info.magnolia.freemarker.FreemarkerHelper.render(FreemarkerHelper.java:152)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.renderer.FreemarkerRenderer.onRender(FreemarkerRenderer.java:100)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.renderer.AbstractRenderer.render(AbstractRenderer.java:155)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.engine.DefaultRenderingEngine.render(DefaultRenderingEngine.java:118)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.engine.DefaultRenderingEngine$$EnhancerByCGLIB$$89664a12.render(<generated>)
[INFO] [talledLocalContainer] 	at info.magnolia.templating.elements.AreaElement.end(AreaElement.java:330)
[INFO] [talledLocalContainer] 	at info.magnolia.templating.freemarker.AbstractDirective.execute(AbstractDirective.java:98)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visit(Environment.java:333)
[INFO] [talledLocalContainer] 	at freemarker.core.UnifiedCall.accept(UnifiedCall.java:100)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visit(Environment.java:257)
[INFO] [talledLocalContainer] 	at freemarker.core.MixedContent.accept(MixedContent.java:57)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visit(Environment.java:257)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.process(Environment.java:235)
[INFO] [talledLocalContainer] 	at freemarker.template.Template.process(Template.java:262)
[INFO] [talledLocalContainer] 	at info.magnolia.freemarker.FreemarkerHelper.render(FreemarkerHelper.java:152)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.renderer.FreemarkerRenderer.onRender(FreemarkerRenderer.java:100)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.renderer.AbstractRenderer.render(AbstractRenderer.java:155)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.engine.DefaultRenderingEngine.render(DefaultRenderingEngine.java:118)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.engine.DefaultRenderingEngine.render(DefaultRenderingEngine.java:99)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.engine.DefaultRenderingEngine.render(DefaultRenderingEngine.java:94)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.engine.DefaultRenderingEngine$$EnhancerByCGLIB$$89664a12.render(<generated>)
[INFO] [talledLocalContainer] 	at info.magnolia.templating.elements.ComponentElement.begin(ComponentElement.java:193)
[INFO] [talledLocalContainer] 	at info.magnolia.templating.freemarker.AbstractDirective.execute(AbstractDirective.java:93)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visit(Environment.java:333)
[INFO] [talledLocalContainer] 	at freemarker.core.UnifiedCall.accept(UnifiedCall.java:100)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visitByHiddingParent(Environment.java:278)
[INFO] [talledLocalContainer] 	at freemarker.core.IteratorBlock$Context.runLoop(IteratorBlock.java:157)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visitIteratorBlock(Environment.java:501)
[INFO] [talledLocalContainer] 	at freemarker.core.IteratorBlock.accept(IteratorBlock.java:67)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visit(Environment.java:257)
[INFO] [talledLocalContainer] 	at freemarker.core.MixedContent.accept(MixedContent.java:57)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visit(Environment.java:257)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.process(Environment.java:235)
[INFO] [talledLocalContainer] 	at freemarker.template.Template.process(Template.java:262)
[INFO] [talledLocalContainer] 	at info.magnolia.freemarker.FreemarkerHelper.render(FreemarkerHelper.java:152)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.renderer.FreemarkerRenderer.onRender(FreemarkerRenderer.java:100)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.renderer.AbstractRenderer.render(AbstractRenderer.java:155)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.engine.DefaultRenderingEngine.render(DefaultRenderingEngine.java:118)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.engine.DefaultRenderingEngine$$EnhancerByCGLIB$$89664a12.render(<generated>)
[INFO] [talledLocalContainer] 	at info.magnolia.templating.elements.AreaElement.end(AreaElement.java:330)
[INFO] [talledLocalContainer] 	at info.magnolia.templating.freemarker.AbstractDirective.execute(AbstractDirective.java:98)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visit(Environment.java:333)
[INFO] [talledLocalContainer] 	at freemarker.core.UnifiedCall.accept(UnifiedCall.java:100)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visit(Environment.java:257)
[INFO] [talledLocalContainer] 	at freemarker.core.MixedContent.accept(MixedContent.java:57)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visitByHiddingParent(Environment.java:278)
[INFO] [talledLocalContainer] 	at freemarker.core.IfBlock.accept(IfBlock.java:48)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visit(Environment.java:257)
[INFO] [talledLocalContainer] 	at freemarker.core.MixedContent.accept(MixedContent.java:57)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visit(Environment.java:257)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.process(Environment.java:235)
[INFO] [talledLocalContainer] 	at freemarker.template.Template.process(Template.java:262)
[INFO] [talledLocalContainer] 	at info.magnolia.freemarker.FreemarkerHelper.render(FreemarkerHelper.java:152)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.renderer.FreemarkerRenderer.onRender(FreemarkerRenderer.java:100)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.renderer.AbstractRenderer.render(AbstractRenderer.java:155)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.engine.DefaultRenderingEngine.render(DefaultRenderingEngine.java:118)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.engine.DefaultRenderingEngine.render(DefaultRenderingEngine.java:99)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.engine.DefaultRenderingEngine.render(DefaultRenderingEngine.java:94)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.engine.DefaultRenderingEngine$$EnhancerByCGLIB$$89664a12.render(<generated>)
[INFO] [talledLocalContainer] 	at info.magnolia.templating.elements.ComponentElement.begin(ComponentElement.java:193)
[INFO] [talledLocalContainer] 	at info.magnolia.templating.freemarker.AbstractDirective.execute(AbstractDirective.java:93)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visit(Environment.java:333)
[INFO] [talledLocalContainer] 	at freemarker.core.UnifiedCall.accept(UnifiedCall.java:100)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visitByHiddingParent(Environment.java:278)
[INFO] [talledLocalContainer] 	at freemarker.core.IteratorBlock$Context.runLoop(IteratorBlock.java:157)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visitIteratorBlock(Environment.java:501)
[INFO] [talledLocalContainer] 	at freemarker.core.IteratorBlock.accept(IteratorBlock.java:67)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visit(Environment.java:257)
[INFO] [talledLocalContainer] 	at freemarker.core.MixedContent.accept(MixedContent.java:57)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visit(Environment.java:257)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.process(Environment.java:235)
[INFO] [talledLocalContainer] 	at freemarker.template.Template.process(Template.java:262)
[INFO] [talledLocalContainer] 	at info.magnolia.freemarker.FreemarkerHelper.render(FreemarkerHelper.java:152)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.renderer.FreemarkerRenderer.onRender(FreemarkerRenderer.java:100)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.renderer.AbstractRenderer.render(AbstractRenderer.java:155)
[INFO] [talledLocalContainer] 	at info.magnolia.module.site.renderer.SiteAwareFreemarkerRenderer.render(SiteAwareFreemarkerRenderer.java:90)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.engine.DefaultRenderingEngine.render(DefaultRenderingEngine.java:118)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.engine.DefaultRenderingEngine$$EnhancerByCGLIB$$89664a12.render(<generated>)
[INFO] [talledLocalContainer] 	at info.magnolia.templating.elements.AreaElement.end(AreaElement.java:330)
[INFO] [talledLocalContainer] 	at info.magnolia.templating.freemarker.AbstractDirective.execute(AbstractDirective.java:98)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visit(Environment.java:333)
[INFO] [talledLocalContainer] 	at freemarker.core.UnifiedCall.accept(UnifiedCall.java:100)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visit(Environment.java:257)
[INFO] [talledLocalContainer] 	at freemarker.core.MixedContent.accept(MixedContent.java:57)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.visit(Environment.java:257)
[INFO] [talledLocalContainer] 	at freemarker.core.Environment.process(Environment.java:235)
[INFO] [talledLocalContainer] 	at freemarker.template.Template.process(Template.java:262)
[INFO] [talledLocalContainer] 	at info.magnolia.freemarker.FreemarkerHelper.render(FreemarkerHelper.java:152)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.renderer.FreemarkerRenderer.onRender(FreemarkerRenderer.java:100)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.renderer.AbstractRenderer.render(AbstractRenderer.java:155)
[INFO] [talledLocalContainer] 	at info.magnolia.module.site.renderer.SiteAwareFreemarkerRenderer.render(SiteAwareFreemarkerRenderer.java:90)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.engine.DefaultRenderingEngine.render(DefaultRenderingEngine.java:118)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.engine.DefaultRenderingEngine$$EnhancerByCGLIB$$89664a12.render(<generated>)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.engine.RenderingFilter.render(RenderingFilter.java:195)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.engine.RenderingFilter.handleTemplateRequest(RenderingFilter.java:140)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.engine.RenderingFilter.doFilter(RenderingFilter.java:94)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:85)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)
[INFO] [talledLocalContainer] 	at info.magnolia.rendering.model.ModelExecutionFilter.doFilter(ModelExecutionFilter.java:100)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.OncePerRequestAbstractMgnlFilter.doFilter(OncePerRequestAbstractMgnlFilter.java:59)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AggregatorFilter.doFilter(AggregatorFilter.java:100)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:85)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.security.BaseSecurityFilter.doFilter(BaseSecurityFilter.java:57)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:85)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.RepositoryMappingFilter.doFilter(RepositoryMappingFilter.java:108)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:85)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.CompositeFilter.doFilter(CompositeFilter.java:65)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:85)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:74)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:81)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:81)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:81)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:81)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:81)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:81)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:81)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:81)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:81)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:81)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:81)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:81)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:81)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:81)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:81)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:81)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:81)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.CompositeFilter.doFilter(CompositeFilter.java:65)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:85)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.VirtualUriFilter.doFilter(VirtualUriFilter.java:69)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:85)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)
[INFO] [talledLocalContainer] 	at info.magnolia.module.cache.executor.Bypass.processCacheRequest(Bypass.java:58)
[INFO] [talledLocalContainer] 	at info.magnolia.module.cache.executor.CompositeExecutor.processCacheRequest(CompositeExecutor.java:67)
[INFO] [talledLocalContainer] 	at info.magnolia.module.cache.filter.CacheFilter.doFilter(CacheFilter.java:174)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.OncePerRequestAbstractMgnlFilter.doFilter(OncePerRequestAbstractMgnlFilter.java:59)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)
[INFO] [talledLocalContainer] 	at info.magnolia.test.fixture.CacheMonitorFilter.doFilter(CacheMonitorFilter.java:108)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.OncePerRequestAbstractMgnlFilter.doFilter(OncePerRequestAbstractMgnlFilter.java:59)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.i18n.I18nContentSupportFilter.doFilter(I18nContentSupportFilter.java:74)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:85)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.RangeSupportFilter.doFilter(RangeSupportFilter.java:84)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:85)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.security.BaseSecurityFilter.doFilter(BaseSecurityFilter.java:57)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:85)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:81)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.security.SecurityCallbackFilter.doFilter(SecurityCallbackFilter.java:80)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.OncePerRequestAbstractMgnlFilter.doFilter(OncePerRequestAbstractMgnlFilter.java:59)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.security.LogoutFilter.doFilter(LogoutFilter.java:94)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.OncePerRequestAbstractMgnlFilter.doFilter(OncePerRequestAbstractMgnlFilter.java:59)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)
[INFO] [talledLocalContainer] 	at info.magnolia.module.site.filters.SiteMergeFilter.doFilter(SiteMergeFilter.java:119)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:85)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MultiChannelFilter.doFilter(MultiChannelFilter.java:83)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:85)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)
[INFO] [talledLocalContainer] 	at info.magnolia.module.cache.filter.GZipFilter.doFilter(GZipFilter.java:73)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.OncePerRequestAbstractMgnlFilter.doFilter(OncePerRequestAbstractMgnlFilter.java:59)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:81)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.security.auth.login.LoginFilter.doFilter(LoginFilter.java:127)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:85)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:81)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.CosMultipartRequestFilter.doFilter(CosMultipartRequestFilter.java:87)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.OncePerRequestAbstractMgnlFilter.doFilter(OncePerRequestAbstractMgnlFilter.java:59)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)
[INFO] [talledLocalContainer] 	at info.magnolia.module.devicedetection.filter.DeviceDetectionFilter.doFilter(DeviceDetectionFilter.java:71)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.OncePerRequestAbstractMgnlFilter.doFilter(OncePerRequestAbstractMgnlFilter.java:59)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.ContentTypeFilter.doFilter(ContentTypeFilter.java:148)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:85)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.ContextFilter.doFilter(ContextFilter.java:128)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:85)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:79)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.CompositeFilter.doFilter(CompositeFilter.java:65)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:85)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.SafeDestroyMgnlFilterWrapper.doFilter(SafeDestroyMgnlFilterWrapper.java:107)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlFilterDispatcher.doDispatch(MgnlFilterDispatcher.java:67)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlMainFilter.doFilter(MgnlMainFilter.java:108)
[INFO] [talledLocalContainer] 	at info.magnolia.cms.filters.MgnlMainFilter.doFilter(MgnlMainFilter.java:94)
[INFO] [talledLocalContainer] 	at org.mortbay.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1212)
[INFO] [talledLocalContainer] 	at org.mortbay.jetty.servlet.ServletHandler.handle(ServletHandler.java:399)
[INFO] [talledLocalContainer] 	at org.mortbay.jetty.security.SecurityHandler.handle(SecurityHandler.java:216)
[INFO] [talledLocalContainer] 	at org.mortbay.jetty.servlet.SessionHandler.handle(SessionHandler.java:182)
[INFO] [talledLocalContainer] 	at org.mortbay.jetty.handler.ContextHandler.handle(ContextHandler.java:766)
[INFO] [talledLocalContainer] 	at org.mortbay.jetty.webapp.WebAppContext.handle(WebAppContext.java:450)
[INFO] [talledLocalContainer] 	at org.mortbay.jetty.handler.ContextHandlerCollection.handle(ContextHandlerCollection.java:230)
[INFO] [talledLocalContainer] 	at org.mortbay.jetty.handler.HandlerCollection.handle(HandlerCollection.java:114)
[INFO] [talledLocalContainer] 	at org.mortbay.jetty.handler.HandlerWrapper.handle(HandlerWrapper.java:152)
[INFO] [talledLocalContainer] 	at org.mortbay.jetty.Server.handle(Server.java:326)
[INFO] [talledLocalContainer] 	at org.mortbay.jetty.HttpConnection.handleRequest(HttpConnection.java:542)
[INFO] [talledLocalContainer] 	at org.mortbay.jetty.HttpConnection$RequestHandler.headerComplete(HttpConnection.java:928)
[INFO] [talledLocalContainer] 	at org.mortbay.jetty.HttpParser.parseNext(HttpParser.java:549)
[INFO] [talledLocalContainer] 	at org.mortbay.jetty.HttpParser.parseAvailable(HttpParser.java:212)
[INFO] [talledLocalContainer] 	at org.mortbay.jetty.HttpConnection.handle(HttpConnection.java:404)
[INFO] [talledLocalContainer] 	at org.mortbay.io.nio.SelectChannelEndPoint.run(SelectChannelEndPoint.java:410)
[INFO] [talledLocalContainer] 	at org.mortbay.thread.BoundedThreadPool$PoolThread.run(BoundedThreadPool.java:451)

See: https://jenkins.magnolia-cms.com/job/magnolia-bundle_trunk/5271/

Comment by Roman Kovařík [ 24/Nov/15 ]

Add tests for UserFormModel.

Comment by Philip Mundt [ 30/Nov/15 ]

publicuserregistration/components/user-login.ftl was deprecared but replacement public-user-registration/templates/components/login.ftl doesn't render a title when provided in the dialog.

Comment by Roman Kovařík [ 02/Dec/15 ]

Reopened:

  • info.magnolia.module.publicuserregistration.password.MailChangePasswordLinkStrategy#setTargetPagePath could be overridden info.magnolia.module.publicuserregistration.password.MailChangePasswordLinkStrategy#setTargetPage -> reproducible with java8 where e.g. the link from mail uses the default configuration.
Generated at Mon Feb 12 06:43:40 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.