Closed (fixed)
Project:
PagosOnline for Ubercart
Version:
6.x-1.1
Component:
Code
Priority:
Critical
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
24 May 2012 at 18:06 UTC
Updated:
30 Oct 2013 at 13:58 UTC
Hola, la variable extra1 debe incluir la dirección del comprador por razones de Garantía Antifraude.
Comments
Comment #1
dvasquez commentedComment #2
juankvillegas commentedThe variable extra1 is already in use.
The address is already being sent to PagosOnline in its own variable.
Where did you read that extra1 must be used to send the address?
Comment #3
juankvillegas commentedComment #4
dvasquez commentedDisculpa la demora.
Instalamos un sitio para un cliente, y el equipo de PagosOnline nos dijo que debemos incluir ahí (en extra1) las variables de dirección.
Tienes la misma información que nosotros?
Comment #5
juankvillegas commentedEl problema es que en este momento la variable extra1 está siendo usada para guardar el ID de la orden. Y más tarde, cuando PagosOnline confirma la transacción de ahí se sabe qué orden se está pagando. Al hacer esa modificación va a haber problemas de consistencia.
PagosOnline ofrece 2 campos adicionales (extra1 y extra2) para que se envíe cualquier información que se pueda necesitar después. El problema es que ahora están tapando un error de diseño usando uno de esos campos (extra1) para la dirección, en lugar de arreglar el problema que tienen al no poder usar la dirección que ya se está enviando en el formulario con los nombres de variables que ellos mismos implementaron (no recuerdo si el campo de dirección se llama address o direccion o algo similar, el caso es que el módulo lo implementa correctamente).
¿Cuál es exactamente el problema por el que necesitan enviar la dirección en el campo extra 1?
Comment #6
dvasquez commentedHola Juank.
Bueno, ellos nos dijeron que en esa variable debía entregarse las direcciones de los compradores, por motivos de algún tipo de ley antifraude. Yo, inicialmente les etregué la variable de ID de orden, más las direcciones que nos da Ubercart de Entregas y Correspondencia, pero me dijeron que les dejara sólo las direcciones.
De todas formas, te agradezco cómo enfocaste la respuesta en #5, y voy a llamarlos para ver qué ocurriría si alguien hace un reclamo para confrontar la orden de envío o si la necesitan por otra variable (posiblemente extra2).
Saludos.
Pdata: Este es el mensaje que me envió:
Comment #7
juankvillegas commentedEl problema ni siquiera alcanza para el caso en el que alguien llame a hacer reclamos. Tengo que revisar el código pero estoy casi seguro que esa variable extra1 se usa al recibir la confirmación de PagosOnline para marcar la orden (que viene en la variable extra1) como pagada, rechazada, etc, etc, etc.
Es posible que al hacer ese cambio el sitio web nunca se entere que una orden fue pagada.
Comment #8
dvasquez commentedBueno, la respuesta que recibí de ellos es que para la orden de compra usan la variable refVenta.
Y nos dijeron que podíamos revisar en el Manual 3.1.
Bueno, Juank, gracias por tus respuestas, realmente agradezco tu ayuda.
Pdata: Revisé el módulo commerce_pagosonline de Grangrupon.com y tienen esto:
Saludos.
Comment #9
juankvillegas commentedYo hice la implementación del módulo usando ese mismo manual que está adjuntando ahí... y como puede ver ahí dice que el campo extra1 puede ser utilizado para enviar cualquier información.
Un saludo.
Comment #10
juankvillegas commentedYo no usé el campo refVenta porque una persona o empresa podría tener más de 1 sitio web vendiendo sobre la misma cuenta de PagosOnline, entonces podría haber más de una orden con el mismo número.
Comment #11
dvasquez commentedCómo se podría solucionar esto Juank, agregando al refVenta no sólo un ID numérico sino un Alfanumérico? Por ejemplo:
Donde $acronymSite es una especie de Acronimo, y para Avianca sería AV, para Lan sería LAN, para VivaColombia sería VIVAC, para cada aeroñínea una diferente. Así tendríamos LAN18, AV18, VIVAC18, etc.
Gracias por tu ayuda?
Comment #12
juankvillegas commented¿Solucionar qué exactamente? si la pregunta no tiene que ver con lo de la variable extra1 entonces mejor escríbame por http://drupal.org/user/568110/contact
Comment #13
dvasquez commentedHola Juank.
Tengo dos inquietudes con respecto a la variable Extra1:
Si lo hago así, ¿el módulo ya no funcionaría correctamente?
Si no voy a usar más la variable Extra1 para enviar refVenta, sino para la dirección que me exigen en PagosOnline, y tengo varias tiendas con la misma cuenta de PagosOnline me va a enviar órdenes de pagos repetidas. Pero, pregunto si podría solucionarse poniendo un prefijo para cada tienda.
¿Podria ser añadiendo un campo de Iniciales en los datos de la tienda por ejemplo?
Saludos.
Pdata: Vuelvo a preguntar: Si decido usar Extra1 para la dirección, funcionaría de forma errónea el módulo?
Comment #14
juankvillegas commentedLamentablemente no he tenido tiempo para revisar el módulo, pero yo diría que sí deja de funcionar correctamente al hacer ese cambio.
Comment #15
dvasquez commentedPuse el estado del issue como necesita ser trabajado, para que quede en cola.
Me gustaría que el módulo funcione según lo que nos solicita PagosOnline. Si alguien puede ayudarte a a trabajarlo me avisas. Yo lo único que podría hacer para ayudar a la comunidad con el módulo sería buscar recursos para aportar, y que así, todos nos beneficienmos. Me parece que fue un muy buen trabajo y yo por mi parte quiero seguir usándolo.
Saludos.
Comment #16
juankvillegas commentedYa pude ver el código y definitivamente hacer el cambio que usted sugiere en #4 dañaría el funcionamiento del módulo. Sin embargo, corregirlo es relativamente sencillo, al menos en la versión 6.
Voy a hacer el cambio ahí y a ponerlo en la versión dev para que lo pueda probar.
Yo aviso por acá una vez esté subida una versión para prueba.
Comment #17
dvasquez commentedMuchas gracias Juan. Es de mucho valor no sólo para mí sino para los que están usando hasta hoy el módulo. Saludos!
Comment #18
juankvillegas commentedPor favor instale la versión 6.x-1.x-dev, debería funcionar sin problemas.
Me avisa si le funciona para subir el cambio a una versión final.
Comment #19
dvasquez commentedEsperando verificación de PagosOnline.
Por nuestra parte, funcionando muy bien.
Gracias juankvillegas.
Comment #20
juankvillegas commented