Hola, la variable extra1 debe incluir la dirección del comprador por razones de Garantía Antifraude.

Comments

dvasquez’s picture

Priority: Major » Critical
juankvillegas’s picture

The 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?

juankvillegas’s picture

Status: Active » Closed (works as designed)
dvasquez’s picture

Disculpa 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.

Línea 67
- 'extra1' => $order_id,
+ 'extra1' => $order->billing_street1 . ', ' . $order->billing_street2,

Tienes la misma información que nosotros?

juankvillegas’s picture

El 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?

dvasquez’s picture

Hola 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ó:

Gracias por su comunicación. Sin embargo, se hizo una revisión de la variable extra1 y si se está enviando la información de la dirección pero al parecer con un dato adicional el cual es el consecutivo de la variable refVenta.
Como pueden observar en la imagen, la dirección que yo ingresé era Clle 93 prueba, y aparece con un 18 adicional.

juankvillegas’s picture

El 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.

dvasquez’s picture

Bueno, 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:

 $form["extra1"]= array('#type' => 'hidden', '#value' =>$billing_address['thoroughfare']);
 $form["extra2"]= array('#type' => 'hidden', '#value' =>$billing_address['country']);

Saludos.

juankvillegas’s picture

Yo 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.

juankvillegas’s picture

Yo 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.

dvasquez’s picture

Cómo se podría solucionar esto Juank, agregando al refVenta no sólo un ID numérico sino un Alfanumérico? Por ejemplo:

'refVenta'          => $acronymSite . uc_pagosonline_get_reference($order_id),

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?

juankvillegas’s picture

¿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

dvasquez’s picture

Hola Juank.
Tengo dos inquietudes con respecto a la variable Extra1:

  1. Según lo que me dijo la asesora de PagosOnline, debemos enviar las direcciones por la variable Extra1.
    Si lo hago así, ¿el módulo ya no funcionaría correctamente?
  2. Me dices en tu respuesta en http://drupal.org/node/1599144#comment-6554822 lo siguiente:

    Yo 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.

    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?

juankvillegas’s picture

Lamentablemente no he tenido tiempo para revisar el módulo, pero yo diría que sí deja de funcionar correctamente al hacer ese cambio.

dvasquez’s picture

Status: Closed (works as designed) » Needs work

Puse 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.

juankvillegas’s picture

Ya 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.

dvasquez’s picture

Muchas gracias Juan. Es de mucho valor no sólo para mí sino para los que están usando hasta hoy el módulo. Saludos!

juankvillegas’s picture

Status: Needs work » Needs review

Por 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.

dvasquez’s picture

Esperando verificación de PagosOnline.
Por nuestra parte, funcionando muy bien.

Gracias juankvillegas.

juankvillegas’s picture

Status: Needs review » Closed (fixed)