Prechecker results: error


Overview

(0 errors, 0 warnings)

This section shows the general problems detected when processing the code [More info]


PHP lint problems

(0 errors, 0 warnings)

This section shows php lint problems in the code detected by php -l [More info]


PHP coding style problems

(107 errors, 131 warnings)

This section shows the coding style problems detected in the code by phpcs [More info]

mod/certifygen/classes/event/certificate_downloaded.php
(#1) No one-line description found in phpdocs for docblock of file certificate_downloaded.php
mod/certifygen/classes/event/certificate_issued.php
(#1) No one-line description found in phpdocs for docblock of file certificate_issued.php
mod/certifygen/classes/event/certificate_revoked.php
(#1) No one-line description found in phpdocs for docblock of file certificate_revoked.php
mod/certifygen/classes/external/get_courses_as_student_external.php
(#1) No one-line description found in phpdocs for docblock of file get_courses_as_student_external.php
(#153) Empty CATCH statement detected
(#154) Inline comments must start with a capital letter, digit or 3-dots sequence
(#154) Inline comments must end in full-stops, exclamation marks, or question marks
(#154) This comment is 60% valid code; is this commented out code?
mod/certifygen/classes/external/get_courses_as_teacher_external.php
(#1) No one-line description found in phpdocs for docblock of file get_courses_as_teacher_external.php
mod/certifygen/classes/external/get_draft_certificate_external.php
(#1) No one-line description found in phpdocs for docblock of file get_draft_certificate_external.php
mod/certifygen/classes/external/get_id_instance_certificate_external.php
(#1) No one-line description found in phpdocs for docblock of file get_id_instance_certificate_external.php
mod/certifygen/classes/external/get_json_certificate_external.php
(#1) No one-line description found in phpdocs for docblock of file get_json_certificate_external.php
(#191) Inline comments must start with a capital letter, digit or 3-dots sequence
(#191) This comment is 57% valid code; is this commented out code?
(#191) No space found before comment text; expected "// $result['json']['data'] = $json;" but found "//$result['json']['data'] = $json;"
(#192) No space found before comment text; expected "// $data = $json;" but found "//$data = $json;"
mod/certifygen/classes/external/get_json_teacher_certificate_external.php
(#1) No one-line description found in phpdocs for docblock of file get_json_teacher_certificate_external.php
mod/certifygen/classes/external/get_pdf_certificate_external.php
(#1) No one-line description found in phpdocs for docblock of file get_pdf_certificate_external.php
mod/certifygen/classes/external/getcoursesnames_external.php
(#1) No one-line description found in phpdocs for docblock of file getcoursesnames_external.php
mod/certifygen/classes/external/getmodellisttable_external.php
(#1) No one-line description found in phpdocs for docblock of file getmodellisttable_external.php
mod/certifygen/classes/external/getmycertificatedata_external.php
(#1) No one-line description found in phpdocs for docblock of file getmycertificatedata_external.php
mod/certifygen/classes/external/getteacherrequestviewdata_external.php
(#1) No one-line description found in phpdocs for docblock of file getteacherrequestviewdata_external.php
mod/certifygen/classes/filters/certifygenfilter.php
(#82) Line exceeds 132 characters; contains 177 characters
(#85) Line exceeds 132 characters; contains 157 characters
mod/certifygen/classes/forms/codeform.php
(#72) File must end with a newline character
mod/certifygen/classes/persistents/certifygen_validations.php
(#168) Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable parameter: $userid.
(#189) Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable parameter: $user.
(#306) Functions must not contain multiple empty lines in a row; found 2 empty lines
mod/certifygen/classes/tables/activityteacher_table.php
(#89) Visibility must be declared on method "__construct"
(#94) Inline doc block type-hinting for '$cm' does not match next code line '[...'
(#362) Functions must not contain multiple empty lines in a row; found 2 empty lines
mod/certifygen/classes/tables/errors_filterset.php
(#1) No one-line description found in phpdocs for docblock of file errors_filterset.php
mod/certifygen/classes/tables/showerrors_table.php
(#1) No one-line description found in phpdocs for docblock of file showerrors_table.php
(#57) Visibility must be declared on method "__construct"
mod/certifygen/classes/task/checkerror.php
(#1) No one-line description found in phpdocs for docblock of file checkerror.php
mod/certifygen/db/upgrade.php
(#70) Functions must not contain multiple empty lines in a row; found 2 empty lines
(#87) Functions must not contain multiple empty lines in a row; found 2 empty lines
mod/certifygen/lib.php
(#1) No one-line description found in phpdocs for docblock of file lib.php
mod/certifygen/report/basic/classes/useofthecoursealgorithm.php
(#309) Whitespace found at end of line within string
(#310) Whitespace found at end of line within string
(#329) The use of the AS keyword to alias tables is bad for cross-db
(#365) Whitespace found at end of line within string
mod/certifygen/report/basic/lang/ca/certifygenreport_basic.php
(#48) Whitespace found at end of line within string
(#49) Whitespace found at end of line within string
mod/certifygen/report/basic/lang/en/certifygenreport_basic.php
(#48) Whitespace found at end of line within string
(#49) Whitespace found at end of line within string
mod/certifygen/report/basic/lang/es/certifygenreport_basic.php
(#48) Whitespace found at end of line within string
(#49) Whitespace found at end of line within string
mod/certifygen/report/basic/lang/eu/certifygenreport_basic.php
(#48) Whitespace found at end of line within string
(#49) Whitespace found at end of line within string
mod/certifygen/report/basic/lang/gl/certifygenreport_basic.php
(#48) Whitespace found at end of line within string
(#49) Whitespace found at end of line within string
mod/certifygen/repository/onedrive/classes/certifygenrepository_onedrive.php
(#1) No one-line description found in phpdocs for docblock of file certifygenrepository_onedrive.php
mod/certifygen/repository/onedrive/classes/onedriveconnection.php
(#430) Inline comments must start with a capital letter, digit or 3-dots sequence
(#430) This comment is 67% valid code; is this commented out code?
(#430) No space found before comment text; expected "// $fields = "folder,id,lastModifiedDateTime,name,size,webUrl,thumbnails";" but found "//$fields = "folder,id,lastModifiedDateTime,name,size,webUrl,thumbnails";"
(#464) Inline comments must start with a capital letter, digit or 3-dots sequence
(#464) This comment is 64% valid code; is this commented out code?
(#464) No space found before comment text; expected "// 'thumbnail' => $OUTPUT->image_url(file_folder_icon(64))->out(false)," but found "//'thumbnail' => $OUTPUT->image_url(file_folder_icon(64))->out(false),"
(#465) No space found before comment text; expected "// 'thumbnail_height' => 64," but found "//'thumbnail_height' => 64,"
(#466) No space found before comment text; expected "// 'thumbnail_width' => 64," but found "//'thumbnail_width' => 64,"
(#467) There should be a comma after the last array item in a multi-line array.
(#475) There should be a comma after the last array item in a multi-line array.
(#483) Inline comments must start with a capital letter, digit or 3-dots sequence
(#483) This comment is 54% valid code; is this commented out code?
(#483) No space found before comment text; expected "// if (empty($remotefile->thumbnails)) {" but found "//if (empty($remotefile->thumbnails)) {"
(#484) Expected 1 space before comment text but found 4; use block comment if you need indentation
(#485) Expected 1 space before comment text but found 4; use block comment if you need indentation
(#486) Expected 1 space before comment text but found 4; use block comment if you need indentation
(#487) Expected 1 space before comment text but found 8; use block comment if you need indentation
(#488) Expected 1 space before comment text but found 8; use block comment if you need indentation
(#489) Expected 1 space before comment text but found 4; use block comment if you need indentation
(#490) Expected 1 space before comment text but found 8; use block comment if you need indentation
(#491) Expected 1 space before comment text but found 8; use block comment if you need indentation
(#492) Expected 1 space before comment text but found 4; use block comment if you need indentation
(#493) Inline comments must end in full-stops, exclamation marks, or question marks
(#493) No space found before comment text; expected "// }" but found "//}"
(#495) Inline comments must start with a capital letter, digit or 3-dots sequence
(#495) This comment is 58% valid code; is this commented out code?
(#495) No space found before comment text; expected "// if (!empty($remotefile->thumbnails)) {" but found "//if (!empty($remotefile->thumbnails)) {"
(#496) Expected 1 space before comment text but found 4; use block comment if you need indentation
(#497) Expected 1 space before comment text but found 4; use block comment if you need indentation
(#498) Expected 1 space before comment text but found 8; use block comment if you need indentation
(#499) Expected 1 space before comment text but found 8; use block comment if you need indentation
(#500) Expected 1 space before comment text but found 12; use block comment if you need indentation
(#501) Expected 1 space before comment text but found 12; use block comment if you need indentation
(#502) Expected 1 space before comment text but found 12; use block comment if you need indentation
(#503) Expected 1 space before comment text but found 8; use block comment if you need indentation
(#504) Expected 1 space before comment text but found 4; use block comment if you need indentation
(#505) Inline comments must end in full-stops, exclamation marks, or question marks
(#505) No space found before comment text; expected "// }" but found "//}"
(#512) Inline comments must start with a capital letter, digit or 3-dots sequence
(#512) This comment is 60% valid code; is this commented out code?
(#512) No space found before comment text; expected "// 'thumbnail' => $thumb," but found "//'thumbnail' => $thumb,"
(#513) No space found before comment text; expected "// 'thumbnail_height' => $thumbwidth," but found "//'thumbnail_height' => $thumbwidth,"
(#514) No space found before comment text; expected "// 'thumbnail_width' => $thumbheight," but found "//'thumbnail_width' => $thumbheight,"
(#519) This comment is 47% valid code; is this commented out code?
(#520) Inline comments must end in full-stops, exclamation marks, or question marks
(#520) No space found before comment text; expected "// $files = array_filter($files, [$this, 'filter']);" but found "//$files = array_filter($files, [$this, 'filter']);"
(#588) Short array syntax must be used to define arrays
(#592) There should be a comma after the last array item in a multi-line array.
mod/certifygen/showerrors.php
(#1) No one-line description found in phpdocs for docblock of file showerrors.php
mod/certifygen/tests/deletemodel_external_test.php
(#51) PHPUnit class "deletemodel_external_test" does not have any namespace. It is recommended to add it to the "mod_certifygen" namespace, using more levels if needed, in order to match the code being tested
(#66) Test method test_deletemodel() is missing any coverage information, own or at class level
(#116) Test method test_deletemodel2() is missing any coverage information, own or at class level
mod/certifygen/tests/deleteteacherrequest_external_test.php
(#53) PHPUnit class "deleteteacherrequest_external_test" does not have any namespace. It is recommended to add it to the "mod_certifygen" namespace, using more levels if needed, in order to match the code being tested
(#67) Test method test_deleteteacherrequest() is missing any coverage information, own or at class level
(#124) Test method test_deleteteacherrequest2() is missing any coverage information, own or at class level
mod/certifygen/tests/downloadcertificate_external_test.php
(#57) PHPUnit class "downloadcertificate_external_test" does not have any namespace. It is recommended to add it to the "mod_certifygen" namespace, using more levels if needed, in order to match the code being tested
(#74) Test method test_downloadcertificate() is missing any coverage information, own or at class level
(#158) Test method test_downloadcertificate2() is missing any coverage information, own or at class level
(#250) Test method test_downloadcertificate_3() is missing any coverage information, own or at class level
mod/certifygen/tests/downloadteachercertificate_external_test.php
(#57) PHPUnit class "downloadteachercertificate_external_test" does not have any namespace. It is recommended to add it to the "mod_certifygen" namespace, using more levels if needed, in order to match the code being tested
(#73) Test method test_downloadteachercertificate() is missing any coverage information, own or at class level
(#142) Test method test_downloadteachercertificate2() is missing any coverage information, own or at class level
(#217) Test method test_downloadteachercertificate_3() is missing any coverage information, own or at class level
mod/certifygen/tests/emitcertificate_external_test.php
(#54) PHPUnit class "emitcertificate_external_test" does not have any namespace. It is recommended to add it to the "mod_certifygen" namespace, using more levels if needed, in order to match the code being tested
(#70) Test method test_emitcertificate() is missing any coverage information, own or at class level
(#139) Test method test_emitcertificate_2() is missing any coverage information, own or at class level
(#209) Test method test_emitcertificate_3() is missing any coverage information, own or at class level
mod/certifygen/tests/emitteacherrequest_external_test.php
(#54) PHPUnit class "emitteacherrequest_external_test" does not have any namespace. It is recommended to add it to the "mod_certifygen" namespace, using more levels if needed, in order to match the code being tested
(#70) Test method test_emitteacherrequest() is missing any coverage information, own or at class level
(#124) Test method test_emitteacherrequest2() is missing any coverage information, own or at class level
(#184) Test method test_emitteacherrequest_3() is missing any coverage information, own or at class level
mod/certifygen/tests/get_courses_as_student_external_test.php
(#51) PHPUnit class "get_courses_as_student_external_test" does not have any namespace. It is recommended to add it to the "mod_certifygen" namespace, using more levels if needed, in order to match the code being tested
(#66) Test method test_get_courses_as_student_nopermission() is missing any coverage information, own or at class level
(#105) Test method test_get_courses_as_student_with_no_certifygen() is missing any coverage information, own or at class level
(#150) Test method test_get_courses_as_student() is missing any coverage information, own or at class level
(#230) Test method test_get_courses_as_student_by_userfield() is missing any coverage information, own or at class level
(#328) Test method test_get_courses_as_student_by_userfield_username() is missing any coverage information, own or at class level
(#436) Test method test_get_courses_as_student_by_lang() is missing any coverage information, own or at class level
mod/certifygen/tests/get_courses_as_teacher_external_test.php
(#52) PHPUnit class "get_courses_as_teacher_external_test" does not have any namespace. It is recommended to add it to the "mod_certifygen" namespace, using more levels if needed, in order to match the code being tested
(#69) Test method test_get_courses_as_teacher_nopermission() is missing any coverage information, own or at class level
(#126) Test method test_get_courses_as_teacher() is missing any coverage information, own or at class level
(#206) Test method test_get_courses_as_teacher_by_userfield_nomodel() is missing any coverage information, own or at class level
(#264) Test method test_get_courses_as_teacher_by_userfield() is missing any coverage information, own or at class level
(#345) Test method test_get_courses_as_teacher_by_userfield_idnumber() is missing any coverage information, own or at class level
(#436) Test method test_get_courses_as_teacher_by_lang() is missing any coverage information, own or at class level
(#511) Test method test_get_courses_as_teacher_not_checked() is missing any coverage information, own or at class level
mod/certifygen/tests/get_draft_certificate_external_test.php
(#58) PHPUnit class "get_draft_certificate_external_test" does not have any namespace. It is recommended to add it to the "mod_certifygen" namespace, using more levels if needed, in order to match the code being tested
(#74) Test method test_1() is missing any coverage information, own or at class level
(#140) Test method test_2() is missing any coverage information, own or at class level
(#208) Test method test_3() is missing any coverage information, own or at class level
(#292) Test method test_4() is missing any coverage information, own or at class level
(#362) Test method test_5() is missing any coverage information, own or at class level
(#433) Test method test_6() is missing any coverage information, own or at class level
mod/certifygen/tests/get_id_instance_certificate_external_test.php
(#51) PHPUnit class "get_id_instance_certificate_external_test" does not have any namespace. It is recommended to add it to the "mod_certifygen" namespace, using more levels if needed, in order to match the code being tested
(#66) Test method test_get_id_instance_certificate_nopermission() is missing any coverage information, own or at class level
(#106) Test method test_get_id_instance_certificate() is missing any coverage information, own or at class level
(#213) Test method test_get_id_instance_certificate_by_userfield() is missing any coverage information, own or at class level
(#341) Test method test_get_id_instance_certificate_by_lang() is missing any coverage information, own or at class level
(#474) Test method test_get_id_instance_certificate_validation_not_checked() is missing any coverage information, own or at class level
mod/certifygen/tests/get_json_teacher_certificate_external_test.php
(#52) PHPUnit class "get_json_teacher_certificate_external_test" does not have any namespace. It is recommended to add it to the "mod_certifygen" namespace, using more levels if needed, in order to match the code being tested
(#69) Test method test_1() is missing any coverage information, own or at class level
(#100) Test method test_2() is missing any coverage information, own or at class level
(#139) Test method test_3() is missing any coverage information, own or at class level
(#174) Test method test_4() is missing any coverage information, own or at class level
(#224) Test method test_5() is missing any coverage information, own or at class level
mod/certifygen/tests/get_pdf_certificate_external_test.php
(#57) PHPUnit class "get_pdf_certificate_external_test" does not have any namespace. It is recommended to add it to the "mod_certifygen" namespace, using more levels if needed, in order to match the code being tested
(#73) Test method test_1() is missing any coverage information, own or at class level
(#151) Test method test_2() is missing any coverage information, own or at class level
(#240) Test method test_3() is missing any coverage information, own or at class level
(#335) Test method test_4() is missing any coverage information, own or at class level
(#415) Test method test_5() is missing any coverage information, own or at class level
(#496) Test method test_6() is missing any coverage information, own or at class level
(#582) Test method test_7() is missing any coverage information, own or at class level
mod/certifygen/tests/get_pdf_teacher_certificate_external_test.php
(#56) PHPUnit class "get_pdf_teacher_certificate_external_test" does not have any namespace. It is recommended to add it to the "mod_certifygen" namespace, using more levels if needed, in order to match the code being tested
(#73) Test method test_1() is missing any coverage information, own or at class level
(#148) Test method test_2() is missing any coverage information, own or at class level
(#237) Test method test_3() is missing any coverage information, own or at class level
(#345) Test method test_4() is missing any coverage information, own or at class level
(#438) Test method test_5() is missing any coverage information, own or at class level
(#532) Test method test_6() is missing any coverage information, own or at class level
(#624) Test method test_7() is missing any coverage information, own or at class level
mod/certifygen/tests/getcoursesnames_external_test.php
(#50) PHPUnit class "getcoursesnames_external_test" does not have any namespace. It is recommended to add it to the "mod_certifygen" namespace, using more levels if needed, in order to match the code being tested
(#63) Test method test_getcoursesnames() is missing any coverage information, own or at class level
mod/certifygen/tests/getmodellisttable_external_test.php
(#52) PHPUnit class "getmodellisttable_external_test" does not have any namespace. It is recommended to add it to the "mod_certifygen" namespace, using more levels if needed, in order to match the code being tested
(#64) Test method test_getmodellisttable_nopermision() is missing any coverage information, own or at class level
(#86) Test method test_getmodellisttable() is missing any coverage information, own or at class level
mod/certifygen/tests/getmycertificatedata_external_test.php
(#52) PHPUnit class "getmycertificatedata_external_test" does not have any namespace. It is recommended to add it to the "mod_certifygen" namespace, using more levels if needed, in order to match the code being tested
(#68) Test method test_getmycertificatedata() is missing any coverage information, own or at class level
mod/certifygen/tests/getteacherrequestviewdata_external_test.php
(#1) No one-line description found in phpdocs for docblock of file getteacherrequestviewdata_external_test.php
(#51) PHPUnit class "getteacherrequestviewdata_external_test" does not have any namespace. It is recommended to add it to the "mod_certifygen" namespace, using more levels if needed, in order to match the code being tested
(#67) Test method test_getteacherrequestviewdata() is missing any coverage information, own or at class level
(#141) Test method test_getteacherrequestviewdata_with_data() is missing any coverage information, own or at class level
mod/certifygen/tests/revokecertificate_external_test.php
(#55) PHPUnit class "revokecertificate_external_test" does not have any namespace. It is recommended to add it to the "mod_certifygen" namespace, using more levels if needed, in order to match the code being tested
(#79) Test method test_revokecertificate() is missing any coverage information, own or at class level
(#159) Test method test_revokecertificate_2() is missing any coverage information, own or at class level
mod/certifygen/tests/searchcategory_external_test.php
(#70) Test method test_searchcategory_nopermission() is missing any coverage information, own or at class level
(#104) Test method test_searchcategory() is missing any coverage information, own or at class level
mod/certifygen/tests/searchcourse_external_test.php
(#51) PHPUnit class "searchcourse_external_test" does not have any namespace. It is recommended to add it to the "mod_certifygen" namespace, using more levels if needed, in order to match the code being tested
(#63) Test method test_searchcourse_nopermission() is missing any coverage information, own or at class level
(#100) Test method test_searchcourse() is missing any coverage information, own or at class level
mod/certifygen/tests/searchmycourses_external_test.php
(#53) PHPUnit class "searchmycourses_external_test" does not have any namespace. It is recommended to add it to the "mod_certifygen" namespace, using more levels if needed, in order to match the code being tested
(#70) Test method test_searchmycourses_notenrolled() is missing any coverage information, own or at class level
(#121) Test method test_searchmycourses_enrolled() is missing any coverage information, own or at class level
mod/certifygen/validation/cmd/classes/certifygenvalidation_cmd.php
(#1) No one-line description found in phpdocs for docblock of file certifygenvalidation_cmd.php
mod/certifygen/validation/csv/classes/certifygenvalidation_csv.php
(#166) Whitespace found at end of line within string
(#203) Whitespace found at end of line within string
(#228) Whitespace found at end of line within string
(#255) Whitespace found at end of line within string
(#508) Whitespace found at end of line within string
mod/certifygen/validation/csv/settings.php
(#56) Line indented incorrectly; expected at least 4 spaces, found 0
(#64) Line indented incorrectly; expected at least 4 spaces, found 0
(#72) Line indented incorrectly; expected at least 4 spaces, found 0
mod/certifygen/validation/electronic/lang/ca/certifygenvalidation_electronic.php
(#1) No one-line description found in phpdocs for docblock of file certifygenvalidation_electronic.php
mod/certifygen/validation/electronic/lang/en/certifygenvalidation_electronic.php
(#1) No one-line description found in phpdocs for docblock of file certifygenvalidation_electronic.php
mod/certifygen/validation/electronic/lang/es/certifygenvalidation_electronic.php
(#1) No one-line description found in phpdocs for docblock of file certifygenvalidation_electronic.php
mod/certifygen/validation/electronic/lang/eu/certifygenvalidation_electronic.php
(#1) No one-line description found in phpdocs for docblock of file certifygenvalidation_electronic.php
mod/certifygen/validation/electronic/lang/gl/certifygenvalidation_electronic.php
(#1) No one-line description found in phpdocs for docblock of file certifygenvalidation_electronic.php
mod/certifygen/validation/none/lang/ca/certifygenvalidation_none.php
(#1) No one-line description found in phpdocs for docblock of file certifygenvalidation_none.php
mod/certifygen/validation/none/lang/en/certifygenvalidation_none.php
(#1) No one-line description found in phpdocs for docblock of file certifygenvalidation_none.php
mod/certifygen/validation/none/lang/es/certifygenvalidation_none.php
(#1) No one-line description found in phpdocs for docblock of file certifygenvalidation_none.php
mod/certifygen/validation/none/lang/eu/certifygenvalidation_none.php
(#1) No one-line description found in phpdocs for docblock of file certifygenvalidation_none.php
mod/certifygen/validation/none/lang/gl/certifygenvalidation_none.php
(#1) No one-line description found in phpdocs for docblock of file certifygenvalidation_none.php
mod/certifygen/validation/webservice/classes/external/get_user_requests_external.php
(#113) Functions must not contain multiple empty lines in a row; found 2 empty lines
mod/certifygen/validation/webservice/lang/ca/certifygenvalidation_webservice.php
(#1) No one-line description found in phpdocs for docblock of file certifygenvalidation_webservice.php
mod/certifygen/validation/webservice/lang/en/certifygenvalidation_webservice.php
(#1) No one-line description found in phpdocs for docblock of file certifygenvalidation_webservice.php
mod/certifygen/validation/webservice/lang/es/certifygenvalidation_webservice.php
(#1) No one-line description found in phpdocs for docblock of file certifygenvalidation_webservice.php
mod/certifygen/validation/webservice/lang/eu/certifygenvalidation_webservice.php
(#1) No one-line description found in phpdocs for docblock of file certifygenvalidation_webservice.php
mod/certifygen/validation/webservice/lang/gl/certifygenvalidation_webservice.php
(#1) No one-line description found in phpdocs for docblock of file certifygenvalidation_webservice.php
mod/certifygen/validation/webservice/tests/change_status_external_test.php
(#51) PHPUnit class "change_status_external_test" does not have any namespace. It is recommended to add it to the "certifygenvalidation_webservice" namespace, using more levels if needed, in order to match the code being tested
(#67) Test method test_1() is missing any coverage information, own or at class level
(#157) Test method test_2() is missing any coverage information, own or at class level
(#239) Test method test_3() is missing any coverage information, own or at class level
(#283) Test method test_4() is missing any coverage information, own or at class level
mod/certifygen/validation/webservice/tests/get_user_requests_external_test.php
(#51) PHPUnit class "get_user_requests_external_test" does not have any namespace. It is recommended to add it to the "certifygenvalidation_webservice" namespace, using more levels if needed, in order to match the code being tested
(#67) Test method test_1() is missing any coverage information, own or at class level
(#139) Test method test_2() is missing any coverage information, own or at class level
(#214) Test method test_3() is missing any coverage information, own or at class level
(#312) Test method test_4() is missing any coverage information, own or at class level
(#413) Test method test_5() is missing any coverage information, own or at class level
(#533) Test method test_6() is missing any coverage information, own or at class level
(#619) Expected 1 space before "-"; 0 found
(#619) Expected 1 space after "-"; 0 found
mod/certifygen/version.php
(#37) Inline comments must start with a capital letter, digit or 3-dots sequence
(#37) This comment is 58% valid code; is this commented out code?
(#37) No space found before comment text; expected "// $plugin->version = 2024100402;" but found "//$plugin->version = 2024100402;"

Javascript coding style problems

(0 errors, 52 warnings)

This section shows the coding style problems detected in the code by eslint [More info]

mod/certifygen/amd/src/activitymanager.js
(#42) Unexpected space before function parentheses. (space-before-function-paren)
(#62) Avoid nesting promises. (promise/no-nesting)
(#80) Expected to return a value at the end of function. (consistent-return)
(#87) Avoid nesting promises. (promise/no-nesting)
(#91) Too many nested callbacks (6). Maximum allowed is 5. (max-nested-callbacks)
(#91) Each then() should return a value or throw (promise/always-return)
(#92) Too many nested callbacks (7). Maximum allowed is 5. (max-nested-callbacks)
(#275) Comments should not begin with a lowercase character. (capitalized-comments)
(#275) Expected space or tab after '//' in comment. (spaced-comment)
(#284) Each then() should return a value or throw (promise/always-return)
mod/certifygen/amd/src/modelcreate.js
(#14) Comments should not begin with a lowercase character. (capitalized-comments)
(#20) Extra space after key 'GET_LIST_TABLE'. (key-spacing)
(#73) Each then() should return a value or throw (promise/always-return)
(#82) A function with a name starting with an uppercase letter should only be used as a constructor. (@babel/new-cap)
mod/certifygen/amd/src/modelmanagement.js
(#66) Expected catch,fail() or return (promise/catch-or-return)
(#66) Each then() should return a value or throw (promise/always-return)
(#98) Unexpected space before function parentheses. (space-before-function-paren)
(#105) Unexpected space before function parentheses. (space-before-function-paren)
(#106) Each then() should return a value or throw (promise/always-return)
(#123) Avoid nesting promises. (promise/no-nesting)
(#138) Expected to return a value at the end of function. (consistent-return)
(#138) Unexpected space before function parentheses. (space-before-function-paren)
(#144) Avoid nesting promises. (promise/no-nesting)
(#148) Each then() should return a value or throw (promise/always-return)
(#150) Too many nested callbacks (6). Maximum allowed is 5. (max-nested-callbacks)
(#159) Unexpected space before function parentheses. (space-before-function-paren)
(#164) A function with a name starting with an uppercase letter should only be used as a constructor. (@babel/new-cap)
mod/certifygen/amd/src/requestmanagement.js
(#92) Unexpected space before function parentheses. (space-before-function-paren)
(#106) Avoid nesting promises. (promise/no-nesting)
(#124) Expected to return a value at the end of function. (consistent-return)
(#131) Avoid nesting promises. (promise/no-nesting)
(#135) Too many nested callbacks (6). Maximum allowed is 5. (max-nested-callbacks)
(#135) Each then() should return a value or throw (promise/always-return)
(#136) Too many nested callbacks (7). Maximum allowed is 5. (max-nested-callbacks)
(#153) Unexpected space before function parentheses. (space-before-function-paren)
(#167) Avoid nesting promises. (promise/no-nesting)
(#185) Expected to return a value at the end of function. (consistent-return)
(#192) Avoid nesting promises. (promise/no-nesting)
(#196) Too many nested callbacks (6). Maximum allowed is 5. (max-nested-callbacks)
(#196) Each then() should return a value or throw (promise/always-return)
(#197) Too many nested callbacks (7). Maximum allowed is 5. (max-nested-callbacks)
(#214) Unexpected space before function parentheses. (space-before-function-paren)
(#233) Each then() should return a value or throw (promise/always-return)
(#252) Avoid nesting promises. (promise/no-nesting)
(#264) There should be no space after '{'. (@babel/object-curly-spacing)
(#266) Expected to return a value at the end of function. (consistent-return)
(#273) Avoid nesting promises. (promise/no-nesting)
(#277) Too many nested callbacks (6). Maximum allowed is 5. (max-nested-callbacks)
(#277) Each then() should return a value or throw (promise/always-return)
(#279) Too many nested callbacks (7). Maximum allowed is 5. (max-nested-callbacks)
(#283) Too many nested callbacks (7). Maximum allowed is 5. (max-nested-callbacks)
(#332) Each then() should return a value or throw (promise/always-return)

CSS problems

(0 errors, 0 warnings)

This section shows CSS problems detected by stylelint [More info]


PHPDocs style problems

(55 errors, 0 warnings)

This section shows the phpdocs problems detected in the code by local_moodlecheck [More info]

mod/certifygen/classes/certifygen.php
(#333) Phpdocs for function certifygen::get_issues_for_course_by_lang has incomplete parameters list
(#699) Phpdocs for function certifygen::can_be_issued has incomplete parameters list
mod/certifygen/classes/certifygen_file.php
(#68) Phpdocs for function certifygen_file::__construct has incomplete parameters list
mod/certifygen/classes/event/certificate_downloaded.php
(#73) Phpdocs for function certificate_downloaded::create_from_validation has incomplete parameters list
mod/certifygen/classes/event/certificate_issued.php
(#72) Phpdocs for function certificate_issued::create_from_validation has incomplete parameters list
mod/certifygen/classes/external/get_id_instance_certificate_external.php
(#79) Phpdocs for function get_id_instance_certificate_external::get_id_instance_certificate has incomplete parameters list
mod/certifygen/classes/external/get_json_teacher_certificate_external.php
(#76) Phpdocs for function get_json_teacher_certificate_external::get_json_teacher_certificate has incomplete parameters list
mod/certifygen/classes/filters/certifygenfilter.php
(#57) Phpdocs for function certifygenfilter::__construct has incomplete parameters list
(#69) Phpdocs for function certifygenfilter::filter has incomplete parameters list
mod/certifygen/classes/forms/modelform.php
(#323) Phpdocs for function modelform::validation has incomplete parameters list
mod/certifygen/classes/forms/teacherrequestform.php
(#187) Phpdocs for function teacherrequestform::validation has incomplete parameters list
mod/certifygen/classes/output/views/activity_view.php
(#112) Phpdocs for function activity_view::export_for_template has incomplete parameters list
mod/certifygen/classes/persistents/certifygen_context.php
(#172) Phpdocs for function certifygen_context::has_course_context has incomplete parameters list
mod/certifygen/classes/plugininfo/certifygenreport.php
(#82) Phpdocs for function certifygenreport::load_settings has incomplete parameters list
mod/certifygen/classes/plugininfo/certifygenrepository.php
(#68) Phpdocs for function certifygenrepository::load_settings has incomplete parameters list
mod/certifygen/classes/plugininfo/certifygenvalidation.php
(#82) Phpdocs for function certifygenvalidation::load_settings has incomplete parameters list
mod/certifygen/classes/tables/activityteacher_table.php
(#159) Phpdocs for function activityteacher_table::col_revoke has incomplete parameters list
(#190) Phpdocs for function activityteacher_table::col_status has incomplete parameters list
(#213) Phpdocs for function activityteacher_table::col_dateissued has incomplete parameters list
(#225) Phpdocs for function activityteacher_table::col_download has incomplete parameters list
(#246) Phpdocs for function activityteacher_table::col_emit has incomplete parameters list
mod/certifygen/classes/tables/modellist_table.php
(#98) Phpdocs for function modellist_table::col_type has incomplete parameters list
(#108) Phpdocs for function modellist_table::col_modelname has incomplete parameters list
(#118) Phpdocs for function modellist_table::col_template has incomplete parameters list
(#131) Phpdocs for function modellist_table::col_lastupdate has incomplete parameters list
(#140) Phpdocs for function modellist_table::col_deletemodel has incomplete parameters list
(#152) Phpdocs for function modellist_table::col_editmodel has incomplete parameters list
(#163) Phpdocs for function modellist_table::col_associatecontexts has incomplete parameters list
mod/certifygen/classes/tables/showerrors_table.php
(#113) Phpdocs for function showerrors_table::col_user has incomplete parameters list
(#139) Phpdocs for function showerrors_table::col_download has incomplete parameters list
(#179) Phpdocs for function showerrors_table::col_type has incomplete parameters list
(#188) Phpdocs for function showerrors_table::col_validation has incomplete parameters list
(#197) Phpdocs for function showerrors_table::col_report has incomplete parameters list
(#209) Phpdocs for function showerrors_table::col_repository has incomplete parameters list
(#222) Phpdocs for function showerrors_table::col_name has incomplete parameters list
(#234) Phpdocs for function showerrors_table::col_status has incomplete parameters list
(#244) Phpdocs for function showerrors_table::col_message has incomplete parameters list
(#252) Phpdocs for function showerrors_table::col_model has incomplete parameters list
(#260) Phpdocs for function showerrors_table::col_validationid has incomplete parameters list
(#268) Phpdocs for function showerrors_table::col_timecreated has incomplete parameters list
mod/certifygen/classes/template.php
(#250) Phpdocs for function template::issue_certificate has incomplete parameters list
mod/certifygen/lib.php
(#97) Phpdocs for function certifygen_update_instance has incomplete parameters list
(#123) Phpdocs for function certifygen_delete_instance has incomplete parameters list
(#295) Phpdocs for function mod_certifygen_myprofile_navigation has incomplete parameters list
(#324) Phpdocs for function mod_certifygen_pluginfile has incomplete parameters list
mod/certifygen/report/basic/classes/certifygenpdf.php
(#53) Phpdocs for function certifygenpdf::set_footer_text has incomplete parameters list
mod/certifygen/report/basic/lib.php
(#34) Phpdocs for function certifygenreport_basic_pluginfile has incomplete parameters list
mod/certifygen/repository/csv/classes/certifygenrepository_csv.php
(#135) Phpdocs for function certifygenrepository_csv::get_file_content has incomplete parameters list
mod/certifygen/repository/localrepository/classes/certifygenrepository_localrepository.php
(#182) Phpdocs for function certifygenrepository_localrepository::get_file_content has incomplete parameters list
mod/certifygen/repository/onedrive/classes/onedriveconnection.php
(#116) Phpdocs for function onedriveconnection::upload_file has incomplete parameters list
(#205) Phpdocs for function onedriveconnection::set_file_sharing_anyone_with_link_can_read has incomplete parameters list
(#334) Phpdocs for function onedriveconnection::delete_file_by_path has incomplete parameters list
(#378) Phpdocs for function onedriveconnection::get_file has incomplete parameters list
mod/certifygen/validation/csv/classes/certifygenvalidation_csv.php
(#360) Phpdocs for function certifygenvalidation_csv::get_file_url_from_external_service has incomplete parameters list
(#450) Phpdocs for function certifygenvalidation_csv::create_file_from_content has incomplete parameters list

Update savepoints problems

(0 errors, 0 warnings)

This section shows problems detected with the handling of upgrade savepoints [More info]


Third party library modification problems

(0 errors, 0 warnings)

This section shows problems detected with the modification of third party libraries [More info]


grunt changes

(4 errors, 0 warnings)

This section shows files built by grunt and not commited [More info]

mod/certifygen/amd/build/certificatemanager.min.js
Uncommitted change detected.
mod/certifygen/amd/build/certificatemanager.min.js.map
Uncommitted change detected.
mod/certifygen/amd/build/studentcertificatemanager.min.js
Uncommitted change detected.
mod/certifygen/amd/build/studentcertificatemanager.min.js.map
Uncommitted change detected.

shifter problems

(0 errors, 0 warnings)

This section shows problems detected by shifter [More info]


Mustache template problems

(4 errors, 5 warnings)

This section shows problems detected in mustache templates [More info]

mod/certifygen/report/basic/templates/report.mustache
(#1) HTML Validation error, line 3: Bad value “” for attribute “src” on element “img”: Must be non-empty. ( <img src="" alt="Logo configurad" width="100" height="100"/> )
(#1) HTML Validation info, line 3: Trailing slash on void elements has no effect and interacts badly with unquoted attribute values. ( <img src="" alt="Logo configurad" width="100" height="100"/> )
(#1) HTML Validation error, line 10: Element “style” not allowed as child of element “body” in this context. (Suppressing further errors from this subtree.) ( </div><style> )
mod/certifygen/templates/activity.mustache
(#1) HTML Validation error, line 13: Element “style” not allowed as child of element “body” in this context. (Suppressing further errors from this subtree.) (</div><style> )
mod/certifygen/templates/associatemodelcontexts.mustache
(#1) Mustache syntax exception: Example context JSON is unparsable, fails with: Syntax error
mod/certifygen/templates/course_list_request.mustache
(#1) Mustache syntax exception: Example context JSON is unparsable, fails with: Syntax error
mod/certifygen/templates/form_category_selector_suggestion.mustache
(#1) Mustache syntax exception: Example context JSON is unparsable, fails with: Syntax error
mod/certifygen/templates/model_list.mustache
(#1) HTML Validation error, line 11: Element “style” not allowed as child of element “body” in this context. (Suppressing further errors from this subtree.) (></div><style> [)
(#1) HTML Validation error, line 18: CSS: Parse Error. ( } }</st)
mod/certifygen/templates/profile_my_certificates.mustache
(#1) Mustache syntax exception: Example context JSON is unparsable, fails with: Syntax error

Gherkin .feature problems

(0 errors, 0 warnings)

This section shows problems detected in behat .feature files [More info]