Project

General

Profile

Bug #7768

Review buy_course plugin style

Added by Yannick Warnier over 4 years ago. Updated about 4 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

Associated revisions

Revision c9acd1f5 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Add class method for get user from course by status - refs #7768

Revision 4e8b2f02 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix save data - refs #7768

Revision 6d66b3a0 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Minor- Fix and format process tpl - refs #7768

Revision a2cf158a (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix process confirm - refs #7768

Revision 7bfddf2d (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Minor - Format code and use Bootstrap tabs - refs #7768

Revision 24b1d2fe (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix courses list for buy - refs #7768

Revision 91e2a9df (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix process confirm template - refs #7768

Revision e78528d1 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Minor - Code format for tpl - refs #7768

Revision 619379f5 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix paymentsetup page - refs #7768

Revision b2152d25 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix Configuration page - refs #7768

Revision 579e0a95 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix Index page - refs #7768

Revision 9a30669c (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix List page - refs #7768

Revision affd4988 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Modify database structure - refs #7768

Revision a3073bdd (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix Payment Setup for new DB structure - refs #7768

Revision d4c1b86a (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Don't limit session list - refs #7768

Revision 7a4bdcf3 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix Configuration page with new DB structure - refs #7768

Revision 8463e6db (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix course catalog with new DB structure - refs #7768

Revision 7cbf4aef (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix DB script install - refs #7768

Revision 43a311f8 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix plugin catalog with new DB structure - refs #7768

Revision c7044634 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix Process page - refs #7768

Revision e1f3bb96 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix PayPal process - refs #7768

Revision 9d925b1e (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix required parameter missing for PayPal process - refs #7768

Revision 821d2209 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix Buy process via transfer - refs #7768

Revision 3d1c8d23 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix Pending Orders pages - refs #7768

Revision 576ef7fb (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Rename file from Pending Orders page - refs #7768

Revision 16b37cf0 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Remove plugin file for inscription - refs #7768

Revision 2d8b2520 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Sort sale list - refs #7768

Revision ae6aebf7 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix get status for user on Course Catalog - refs #7768

Revision 4bc75254 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Minor - Fix messsage buying course with PayPal - refs #7768

Revision 890a3794 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Remove unnecessaty JS/Ajax code - refs #7768

Revision f76c9ebd (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Remove unnecessary CSS code and images - refs #7768

Revision 20560a12 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Improve course list on sale - refs #7768

Revision 0e60079a (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix url in breadcrumbs - refs #7768

Revision abada939 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Remove unncessary ajax file, show general coach, fix messages - refs #7768

Revision 251bcd9d (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix filters in course/session catalog - refs #7768

Revision 9003ff35 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Validate enabled payments types - refs #7768

Revision 289477d9 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Restore generation of order reference - refs #7768

Revision 1a0a797b (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Remove unnecesary code - refs #7768

Revision 65cf78cd (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix lang variables - refs #7768

Revision a9fce617 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Update plugin changelog and plugin description - refs #7768

Revision 974fc6c8 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Improve showing error message - refs #7768

Revision 18b23add (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Minor - Update language files - refs #7768

Revision 970814a1 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fill with empty data the initial PayPal account - refs #7768

Revision 37c815b0 (diff)
Added by Yannick Warnier about 4 years ago

Align buttons at the center of their boxes in buycourses plugin - refs #7768

Revision b1869332 (diff)
Added by Yannick Warnier about 4 years ago

Blank $sessionData['courses'] before treatment to avoid echo - refs #7768

Revision 5f30e5a6 (diff)
Added by Yannick Warnier about 4 years ago

Improve French translation of buycourses module - refs #7768

Revision 339b2903 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix and improve HTML code on Buy Courses index - refs #7768

Revision edf83bdc (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Add filter by user on sales report - refs #7768

Revision 1ebec60d (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Minor - Format code and PHP Doc - refs #7768

Revision 5b26584f (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Display payment method on sale report - refs #7768

Revision df5a2eb3 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Register datetime of sales - refs #7768

Revision 8f67cf68 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Allow register beneficiaries with the sales of items - refs #7768

Revision 30bc15bd (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Only show courses details on configuration of available courses - refs #7768

Revision e159a227 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Minor - Update changelog and plugin description - refs #7768

Revision d582b66f (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Set all course teachers or all session course coaches as beneficiaries by default - refs #7768

Revision 58341506 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix the removal process of tables in the database - refs #7768

Revision 4ceba142 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Minor - Update language files - refs #7768

Revision dc970620 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Show product type on configuration course - refs #7768

Revision e406e2ee (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Fix install process to avoid re-install plugin - refs #7768

Revision 0ed8c25f
Added by Angel Fernando Quiroz Campos about 4 years ago

Merge pull request #851 from AngelFQC/7768

Add filter to sale report, register sale datetime, register beneficiaries - refs #7768

Revision 7ec1e323 (diff)
Added by Angel Fernando Quiroz Campos about 4 years ago

Minor - Update language files from buycourses plugin - refs #7768

History

#1

Updated by Alex Aragon over 4 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

#2

Updated by Yannick Warnier over 4 years ago

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

#3

Updated by Yannick Warnier about 4 years ago

Angel, como estamos con esta tarea?

#4

Updated by Angel Quiroz about 4 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.

#5

Updated by Angel Quiroz about 4 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

#6

Updated by Angel Quiroz about 4 years ago

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

#7

Updated by Angel Quiroz about 4 years ago

  • % Done changed from 0 to 60
#8

Updated by Angel Quiroz about 4 years ago

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

Falta filtro de cursos/sessiones por precio

#9

Updated by Angel Quiroz about 4 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

#10

Updated by Angel Quiroz about 4 years ago

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

Estoy corrigiendo las variables de idioma

#11

Updated by Angel Quiroz about 4 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

#12

Updated by Angel Quiroz about 4 years ago

  • Assignee changed from Angel Quiroz to Yannick Warnier
#13

Updated by Yannick Warnier about 4 years ago

Las reviso este domingo.

#14

Updated by Yannick Warnier about 4 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
#15

Updated by Yannick Warnier about 4 years ago

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

Updated by Angel Quiroz about 4 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.

#17

Updated by Yannick Warnier about 4 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.

#18

Updated by Yannick Warnier about 4 years ago

  • % Done changed from 70 to 100

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

#19

Updated by Yannick Warnier about 4 years ago

  • Status changed from Assigned to Bug resolved
#20

Updated by Angel Quiroz about 4 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

Also available in: Atom PDF