Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0043851Openbravo ERPA. Platformpublic2020-04-23 19:542022-02-01 07:24
caristu 
Triage Platform Base 
normalminorhave not tried
newopen 
5
 
 
Core
No
0043851: Wrong error handling if the donwload of a ProcessUploadedFile process fails
In case an error is thrown while dowloading the result of a ProcessUploadedFile, the error is not being handled properly.
1) Apply the attached patch which forces an error while dowloading a file with a ProcessUploadedFile
2) Go to the "Business Partner Set" window
3) Click in the "upload file" toolbar button and upload the attached file (bpSets.txt)
4) Note that after our forced error is thrown, it is not being managed properly. The following stack trace is raised in the log:
org.openbravo.base.exception.OBException: Error uploading file JSONObject["viewID"] not found.
    at org.openbravo.client.application.businesslogic.ProcessUploadedFile.post(ProcessUploadedFile.java:98) ~[classes/:?]
    at org.openbravo.client.application.businesslogic.ProcessUploadedFile.doPost(ProcessUploadedFile.java:65) ~[classes/:?]
    at javax.servlet.http.HttpServlet.service(HttpServlet.java:661) ~[servlet-api.jar:?]
....
Note that the ProcessUploadedFile already has a mechanism to handle the error and display it in the toolbar. The problem is that the viewID is not being sent in the request to download the resulting file.

Adding the viewID in the request fixes the problem.
No tags attached.
txt bpSets.txt (9) 2020-04-23 19:54
https://issues.openbravo.com/file_download.php?file_id=14373&type=bug
diff forcedError.diff (884) 2020-04-23 19:55
https://issues.openbravo.com/file_download.php?file_id=14374&type=bug
Issue History
2020-04-23 19:54caristuNew Issue
2020-04-23 19:54caristuAssigned To => platform
2020-04-23 19:54caristuFile Added: bpSets.txt
2020-04-23 19:54caristuModules => Core
2020-04-23 19:54caristuTriggers an Emergency Pack => No
2020-04-23 19:55caristuFile Added: forcedError.diff
2022-02-01 07:24alostaleAssigned Toplatform => Triage Platform Base

There are no notes attached to this issue.