Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0041747Openbravo ERPA. Platformpublic2019-09-04 12:152019-09-10 10:29
Reportersalvador_campanella 
Assigned Toalostale 
PriorityimmediateSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version3.0PR19Q3 
Target VersionpiFixed in Version3.0PR19Q4 
Merge Request Status
Review Assigned Tocaristu
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression levelProduction - QA Approved
Regression date2019-04-26
Regression introduced in release3.0PR19Q3
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/cbb150e17b8b69ad6f243c526ba64ef744ed8ba2 [^]
Triggers an Emergency PackNo
Summary0041747: Negative values have string format when they are exported to csv file
DescriptionWhen the data of any grid is exported as csv using the button "Export to Spreadsheet" the negative values have a prefix "tabulation Character" this produces that the spreadsheet program format the field as text. This causes that these values don't be used to do autosum or in formulas.

Also, if the spreadsheet program has the tab character as separator the field with negative value have a shift to the right column

This prefix character was introduced in the issue
https://issues.openbravo.com/view.php?id=40647 [^]
Steps To Reproduce- Export any grid than contains negative values
- Open the CSV file with a spreadsheet program

a.- Leave the tab character selected as a separator
- Result all the values negatives will be a shift to the right column

b.- Deselect the tab character
- Do an Autosum with all the column that contains negative values
- Save the value
- Edit any negative value erasing the tab character
- Do the autosum again
- Result if different.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
depends on backport 00417543.0PR19Q3.1 closed alostale Negative values have string format when they are exported to csv file 
caused by defect 0040647 closed alostale escape formulas when exporting csv 
Attached Filespng Screenshot from 2019-09-04 12-03-00.png (24,753) 2019-09-04 12:15
https://issues.openbravo.com/file_download.php?file_id=13281&type=bug
png

png Screenshot from 2019-09-04 12-04-12.png (12,856) 2019-09-04 12:15
https://issues.openbravo.com/file_download.php?file_id=13282&type=bug
png
Issue History
Date ModifiedUsernameFieldChange
2019-09-04 12:15salvador_campanellaNew Issue
2019-09-04 12:15salvador_campanellaAssigned To => platform
2019-09-04 12:15salvador_campanellaFile Added: Screenshot from 2019-09-04 12-03-00.png
2019-09-04 12:15salvador_campanellaModules => Core
2019-09-04 12:15salvador_campanellaResolution time => 1569362400
2019-09-04 12:15salvador_campanellaTriggers an Emergency Pack => No
2019-09-04 12:15salvador_campanellaFile Added: Screenshot from 2019-09-04 12-04-12.png
2019-09-04 12:18PracticsIssue Monitored: Practics
2019-09-04 12:20PracticsIssue End Monitor: Practics
2019-09-04 12:20PracticsIssue Monitored: Practics
2019-09-05 08:27alostaleAssigned Toplatform => alostale
2019-09-05 08:27alostaleRelationship addedcaused by 0040647
2019-09-05 08:30alostaleReview Assigned To => caristu
2019-09-05 08:30alostaleRegression level => Production - QA Approved
2019-09-05 08:30alostaleRegression date => 2019-04-26
2019-09-05 08:30alostaleRegression introduced in release => 3.0PR19Q3
2019-09-05 08:30alostaleRegression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/cbb150e17b8b69ad6f243c526ba64ef744ed8ba2 [^]
2019-09-05 08:30alostalePrioritynormal => immediate
2019-09-05 08:32alostaleStatusnew => scheduled
2019-09-05 08:47hgbotCheckin
2019-09-05 08:47hgbotNote Added: 0114405
2019-09-05 08:47hgbotStatusscheduled => resolved
2019-09-05 08:47hgbotResolutionopen => fixed
2019-09-05 08:47hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/5879f2488432819dc36b197421e94aca5f804314 [^]
2019-09-05 18:16hudsonbotCheckin
2019-09-05 18:16hudsonbotNote Added: 0114427
2019-09-10 10:29caristuNote Added: 0114486
2019-09-10 10:29caristuStatusresolved => closed
2019-09-10 10:29caristuFixed in Version => 3.0PR19Q4

Notes
(0114405)
hgbot   
2019-09-05 08:47   
Repository: erp/devel/pi
Changeset: 5879f2488432819dc36b197421e94aca5f804314
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Thu Sep 05 08:47:04 2019 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/5879f2488432819dc36b197421e94aca5f804314 [^]

fixed bug 41747: negative numeric values exported as string in csv

  Whenever a value is exported to csv, it is prepended a tab if it starts with
  any character that can be used for formulas (issue 0040647) in order to prevent
  formula injection. This causes negative numeric values to be also escaped being
  treated as strings.

  Numeric values don't require to be escaped at all as they are cannot be crafted
  for formula injection.

---
M modules/org.openbravo.service.datasource/src/org/openbravo/service/datasource/DataSourceServlet.java
---
(0114427)
hudsonbot   
2019-09-05 18: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/494bf1e23c6a [^]
Maturity status: Test
(0114486)
caristu   
2019-09-10 10:29   
Reviewed + tested OK.