Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0040647Openbravo ERPA. Platformpublic2019-04-18 15:362019-05-21 15:16
Reporteralostale 
Assigned Toalostale 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version3.0PR19Q3 
Merge Request Status
Review Assigned Tocaristu
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0040647: escape formulas when exporting csv
DescriptionWhen exporting to cvs, if a value starts with equals it should be escaped not to treat it as a formula when opened with as spreadsheet.
Steps To Reproduce-
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 0040646 closed nonofrancisco use proper html escape in old selectors 
causes defect 0041747pi closed alostale Negative values have string format when they are exported to csv file 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2019-04-18 15:36alostaleNew Issue
2019-04-18 15:36alostaleAssigned To => platform
2019-04-18 15:36alostaleModules => Core
2019-04-18 15:36alostaleTriggers an Emergency Pack => No
2019-04-18 15:37alostaleReview Assigned To => caristu
2019-04-18 15:37alostaleAssigned Toplatform => alostale
2019-04-18 15:55alostaleRelationship addedrelated to 0040646
2019-04-18 15:57hgbotCheckin
2019-04-18 15:57hgbotNote Added: 0111210
2019-04-18 15:57hgbotStatusnew => resolved
2019-04-18 15:57hgbotResolutionopen => fixed
2019-04-18 15:57hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/2986020672639f14f8c2cbd9d66e5d9577b38d10 [^]
2019-04-26 10:27hgbotCheckin
2019-04-26 10:27hgbotNote Added: 0111353
2019-04-26 15:09caristuNote Added: 0111363
2019-04-26 15:09caristuStatusresolved => closed
2019-04-26 15:09caristuFixed in Version => 3.0PR19Q3
2019-05-21 15:16hudsonbotCheckin
2019-05-21 15:16hudsonbotNote Added: 0111990
2019-05-21 15:16hudsonbotCheckin
2019-05-21 15:16hudsonbotNote Added: 0112002
2019-09-05 08:27alostaleRelationship addedcauses 0041747

Notes
(0111210)
hgbot   
2019-04-18 15:57   
Repository: erp/devel/pi
Changeset: 2986020672639f14f8c2cbd9d66e5d9577b38d10
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Thu Apr 18 15:42:05 2019 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/2986020672639f14f8c2cbd9d66e5d9577b38d10 [^]

fixed bug 40647: escape formulas when exporting csv

  When exporting to csv:
    - If value starts with equals, escape it so that it is not treated as a formula
    - Use JDK's String.replace instead of OB's

---
M modules/org.openbravo.service.datasource/src/org/openbravo/service/datasource/DataSourceServlet.java
---
(0111353)
hgbot   
2019-04-26 10:27   
Repository: erp/devel/pi
Changeset: cbb150e17b8b69ad6f243c526ba64ef744ed8ba2
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Fri Apr 26 10:23:56 2019 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/cbb150e17b8b69ad6f243c526ba64ef744ed8ba2 [^]

related to issue 40647: escape formulas when exporting csv

  - Use tab instead of quoute to escape forumulas as latter is not visible
  - Escape also other characters that prefix formulas (+,-,@)

---
M modules/org.openbravo.service.datasource/src/org/openbravo/service/datasource/DataSourceServlet.java
---
(0111363)
caristu   
2019-04-26 15:09   
Code reviewed + tested OK
(0111990)
hudsonbot   
2019-05-21 15:16   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/9b8f37d9d85e [^]
Maturity status: Test
(0112002)
hudsonbot   
2019-05-21 15:16   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/9b8f37d9d85e [^]
Maturity status: Test