Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0021138
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 07. Sales managementmajoralways2012-07-18 12:112012-07-26 02:42
ReportermiruritaView Statuspublic 
Assigned Tomirurita 
PriorityhighResolutionfixedFixed in Versionpi
StatusclosedFix in branchFixed in SCM revisiond055a5224b99
ProjectionnoneETAnoneTarget Version
OSLinux 32 bitDatabasePostgreSQLJava version1.6.0_18
OS VersionCommunity ApplianceDatabase version8.3.9Ant version1.7.1
Product VersionpiSCM revision 
Review Assigned To
Web browser
ModulesCore
Regression levelProduction - Confirmed Stable
Regression date2012-03-13
Regression introduced in release
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/3ab535c40a07 [^]
Triggers an Emergency Packyes
Summary

0021138: Copy Lines not working using comma (,) as decimal separator

DescriptionCopy Lines does not work with commas as decimal separator.

If you select an existing order line and try to copy you will get an error because the "Last Price" and "Quantity" contains illegal values.

The validation for numeric values is failing with numbers using commas as decimal separator.

CopyFormOrder.html validateNumber function
var invalid = isNaN(Number(input.value));
Steps To Reproduce1) Create a new Sales Order header for Alimentos y Supermercados, S.A with
2) Create a new line
  + Product: Agua sin Gas 1L
  + Quantity: 1
3) Book the order
4) Create a new Sales Order for Alimentos y Supermercados, S.A with
5) Click on "Copy Lines" button
6) Select previously created sales order line (checkbox in the left side)
7) Edit the quantity to = 2
8) Click on OK button
9) You will get an error: * The value entered is not valid.


TagsNo tags attached.
Attached Filespng file icon CopyLines_comma_decimal.png [^] (94,601 bytes) 2012-07-18 12:13

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
(0050738)
hgbot (developer)
2012-07-18 18:10

Repository: erp/devel/pi
Changeset: d055a5224b9952feddd6146b4003c8631766b678
Author: Mikel Irurita <mikel.irurita <at> openbravo.com>
Date: Wed Jul 18 16:37:40 2012 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/d055a5224b9952feddd6146b4003c8631766b678 [^]

Fixes issue 21138: Copy Lines not working using comma as decimal separator

---
M src/org/openbravo/erpCommon/ad_actionButton/CopyFromOrder.html
---
(0050846)
dmiguelez (developer)
2012-07-25 18:56

Code Review + Test OK
(0050854)
hudsonbot (developer)
2012-07-26 02:42

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/5edf203d5780 [^]

Maturity status: Test

- Issue History
Date Modified Username Field Change
2012-07-18 12:11 mirurita New Issue
2012-07-18 12:11 mirurita Assigned To => mirurita
2012-07-18 12:11 mirurita Modules => Core
2012-07-18 12:12 mirurita Regression level => Production - Confirmed Stable
2012-07-18 12:12 mirurita Regression date => 2012-03-13
2012-07-18 12:12 mirurita Regression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/3ab535c40a07 [^]
2012-07-18 12:13 mirurita File Added: CopyLines_comma_decimal.png
2012-07-18 18:10 hgbot Checkin
2012-07-18 18:10 hgbot Note Added: 0050738
2012-07-18 18:10 hgbot Status new => resolved
2012-07-18 18:10 hgbot Resolution open => fixed
2012-07-18 18:10 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/d055a5224b9952feddd6146b4003c8631766b678 [^]
2012-07-25 18:56 dmiguelez Note Added: 0050846
2012-07-25 18:56 dmiguelez Status resolved => closed
2012-07-25 18:56 dmiguelez Fixed in Version => pi
2012-07-26 02:42 hudsonbot Checkin
2012-07-26 02:42 hudsonbot Note Added: 0050854


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker