details:   https://code.openbravo.com/erp/devel/pi/rev/d6181edcf7b3
changeset: 23040:d6181edcf7b3
user:      Augusto Mauch <augusto.mauch <at> openbravo.com>
date:      Thu May 15 12:11:43 2014 +0200
summary:   Fixes issue 26555: Bad initialization of CanvasItem in grid view.

The problem was the following:
- The shouldSaveValue of the CanvasItems have always been false, but until this 
Smartclient upgrade this was not enforced.
- After the upgrade of Smartclient the shouldSaveValue is actually being 
enforced, the value returned from the FIC for this column is not being saved in 
the form values.

To fix this, the shouldSaveValue of the CanvasItem class has been set to true. 
The only consequence of this change is that the value of the CanvasItems should 
now be stored in the form values, see 
http://www.smartclient.com/docs/9.1/a/b/c/go.html#attr..CanvasItem.shouldSaveValue

diffstat:

 
modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-canvas.js
 |  4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

diffs (14 lines):

diff -r cb4a3042e3a5 -r d6181edcf7b3 
modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-canvas.js
--- 
a/modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-canvas.js
      Thu May 15 11:53:24 2014 +0200
+++ 
b/modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-canvas.js
      Thu May 15 12:11:43 2014 +0200
@@ -24,7 +24,9 @@
 
 isc.OBClientClassCanvasItem.addProperties({
   autoDestroy: true,
-
+  // See issue https://issues.openbravo.com/view.php?id=26555
+  // 
http://www.smartclient.com/docs/9.1/a/b/c/go.html#attr..CanvasItem.shouldSaveValue
+  shouldSaveValue: true,
   // if the canvas is used somewhere else (in the statusbar) then
   // don't do placeCanvas.
   placeCanvas: function () {

------------------------------------------------------------------------------
"Accelerate Dev Cycles with Automated Cross-Browser Testing - For FREE
Instantly run your Selenium tests across 300+ browser/OS combos.
Get unparalleled scalability from the best Selenium testing platform available
Simple to use. Nothing to install. Get started now for free."
http://p.sf.net/sfu/SauceLabs
_______________________________________________
Openbravo-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openbravo-commits

Reply via email to