UNIMOODLE / moodle-mod_certifygen

GNU General Public License v3.0
1 stars 1 forks source link

Error cuando vas al /view.php de la actividad siendo profesor o administrador #80

Closed tmas0 closed 2 weeks ago

tmas0 commented 2 weeks ago

Buenas @xpazv @elena3ip,

En la última versión subida estamos viendo un error a la hora de mostrar la tabla activityteacher_table.

Por una parte, los headers definidos son:

$headers = [
            get_string('fullname'),
            get_string('status', 'mod_certifygen'),
            get_string('date'),
            '',
            '',
            '',
        ];

Y la query que ejecuta dicha table (https://github.com/UNIMOODLE/moodle-mod_certifygen/blob/22faaa25a8014db33874794d1f9e4624b187df48/classes/certifygen.php#L347), es la siguiente:

SELECT DISTINCT us.id as userid
                              , ci.id as issueid
                              , ci.code
                              , ci.emailed
                              , ci.timecreated as ctimecreated
                              , ci.templateid
                              , ci.expires
                              , ci.courseid
                              , ci.archived
                              , cv.lang as clang
                              , cv.status as cstatus
                              , cv.id as validationid
                              , us.*
                              , ci.courseid as courseid
                              , ci.archived
                              , cv.lang as clang
                              , cv.status as cstatus
                              , cv.id as validationid
                              , us.* FROM {user} us
             LEFT JOIN {certifygen_validations} cv
                        ON (cv.userid = us.id AND cv.lang = :lang AND cv.certifygenid = :certifygenid)
             LEFT JOIN {tool_certificate_issues} ci
                        ON (ci.userid = us.id AND cv.issueid = ci.id
                            AND ci.templateid = :templateid AND ci.component = :component AND ci.courseid = :courseid)
             WHERE us.id = :param20

No se asemeja una cosa con la otra. ¿Podéis revisar?

Un saludo

tmas0 commented 2 weeks ago

Buenas,

Lo he obviado, pero en dicha query que escapó un copy&paste doble, por favor, ya que estamos, también lo podéis arreglar.

Un saludo

elena3ip commented 2 weeks ago

Ya está modificado el select en la rama develop y main.

tmas0 commented 2 weeks ago

Comprobado.