Skip to content

Conversation

@juan-cortizas-ponte
Copy link

En algunos casos desde el listado de anuncios, al acceder a uno de ellos y volver atrás con el icono de volver al listado, este se vuelve a cargar pero desactivando los iconos de acciones de cada anuncio

imagen

imagen

Concretando más se da en cursos donde tenemos algún grupo de usuarios creado, por lo que si vamos a la herramienta de anuncios ...

main/announcements/announcements.php?cidReq={course_code}&id_session={session_code}&gradebook=0&origin=&gidReq=0

Obtendremos un listado con todos los anuncios, con las acciones habilitadas en cada uno de los anuncios ...

Pero si entramos en el detalle de un de los anuncio que fue creado para un grupo en concreto y volvemos al listado pulsando el icono de acción de volver al listado, veremos como en el listado aparece con alguno de los anuncios con las opciones deshabilitadas.

La causa está en que en el enlace del icono de volver al listado que aparece en el detalle de un anuncio agrega el parámetro "gidReq" de código de grupo al que pertenece el propio anuncio, por lo que, al volver atrás, se evaluará ese parámetro y deshabilitará las acciones en los anuncios que no pertenecen al código de grupo del parámetro.

https://github.com/chamilo/chamilo-lms/blob/1.11.x/main/inc/lib/AnnouncementManager.php#L1915-L1916

Interpreto que no hace falta que el enlace de volver atrás al listado necesite ese parámetro, por lo que este pull request lo quita.

@ywarnier
Copy link
Member

Tiene que ser un poco más contextual que esto, porque si lo quitas para todos los casos, cuando entres al evento desde la interfaz del grupo, al regresar te lanza a la página de anuncios generales del curso (te saca del contexto del grupo).

Copy link
Member

@ywarnier ywarnier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See comment. Use $group_id instead of just removing $row['to_group_id']

@ywarnier ywarnier merged commit 92aa4a9 into chamilo:1.11.x Dec 30, 2022
@ywarnier
Copy link
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants