[MGNLUI-6817] Exception throwing use-case when using upload field within $tabbedForm layout Created: 12/Aug/21  Updated: 13/Aug/21  Resolved: 13/Aug/21

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

Type: Initiative Priority: Neutral
Reporter: Adam Siska Assignee: Unassigned
Resolution: Obsolete Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Template:

 Description   

Reproduction steps:

  • Add upload field with maxUploadSize property to tabbed form
  • upload a large enough file (btw. there is no indicators here to let user know file is uploading, just upload button is disabled)
  • switch to other tab and back, now the upload button is enabled again
  • upload another file

Expected (before MGNLUI-6390):

  • last selected file is eventually uploaded in field

Actual (after MGNLUI-6390):

  • exception is thrown

Dev notes:

com.vaadin.server.UploadException: Upload failed
 at com.vaadin.server.communication.FileUploadHandler.streamToReceiver(FileUploadHandler.java:642) ~[vaadin-server-8.13.1.jar:8.13.1]
 at com.vaadin.server.communication.FileUploadHandler.handleFileUploadValidationAndData(FileUploadHandler.java:469) [vaadin-server-8.13.1.jar:8.13.1]
 at com.vaadin.server.communication.FileUploadHandler.doHandleSimpleMultipartFileUpload(FileUploadHandler.java:422) [vaadin-server-8.13.1.jar:8.13.1]
 at com.vaadin.server.communication.FileUploadHandler.handleRequest(FileUploadHandler.java:299) [vaadin-server-8.13.1.jar:8.13.1]
 at com.vaadin.server.VaadinService.handleRequest(VaadinService.java:1608) [vaadin-server-8.13.1.jar:8.13.1]
 at com.vaadin.server.VaadinServlet.service(VaadinServlet.java:448) [vaadin-server-8.13.1.jar:8.13.1]
 at info.magnolia.admincentral.AdmincentralServlet.service(AdmincentralServlet.java:119) [magnolia-admincentral-6.2.12-SNAPSHOT.jar:?]
 at javax.servlet.http.HttpServlet.service(HttpServlet.java:741) [servlet-api.jar:?]
 at info.magnolia.cms.filters.ServletDispatchingFilter.doFilter(ServletDispatchingFilter.java:148) [magnolia-core-6.2.12-SNAPSHOT.jar:?]
 at info.magnolia.cms.filters.AbstractMgnlFilter.doFilter(AbstractMgnlFilter.java:85) [magnolia-core-6.2.12-SNAPSHOT.jar:?]
 ...
 at java.lang.Thread.run(Thread.java:748) [?:1.8.0_212]
 Caused by: java.lang.IllegalStateException: uploading already started
 at com.vaadin.ui.Upload.startUpload(Upload.java:900) ~[vaadin-server-8.13.1.jar:8.13.1]
 at com.vaadin.ui.Upload$2.streamingStarted(Upload.java:1136) ~[vaadin-server-8.13.1.jar:8.13.1]
 at com.vaadin.server.communication.FileUploadHandler.streamToReceiver(FileUploadHandler.java:559) ~[vaadin-server-8.13.1.jar:8.13.1]
 ... 91 more

see https://git.magnolia-cms.com/projects/PLATFORM/repos/ui/pull-requests/1841/overview?commentId=83510



 Comments   
Comment by Adam Siska [ 13/Aug/21 ]

obsolete after https://git.magnolia-cms.com/projects/PLATFORM/repos/ui/pull-requests/1851/overview

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