From 5ab7df2fd859317d5f1dcd0219eea2841233c7dd Mon Sep 17 00:00:00 2001
From: Mauricio Baeza <git@elmau.net>
Date: Sat, 30 Dec 2017 17:15:08 -0600
Subject: [PATCH] Editar clientes y productos con doble click

---
 source/static/js/controller/main.js     | 18 +-----------------
 source/static/js/controller/partners.js | 24 +++++++++++++++++++++++-
 source/static/js/controller/products.js |  4 ++--
 3 files changed, 26 insertions(+), 20 deletions(-)

diff --git a/source/static/js/controller/main.js b/source/static/js/controller/main.js
index d718cd6..9d7fdf9 100644
--- a/source/static/js/controller/main.js
+++ b/source/static/js/controller/main.js
@@ -33,23 +33,6 @@ var controllers = {
         $$('menu_user').attachEvent('onMenuItemClick', menu_user_click);
         configuracion_inicial()
 
-        //~ Partner
-        $$('cmd_new_partner').attachEvent('onItemClick', cmd_new_partner_click);
-        $$('cmd_new_contact').attachEvent('onItemClick', cmd_new_contact_click);
-        $$('cmd_edit_partner').attachEvent('onItemClick', cmd_edit_partner_click);
-        $$('cmd_delete_partner').attachEvent('onItemClick', cmd_delete_partner_click);
-        $$('cmd_save_partner').attachEvent('onItemClick', cmd_save_partner_click);
-        $$('cmd_cancel_partner').attachEvent('onItemClick', cmd_cancel_partner_click);
-        $$('cmd_cancel_contact').attachEvent('onItemClick', cmd_cancel_contact_click);
-        $$('codigo_postal').attachEvent('onKeyPress', postal_code_key_press);
-        $$('codigo_postal').attachEvent('onTimedKeyPress', postal_code_key_up);
-        $$('colonia').attachEvent('onFocus', colonia_on_focus)
-        $$("tipo_persona").attachEvent( "onChange", opt_tipo_change)
-        $$("es_cliente").attachEvent( "onChange", is_client_change)
-        $$("es_proveedor").attachEvent( "onChange", is_supplier_change)
-        $$("rfc").attachEvent( "onBlur", rfc_lost_focus)
-        $$('multi').attachEvent('onViewChange', multi_change)
-
         //~ Invoices
         $$('cmd_new_invoice').attachEvent("onItemClick", cmd_new_invoice_click)
         $$('cmd_refacturar').attachEvent("onItemClick", cmd_refacturar_click)
@@ -85,6 +68,7 @@ var controllers = {
         $$('cmd_facturar_preinvoice').attachEvent('onItemClick', cmd_facturar_preinvoice_click)
         $$('grid_preinvoices').attachEvent('onItemClick', grid_preinvoices_click)
 
+        partners_controllers.init()
         products_controllers.init()
         bancos_controllers.init()
         invoices_controllers.init()
diff --git a/source/static/js/controller/partners.js b/source/static/js/controller/partners.js
index 74185e4..7848c7e 100644
--- a/source/static/js/controller/partners.js
+++ b/source/static/js/controller/partners.js
@@ -1,5 +1,27 @@
 
 
+var partners_controllers = {
+    init: function(){
+        $$('cmd_new_partner').attachEvent('onItemClick', cmd_new_partner_click);
+        $$('cmd_new_contact').attachEvent('onItemClick', cmd_new_contact_click);
+        $$('cmd_edit_partner').attachEvent('onItemClick', cmd_edit_partner_click);
+        $$('cmd_delete_partner').attachEvent('onItemClick', cmd_delete_partner_click);
+        $$('cmd_save_partner').attachEvent('onItemClick', cmd_save_partner_click);
+        $$('cmd_cancel_partner').attachEvent('onItemClick', cmd_cancel_partner_click);
+        $$('cmd_cancel_contact').attachEvent('onItemClick', cmd_cancel_contact_click);
+        $$('codigo_postal').attachEvent('onKeyPress', postal_code_key_press);
+        $$('codigo_postal').attachEvent('onTimedKeyPress', postal_code_key_up);
+        $$('colonia').attachEvent('onFocus', colonia_on_focus)
+        $$("tipo_persona").attachEvent( "onChange", opt_tipo_change)
+        $$("es_cliente").attachEvent( "onChange", is_client_change)
+        $$("es_proveedor").attachEvent( "onChange", is_supplier_change)
+        $$("rfc").attachEvent( "onBlur", rfc_lost_focus)
+        $$('multi').attachEvent('onViewChange', multi_change)
+        $$('grid_partners').attachEvent('onItemDblClick', cmd_edit_partner_click)
+    }
+}
+
+
 function get_condicion_pago(){
     webix.ajax().get('/values/condicionespago', {
         error: function(text, data, xhr) {
@@ -34,7 +56,7 @@ function cmd_new_contact_click(id, e, node){
 }
 
 
-function cmd_edit_partner_click(id, e, node){
+function cmd_edit_partner_click(){
     var msg = ''
     var row = $$('grid_partners').getSelectedItem()
 
diff --git a/source/static/js/controller/products.js b/source/static/js/controller/products.js
index 62a30e5..5d8c887 100644
--- a/source/static/js/controller/products.js
+++ b/source/static/js/controller/products.js
@@ -9,9 +9,9 @@ var products_controllers = {
         $$("cmd_cancel_product").attachEvent("onItemClick", cmd_cancel_product_click)
         $$("chk_automatica").attachEvent("onChange", chk_automatica_change)
         $$("valor_unitario").attachEvent("onChange", valor_unitario_change)
-        //~ $$("clave_sat").attachEvent('onSearchIconClick', clave_sat_icon_click)
         $$('precio_con_impuestos').attachEvent('onChange', precio_con_impuestos_change)
         $$('precio_con_impuestos').attachEvent('onTimedKeyPress', precio_con_impuestos_key_up);
+        $$('grid_products').attachEvent('onItemDblClick', cmd_edit_product_click)
     }
 }
 
@@ -57,7 +57,7 @@ function cmd_new_product_click(id, e, node){
 }
 
 
-function cmd_edit_product_click(id, e, node){
+function cmd_edit_product_click(){
     configurar_productos()
     var grid = $$('grid_products')
     var row = grid.getSelectedItem()