details:   https://code.openbravo.com/erp/devel/pi/rev/1f55c40cadb9
changeset: 21088:1f55c40cadb9
user:      Asier Lostalé <asier.lostale <at> openbravo.com>
date:      Thu Sep 05 18:35:18 2013 +0200
summary:   fixed bug 24702: can't edit in grid product with characteristics

diffstat:

 
modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-characteristics.js
 |  19 +++++++++-
 
modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/reference/CharacteristicsUIDefinition.java
                |   5 ++
 2 files changed, 23 insertions(+), 1 deletions(-)

diffs (50 lines):

diff -r 6522c822ca55 -r 1f55c40cadb9 
modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-characteristics.js
--- 
a/modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-characteristics.js
     Thu Sep 05 17:33:33 2013 +0200
+++ 
b/modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-characteristics.js
     Thu Sep 05 18:35:18 2013 +0200
@@ -354,7 +354,6 @@
 
 isc.ClassFactory.defineClass('OBCharacteristicsFilterItem', isc.OBTextItem);
 
-
 isc.OBCharacteristicsFilterItem.addClassProperties({
   getDisplayValue: function (displayValue) {
     var c, characteristic, v, value, hasAny = false,
@@ -508,4 +507,22 @@
     this.Super('destroy', arguments);
 
   }
+});
+
+// == OBCharacteristicsGridItem ==
+// If the Form Item used when editing in grid characteristics, it is an 
OBTextItem
+// but its value is not complete (js object with information about all 
characteristics)
+// but just database value.
+isc.ClassFactory.defineClass('OBCharacteristicsGridItem', isc.OBTextItem);
+
+isc.OBCharacteristicsGridItem.addProperties({
+  setValue: function (value) {
+    // forget about complex object value and use just what is in DB
+    if (!value || !value.characteristics || !value.dbValue) {
+      this.Super('setValue', arguments);
+      return;
+    }
+
+    this.setValue(value.dbValue);
+  }
 });
\ No newline at end of file
diff -r 6522c822ca55 -r 1f55c40cadb9 
modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/reference/CharacteristicsUIDefinition.java
--- 
a/modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/reference/CharacteristicsUIDefinition.java
    Thu Sep 05 17:33:33 2013 +0200
+++ 
b/modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/reference/CharacteristicsUIDefinition.java
    Thu Sep 05 18:35:18 2013 +0200
@@ -44,6 +44,11 @@
   }
 
   @Override
+  public String getGridEditorType() {
+    return "OBCharacteristicsGridItem";
+  }
+
+  @Override
   public String getFieldProperties(Field field, boolean getValueFromSession) {
     String result = super.getFieldProperties(field, getValueFromSession);
     OBContext.setAdminMode(true);

------------------------------------------------------------------------------
Learn the latest--Visual Studio 2012, SharePoint 2013, SQL 2012, more!
Discover the easy way to master current and previous Microsoft technologies
and advance your career. Get an incredible 1,500+ hours of step-by-step
tutorial videos with LearnDevNow. Subscribe today and save!
http://pubads.g.doubleclick.net/gampad/clk?id=58041391&iu=/4140/ostg.clktrk
_______________________________________________
Openbravo-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openbravo-commits

Reply via email to