Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0049398Openbravo ERPB. User interfacepublic2022-05-24 21:052022-06-27 11:34
Reporteraxelmercado 
Assigned ToAugustoMauch 
PriorityhighSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version3.0PR19Q4.3 
Target VersionFixed in VersionPR22Q3 
Merge Request Statusapproved
Review Assigned To
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0049398: Error displaying a time type field in grid view and form view in a property field
DescriptionWhen displaying information in a field of type time stored in a property field, it is displayed correctly in grid view, but in form view the values are changed.
Steps To Reproduce1-Identify a table that contains a time column or create a new new. I.e. you can use the ad_org.EM_Obpos2_Default_Deliverytime column (Default Delivery Time field)
2-Call this column through a property field in another table. I.e. create a property field in the Unit of Measure header tab, using organization.obpos2DefaultDeliverytime as property value.
3-Give a value to the Default Delivery Time field of an organization
3-Open Unit of Measure, and notice that the values for the property field in the form and grid views are different


This will show that the values in both views are different.
Proposed SolutionWe believe that this is due to the fact that in grid view the value is shown with time zone and in formulary view it is not.
Additional Information
TagsNo tags attached.
Relationships
Attached Filesjpeg cap1.jpeg (70,096) 2022-05-24 21:05
https://issues.openbravo.com/file_download.php?file_id=17056&type=bug
jpeg

jpeg cap2.jpeg (51,356) 2022-05-24 21:06
https://issues.openbravo.com/file_download.php?file_id=17057&type=bug
jpeg

png exampWin,tab&fie.png (108,585) 2022-06-16 15:12
https://issues.openbravo.com/file_download.php?file_id=17176&type=bug
png

png exampWin,tab&fie2.png (260,914) 2022-06-16 15:13
https://issues.openbravo.com/file_download.php?file_id=17177&type=bug
png

png exampBPLocationHour.png (135,734) 2022-06-16 15:13
https://issues.openbravo.com/file_download.php?file_id=17178&type=bug
png

png exampWindHour.png (57,905) 2022-06-16 15:14
https://issues.openbravo.com/file_download.php?file_id=17179&type=bug
png

png exampWindHourFormView.png (69,585) 2022-06-16 15:14
https://issues.openbravo.com/file_download.php?file_id=17180&type=bug
png
Issue History
Date ModifiedUsernameFieldChange
2022-05-24 21:05axelmercadoNew Issue
2022-05-24 21:05axelmercadoAssigned To => Triage Platform Base
2022-05-24 21:05axelmercadoFile Added: cap1.jpeg
2022-05-24 21:05axelmercadoModules => Core
2022-05-24 21:05axelmercadoTriggers an Emergency Pack => No
2022-05-24 21:06axelmercadoFile Added: cap2.jpeg
2022-05-24 21:08ivazquezOBNetwork customer => No
2022-05-24 21:14eugeniIssue Monitored: eugeni
2022-06-16 15:12axelmercadoFile Added: exampWin,tab&fie.png
2022-06-16 15:13axelmercadoFile Added: exampWin,tab&fie2.png
2022-06-16 15:13axelmercadoFile Added: exampBPLocationHour.png
2022-06-16 15:14axelmercadoFile Added: exampWindHour.png
2022-06-16 15:14axelmercadoFile Added: exampWindHourFormView.png
2022-06-24 12:02AugustoMauchAssigned ToTriage Platform Base => AugustoMauch
2022-06-24 12:02AugustoMauchStatusnew => scheduled
2022-06-24 13:51hgbotMerge Request Status => open
2022-06-24 13:51hgbotNote Added: 0138754
2022-06-27 10:03hgbotMerge Request Statusopen => approved
2022-06-27 10:04hgbotMerge Request Statusapproved => open
2022-06-27 10:13AugustoMauchSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=24358#r24358
2022-06-27 11:34hgbotMerge Request Statusopen => approved
2022-06-27 11:34hgbotResolutionopen => fixed
2022-06-27 11:34hgbotStatusscheduled => closed
2022-06-27 11:34hgbotFixed in Version => PR22Q3
2022-06-27 11:34hgbotNote Added: 0138776
2022-06-27 11:34hgbotNote Added: 0138777

Notes
(0138754)
hgbot   
2022-06-24 13:51   
Merge Request created: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/628 [^]
(0138776)
hgbot   
2022-06-27 11:34   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/openbravo [^]
Changeset: 1858d262a1e4e4e8b35f0bd025b89f5705831f8a
Author: Augusto Mauch <augusto.mauch@openbravo.com>
Date: 27-06-2022 11:32:43
URL: https://gitlab.com/openbravo/product/openbravo/-/commit/1858d262a1e4e4e8b35f0bd025b89f5705831f8a [^]

Fixes ISSUE-49398: Time property fields are properly displayed in form view

The TimeUIDefinition.getFieldProperties expects to receive the UTC value as a request parameter. When the field is
not a property field, the request parameter is named like this: "inp" + Sqlc.TransformaNombreColumna(field.getColumn().getDBColumnName()),
but when it is a property field the naming of the parameter is different. As a result, the current value was not found, so it was not
being converted from UTC to local time.

To fix it, the getFieldProperties method will first look for the standard parameter and if not found it will look for it using the property
field name convention

---
M modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/reference/TimeUIDefinition.java
---
(0138777)
hgbot   
2022-06-27 11:34   
Merge request merged: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/628 [^]