Project

General

Profile

Bug #7533

Contador del número de cursos por sesión

Added by José Angel Ruiz (Nosolored) over 4 years ago. Updated over 4 years ago.

Status:
Bug resolved
Priority:
Normal
Assignee:
Category:
Sessions
Target version:
Start date:
17/02/2015
Due date:
% Done:

100%

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

Description

En el resumen de una sesión de formación (/main/admin/resume_session.php) si damos al botón de eliminar (aspa roja), se elimina el curso de la sesión pero no actualiza correctamente el contador que indica el número de cursos asignados a la sesión (incrementa uno en vez de decrementar).

En la siguiente url muestro la modificación para corregir este comportamiento:
https://github.com/nosolored/chamilo-lms/commit/e7ab0b84d526f29967889ac4d6a7cebd1f75ac9a

Associated revisions

Revision c0139a78
Added by Angel Fernando Quiroz Campos over 4 years ago

Merge pull request #518 from AngelFQC/7533

Course correction counter in the list of sessions - refs #7533

History

#1

Updated by Yannick Warnier over 4 years ago

  • Status changed from New to Assigned
  • Assignee set to Angel Quiroz
  • Target version set to 1.10.0

Hola José. Gracias. Verifiqué tu commit y estoy de acuerdo. Trata de siempre desarrollar estos parches en una rama específica de tu repo y mandarlos como PR. Ayuda mucho en el proceso.
Angel, cuando mandan un cambio así, lo más fácil (para mantener la autoría del parche) es agregar su repo como remote, hacer un fetch de su repo (en una rama tuya) y luego un cherry-pick del cambio (por el ID del commit). De ahí haces merge con 1.10.x y push (una vez integrado) y debería mandar todo bien.

#2

Updated by Angel Quiroz over 4 years ago

  • Status changed from Assigned to Bug resolved
  • % Done changed from 0 to 100

Acabo de hacer el cherry-pick y actualzar 1.10.x

Also available in: Atom PDF