From 2201c3c0b57ad977be63a55838cda34d412390c8 Mon Sep 17 00:00:00 2001
From: ajlopez <webmaster@ajlopez.com>
Date: Mon, 3 Mar 2014 17:20:38 -0300
Subject: [PATCH] define brands, companies, as separate function

---
 samples/preciosa/preciosa.js      | 22 +++++++++++++---------
 samples/preciosa/test/marcas.js   | 10 ++++++++++
 samples/preciosa/test/preciosa.js |  1 +
 3 files changed, 24 insertions(+), 9 deletions(-)

diff --git a/samples/preciosa/preciosa.js b/samples/preciosa/preciosa.js
index 31d08c4..5e5b431 100644
--- a/samples/preciosa/preciosa.js
+++ b/samples/preciosa/preciosa.js
@@ -20,7 +20,7 @@ function initialize() {
 }
 
 function loadRemoteMarcas(cb) {
-    var url = 'http://preciosdeargentina.com.ar/api/v1/marcas/?page=1&format=json&page_size=100';
+    var url = 'http://preciosdeargentina.com.ar/api/v1/marcas/?page=1&format=json&page_size=5000';
 
     http.get(url, function(res) {
         var body = '';
@@ -57,14 +57,6 @@ function loadMarcasFabricantes() {
         else if (row.model == 'precios.marca')
             addMarca(row);
     });
-    
-    fabricantes.forEach(function (item) {
-        defineFabricante(item);
-    });
-    
-    marcas.forEach(function (item) {
-        defineMarca(item);
-    });
 }
 
 function addFabricante(row) {
@@ -77,6 +69,16 @@ function addFabricante(row) {
     fabricantes[item.id] = item;
 }
 
+function defineMarcasFabricantes() {    
+    fabricantes.forEach(function (item) {
+        defineFabricante(item);
+    });
+    
+    marcas.forEach(function (item) {
+        defineMarca(item);
+    });
+}
+
 function defineFabricante(item) {
     var item2 = {
         id: item.id,
@@ -320,6 +322,8 @@ function getProvinciaPais(fullname) {
 module.exports = {
     initialize: initialize,
     loadMarcasFabricantes: loadMarcasFabricantes,
+    loadRemoteMarcas: loadRemoteMarcas,
+    defineMarcasFabricantes: defineMarcasFabricantes,
     loadCategorias: loadCategorias,
     loadProductos: loadProductos,
     loadCiudades: loadCiudades,
diff --git a/samples/preciosa/test/marcas.js b/samples/preciosa/test/marcas.js
index 89ed872..3f105dd 100644
--- a/samples/preciosa/test/marcas.js
+++ b/samples/preciosa/test/marcas.js
@@ -21,4 +21,14 @@ exports['Load brands, companies'] = function (test) {
     test.ok(result.id);
     test.ok(result.nombre);
 };
+
+exports['Load remote brands'] = function (test) {
+    test.async();
+    
+    preciosa.loadRemoteMarcas(function (err, data) {
+        test.equal(err, null);
+        test.done();
+    });
+};
+
 
diff --git a/samples/preciosa/test/preciosa.js b/samples/preciosa/test/preciosa.js
index 6ec2b82..0cdc1ab 100644
--- a/samples/preciosa/test/preciosa.js
+++ b/samples/preciosa/test/preciosa.js
@@ -7,6 +7,7 @@ exports['initialize'] = function (test) {
 
 exports['load brands and companies'] = function (test) {
     preciosa.loadMarcasFabricantes();
+    preciosa.defineMarcasFabricantes();
     
     var marca = preciosa.getMarca(1);
     test.ok(marca);