0039777: Combo popup successCallbackProduct should be extensible

diff --git a/web/org.openbravo.retail.discounts.combo/source/combo-popup.js b/web/org.openbravo.retail.discounts.combo/source/combo-popup.js
--- a/web/org.openbravo.retail.discounts.combo/source/combo-popup.js
+++ b/web/org.openbravo.retail.discounts.combo/source/combo-popup.js
@@ -654,6 +654,36 @@
     OB.UTIL.adaptFontSize(this.$.cont.id, this.$.cont.minFontSize, this.$.cont.maxFontSize, this.$.cont.maxHeight, this.$.cont.minLnCalc);
 
   },
+  successCallbackProduct: function (dataProducts) {
+    if (dataProducts.length === 1) {
+      var model = dataProducts.models[0],
+          name = model.get('_identifier'),
+          price = OB.I18N.formatCurrency(model.get('standardPrice')),
+          image;
+
+      //Issue 28397: If the user changes very fast between families, it may happen
+      //that the callback is executed after the button has been destroyed. In that
+      //case all components have been destroyed and therefore we should do nothing
+      if (this.destroyed) {
+        return;
+      }
+      this.$.identifier.setContent(name);
+      this.$.price.setContent(price);
+      if (OB.MobileApp.model.get('permissions')['OBPOS_retail.productImages']) {
+        image = OB.UTIL.getImageURL(model.get('id'));
+        this.$.thumbnail.setImgUrl(image);
+      } else {
+        image = model.get('img');
+        this.$.thumbnail.setImg(image);
+      }
+      OB.UTIL.adaptFontSize(this.$.cont.id, this.$.cont.minFontSize, this.$.cont.maxFontSize, this.$.cont.maxHeight, this.$.cont.minLnCalc);
+    } else {
+      this.doHidePopup({
+        popup: 'OBCOMBO_Popup'
+      });
+      OB.UTIL.showError(OB.I18N.getLabel('OBCOMBO_PRODUCT_NOTIN_ASSORTMENT'));
+    }
+  },
   initComponents: function () {
     this.inherited(arguments);
     var me = this,
@@ -663,36 +693,6 @@
       OB.UTIL.showError('OBDAL error: ' + error);
     }
 
-    function successCallbackProduct(dataProducts) {
-      if (dataProducts.length === 1) {
-        var model = dataProducts.models[0],
-            name = model.get('_identifier'),
-            price = OB.I18N.formatCurrency(model.get('standardPrice')),
-            image;
-
-        //Issue 28397: If the user changes very fast between families, it may happen
-        //that the callback is executed after the button has been destroyed. In that
-        //case all components have been destroyed and therefore we should do nothing
-        if (me.destroyed) {
-          return;
-        }
-        me.$.identifier.setContent(name);
-        me.$.price.setContent(price);
-        if (OB.MobileApp.model.get('permissions')['OBPOS_retail.productImages']) {
-          image = OB.UTIL.getImageURL(model.get('id'));
-          me.$.thumbnail.setImgUrl(image);
-        } else {
-          image = model.get('img');
-          me.$.thumbnail.setImg(image);
-        }
-        OB.UTIL.adaptFontSize(me.$.cont.id, me.$.cont.minFontSize, me.$.cont.maxFontSize, me.$.cont.maxHeight, me.$.cont.minLnCalc);
-      } else {
-        me.doHidePopup({
-          popup: 'OBCOMBO_Popup'
-        });
-        OB.UTIL.showError(OB.I18N.getLabel('OBCOMBO_PRODUCT_NOTIN_ASSORTMENT'));
-      }
-    }
     var criteria;
     if (!OB.MobileApp.model.hasPermission('OBPOS_remote.product', true)) {
       criteria = {
@@ -711,7 +711,9 @@
       criteria.remoteFilters = remoteCriteria;
     }
 
-    OB.Dal.find(OB.Model.Product, criteria, successCallbackProduct, errorCallback);
+    OB.Dal.find(OB.Model.Product, criteria, function (dataProducts) {
+      me.successCallbackProduct(dataProducts);
+    }, errorCallback);
 
     this.$.countAndRemove.updateCount(this.owner.owner.owner.owner.owner.owner.getCProductQty(this.model.get('id')), true);
   }