Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0022119Openbravo ERPA. Platformpublic2012-10-26 10:032012-11-27 11:19
Reporteriperdomo 
Assigned Toshankarb 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS30OS Version3.5.4-1-ARCH
Product Version 
Target VersionFixed in Version3.0MP18 
Merge Request Status
Review Assigned ToAugustoMauch
OBNetwork customerNo
Web browserGoogle Chrome
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0022119: convertToClassicString in TimeUIDefinition is not in sync with the UTC normalization logic
DescriptionThe TimeUIDefinition class has some normalization logic to convert times to UTC back and forth.

The converToClassicString doesn't have this logic.
Steps To ReproduceMake a FIC request to a tab/record that has a time field, e.g.
* Login as System Administrator
* Make a GET request:
http://localhost:8080/openbravo/org.openbravo.client.kernel?MODE=EDIT&PARENT_ID=null&TAB_ID=CD573DF1E351485EA2B2DE487DCACA6F&ROW_ID=5C13C452732D414A8419BB2C5F8FCE39&_action=org.openbravo.client.application.window.FormInitializationComponent [^]
* See the log
Proposed Solution* converToClassicString must have the same normalization to UTC time
Additional Information
TagsNo tags attached.
Relationships
Attached Filesdiff FixIssue22119.diff (3,936) 2012-11-09 09:26
https://issues.openbravo.com/file_download.php?file_id=5718&type=bug
Issue History
Date ModifiedUsernameFieldChange
2012-10-26 10:03iperdomoNew Issue
2012-10-26 10:03iperdomoAssigned To => AugustoMauch
2012-10-26 10:03iperdomoWeb browser => Google Chrome
2012-10-26 10:03iperdomoModules => Core
2012-10-26 10:03iperdomoOBNetwork customer => No
2012-10-29 17:49AugustoMauchAssigned ToAugustoMauch => shankarb
2012-11-09 09:26shankarbFile Added: FixIssue22119.diff
2012-11-14 14:37shankarbReview Assigned To => dbaz
2012-11-14 14:37shankarbWeb browserGoogle Chrome => Google Chrome
2012-11-14 14:37shankarbIssue Monitored: dbaz
2012-11-14 14:38hgbotCheckin
2012-11-14 14:38hgbotNote Added: 0054232
2012-11-14 14:38hgbotStatusnew => resolved
2012-11-14 14:38hgbotResolutionopen => fixed
2012-11-14 14:38hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/ead23f919c59e97b82c9a678c44258dfe3920497 [^]
2012-11-20 20:31dbazNote Added: 0054352
2012-11-20 20:31dbazStatusresolved => new
2012-11-20 20:31dbazResolutionfixed => open
2012-11-20 20:31dbazReview Assigned Todbaz => AugustoMauch
2012-11-20 20:31dbazWeb browserGoogle Chrome => Google Chrome
2012-11-20 20:32dbazIssue End Monitor: dbaz
2012-11-20 20:32dbazIssue Monitored: AugustoMauch
2012-11-20 20:32dbazStatusnew => scheduled
2012-11-20 20:32dbazfix_in_branch => pi
2012-11-20 20:32dbazStatusscheduled => resolved
2012-11-20 20:32dbazResolutionopen => fixed
2012-11-21 06:06hudsonbotCheckin
2012-11-21 06:06hudsonbotNote Added: 0054383
2012-11-27 11:19AugustoMauchNote Added: 0054518
2012-11-27 11:19AugustoMauchStatusresolved => closed
2012-11-27 11:19AugustoMauchFixed in Version => 3.0MP18

Notes
(0054232)
hgbot   
2012-11-14 14:38   
Repository: erp/devel/pi
Changeset: ead23f919c59e97b82c9a678c44258dfe3920497
Author: Shankar Balachandran <shankar.balachandran <at> openbravo.com>
Date: Wed Nov 14 14:36:31 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/ead23f919c59e97b82c9a678c44258dfe3920497 [^]

Fixes Issue 0022119: Implemented convertToClassicString in TimeUIDefinition.

Implemented defined method convertToClassicString and refactored code to reuse time normalization logic.

---
M modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/reference/TimeUIDefinition.java
---
(0054352)
dbaz   
2012-11-20 20:31   
Reopened to modify the field "Review Assigned To"
(0054383)
hudsonbot   
2012-11-21 06:06   
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/c3c5e61b0164 [^]

Maturity status: Test
(0054518)
AugustoMauch   
2012-11-27 11:19   
Code reviewed and verified in pi@b43e48d603a9