Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0019114Openbravo ERP03. Procurement managementpublic2011-11-21 09:142011-12-13 16:43
Reporterjonalegriaesarte 
Assigned Tomtaal 
PriorityurgentSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Versionpi 
Target Version3.0MP7Fixed in Version3.0MP7 
Merge Request Status
Review Assigned To
OBNetwork customerOBPS
Web browser
ModulesCore
Support ticket13306
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0019114: Amounts not formatted properly in grid view
DescriptionAmounts not formatted properly in grid view. They do not follow the format defined in Format.xml
Steps To Reproduce- login
- go to Purchase Order screen and view it in grid mode
- see that the Total Gross Amount is NOT formatted using two fixed decimals as defined in Format.xml, 224.2 should be shown as 224.20
- if you edit (inline or in full edit) the amounts display correctly
Proposed Solution
Additional Information
TagsRegression
Relationships
related to defect 00175313.0MP4 closed mtaal Description field with 2 rows looks nasty in grid view 
has duplicate defect 00192453.0MP7 closed mtaal Re-introduce 1000 separators 
related to defect 0018992 closed alostale Monetary amounts are not properly formatted in grid view 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2011-11-21 09:14jonalegriaesarteNew Issue
2011-11-21 09:14jonalegriaesarteAssigned To => alostale
2011-11-21 09:14jonalegriaesarteModules => Core
2011-11-21 09:14jonalegriaesarteOBNetwork customer => Yes
2011-11-21 09:14jonalegriaesarteSupport ticket => 13306
2011-11-21 09:16jonalegriaesarteSeveritymajor => minor
2011-11-21 09:16jonalegriaesarteTarget Version3.0MP6 => 3.0MP7
2011-11-23 08:59alostaleRelationship addedrelated to 0018992
2011-11-23 09:01alostaleStatusnew => scheduled
2011-11-23 09:01alostalefix_in_branch => pi
2011-12-01 12:01alostaleRelationship addedrelated to 0017531
2011-12-01 13:36alostaleNote Added: 0043651
2011-12-12 09:57mtaalAssigned Toalostale => mtaal
2011-12-12 11:23hgbotCheckin
2011-12-12 11:23hgbotNote Added: 0043787
2011-12-12 11:23hgbotStatusscheduled => resolved
2011-12-12 11:23hgbotResolutionopen => fixed
2011-12-12 11:23hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/18cf176c087fef14aa0e9475a24af739f7b77c63 [^]
2011-12-12 11:24mtaalRelationship addedhas duplicate 0019245
2011-12-12 12:11alostaleNote Added: 0043796
2011-12-12 12:11alostaleStatusresolved => closed
2011-12-12 12:11alostaleFixed in Version => 3.0MP7
2011-12-12 12:11alostaleTag Attached: Regression
2011-12-13 16:43hudsonbotCheckin
2011-12-13 16:43hudsonbotNote Added: 0043841

Notes
(0043651)
alostale   
2011-12-01 13:36   
Fix for 0017531 adds formatCellValue method to OBViewGrid. The problem is this methods returns the value without formatting it in case it is not a String with \n.

Completely removing this method solved this issue though 0017531 appears again. Returning super formatCellValue does not work...
(0043787)
hgbot   
2011-12-12 11:23   
Repository: erp/devel/pi
Changeset: 18cf176c087fef14aa0e9475a24af739f7b77c63
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Mon Dec 12 11:22:51 2011 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/18cf176c087fef14aa0e9475a24af739f7b77c63 [^]

Fixes issue 19114: Amounts not formatted properly in grid view

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-view-grid.js
---
(0043796)
alostale   
2011-12-12 12:11   
Tested pi@45ff7d718f06

Code review.
Verified it fixes the reported issue.
Verified 0017531 is still fixed.

No other regression possibility figured out.
(0043841)
hudsonbot   
2011-12-13 16:43   
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/8a06361cc265 [^]

Maturity status: Test