Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0023260Openbravo ERPA. Platformpublic2013-03-07 17:442013-03-13 14:44
Reporteregoitz 
Assigned ToAugustoMauch 
PriorityimmediateSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version3.0MP20 
Target Version3.0MP22Fixed in Version3.0MP22 
Merge Request Status
Review Assigned Toshankarb
OBNetwork customerOBPS
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0023260: The time fields are not correctly exported to csv
DescriptionWhen exporting a grid to csv with a column defined as time, the time is exported incorrectly.
Steps To Reproduce-Go to process request window
-Add the start time on the grd
-Export the grid
*See that starttime columns is exported with incorrect time.
In my case with 1 hours difference. It could be related with the timezone of the client.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 00218103.0MP19 closed AugustoMauch The times are incorrectly exported when exporting the grid to csv 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2013-03-07 17:44egoitzNew Issue
2013-03-07 17:44egoitzAssigned To => AugustoMauch
2013-03-07 17:44egoitzModules => Core
2013-03-07 17:44egoitzOBNetwork customer => Yes
2013-03-07 17:44egoitzTriggers an Emergency Pack => No
2013-03-07 17:44egoitzRelationship addedrelated to 0021810
2013-03-07 18:10AugustoMauchIssue Monitored: shankarb
2013-03-07 18:10AugustoMauchReview Assigned To => shankarb
2013-03-07 18:16hgbotCheckin
2013-03-07 18:16hgbotNote Added: 0057151
2013-03-07 18:16hgbotStatusnew => resolved
2013-03-07 18:16hgbotResolutionopen => fixed
2013-03-07 18:16hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/141b8d2ba8dca36f64a783d88aab7489b775b29e [^]
2013-03-07 18:19AugustoMauchNote Added: 0057152
2013-03-13 14:44shankarbNote Added: 0057228
2013-03-13 14:44shankarbStatusresolved => closed
2013-03-13 14:44shankarbFixed in Version => 3.0MP22

Notes
(0057151)
hgbot   
2013-03-07 18:16   
Repository: erp/devel/pi
Changeset: 141b8d2ba8dca36f64a783d88aab7489b775b29e
Author: Augusto Mauch <augusto.mauch <at> openbravo.com>
Date: Thu Mar 07 18:14:20 2013 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/141b8d2ba8dca36f64a783d88aab7489b775b29e [^]

Fixes issue 23260: Time fields are exported properly to CSV

In the fix of the issue 21810 [1] a change was done to export the DateTime columns properly to CSV. This fix did not apply to Time fields, so when a time column was exported to CSV it could belong to the wrong timezone.

This fix applies the fix of the DateTime columns to the Time columns.

[1] https://issues.openbravo.com/view.php?id=21810 [^]

---
M modules/org.openbravo.service.datasource/src/org/openbravo/service/datasource/DataSourceServlet.java
M src/org/openbravo/base/model/Property.java
---
(0057152)
AugustoMauch   
2013-03-07 18:19   
Test plan:

As System Administrator
- Open the Preferences window
- Add a preference to the property "Local Timezone ID", choose the proper timezone from this list [1].
- Log out and log in again to load the new preference.
- Open the Process Definition window.
- Make visible the Start Time and the Creation Date columns
- Export the grid to CSV
- Check that the values of the Creation Date and Start Time columns are the same in the CSV and in the grid

[1] http://www.java2s.com/Tutorial/Java/0120__Development/GettingallthetimezonesIDs.htm [^]
(0057228)
shankarb   
2013-03-13 14:44   
Code reviewed and verified in pi changeset 4b43f5a6429b