[MGNLUI-3213] SwitchableField containing MultiValueField throws IllegalArgumentException on commit Created: 22/Oct/14  Updated: 09/Mar/21  Resolved: 09/Mar/21

Status: Closed
Project: Magnolia UI
Component/s: dialogs
Affects Version/s: 5.3.4
Fix Version/s: None

Type: Bug Priority: Neutral
Reporter: Diana Racho Assignee: Unassigned
Resolution: Workaround exists Votes: 10
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: PNG File DelegatingMultiValueFieldTransformer.png     JPEG File switchable_with_multivalue.jpg    
Issue Links:
Relates
relates to MGNLUI-3533 Using Switchable field with Composite... Closed
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Date of First Response:

 Description   

When using MultiValueField within a SwitchableField we get following exception:

14-10-22 14:11:26 ERROR AUTHOR[http-bio-8001-exec-6]info.magnolia.ui.dialog.formdialog.FormDialogPresenterImpl: An error occurred while executing an action.
info.magnolia.ui.api.action.ActionExecutionException: Action execution failed for action: commit
        at info.magnolia.ui.api.action.AbstractActionExecutor.execute(AbstractActionExecutor.java:64)
        at info.magnolia.ui.dialog.BaseDialogPresenter.executeAction(BaseDialogPresenter.java:198)
        at info.magnolia.ui.dialog.BaseDialogPresenter.onActionFired(BaseDialogPresenter.java:189)
        at info.magnolia.ui.dialog.actionarea.renderer.DefaultEditorActionRenderer$DefaultActionView$1.buttonClick(DefaultEditorActionRenderer.java:66)
        at sun.reflect.GeneratedMethodAccessor539.invoke(Unknown Source)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:606)
        at com.vaadin.event.ListenerMethod.receiveEvent(ListenerMethod.java:508)
        at com.vaadin.event.EventRouter.fireEvent(EventRouter.java:167)
        at com.vaadin.server.AbstractClientConnector.fireEvent(AbstractClientConnector.java:969)
        at com.vaadin.ui.Button.fireClick(Button.java:368)
        at com.vaadin.ui.Button$1.click(Button.java:57)
        at sun.reflect.GeneratedMethodAccessor538.invoke(Unknown Source)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:606)
        at com.vaadin.server.ServerRpcManager.applyInvocation(ServerRpcManager.java:168)
        at com.vaadin.server.ServerRpcManager.applyInvocation(ServerRpcManager.java:118)
        at com.vaadin.server.communication.ServerRpcHandler.handleBurst(ServerRpcHandler.java:214)
        at com.vaadin.server.communication.ServerRpcHandler.handleRpc(ServerRpcHandler.java:111)
        at com.vaadin.server.communication.UidlRequestHandler.synchronizedHandleRequest(UidlRequestHandler.java:91)
        at com.vaadin.server.SynchronizedRequestHandler.handleRequest(SynchronizedRequestHandler.java:37)
        at com.vaadin.server.VaadinService.handleRequest(VaadinService.java:1371)
        at com.vaadin.server.VaadinServlet.service(VaadinServlet.java:238)
        at info.magnolia.ui.admincentral.AdmincentralVaadinServlet.service(AdmincentralVaadinServlet.java:132)
        at javax.servlet.http.HttpServlet.service(HttpServlet.java:722)
        at info.magnolia.cms.filters.ServletDispatchingFilter.doFilter(ServletDispatchingFilter.java:148)
        at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:89)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:82)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:82)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:82)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:82)
        at info.magnolia.cms.filters.CompositeFilter.doFilter(CompositeFilter.java:65)
        at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:89)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
        at info.magnolia.cms.filters.VirtualUriFilter.doFilter(VirtualUriFilter.java:68)
        at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:89)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
        at info.magnolia.module.cache.executor.Bypass.processCacheRequest(Bypass.java:58)
        at info.magnolia.module.cache.executor.CompositeExecutor.processCacheRequest(CompositeExecutor.java:66)
        at info.magnolia.module.cache.filter.CacheFilter.doFilter(CacheFilter.java:153)
        at info.magnolia.cms.filters.OncePerRequestAbstractMgnlFilter.doFilter(OncePerRequestAbstractMgnlFilter.java:60)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
        at info.magnolia.cms.i18n.I18nContentSupportFilter.doFilter(I18nContentSupportFilter.java:73)
        at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:89)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
        at info.magnolia.cms.filters.RangeSupportFilter.doFilter(RangeSupportFilter.java:84)
        at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:89)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
        at info.magnolia.cms.security.BaseSecurityFilter.doFilter(BaseSecurityFilter.java:57)
        at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:89)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:82)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:82)
        at info.magnolia.cms.security.SecurityCallbackFilter.doFilter(SecurityCallbackFilter.java:83)
        at info.magnolia.cms.filters.OncePerRequestAbstractMgnlFilter.doFilter(OncePerRequestAbstractMgnlFilter.java:60)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
        at info.magnolia.cms.security.LogoutFilter.doFilter(LogoutFilter.java:94)
        at info.magnolia.cms.filters.OncePerRequestAbstractMgnlFilter.doFilter(OncePerRequestAbstractMgnlFilter.java:60)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
        at info.magnolia.module.templatingkit.filters.SiteMergeFilter.doFilter(SiteMergeFilter.java:112)
        at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:89)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
        at info.magnolia.multisite.filters.MultiSiteFilter.doFilter(MultiSiteFilter.java:104)
        at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:89)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
        at info.magnolia.cms.filters.MultiChannelFilter.doFilter(MultiChannelFilter.java:82)
        at info.magnolia.cms.filters.OncePerRequestAbstractMgnlFilter.doFilter(OncePerRequestAbstractMgnlFilter.java:60)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
        at info.magnolia.module.cache.filter.GZipFilter.doFilter(GZipFilter.java:73)
        at info.magnolia.cms.filters.OncePerRequestAbstractMgnlFilter.doFilter(OncePerRequestAbstractMgnlFilter.java:60)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:82)
        at info.magnolia.cms.security.auth.login.LoginFilter.doFilter(LoginFilter.java:104)
        at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:89)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
        at info.magnolia.enterprise.registration.RegistrationFilter.doFilter(RegistrationFilter.java:56)
        at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:89)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:82)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:82)
        at info.magnolia.module.devicedetection.filter.DeviceDetectionFilter.doFilter(DeviceDetectionFilter.java:71)
        at info.magnolia.cms.filters.OncePerRequestAbstractMgnlFilter.doFilter(OncePerRequestAbstractMgnlFilter.java:60)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
        at info.magnolia.cms.filters.ContentTypeFilter.doFilter(ContentTypeFilter.java:103)
        at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:89)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
        at info.magnolia.cms.filters.ContextFilter.doFilter(ContextFilter.java:129)
        at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:89)
        at info.magnolia.cms.filters.MgnlFilterChain.doFilter(MgnlFilterChain.java:80)
        at info.magnolia.cms.filters.CompositeFilter.doFilter(CompositeFilter.java:65)
        at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:89)
        at info.magnolia.cms.filters.SafeDestroyMgnlFilterWrapper.doFilter(SafeDestroyMgnlFilterWrapper.java:106)
        at info.magnolia.cms.filters.MgnlFilterDispatcher.doDispatch(MgnlFilterDispatcher.java:66)
        at info.magnolia.cms.filters.MgnlMainFilter.doFilter(MgnlMainFilter.java:107)
        at info.magnolia.cms.filters.MgnlMainFilter.doFilter(MgnlMainFilter.java:93)
        at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:243)
        at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:210)
        at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:222)
        at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:123)
        at org.apache.catalina.authenticator.AuthenticatorBase.invoke(AuthenticatorBase.java:472)
        at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:168)
        at org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:99)
        at org.apache.catalina.valves.AccessLogValve.invoke(AccessLogValve.java:929)
        at org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:118)
        at org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:407)
        at org.apache.coyote.http11.AbstractHttp11Processor.process(AbstractHttp11Processor.java:1002)
        at org.apache.coyote.AbstractProtocol$AbstractConnectionHandler.process(AbstractProtocol.java:585)
        at org.apache.tomcat.util.net.JIoEndpoint$SocketProcessor.run(JIoEndpoint.java:312)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
        at java.lang.Thread.run(Thread.java:744)
Caused by: java.lang.IllegalArgumentException: Cannot set property to a value of type class com.vaadin.data.util.PropertysetItem
        at info.magnolia.jcr.util.PropertyUtil.setProperty(PropertyUtil.java:132)
        at info.magnolia.ui.vaadin.integration.jcr.AbstractJcrNodeAdapter.updateProperty(AbstractJcrNodeAdapter.java:266)
        at info.magnolia.ui.vaadin.integration.jcr.AbstractJcrAdapter.updateProperties(AbstractJcrAdapter.java:126)
        at info.magnolia.ui.vaadin.integration.jcr.AbstractJcrNodeAdapter.updateProperties(AbstractJcrNodeAdapter.java:219)
        at info.magnolia.ui.vaadin.integration.jcr.JcrNewNodeAdapter.applyChanges(JcrNewNodeAdapter.java:124)
        at info.magnolia.ui.admincentral.dialog.action.SaveDialogAction.execute(SaveDialogAction.java:83)
        at info.magnolia.ui.api.action.AbstractActionExecutor.execute(AbstractActionExecutor.java:62)
        ... 111 more


 Comments   
Comment by Cédric Eberhardt [ 04/Oct/17 ]

We still have the problem with 5.5.6. Any news about it?

Comment by Antti Hietala [ 20/Jun/18 ]

ceberhardt, the workaround given in MGNLUI-3533 also works here. Set the transformerClass property under your switchable field to info.magnolia.ui.form.field.transformer.composite.DelegatingCompositeFieldTransformer.

You can also use info.magnolia.ui.form.field.transformer.multi.DelegatingMultiValueFieldTransformer as recommended in documentation: DelegatingMultiValueTransformer

Here's how to apply DelegatingMultiValueFieldTransformer to the example reported in this issue:

Can you try those options and report back whether it solves the error and stores values in the way you would expect? 

Comment by Roman Kovařík [ 09/Mar/21 ]

Closing this as M5.3 has reached EOL.

https://docs.magnolia-cms.com/product-docs/Support/End-of-life-policy.html#_end_of_life

Generated at Mon Feb 12 09:04:20 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.