Browse Source

refactor of getProducts

Tati 9 years ago
parent
commit
b0ba3d0a14
1 changed files with 19 additions and 9 deletions
  1. 19 9
      client/js/app.js

+ 19 - 9
client/js/app.js

@@ -35,15 +35,26 @@ var sections = [
     'productos'
     'productos'
 ];
 ];
 
 
+function fetchProducts(category) {
+    return new Promise(function(resolve, reject) {
+        var products = [];
+        if (R.isNil(data.allProducts[category]) || R.isEmpty(data.allProducts[category])) {
+            http(BASE_URL + 'products/category/' + category).then(function(xs) {
+                resolve(xs)
+            }, function(err) {
+                reject(err);
+            })
+        } else {
+            resolve(data.allProducts[category]);
+        }
+    })
+}
+
 function getProducts(category) {
 function getProducts(category) {
-    if (R.isNil(data.allProducts[category]) || R.isEmpty(data.allProducts[category])) {
-        http(BASE_URL + 'products/category/' + category).then(function(xs) {
-            data.allProducts[category] = xs;
-            data.products = xs;
-        }, logError)
-    } else {
-        data.products = data.allProducts[category];
-    }
+    fetchProducts(category).then(function(xs) {
+        data.allProducts[category] = xs;
+        data.products = xs;
+    })
 }
 }
 
 
 var selectedSection_ = {
 var selectedSection_ = {
@@ -115,7 +126,6 @@ function chooseSection(section) {
 var elements = [];
 var elements = [];
 
 
 function addSelectedClass(elemList, x, opt) {
 function addSelectedClass(elemList, x, opt) {
-    console.log('elemList: ', elemList);
     elemList.forEach(function(el) {
     elemList.forEach(function(el) {
         var isSelected = el.text === x;
         var isSelected = el.text === x;
         el.className = isSelected ? 'selected' : '';
         el.className = isSelected ? 'selected' : '';