Bug #7768
Review buy_course plugin style
Description
The style is broken in 1.10.x
Files
Associated revisions
Fix save data - refs #7768
Minor- Fix and format process tpl - refs #7768
Fix process confirm - refs #7768
Minor - Format code and use Bootstrap tabs - refs #7768
Fix courses list for buy - refs #7768
Fix process confirm template - refs #7768
Minor - Code format for tpl - refs #7768
Fix paymentsetup page - refs #7768
Fix Configuration page - refs #7768
Fix Index page - refs #7768
Fix List page - refs #7768
Modify database structure - refs #7768
Fix Payment Setup for new DB structure - refs #7768
Don't limit session list - refs #7768
Fix Configuration page with new DB structure - refs #7768
Fix course catalog with new DB structure - refs #7768
Fix DB script install - refs #7768
Fix plugin catalog with new DB structure - refs #7768
Fix Process page - refs #7768
Fix PayPal process - refs #7768
Fix required parameter missing for PayPal process - refs #7768
Fix Buy process via transfer - refs #7768
Fix Pending Orders pages - refs #7768
Rename file from Pending Orders page - refs #7768
Remove plugin file for inscription - refs #7768
Sort sale list - refs #7768
Fix get status for user on Course Catalog - refs #7768
Minor - Fix messsage buying course with PayPal - refs #7768
Remove unnecessaty JS/Ajax code - refs #7768
Remove unnecessary CSS code and images - refs #7768
Improve course list on sale - refs #7768
Fix url in breadcrumbs - refs #7768
Remove unncessary ajax file, show general coach, fix messages - refs #7768
Fix filters in course/session catalog - refs #7768
Validate enabled payments types - refs #7768
Restore generation of order reference - refs #7768
Remove unnecesary code - refs #7768
Fix lang variables - refs #7768
Update plugin changelog and plugin description - refs #7768
Improve showing error message - refs #7768
Minor - Update language files - refs #7768
Fill with empty data the initial PayPal account - refs #7768
Align buttons at the center of their boxes in buycourses plugin - refs #7768
Blank $sessionData['courses'] before treatment to avoid echo - refs #7768
Improve French translation of buycourses module - refs #7768
Fix and improve HTML code on Buy Courses index - refs #7768
Add filter by user on sales report - refs #7768
Minor - Format code and PHP Doc - refs #7768
Display payment method on sale report - refs #7768
Register datetime of sales - refs #7768
Allow register beneficiaries with the sales of items - refs #7768
Only show courses details on configuration of available courses - refs #7768
Minor - Update changelog and plugin description - refs #7768
Set all course teachers or all session course coaches as beneficiaries by default - refs #7768
Fix the removal process of tables in the database - refs #7768
Minor - Update language files - refs #7768
Show product type on configuration course - refs #7768
Fix install process to avoid re-install plugin - refs #7768
Minor - Update language files from buycourses plugin - refs #7768
History
Updated by Alex Aragon over 5 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
Updated by Yannick Warnier over 5 years ago
Angel, esto queda pendiente de ti... (desde hace mucho tiempo ya)
Updated by Angel Quiroz over 5 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.
Updated by Angel Quiroz over 5 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
Updated by Angel Quiroz over 5 years ago
Ya he corregido el proceso de compra con PayPal.
Me falta corregir mediante transferencia
Y los ventas pendiente de pago.
Updated by Angel Quiroz over 5 years ago
Procesos de compra con PayPal y transferencia
Y lista de ventas por estado
Falta filtro de cursos/sessiones por precio
Updated by Angel Quiroz over 5 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
Updated by Angel Quiroz over 5 years ago
Agregué la opción de 'referencia de venta'
Estoy corrigiendo las variables de idioma
Updated by Angel Quiroz over 5 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
Updated by Yannick Warnier over 5 years ago
- File sesiones-dupli.png sesiones-dupli.png added
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.
- 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
Updated by Yannick Warnier over 5 years ago
- Assignee changed from Yannick Warnier to Angel Quiroz
- % Done changed from 60 to 70
Updated by Angel Quiroz over 5 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.
Updated by Yannick Warnier over 5 years ago
- File buy-course-index.png buy-course-index.png added
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.
Updated by Yannick Warnier over 5 years ago
- % Done changed from 70 to 100
Arreglé los dos que faltaban. Cierro esta tarea. Buen trabajo!
Updated by Angel Quiroz over 5 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
Add class method for get user from course by status - refs #7768