Skip to content

Optimizamos código, además de hacerlo más mantenible a futuro#166

Open
tmas0 wants to merge 2 commits intoUNIMOODLE:main_oldfrom
tmas0:main
Open

Optimizamos código, además de hacerlo más mantenible a futuro#166
tmas0 wants to merge 2 commits intoUNIMOODLE:main_oldfrom
tmas0:main

Conversation

@tmas0
Copy link
Copy Markdown

@tmas0 tmas0 commented Apr 14, 2025

Buenas,

Aprovechando el ticket #151 que empezamos en la MootES25, y viendo que @fpano ha empezado a arreglar la parte de ese ticket, además de eso vimos una serie de cosillas que se podían mejorar. Os las listo y hacemos el PR para que se incorpore:

  1. Aplicamos SQL coding Style.
  2. Cuando el filtro solo atañe al curso en sí, pasamos la global $COURSE, y tiramos de ella para sacar el fullname del curso. Evitamos el foreach, y luego el get_course.
  3. Usamos la función fullname para sacar el fullname del usurio. Se había colado un caso que no se usaba.

Un saludo

@tmas0 tmas0 requested a review from fpano April 14, 2025 08:27
@juacas
Copy link
Copy Markdown
Contributor

juacas commented May 21, 2025

@fpano por favor revisa el PR y dime si lo ves inocuo y conveniente para incorporarlo.
Un saludo.

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.

3 participants