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

(175 errors, 303 warnings)

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

mod/simplecertificate/backup/moodle2/backup_simplecertificate_activity_task.class.php
(#1) No one-line description found in phpdocs for docblock of file backup_simplecertificate_activity_task.class.php
(#18) Incorrect @package tag for file backup_simplecertificate_activity_task.class.php. Expected mod_simplecertificate, found moodlecore.
(#59) The static declaration must come after the visibility declaration
mod/simplecertificate/backup/moodle2/backup_simplecertificate_settingslib.php
(#1) No one-line description found in phpdocs for docblock of file backup_simplecertificate_settingslib.php
(#18) Incorrect @package tag for file backup_simplecertificate_settingslib.php. Expected mod_simplecertificate, found moodlecore.
(#28) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
(#28) File must end with a newline character
mod/simplecertificate/backup/moodle2/backup_simplecertificate_stepslib.php
(#1) No one-line description found in phpdocs for docblock of file backup_simplecertificate_stepslib.php
(#18) Incorrect @package tag for file backup_simplecertificate_stepslib.php. Expected mod_simplecertificate, found moodlecore.
(#35) Missing docblock for class backup_simplecertificate_activity_structure_step
(#37) Missing docblock for function define_structure
(#44) Short array syntax must be used to define arrays
(#44) Short array syntax must be used to define arrays
(#54) Short array syntax must be used to define arrays
(#55) Short array syntax must be used to define arrays
(#62) Short array syntax must be used to define arrays
(#66) Short array syntax must be used to define arrays
mod/simplecertificate/backup/moodle2/restore_simplecertificate_activity_task.class.php
(#19) Incorrect @package tag for file restore_simplecertificate_activity_task.class.php. Expected mod_simplecertificate, found moodlecore.
(#56) The static declaration must come after the visibility declaration
(#66) The static declaration must come after the visibility declaration
(#80) The static declaration must come after the visibility declaration
(#105) The static declaration must come after the visibility declaration
(#117) Missing docblock for function after_restore
mod/simplecertificate/backup/moodle2/restore_simplecertificate_stepslib.php
(#1) No one-line description found in phpdocs for docblock of file restore_simplecertificate_stepslib.php
(#18) Incorrect @package tag for file restore_simplecertificate_stepslib.php. Expected mod_simplecertificate, found moodlecore.
(#23) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
(#33) Missing docblock for function define_structure
(#35) Short array syntax must be used to define arrays
(#48) Missing docblock for function process_simplecertificate
(#98) Missing docblock for function process_simplecertificate_issue
(#116) Missing docblock for function after_execute
mod/simplecertificate/classes/certificates.php
(#27) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
(#150) There should be a comma after the last array item in a multi-line array.
(#213) There should be a comma after the last array item in a multi-line array.
(#247) There should be a comma after the last array item in a multi-line array.
mod/simplecertificate/classes/event/certificate_verified.php
(#19) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
(#24) Incorrect @package tag for class certificate_verified. Expected mod_simplecertificate, found mod.
(#30) Expected 1 space before opening brace; found 2
(#32) Missing docblock for function init
(#33) This comment is 48% valid code; is this commented out code?
(#38) Missing docblock for function get_name
(#42) Missing docblock for function get_description
(#46) There should be a comma after the last array item in a multi-line array.
(#50) Missing docblock for function get_url
(#62) File must end with a newline character
mod/simplecertificate/classes/event/course_module_viewed.php
(#19) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
(#24) Incorrect @package tag for class course_module_viewed. Expected mod_simplecertificate, found mod.
mod/simplecertificate/classes/external/get_issues.php
(#51) There must not be a space before the colon in a return type declaration
(#103) Line indented incorrectly; expected at least 20 spaces, found 16
(#104) Line indented incorrectly; expected at least 20 spaces, found 16
mod/simplecertificate/classes/observer.php
(#19) Missing @copyright tag
(#34) "require_once" must be immediately followed by an open parenthesis
mod/simplecertificate/classes/output/certificates.php
(#78) File must end with a newline character
mod/simplecertificate/classes/privacy/provider.php
(#118) Missing docblock for function export_user_preferences
(#126) Line exceeds 132 characters; contains 133 characters
(#130) Line exceeds 132 characters; contains 134 characters
(#184) Variable "in_sql" must not contain underscores.
(#184) Variable "in_params" must not contain underscores.
(#185) Variable "in_params" must not contain underscores.
(#186) Variable "in_sql" must not contain underscores.
mod/simplecertificate/db/access.php
(#1) Missing @copyright tag
(#1) Missing @license tag
(#1) Missing docblock for file access.php
(#49) Short array syntax must be used to define arrays
(#51) Short array syntax must be used to define arrays
(#55) Short array syntax must be used to define arrays
(#57) There should be a comma after the last array item in a multi-line array.
(#59) There should be a comma after the last array item in a multi-line array.
(#62) Short array syntax must be used to define arrays
(#66) Short array syntax must be used to define arrays
(#70) There should be a comma after the last array item in a multi-line array.
(#71) There should be a comma after the last array item in a multi-line array.
(#74) Short array syntax must be used to define arrays
(#78) Short array syntax must be used to define arrays
(#81) There should be a comma after the last array item in a multi-line array.
(#82) There should be a comma after the last array item in a multi-line array.
(#84) File must end with a newline character
mod/simplecertificate/db/events.php
(#17) Missing @copyright tag
(#20) Incorrect @package tag for file events.php. Expected mod_simplecertificate, found simplecertificate.
(#26) Short array syntax must be used to define arrays
(#27) Spaces must be used to indent lines; tabs are not allowed
(#27) Tab found within whitespace
(#27) Short array syntax must be used to define arrays
(#28) Spaces must be used to indent lines; tabs are not allowed
(#28) Tab found within whitespace
(#29) There should be a comma after the last array item in a multi-line array.
(#30) There should be a comma after the last array item in a multi-line array.
(#31) File must end with a newline character
mod/simplecertificate/db/log.php
(#20) Incorrect @package tag for file log.php. Expected mod_simplecertificate, found mod.
(#28) Short array syntax must be used to define arrays
(#29) Short array syntax must be used to define arrays
(#30) Short array syntax must be used to define arrays
(#31) Short array syntax must be used to define arrays
(#32) Short array syntax must be used to define arrays
(#33) Short array syntax must be used to define arrays
(#34) File must end with a newline character
mod/simplecertificate/db/upgrade.php
(#33) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
(#35) Missing @copyright tag
(#35) Missing @license tag
(#35) Missing docblock for function xmldb_simplecertificate_upgrade
(#113) Short array syntax must be used to define arrays
(#116) Short array syntax must be used to define arrays
(#119) Short array syntax must be used to define arrays
(#122) Short array syntax must be used to define arrays
(#125) Short array syntax must be used to define arrays
(#145) Short array syntax must be used to define arrays
(#212) Short array syntax must be used to define arrays
(#222) Short array syntax must be used to define arrays
(#223) Short array syntax must be used to define arrays
(#236) Short array syntax must be used to define arrays
(#296) This comment is 67% valid code; is this commented out code?
(#332) Short array syntax must be used to define arrays
(#336) Short array syntax must be used to define arrays
(#345) Short array syntax must be used to define arrays
(#376) Line exceeds 132 characters; contains 142 characters
(#396) This comment is 50% valid code; is this commented out code?
(#402) This comment is 42% valid code; is this commented out code?
(#413) Short array syntax must be used to define arrays
(#417) Short array syntax must be used to define arrays
(#419) Short array syntax must be used to define arrays
(#429) Line exceeds 132 characters; contains 146 characters
mod/simplecertificate/externallib.php
(#29) Missing docblock for class mod_simplecertificate_external
(#37) Short array syntax must be used to define arrays
(#38) There should be a comma after the last array item in a multi-line array.
(#44) Missing docblock for function verify_code
(#48) Short array syntax must be used to define arrays
(#56) Short array syntax must be used to define arrays
(#77) File must end with a newline character
mod/simplecertificate/index.php
(#20) Incorrect @package tag for file index.php. Expected mod_simplecertificate, found mod.
(#32) Short array syntax must be used to define arrays
(#53) Short array syntax must be used to define arrays
(#76) Short array syntax must be used to define arrays
(#78) Short array syntax must be used to define arrays
(#84) Short array syntax must be used to define arrays
(#88) Short array syntax must be used to define arrays
(#101) Short array syntax must be used to define arrays
(#108) Short array syntax must be used to define arrays
(#110) Short array syntax must be used to define arrays
(#117) File must end with a newline character
mod/simplecertificate/lang/en/simplecertificate.php
(#20) Incorrect @package tag for file simplecertificate.php. Expected mod_simplecertificate, found mod.
(#30) The string key "summaryofattempts" is not in the correct order, it should be before "viewcertificateviews"
(#31) The string key "issued" is not in the correct order, it should be before "summaryofattempts"
(#32) The string key "coursegrade" is not in the correct order, it should be before "issued"
(#34) The string key "awardedto" is not in the correct order, it should be before "getcertificate"
(#36) The string key "grade" is not in the correct order, it should be before "receiveddate"
(#37) The string key "code" is not in the correct order, it should be before "grade"
(#39) The string key "opendownload" is not in the correct order, it should be before "report"
(#42) The string key "hours" is not in the correct order, it should be before "openwindow"
(#45) The string key "deletissuedcertificates" is not in the correct order, it should be before "pluginadministration"
(#48) Unexpected comment found. Auto-fixing will not work after this comment
(#49) The string key "certificatename" is not in the correct order, it should be before "nocertificatesissued"
(#50) The string key "certificateimage" is not in the correct order, it should be before "certificatename"
(#56) The string key "coursename" is not in the correct order, it should be before "width"
(#59) The string key "printdate" is not in the correct order, it should be before "printoutcome"
(#61) Unexpected comment found. Auto-fixing will not work after this comment
(#63) The string key "enablesecondpage" is not in the correct order, it should be before "secondpageoptions"
(#74) Unexpected comment found. Auto-fixing will not work after this comment
(#75) The string key "printqrcode" is not in the correct order, it should be before "secondtextposition_help"
(#77) The string key "codex" is not in the correct order, it should be before "printqrcode_help"
(#81) The string key "defaultcodex" is not in the correct order, it should be before "qrcodeposition_help"
(#84) Unexpected comment found. Auto-fixing will not work after this comment
(#86) The string key "completiondate" is not in the correct order, it should be before "issueddate"
(#90) Unexpected comment found. Auto-fixing will not work after this comment
(#93) The string key "printgrade" is not in the correct order, it should be before "userdateformat"
(#94) The string key "gradefmt" is not in the correct order, it should be before "printgrade"
(#95) Unexpected comment found. Auto-fixing will not work after this comment
(#99) The string key "coursetimereq" is not in the correct order, it should be before "gradepoints"
(#101) The string key "emailothers" is not in the correct order, it should be before "emailteachers"
(#102) The string key "emailfrom" is not in the correct order, it should be before "emailothers"
(#103) The string key "delivery" is not in the correct order, it should be before "emailfrom"
(#104) Unexpected comment found. Auto-fixing will not work after this comment
(#106) The string key "download" is not in the correct order, it should be before "openbrowser"
(#109) The string key "emailoncompletion" is not in the correct order, it should be before "nodelivering"
(#110) Unexpected comment found. Auto-fixing will not work after this comment
(#111) The string key "certificatename_help" is not in the correct order, it should be before "emailoncompletion"
(#145) The string key "size" is not in the correct order, it should be before "textposition_help"
(#147) The string key "coursename_help" is not in the correct order, it should be before "size_help"
(#148) The string key "certificateimage_help" is not in the correct order, it should be before "coursename_help"
(#151) The string key "printdate_help" is not in the correct order, it should be before "printoutcome_help"
(#152) The string key "datefmt_help" is not in the correct order, it should be before "printdate_help"
(#154) The string key "gradefmt_help" is not in the correct order, it should be before "printgrade_help"
(#161) The string key "coursetimereq_help" is not in the correct order, it should be before "gradefmt_help"
(#163) The string key "emailothers_help" is not in the correct order, it should be before "emailteachers_help"
(#164) The string key "emailfrom_help" is not in the correct order, it should be before "emailothers_help"
(#165) The string key "delivery_help" is not in the correct order, it should be before "emailfrom_help"
(#173) Unexpected comment found. Auto-fixing will not work after this comment
(#175) The string key "designoptions" is not in the correct order, it should be before "issueoptions"
(#177) Unexpected comment found. Auto-fixing will not work after this comment
(#203) Unexpected comment found. Auto-fixing will not work after this comment
(#204) The string key "defaultwidth" is not in the correct order, it should be before "emailteachermailhtml"
(#205) The string key "defaultheight" is not in the correct order, it should be before "defaultwidth"
(#206) The string key "defaultcertificatetextx" is not in the correct order, it should be before "defaultheight"
(#209) Unexpected comment found. Auto-fixing will not work after this comment
(#212) The string key "cantdeleteissue" is not in the correct order, it should be before "invalidcode"
(#214) The string key "invalidfilterfield" is not in the correct order, it should be before "requiredtimenotmet"
(#216) Unexpected comment found. Auto-fixing will not work after this comment
(#217) The string key "certificateverification" is not in the correct order, it should be before "invalidfilterfield"
(#219) Unexpected comment found. Auto-fixing will not work after this comment
(#225) The string key "getcertificate" is duplicated
(#228) The string key "qrcodefirstpage" is not in the correct order, it should be before "verifycertificate"
(#231) The string key "customfilename" is not in the correct order, it should be before "qrcodefirstpage_help"
(#253) Unexpected comment found. Auto-fixing will not work after this comment
(#255) The string key "issuedview" is not in the correct order, it should be before "standardview"
(#256) The string key "bulkview" is not in the correct order, it should be before "issuedview"
(#260) Unexpected comment found. Auto-fixing will not work after this comment
(#262) The string key "multipdf" is not in the correct order, it should be before "onepdf"
(#265) The string key "completedusers" is not in the correct order, it should be before "showusers"
(#266) The string key "allusers" is not in the correct order, it should be before "completedusers"
(#273) The string key "defaultperpage" is not in the correct order, it should be before "issueddownload"
(#276) Unexpected comment found. Auto-fixing will not work after this comment
(#283) The string key "coursenotfound" is not in the correct order, it should be before "usernotfound"
(#285) The string key "awardedsubject" is not in the correct order, it should be before "issuedcertificatenotfound"
(#290) The string key "certificatecopy" is not in the correct order, it should be before "timestartdatefmt_help"
(#292) The string key "notreceived" is not in the correct order, it should be before "upgradeerror"
(#294) Unexpected comment found. Auto-fixing will not work after this comment
(#295) The string key "eventcertificate_verified" is not in the correct order, it should be before "notreceived"
(#297) The string key "deleteall" is not in the correct order, it should be before "eventcertificate_verified_description"
(#300) The string key "certificates" is not in the correct order, it should be before "deleteselected"
(#304) Unexpected comment found. Auto-fixing will not work after this comment
(#307) The string key "signame_help" is not in the correct order, it should be before "signname"
(#308) The string key "certificatepath" is not in the correct order, it should be before "signame_help"
(#316) The string key "signheight" is not in the correct order, it should be before "signwidth_help"
(#323) The string key "privacy:metadata:simplecertificate_issues" is not in the correct order, it should be before "signinfo_help"
(#327) The string key "privacy:metadata:simplecertificate_issues:code" is not in the correct order, it should be before "privacy:metadata:simplecertificate_issues:coursename"
mod/simplecertificate/lib.php
(#20) Incorrect @package tag for file lib.php. Expected mod_simplecertificate, found mod.
(#98) Short array syntax must be used to define arrays
(#102) Short array syntax must be used to define arrays
(#106) Short array syntax must be used to define arrays
(#115) Short array syntax must be used to define arrays
(#121) Short array syntax must be used to define arrays
(#122) Short array syntax must be used to define arrays
(#148) Short array syntax must be used to define arrays
(#166) Short array syntax must be used to define arrays
(#191) Short array syntax must be used to define arrays
(#267) Short array syntax must be used to define arrays
(#285) Short array syntax must be used to define arrays
(#309) Short array syntax must be used to define arrays
(#333) Short array syntax must be used to define arrays
(#359) Short array syntax must be used to define arrays
(#368) Short array syntax must be used to define arrays
(#377) Short array syntax must be used to define arrays
(#402) Short array syntax must be used to define arrays
(#405) There should be a comma after the last array item in a multi-line array.
(#418) Short array syntax must be used to define arrays
(#425) Short array syntax must be used to define arrays
(#426) Short array syntax must be used to define arrays
(#497) File must end with a newline character
mod/simplecertificate/locallib.php
(#20) Incorrect @package tag for file locallib.php. Expected mod_simplecertificate, found mod.
(#41) Missing docblock for class simplecertificate
(#89) Missing docblock for constant simplecertificate::CERT_ISSUE_DATE
(#90) Missing docblock for constant simplecertificate::COURSE_COMPLETATION_DATE
(#91) Missing docblock for constant simplecertificate::COURSE_START_DATE
(#94) Missing docblock for constant simplecertificate::NO_GRADE
(#95) Missing docblock for constant simplecertificate::COURSE_GRADE
(#98) Missing docblock for constant simplecertificate::DEFAULT_VIEW
(#99) Missing docblock for constant simplecertificate::ISSUED_CERTIFCADES_VIEW
(#100) Missing docblock for constant simplecertificate::BULK_ISSUE_CERTIFCADES_VIEW
(#103) Missing docblock for constant simplecertificate::SIMPLECERT_MAX_PER_PAGE
(#578) Inline comments must end in full-stops, exclamation marks, or question marks
(#582) There should be a comma after the last array item in a multi-line array.
(#619) There should be a comma after the last array item in a multi-line array.
(#836) Missing docblock for function get_formated_grade
(#882) Function utf8_decode() is deprecated since PHP 8.2; Use mb_convert_encoding(), UConverter::transcode() or iconv instead
(#1089) Line exceeds 132 characters; contains 133 characters
(#1317) Opening brace must be the last content on the line
(#1317) Inline comments must start with a capital letter, digit or 3-dots sequence
(#1317) Inline comments must end in full-stops, exclamation marks, or question marks
(#1616) Missing docblock for function remove_links
(#1644) Missing docblock for function remove_user_image
(#1733) Line indented incorrectly; expected at least 12 spaces, found 8
(#1740) Line indented incorrectly; expected at least 12 spaces, found 8
(#1857) Missing docblock for function create_temp_file
(#1864) Missing docblock for function get_user_profile_fields
(#1956) Expected 1 space after ".="; 2 found
(#1962) Expected 1 space after ".="; 2 found
(#2008) Missing docblock for function show_tabs
(#2032) Missing docblock for function view_default
(#2094) Missing docblock for function get_issued_certificate_users
(#2146) Missing docblock for function view_issued_certificates
(#2146) Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable parameter: $selectedusers.
(#2338) Inline comments must start with a capital letter, digit or 3-dots sequence
(#2338) This comment is 60% valid code; is this commented out code?
(#2338) No space found before comment text; expected "// $pos = 5;" but found "//$pos = 5;"
(#2440) Missing docblock for function view_bulk_certificates
(#2440) Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable parameter: $selectedusers.
mod/simplecertificate/mod_form.php
(#17) Old MOODLE_INTERNAL check detected. Replace it by "defined('MOODLE_INTERNAL') || die();"
(#26) Missing @copyright tag
(#26) Missing @license tag
(#26) Missing docblock for class mod_simplecertificate_mod_form
(#28) Missing docblock for function definition
(#36) Short array syntax must be used to define arrays
(#67) Short array syntax must be used to define arrays
(#74) Short array syntax must be used to define arrays
(#81) Short array syntax must be used to define arrays
(#88) Short array syntax must be used to define arrays
(#117) Short array syntax must be used to define arrays
(#125) Short array syntax must be used to define arrays
(#135) Short array syntax must be used to define arrays
(#173) Short array syntax must be used to define arrays
(#175) There should be a comma after the last array item in a multi-line array.
(#186) Short array syntax must be used to define arrays
(#192) Short array syntax must be used to define arrays
(#213) Short array syntax must be used to define arrays
(#219) Short array syntax must be used to define arrays
(#226) Short array syntax must be used to define arrays
(#230) Commas (,) must be followed by white space.
(#278) Short array syntax must be used to define arrays
(#293) Short array syntax must be used to define arrays
(#295) Short array syntax must be used to define arrays
(#298) Short array syntax must be used to define arrays
(#299) Short array syntax must be used to define arrays
(#307) Missing docblock for function add_completion_rules
(#310) Short array syntax must be used to define arrays
(#314) Short array syntax must be used to define arrays
(#316) Short array syntax must be used to define arrays
(#321) Short array syntax must be used to define arrays
(#324) Missing docblock for function completion_rule_enabled
(#328) Missing docblock for function data_postprocessing
(#370) Missing docblock for function check_has_files
(#385) Missing docblock for function get_filemanager_options_array
(#385) Expected 0 spaces after "get_filemanager_options_array"; 1 found
(#388) Short array syntax must be used to define arrays
(#389) Short array syntax must be used to define arrays
(#392) File must end with a newline character
mod/simplecertificate/report.php
(#20) Incorrect @package tag for file report.php. Expected mod_simplecertificate, found mod.
(#52) Short array syntax must be used to define arrays
mod/simplecertificate/settings.php
(#20) Incorrect @package tag for file settings.php. Expected mod_simplecertificate, found mod.
mod/simplecertificate/tests/base_test.php
(#21) Invalid @category tag value "phpunit".
(#38) Testcase mod_simplecertificate_base_testcase should be declared as abstract.
(#38) PHPUnit testcase name "mod_simplecertificate_base_testcase" does not match file name "base_test"
(#38) PHPUnit class "mod_simplecertificate_base_testcase" does not have any namespace. It is recommended to add it to the "mod_simplecertificate" namespace, using more levels if needed, in order to match the code being tested
(#40) Invalid docblock tag "@const".
(#42) Invalid docblock tag "@const".
(#44) Invalid docblock tag "@const".
(#46) Invalid docblock tag "@const".
(#48) Invalid docblock tag "@const".
(#50) Invalid docblock tag "@const".
(#52) Invalid docblock tag "@const".
(#54) Invalid docblock tag "@const".
(#87) The setUp() method in unit tests must always call to parent::setUp().
(#93) Short array syntax must be used to define arrays
(#98) Short array syntax must be used to define arrays
(#103) Short array syntax must be used to define arrays
(#108) Short array syntax must be used to define arrays
(#110) Short array syntax must be used to define arrays
(#113) Short array syntax must be used to define arrays
(#121) Short array syntax must be used to define arrays
(#129) Short array syntax must be used to define arrays
(#141) Missing docblock for function create_extra_users in testcase
(#143) Short array syntax must be used to define arrays
(#148) Short array syntax must be used to define arrays
(#153) Short array syntax must be used to define arrays
(#158) Short array syntax must be used to define arrays
(#163) Short array syntax must be used to define arrays
(#171) Short array syntax must be used to define arrays
(#179) Short array syntax must be used to define arrays
(#205) Short array syntax must be used to define arrays
(#216) Test method test_create_instance() is missing any coverage information, own or at class level
(#216) Test method test_create_instance() is missing a return type
(#227) Missing docblock for constant testable_simplecertificate::PLUGIN_VERSION
(#239) Short array syntax must be used to define arrays
(#243) There should be a comma after the last array item in a multi-line array.
(#291) Missing docblock for function check_user_can_access_certificate_instance in testcase
(#295) Missing docblock for function testable_check_user_can_access_certificate_instance in testcase
(#321) Missing docblock for function testable_create_pdf in testcase
(#349) Missing docblock for function testable_get_issue_file in testcase
(#349) Expected 0 spaces after "testable_get_issue_file"; 1 found
(#353) Missing docblock for function testable_get_certificate_text in testcase
(#414) Missing docblock for function testable_get_issued_certificate_users in testcase
(#414) Expected 0 spaces after "testable_get_issued_certificate_users"; 1 found
mod/simplecertificate/tests/behat/behat_mod_simplecertificate.php
(#34) Missing docblock for class behat_mod_simplecertificate
(#35) This comment is 60% valid code; is this commented out code?
(#75) Line exceeds maximum limit of 180 characters; contains 181 characters
(#81) Short array syntax must be used to define arrays
(#83) Short array syntax must be used to define arrays
(#94) Short array syntax must be used to define arrays
(#98) Short array syntax must be used to define arrays
(#101) Short array syntax must be used to define arrays
(#113) Short array syntax must be used to define arrays
(#118) Short array syntax must be used to define arrays
(#146) Missing docblock for function get_issue_certificate_by_simplecertificate_name
(#150) Short array syntax must be used to define arrays
(#179) Short array syntax must be used to define arrays
(#199) Short array syntax must be used to define arrays
(#207) Short array syntax must be used to define arrays
(#214) Line exceeds 132 characters; contains 134 characters
(#221) Short array syntax must be used to define arrays
(#236) No one-line description found in phpdocs for docblock of function i_check_on_list
(#247) No one-line description found in phpdocs for docblock of function certificate_status_should_be
mod/simplecertificate/tests/generator/lib.php
(#26) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
(#38) Missing docblock for function create_instance
(#38) Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable parameter: $options.
(#41) Short array syntax must be used to define arrays
(#43) Short array syntax must be used to define arrays
(#47) There should be a comma after the last array item in a multi-line array.
(#50) Short array syntax must be used to define arrays
(#65) There should be a comma after the last array item in a multi-line array.
mod/simplecertificate/tests/locallib_test.php
(#21) Invalid @category tag value "phpunit".
(#40) Inline comments must start with a capital letter, digit or 3-dots sequence
(#40) Inline comments must end in full-stops, exclamation marks, or question marks
(#41) Unit test locallib_test should be declared as final.
(#41) PHPUnit class "locallib_test" does not have any namespace. It is recommended to add it to the "mod_simplecertificate" namespace, using more levels if needed, in order to match the code being tested
(#41) Missing docblock for class locallib_test
(#43) Test method test_create_instance() is missing any coverage information, own or at class level
(#43) Test method test_create_instance() is missing a return type
(#47) Commas (,) must be followed by white space.
(#49) Commas (,) must be followed by white space.
(#54) Expected 1 space after "="; 0 found
(#56) Commas (,) must be followed by white space.
(#63) Test method test_update_instance() is missing any coverage information, own or at class level
(#63) Test method test_update_instance() is missing a return type
(#73) Commas (,) must be followed by white space.
(#78) Test method test_delete_instance() is missing any coverage information, own or at class level
(#78) Test method test_delete_instance() is missing a return type
(#84) Commas (,) must be followed by white space.
(#87) Test method test_certificate_images() is missing any coverage information, own or at class level
(#87) Test method test_certificate_images() is missing a return type
(#128) Test method test_certificate_texts() is missing any coverage information, own or at class level
(#128) Test method test_certificate_texts() is missing a return type
(#141) Test method test_create_issue_instance() is missing any coverage information, own or at class level
(#141) Test method test_create_issue_instance() is missing a return type
(#190) Test method test_create_issue_code() is missing any coverage information, own or at class level
(#190) Test method test_create_issue_code() is missing a return type
(#204) Test method test_update_instace_update_haschange_issues() is missing any coverage information, own or at class level
(#204) Test method test_update_instace_update_haschange_issues() is missing a return type
(#234) Test method test_create_pdf_file() is missing any coverage information, own or at class level
(#234) Test method test_create_pdf_file() is missing a return type
(#237) Test method test_detete_instace_update_timedelete_issues() is missing any coverage information, own or at class level
(#237) Test method test_detete_instace_update_timedelete_issues() is missing a return type
(#252) Commas (,) must be followed by white space.
(#259) Test method test_delivery_email() is missing any coverage information, own or at class level
(#259) Test method test_delivery_email() is missing a return type
(#262) Test method test_email_notifications() is missing any coverage information, own or at class level
(#262) Test method test_email_notifications() is missing a return type
(#270) Expected 1 space after "="; 0 found
(#292) Test method test_can_issue_user_without_grade_restrinction() is missing any coverage information, own or at class level
(#292) Test method test_can_issue_user_without_grade_restrinction() is missing a return type
mod/simplecertificate/verify.php
(#20) Incorrect @package tag for file verify.php. Expected mod_simplecertificate, found mod.
(#26) Expected login check (require_login, require_course_login, require_admin, admin_externalpage_setup) following config inclusion. None found.
(#33) Short array syntax must be used to define arrays
(#44) Short array syntax must be used to define arrays
(#52) Short array syntax must be used to define arrays
(#64) Short array syntax must be used to define arrays
(#65) Short array syntax must be used to define arrays
(#67) Short array syntax must be used to define arrays
(#86) Missing docblock for function get_issued_cert
(#89) Short array syntax must be used to define arrays
(#110) Short array syntax must be used to define arrays
mod/simplecertificate/verify_form.php
(#17) Old MOODLE_INTERNAL check detected. Replace it by "defined('MOODLE_INTERNAL') || die();"
(#24) Missing @copyright tag
(#24) Missing @license tag
(#24) Missing docblock for class verify_form
(#27) Missing docblock for function definition
(#27) Expected 0 spaces after "definition"; 1 found
(#31) Short array syntax must be used to define arrays
(#45) Missing docblock for function validation
(#61) Missing docblock for function is_recaptcha_enabled
(#66) File must end with a newline character
mod/simplecertificate/version.php
(#20) Incorrect @package tag for file version.php. Expected mod_simplecertificate, found mod.
mod/simplecertificate/view.php
(#1) Missing @copyright tag
(#1) Missing @license tag
(#1) Missing docblock for file view.php
mod/simplecertificate/wmsendfile.php
(#20) Incorrect @package tag for file wmsendfile.php. Expected mod_simplecertificate, found mod.
(#27) Expected login check (require_login, require_course_login, require_admin, admin_externalpage_setup) following config inclusion. None found.
(#30) Short array syntax must be used to define arrays
(#37) Missing docblock for function send_certificate_file
(#102) Missing docblock for function put_watermark
(#108) Inline comments must start with a capital letter, digit or 3-dots sequence
(#108) This comment is 70% valid code; is this commented out code?
(#109) Inline comments must end in full-stops, exclamation marks, or question marks
(#110) Functions must not contain multiple empty lines in a row; found 2 empty lines
(#127) Short array syntax must be used to define arrays
(#131) Short array syntax must be used to define arrays
(#153) Short array syntax must be used to define arrays
(#153) Short array syntax must be used to define arrays
(#156) There should be a comma after the last array item in a multi-line array.
(#168) Short array syntax must be used to define arrays
(#178) This comment is 60% valid code; is this commented out code?

Javascript coding style problems

(0 errors, 0 warnings)

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


CSS problems

(0 errors, 0 warnings)

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


PHPDocs style problems

(34 errors, 0 warnings)

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

mod/simplecertificate/lib.php
(#34) Phpdocs for function simplecertificate_add_instance has incomplete parameters list
(#52) Phpdocs for function simplecertificate_update_instance has incomplete parameters list
(#91) Phpdocs for function simplecertificate_reset_userdata has incomplete parameters list
(#133) Phpdocs for function simplecertificate_reset_course_form_definition has incomplete parameters list
(#277) Phpdocs for function simplecertificate_pluginfile has incomplete parameters list
(#371) Phpdocs for function simplecertificate_send_event has incomplete parameters list
(#398) Phpdocs for function simplecertificate_get_editor_options has incomplete parameters list
mod/simplecertificate/locallib.php
(#165) Phpdocs for function simplecertificate::add_instance has incomplete parameters list
(#249) Phpdocs for function simplecertificate::remove_issues has incomplete parameters list
(#479) Phpdocs for function simplecertificate::populate_simplecertificate_instance has incomplete parameters list
(#626) Phpdocs for function simplecertificate::get_issue has incomplete parameters list
(#713) Phpdocs for function simplecertificate::get_attempts has incomplete parameters list
(#1315) Phpdocs for function simplecertificate::send_certificade_email has incomplete parameters list
(#1385) Phpdocs for function simplecertificate::get_course_time has incomplete parameters list
(#1657) Phpdocs for function simplecertificate::get_user_image_url has incomplete parameters list
(#1712) Phpdocs for function simplecertificate::get_date has incomplete parameters list
(#1832) Phpdocs for function simplecertificate::get_outcome has incomplete parameters list
(#1899) Phpdocs for function simplecertificate::can_issue has incomplete parameters list
mod/simplecertificate/mod_form.php
(#355) Phpdocs for function mod_simplecertificate_mod_form::validation has incomplete parameters list
mod/simplecertificate/tests/base_test.php
(#202) Phpdocs for function mod_simplecertificate_base_testcase::create_instance has incomplete parameters list
(#229) Phpdocs for function testable_simplecertificate::update_instance has incomplete parameters list
(#328) Phpdocs for function testable_simplecertificate::testable_save_pdf has incomplete parameters list
(#340) Phpdocs for function testable_simplecertificate::testable_send_certificade_email has incomplete parameters list
(#360) Phpdocs for function testable_simplecertificate::testable_get_date has incomplete parameters list
(#379) Phpdocs for function testable_simplecertificate::testable_get_outcome has incomplete parameters list
(#392) Phpdocs for function testable_simplecertificate::testable_can_issue has incomplete parameters list
(#401) Phpdocs for function testable_simplecertificate::testable_issue_file_exists has incomplete parameters list
mod/simplecertificate/tests/behat/behat_mod_simplecertificate.php
(#44) Phpdocs for function behat_mod_simplecertificate::should_see_popup has incomplete parameters list
(#77) Phpdocs for function behat_mod_simplecertificate::add_grade_restriction has incomplete parameters list
(#193) Phpdocs for function behat_mod_simplecertificate::set_certificate_code has incomplete parameters list
(#215) Phpdocs for function behat_mod_simplecertificate::verify_certificate_code has incomplete parameters list
(#233) Phpdocs for function behat_mod_simplecertificate::i_check_on_list has incomplete parameters list
(#244) Phpdocs for function behat_mod_simplecertificate::certificate_status_should_be has incomplete parameters list
mod/simplecertificate/verify.php
(#99) Phpdocs for function get_course_name 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

(0 errors, 0 warnings)

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


shifter problems

(0 errors, 0 warnings)

This section shows problems detected by shifter [More info]


Mustache template problems

(0 errors, 1 warnings)

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

mod/simplecertificate/templates/certificates.mustache
(#1) HTML Validation error, line 7: Bad value “http://certificateurl...” for attribute “href” on element “a”: Invalid host: A non-final domain name label (or the whole domain name) is empty.. (ard-text"><a target="_blank" href="http://certificateurl..."><img s)

Gherkin .feature problems

(50 errors, 0 warnings)

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

mod/simplecertificate/tests/behat/issued_certificate_tab.feature
(#29) Wrong indentation for "and", expected indentation level of 4, but got 1
(#66) Trailing spaces are not allowed
(#67) Trailing spaces are not allowed
(#69) Trailing spaces are not allowed
(#71) New line at EOF(end of file) is required
(#71) Trailing spaces are not allowed
mod/simplecertificate/tests/behat/bulk_operations_tab.feature
(#5) Trailing spaces are not allowed
(#29) Trailing spaces are not allowed
(#45) Trailing spaces are not allowed
(#46) Trailing spaces are not allowed
(#48) Wrong indentation for "given", expected indentation level of 4, but got 3
(#72) New line at EOF(end of file) is required
mod/simplecertificate/tests/behat/add_certificate.feature
(#28) Trailing spaces are not allowed
(#77) Trailing spaces are not allowed
(#78) Multiple empty lines are not allowed
(#78) Trailing spaces are not allowed
(#79) New line at EOF(end of file) is required
(#79) Multiple empty lines are not allowed
(#79) Trailing spaces are not allowed
mod/simplecertificate/tests/behat/verify_certificate.feature
(#3) Trailing spaces are not allowed
(#5) Trailing spaces are not allowed
(#25) Wrong indentation for "and", expected indentation level of 4, but got 1
(#26) Wrong indentation for "and", expected indentation level of 4, but got 1
(#30) Trailing spaces are not allowed
(#35) New line at EOF(end of file) is required
(#35) Trailing spaces are not allowed
mod/simplecertificate/tests/behat/remove_issued_certificate.feature
(#7) Trailing spaces are not allowed
(#40) Wrong indentation for "given", expected indentation level of 4, but got 3
(#48) Multiple empty lines are not allowed
(#48) Trailing spaces are not allowed
(#49) Trailing spaces are not allowed
(#60) Trailing spaces are not allowed
(#61) Wrong indentation for "Scenario", expected indentation level of 2, but got 1
(#69) Trailing spaces are not allowed
(#70) Multiple empty lines are not allowed
(#70) Trailing spaces are not allowed
(#71) Multiple empty lines are not allowed
(#72) Multiple empty lines are not allowed
(#72) Trailing spaces are not allowed
(#73) Multiple empty lines are not allowed
(#73) Trailing spaces are not allowed
(#74) Multiple empty lines are not allowed
(#74) Trailing spaces are not allowed
(#75) Multiple empty lines are not allowed
(#75) Trailing spaces are not allowed
(#76) Multiple empty lines are not allowed
(#76) Trailing spaces are not allowed
(#77) New line at EOF(end of file) is required
(#77) Multiple empty lines are not allowed
(#77) Trailing spaces are not allowed