Refactorizando la llamada a los webservices del PAC

LogicalBricks · Jan 10, 2014 · 5dc8e20 · 5dc8e20
1 parent 5064a64
commit 5dc8e20
Showing 1 changed file with 15 additions and 33 deletions.
diff --git a/lib/fm_timbrado_cfdi/fm_cliente.rb b/lib/fm_timbrado_cfdi/fm_cliente.rb
@@ -25,52 +25,34 @@ def initialize
     def timbrar(rfc, documento, opciones={})
       text_to_cfdi = Base64::encode64( documento )
       # Realizamos la peticion
-      configurar_cliente
-      response = @client.call(:request_timbrar_cfdi,
-                              message:
-                              { "param0" => {
-                                "UserPass" => user_pass,
-                                "UserID" => user_id,
-                                "emisorRFC" => rfc,
-                                "text2CFDI" => text_to_cfdi,
-                              }.merge(opciones)
-                              }
-                             )
-      FmRespuesta.new(response)
+      respuesta = webservice_call(:request_timbrar_cfdi, rfc, {"text2CFDI" => text_to_cfdi}.merge(opciones))
+      FmRespuesta.new(respuesta)
     end
 
     def subir_certificado(rfc, certificado, llave, password, opciones = {})
-      configurar_cliente
-      respuesta = @client.call(:activar_cancelacion,
-                   message:
-                   { "param0" => {
-                     "UserPass" => user_pass,
-                     "UserID" => user_id,
-                     "emisorRFC" => rfc,
-                     "archivoCer" => Base64::encode64(certificado),
+      parametros = { "archivoCer" => Base64::encode64(certificado),
                      "archivoKey" => Base64::encode64(llave),
-                     "clave" => password,
-                   }.merge(opciones)
-                   })
+                     "clave" => password }
+      respuesta = webservice_call(:activar_cancelacion, rfc, parametros.merge(opciones))
       FmRespuestaCancelacion.new(respuesta)
     end
 
     def cancelar(rfc, uuid, opciones = {})
-      configurar_cliente
-      respuesta = @client.call(:request_cancelar_cfdi,
-                   message:
-                   { "param0" => {
-                     "UserPass" => user_pass,
-                     "UserID" => user_id,
-                     "emisorRFC" => rfc,
-                     "uuid" => uuid,
-                   }.merge(opciones)
-                   })
+      respuesta = webservice_call(:request_cancelar_cfdi, rfc, {uuid: uuid}.merge(opciones))
       FmRespuestaCancelacion.new(respuesta)
     end
 
 
     private
+
+    def webservice_call(accion, rfc, opciones)
+      configurar_cliente
+      parametros= { "param0" => { "UserPass" => user_pass, "UserID" => user_id, "emisorRFC" => rfc }.merge(opciones) }
+      @client.call(accion,
+                   message: parametros
+                   )
+    end
+
     def configurar_cliente
       @client  = Savon.client(
         ssl_verify_mode: ssl_verify_mode,