0031404: setItemValue() function should be update "_textualValue" too

Openbravo Issue Tracking System - Openbravo ERP

View Issue Details

Project

Category

View Status

Date Submitted

Last Update

0031404

Openbravo ERP

A. Platform

public

2015-11-09 10:38

2016-01-25 19:03

Reporter

inigosanchez

Assigned To

caristu

Priority

low

Severity

minor

Reproducibility

have not tried

Status

closed

Resolution

no change required

Platform

OS Version

Product Version

Target Version

Fixed in Version

Merge Request Status

Review Assigned To

caristu

OBNetwork customer

Web browser

Modules

Core

Support ticket

Regression level

Regression date

Regression introduced in release

Regression introduced by commit

Triggers an Emergency Pack

Summary

0031404: setItemValue() function should be update "_textualValue" too

Description

In order to simplify the way in which fields are recalculated using setItemValue() the function should update "_textualValue" value too.

Steps To Reproduce

Create a callout in client side (as attached file):
- Copy callout3 file at the end of ob-onchange-functions.js file.
- Login as "Openbravo Admin".
- Go to "Windows, Tabs and Fields" window and select "Price List" window.
- Select "Product Price" tab and go to "Field" subtab.
- Edit in form "Price Limit" field:
- Check "Displayed" and "Show in Grid View" checkbox for testing later.
- Also updated "On Change Function" field with following text: OB.OnChange.calcularpreciodia.
- Save the new configuration of "Price Limit" field.
- Logout and Compile.

- Login with "F&B Internation Admin Group".
- Go to "Price List" window and select "Tarifa de Ventas" record.
- Navigate to "Product price" subtab.
- Edit in grid any record and change "Unit Price" value to 2 and "List Price" value to 4.
- Change "Price Limit" value to 3 and change the focus with the mouse or TAB key. Note than "Unit Price" is changed to 4.
- Save the record and note than value in "Unit Price" is wrong because it is the old value.

Proposed Solution

If it is added this line at the end of callout3 function (attached file) the callout works properly:

form.setTextualValue('standardPrice', OB.Utilities.Number.OBPlainToOBMasked(lp.getValue(), lp.typeInstance.maskNumeric, lp.typeInstance.decSeparator, lp.typeInstance.groupSeparator, OB.Format.defaultGroupingSize), lp.typeInstance)

This logic should be added in setItemValue() function.

Additional Information

Tags

No tags attached.

Relationships

related to	defect	0031189		closed	inigosanchez	on change function is not working fine when the event is triggered by "TAB" key
related to	defect	0031901		closed	caristu	decimal value can't be changed by trigger/observer after manual edition in form view