Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0038451Openbravo ERPA. Platformpublic2018-04-25 14:182018-04-27 13:09
Reportercaristu 
Assigned Tocaristu 
PriorityhighSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target Version3.0PR18Q2Fixed in Version3.0PR18Q2 
Merge Request Status
Review Assigned Toalostale
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0038451: Allow to download attachments in temporary files using the attachment name without concurrency problems
DescriptionAllow to download attachments in temporary files using the attachment name itself without having (possible) concurrency problems in case two users download the same attachment at the same time.

Steps To ReproduceIn description
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0038450 closed caristu Allow to download attachments in temporary files using the attachment name without concurrency problems 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2018-04-25 14:18caristuTypedefect => backport
2018-04-25 14:18caristuTarget Version => 3.0PR18Q2
2018-04-25 16:02hgbotCheckin
2018-04-25 16:02hgbotNote Added: 0104081
2018-04-25 16:02hgbotStatusscheduled => resolved
2018-04-25 16:02hgbotResolutionopen => fixed
2018-04-25 16:02hgbotFixed in SCM revision => http://code.openbravo.com/erp/backports/3.0PR18Q2/rev/b139b9c81651c2bac8e03bf0483a997970cdd13d [^]
2018-04-25 16:03caristuAssigned Toplatform => caristu
2018-04-25 16:04caristuReview Assigned To => alostale
2018-04-25 16:16hgbotCheckin
2018-04-25 16:16hgbotNote Added: 0104083
2018-04-27 13:09alostaleNote Added: 0104159
2018-04-27 13:09alostaleStatusresolved => closed
2018-04-27 13:09alostaleFixed in Version => 3.0PR18Q2

Notes
(0104081)
hgbot   
2018-04-25 16:02   
Repository: erp/backports/3.0PR18Q2
Changeset: b139b9c81651c2bac8e03bf0483a997970cdd13d
Author: Carlos Aristu <carlos.aristu <at> openbravo.com>
Date: Wed Apr 25 15:50:10 2018 +0200
URL: http://code.openbravo.com/erp/backports/3.0PR18Q2/rev/b139b9c81651c2bac8e03bf0483a997970cdd13d [^]

fixes bug 38451: Remove temp file's directory used to download the attachment

  Now the AttachImplementationManager deletes also the temp file's directory used to download the attachment. This change only affects to those attach implementations which return the downloaded attachment inside a temporary file.

  With this change now that attach implementations are able generate those temporary files with any name by creating them inside folders with random names. This is intended to avoid concurrency problems in case two users download the same attachment at the same time.

---
M modules/org.openbravo.client.application/src/org/openbravo/client/application/attachment/AttachImplementationManager.java
---
(0104083)
hgbot   
2018-04-25 16:16   
Repository: erp/backports/3.0PR18Q2
Changeset: 5e0d86cedd3380dec7aa7df9fd9692367c692053
Author: Carlos Aristu <carlos.aristu <at> openbravo.com>
Date: Wed Apr 25 16:14:22 2018 +0200
URL: http://code.openbravo.com/erp/backports/3.0PR18Q2/rev/5e0d86cedd3380dec7aa7df9fd9692367c692053 [^]

related to issue 38451: apply the logic on "Download All" also

---
M modules/org.openbravo.client.application/src/org/openbravo/client/application/attachment/AttachImplementationManager.java
---
(0104159)
alostale   
2018-04-27 13:09   
reviewed + tested