Project

General

Profile

Actions

Bug #7768

closed

Review buy_course plugin style

Added by Yannick Warnier about 8 years ago. Updated about 8 years ago.

Status:
Bug resolved
Priority:
Urgent
Assignee:
Category:
Plugins
Target version:
Start date:
07/07/2015
Due date:
08/07/2015
% Done:

100%

Estimated time:
Spent time:
Complexity:
Normal
SCRUM pts - complexity:
?

Description

The style is broken in 1.10.x


Files

buy1.png (35.4 KB) buy1.png Yannick Warnier, 07/07/2015 08:11
buy2.png (29.5 KB) buy2.png Yannick Warnier, 07/07/2015 08:11
sesiones-dupli.png (25.8 KB) sesiones-dupli.png Yannick Warnier, 16/09/2015 02:27
buy-course-index.png (86.3 KB) buy-course-index.png Yannick Warnier, 17/09/2015 20:02
Actions #1

Updated by Alex Aragon about 8 years ago

  • Assignee changed from Alex Aragon to Angel Quiroz

Los cambios de tpl esta en la siguinte rama.

https://github.com/aragonc/chamilo-lms/tree/CT%23BUYCOURSE

La estructura estaba algo complicada, asi que cambie el orden de las etiquetas y actualizado a bootstrap3

Actions #2

Updated by Yannick Warnier about 8 years ago

Angel, esto queda pendiente de ti... (desde hace mucho tiempo ya)

Actions #3

Updated by Yannick Warnier about 8 years ago

Angel, como estamos con esta tarea?

Actions #4

Updated by Angel Quiroz about 8 years ago

Yannick Warnier wrote:

Angel, como estamos con esta tarea?

Se han hecho correciones de diseño. Y del registro de los cursos y sesiones. El proceso de compra me funcionó (al fin), pero tengo un error en process_success, quizá por el cambio de tablas que hice.

Actions #5

Updated by Angel Quiroz about 8 years ago

En el sandbox de Paypal me aparece que la transacción se realizó, pero en el código del plugin se verifica PAYMENTINFO_0_REASONCODE == Completed
A mi, PAYMENTINFO_0_REASONCODE, me devuelve None
Y no se puede registrar al usuario al curso/sesion

Actions #6

Updated by Angel Quiroz about 8 years ago

Ya he corregido el proceso de compra con PayPal.
Me falta corregir mediante transferencia
Y los ventas pendiente de pago.

Actions #7

Updated by Angel Quiroz about 8 years ago

  • % Done changed from 0 to 60
Actions #8

Updated by Angel Quiroz about 8 years ago

Procesos de compra con PayPal y transferencia
Y lista de ventas por estado

Falta filtro de cursos/sessiones por precio

Actions #9

Updated by Angel Quiroz about 8 years ago

Cambios hasta el momento https://github.com/chamilo/chamilo-lms/compare/chamilo:1.10.x...AngelFQC:7768

Borré sin darme cuenta la opción de generar una 'código de referencia' de la venta.
Estoy agregándola

Actions #10

Updated by Angel Quiroz about 8 years ago

Agregué la opción de 'referencia de venta'

Estoy corrigiendo las variables de idioma

Actions #11

Updated by Angel Quiroz about 8 years ago

Correcciones hechas!

Yannick podrías revisar las funcionalidades antes de enviar el PR, por favor?
https://github.com/chamilo/chamilo-lms/compare/chamilo:1.10.x...AngelFQC:7768 será un PR grande

Actions #12

Updated by Angel Quiroz about 8 years ago

  • Assignee changed from Angel Quiroz to Yannick Warnier
Actions #13

Updated by Yannick Warnier about 8 years ago

Las reviso este domingo.

Actions #14

Updated by Yannick Warnier about 8 years ago

Ante todo: es un buen trabajo.
Hay una serie de puntos adicionales que me gustaría tener implementados (no creo que todos reunidos tomen un día, pero aquí va):

  • Los botones en cada celda del menú dentro de la página de plugin están alineados a la izquierda. Deberían estar centrados.
  • Debería ser posible editar una cuenta bancaria y esconderla, pero no borrarla
  • La desinstalación del plugin debería incluir un guardado (dump?) de las tablas antes de borrarlas. Si se gestionan los pagos a través de ellas, es muy peligroso dejar que se borren por inatención.
  • en la parte de formulario de configuración de paypal, los datos que llenan los campos no desaparecen automáticamente cuando se les da clic, pero tienen un <label> por lo que este texto de llenado no es necesario...
  • cuando configuro y guardo un precio para una sesión, la línea de esta aparece en verde. Cuando configuro y guardo un precio para un curso, no se ve ningun cambio. El cambio a verde me ayuda entender que se guardó.
  • cuando selecciono (en la página de configuración/activación del plugin) que la venta tiene que ser disponible para los usuarios sin cuenta en el portal y que he seleccionado mostrar una pestaña en el menú, debería aparecer la pestaña para los usuarios anónimos.
  • en Inglés, la traduccion dice "Purchasse data" cuando debería decir "Purchase data" (una sola "s"). En francés es "Détails d'achat"
  • en los reportes, debe existir una posibilidad para buscar por nombre del comprador, por nombre de producto y por rango de fechas (como lo hace PayPal mismo), y una de las opciones del desplegable de tipo de transacciones, debe ofrecer una posibilidad para ver todas las transacciones pasadas. La tabla de reporte tb debería dar una columna "Tipo de pago" donde aparezca "PayPal" o "Transferencia". La columna "Order date" debe incluir la hora (no solo la fecha).
  • tengo 4 sesiones que pongo en venta. Cada una tiene un solo curso registrado. Cuando miro el catálogo de compras de sesiones, cada sesión aparece con el curso 4 veces (ver imagen). En la descripción de la sesión (pantalla siguiente en el proceso de compra) aparece 2 veces también.

He probado:
  • activar/desactivar el plugin: OK
  • acceder como anónimo (fail: no me aparece la pestaña)
  • configurar PayPal: OK
  • configurar una cuenta bancaria: OK
  • configurar los precios de cursos y sesiones con decimales: OK
  • comprar, como alumno, 2 sesiones y 1 curso (comprado por transferencia): OK
  • validar una compra por transferencia: OK
  • verificar que el usuario tiene acceso inmediato a los cursos y sesiones: OK
  • verificar que las ventas aparecen en la lista: OK
Actions #15

Updated by Yannick Warnier about 8 years ago

  • Assignee changed from Yannick Warnier to Angel Quiroz
  • % Done changed from 60 to 70
Actions #16

Updated by Angel Quiroz about 8 years ago

  • Los botones en cada celda del menú dentro de la página de plugin están alineados a la izquierda. Deberían estar centrados.

En cuál página?

  • en la parte de formulario de configuración de paypal, los datos que llenan los campos no desaparecen automáticamente cuando se les da clic, pero tienen un <label> por lo que este texto de llenado no es necesario...

Corregido

  • cuando configuro y guardo un precio para una sesión, la línea de esta aparece en verde. Cuando configuro y guardo un precio para un curso, no se ve ningun cambio. El cambio a verde me ayuda entender que se guardó.

A mi sí me resalta la fila en color verde. Quizá olvidaste hacer clic en el check, o los datos eran los mismo y no se actualizó.

  • cuando selecciono (en la página de configuración/activación del plugin) que la venta tiene que ser disponible para los usuarios sin cuenta en el portal y que he seleccionado mostrar una pestaña en el menú, debería aparecer la pestaña para los usuarios anónimos.

Creo que tendría que modificar Chamilo para que los plugins agregen tabs para anónimos. Sólo lo hacen para usuarios registrados.

  • en Inglés, la traduccion dice "Purchasse data" cuando debería decir "Purchase data" (una sola "s"). En francés es "Détails d'achat"

Corregido

  • tengo 4 sesiones que pongo en venta. Cada una tiene un solo curso registrado. Cuando miro el catálogo de compras de sesiones, cada sesión aparece con el curso 4 veces (ver imagen). En la descripción de la sesión (pantalla siguiente en el proceso de compra) aparece 2 veces también.

Acabo de darme cuenta, pero a mi me aparecen 8 veces el mismo curso. Estoy usando las entidades $session->getCourses() que retorna los recursos de session_rel_course, sigo sin saber por qué ocurre.

Actions #17

Updated by Yannick Warnier about 8 years ago

Angel Quiroz wrote:

  • Los botones en cada celda del menú dentro de la página de plugin están alineados a la izquierda. Deberían estar centrados.

En cuál página?

plugin/buycourses/index.php

  • en la parte de formulario de configuración de paypal, los datos que llenan los campos no desaparecen automáticamente cuando se les da clic, pero tienen un <label> por lo que este texto de llenado no es necesario...

Corregido

OK (no lo verifiqué - confio en ti)

  • cuando configuro y guardo un precio para una sesión, la línea de esta aparece en verde. Cuando configuro y guardo un precio para un curso, no se ve ningun cambio. El cambio a verde me ayuda entender que se guardó.

A mi sí me resalta la fila en color verde. Quizá olvidaste hacer clic en el check, o los datos eran los mismo y no se actualizó.

Parece que solo se pone verde cuando el checkbox (de visibilidad) está seleccionado. Debería hacerlo siempre.

  • cuando selecciono (en la página de configuración/activación del plugin) que la venta tiene que ser disponible para los usuarios sin cuenta en el portal y que he seleccionado mostrar una pestaña en el menú, debería aparecer la pestaña para los usuarios anónimos.

Creo que tendría que modificar Chamilo para que los plugins agregen tabs para anónimos. Sólo lo hacen para usuarios registrados.

Sí, no pasa nada para esta. Veo que igual con la dirección de la página de compras, puedes cargarla como anónimo y cuando quieres comprar te manda al formulario de registro. Está bien.

  • en Inglés, la traduccion dice "Purchasse data" cuando debería decir "Purchase data" (una sola "s"). En francés es "Détails d'achat"

Corregido

OK. Verificado.

  • tengo 4 sesiones que pongo en venta. Cada una tiene un solo curso registrado. Cuando miro el catálogo de compras de sesiones, cada sesión aparece con el curso 4 veces (ver imagen). En la descripción de la sesión (pantalla siguiente en el proceso de compra) aparece 2 veces también.

Acabo de darme cuenta, pero a mi me aparecen 8 veces el mismo curso. Estoy usando las entidades $session->getCourses() que retorna los recursos de session_rel_course, sigo sin saber por qué ocurre.

OK, habrás que seguir investigando. getCourses() no debería nunca mandar duplicados, en todo caso.

Actions #18

Updated by Yannick Warnier about 8 years ago

  • % Done changed from 70 to 100

Arreglé los dos que faltaban. Cierro esta tarea. Buen trabajo!

Actions #19

Updated by Yannick Warnier about 8 years ago

  • Status changed from Assigned to Bug resolved
Actions #20

Updated by Angel Quiroz about 8 years ago

Se mejoraron los reportes de ventas, agregando un filtro por usuario
Se agregó la opción de asignar beneficiarios con la venta del curso/sesión
Se corrigieron algunos problemas en la instalación y desinstalación del plugin

PR: https://github.com/chamilo/chamilo-lms/pull/851

Actions

Also available in: Atom PDF