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

(140 errors, 228 warnings)

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

mod/accredible/classes/local/formhelper.php: (#89) There should be a comma after the last array item in a multi-line array.; (#116) There should be a comma after the last array item in a multi-line array.; (#118) Short array syntax must be used to define arrays; (#143) There should be a comma after the last array item in a multi-line array.; (#145) Short array syntax must be used to define arrays
mod/accredible/lang/en/accredible.php: (#28) The string key "accrediblecustomattributename" is not in the correct order, it should be before "activityname"; (#32) The string key "additionalactivitiesthree" is not in the correct order, it should be before "additionalactivitiestwo"; (#34) The string key "apikeylabel" is not in the correct order, it should be before "autoissueheader"; (#35) The string key "apikeyhelp" is not in the correct order, it should be before "apikeylabel"; (#37) The string key "attributemappingcoursecustomfields" is not in the correct order, it should be before "attributemappingcoursefields"; (#40) The string key "euhelp" is not in the correct order, it should be before "eulabel"; (#41) The string key "accredible:addinstance" is not in the correct order, it should be before "euhelp"; (#49) The string key "completionissuecheckbox" is not in the correct order, it should be before "completionissueheader"; (#52) The string key "includegradeattributecheckbox" is not in the correct order, it should be before "includegradeattributedescription"; (#53) The string key "gradeattributegradeitemselect" is not in the correct order, it should be before "includegradeattributecheckbox"; (#55) The string key "dashboardlink" is not in the correct order, it should be before "gradeattributekeynameselect"; (#73) The string key "moodlecoursecustomfield" is not in the correct order, it should be before "moodlecoursefield"; (#82) The string key "groupselect" is not in the correct order, it should be before "templatename"; (#84) The string key "unissueddescription" is not in the correct order, it should be before "unissuedheader"; (#90) The string key "viewsubheader" is not in the correct order, it should be before "viewsubheaderold"; (#92) The string key "gotodashboard" is not in the correct order, it should be before "viewsubheader"; (#94) The string key "activitygroupdescription" is not in the correct order, it should be before "overview"; (#95) The string key "accrediblegroup" is not in the correct order, it should be before "activitygroupdescription"; (#102) The string key "nouserswarning" is not in the correct order, it should be before "privacy:metadata:accredible:quizgrade"
mod/accredible/lang/en/error.php: (#29) The string key "getcredentialserror" is not in the correct order, it should be before "getissuederror"; (#43) The string key "evidenceadderror" is not in the correct order, it should be before "manualadderror:edit"
mod/accredible/tests/apirest/mod_accredible_apirest_test.php: (#31) Unit test mod_accredible_apirest_test should be declared as final.; (#35) The setUp() method in unit tests must always call to parent::setUp().; (#65) Test method test_api_endpoint() is missing a return type; (#85) Test method test_get_credential() is missing a return type; (#153) Test method test_get_credentials() is missing a return type; (#223) Test method test_recipient_sso_link() is missing a return type; (#235) Short array syntax must be used to define arrays; (#262) Short array syntax must be used to define arrays; (#264) There should be a comma after the last array item in a multi-line array.; (#282) Test method test_create_credential() is missing a return type; (#294) Short array syntax must be used to define arrays; (#295) Short array syntax must be used to define arrays; (#297) Short array syntax must be used to define arrays; (#299) There should be a comma after the last array item in a multi-line array.; (#303) There should be a comma after the last array item in a multi-line array.; (#304) There should be a comma after the last array item in a multi-line array.; (#328) Short array syntax must be used to define arrays; (#329) Short array syntax must be used to define arrays; (#331) Short array syntax must be used to define arrays; (#333) There should be a comma after the last array item in a multi-line array.; (#337) There should be a comma after the last array item in a multi-line array.; (#338) There should be a comma after the last array item in a multi-line array.; (#356) Test method test_create_credential_legacy() is missing a return type; (#368) Short array syntax must be used to define arrays; (#369) Short array syntax must be used to define arrays; (#371) Short array syntax must be used to define arrays; (#373) There should be a comma after the last array item in a multi-line array.; (#380) There should be a comma after the last array item in a multi-line array.; (#381) There should be a comma after the last array item in a multi-line array.; (#421) Test method test_search_groups() is missing a return type; (#430) Short array syntax must be used to define arrays; (#453) Short array syntax must be used to define arrays; (#477) Short array syntax must be used to define arrays; (#498) Test method test_create_evidence_item() is missing a return type; (#509) Short array syntax must be used to define arrays; (#510) Short array syntax must be used to define arrays; (#514) There should be a comma after the last array item in a multi-line array.; (#515) There should be a comma after the last array item in a multi-line array.; (#581) Test method test_create_evidence_item_duration() is missing a return type; (#594) Short array syntax must be used to define arrays; (#597) There should be a comma after the last array item in a multi-line array.; (#599) Short array syntax must be used to define arrays; (#600) Short array syntax must be used to define arrays; (#604) There should be a comma after the last array item in a multi-line array.; (#605) There should be a comma after the last array item in a multi-line array.; (#629) Short array syntax must be used to define arrays; (#632) There should be a comma after the last array item in a multi-line array.; (#634) Short array syntax must be used to define arrays; (#635) Short array syntax must be used to define arrays; (#639) There should be a comma after the last array item in a multi-line array.; (#640) There should be a comma after the last array item in a multi-line array.; (#675) Test method test_update_evidence_item_grade() is missing a return type; (#776) Test method test_get_group() is missing a return type; (#824) Test method test_get_groups() is missing a return type; (#891) Test method test_search_attribute_keys() is missing a return type; (#902) Short array syntax must be used to define arrays; (#924) Short array syntax must be used to define arrays; (#947) Short array syntax must be used to define arrays
mod/accredible/tests/client/mod_accredible_client_test.php: (#30) Unit test mod_accredible_client_test should be declared as final.; (#34) The setUp() method in unit tests must always call to parent::setUp().; (#46) Test method test_get() is missing a return type; (#48) Short array syntax must be used to define arrays; (#51) Short array syntax must be used to define arrays; (#54) There should be a comma after the last array item in a multi-line array.; (#55) There should be a comma after the last array item in a multi-line array.; (#78) Test method test_post() is missing a return type; (#80) Short array syntax must be used to define arrays; (#83) Short array syntax must be used to define arrays; (#86) There should be a comma after the last array item in a multi-line array.; (#87) There should be a comma after the last array item in a multi-line array.; (#111) Test method test_put() is missing a return type; (#113) Short array syntax must be used to define arrays; (#116) Short array syntax must be used to define arrays; (#119) There should be a comma after the last array item in a multi-line array.; (#120) There should be a comma after the last array item in a multi-line array.; (#144) Test method test_error() is missing a return type; (#146) Short array syntax must be used to define arrays; (#149) Short array syntax must be used to define arrays; (#152) There should be a comma after the last array item in a multi-line array.; (#153) There should be a comma after the last array item in a multi-line array.
mod/accredible/tests/local/mod_accredible_accredible_test.php: (#28) Unit test mod_accredible_accredible_test should be declared as final.; (#37) The setUp() method in unit tests must always call to parent::setUp().; (#46) Test method test_save_record() is missing a return type; (#106) There should be a comma after the last array item in a multi-line array.; (#107) There should be a comma after the last array item in a multi-line array.; (#112) There should be a comma after the last array item in a multi-line array.; (#113) There should be a comma after the last array item in a multi-line array.; (#118) There should be a comma after the last array item in a multi-line array.; (#119) There should be a comma after the last array item in a multi-line array.; (#188) Test method test_load_credential_custom_attributes() is missing a return type; (#192) Short array syntax must be used to define arrays; (#241) There should be a comma after the last array item in a multi-line array.; (#242) 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.; (#248) There should be a comma after the last array item in a multi-line array.; (#253) There should be a comma after the last array item in a multi-line array.; (#254) There should be a comma after the last array item in a multi-line array.; (#269) There should be a comma after the last array item in a multi-line array.; (#270) There should be a comma after the last array item in a multi-line array.; (#278) There should be a comma after the last array item in a multi-line array.; (#287) There should be a comma after the last array item in a multi-line array.; (#288) There should be a comma after the last array item in a multi-line array.; (#296) There should be a comma after the last array item in a multi-line array.; (#305) There should be a comma after the last array item in a multi-line array.; (#306) There should be a comma after the last array item in a multi-line array.; (#314) There should be a comma after the last array item in a multi-line array.; (#342) There should be a comma after the last array item in a multi-line array.; (#343) There should be a comma after the last array item in a multi-line array.; (#351) There should be a comma after the last array item in a multi-line array.; (#381) There should be a comma after the last array item in a multi-line array.; (#382) There should be a comma after the last array item in a multi-line array.; (#390) There should be a comma after the last array item in a multi-line array.; (#399) There should be a comma after the last array item in a multi-line array.; (#400) There should be a comma after the last array item in a multi-line array.; (#408) There should be a comma after the last array item in a multi-line array.; (#437) There should be a comma after the last array item in a multi-line array.; (#438) There should be a comma after the last array item in a multi-line array.; (#446) There should be a comma after the last array item in a multi-line array.; (#455) There should be a comma after the last array item in a multi-line array.; (#456) There should be a comma after the last array item in a multi-line array.; (#461) There should be a comma after the last array item in a multi-line array.; (#462) There should be a comma after the last array item in a multi-line array.; (#467) There should be a comma after the last array item in a multi-line array.; (#468) There should be a comma after the last array item in a multi-line array.; (#478) There should be a comma after the last array item in a multi-line array.; (#488) Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable parameter: $overrides.; (#502) There should be a comma after the last array item in a multi-line array.
mod/accredible/tests/local/mod_accredible_attribute_keys_test.php: (#30) Unit test mod_accredible_attribute_keys_test should be declared as final.; (#34) The setUp() method in unit tests must always call to parent::setUp().; (#63) Test method test_get_attribute_keys() is missing a return type; (#76) Short array syntax must be used to define arrays; (#77) Short array syntax must be used to define arrays; (#88) Short array syntax must be used to define arrays; (#92) There should be a comma after the last array item in a multi-line array.; (#141) Short array syntax must be used to define arrays; (#148) Short array syntax must be used to define arrays; (#149) Short array syntax must be used to define arrays; (#160) Short array syntax must be used to define arrays; (#164) There should be a comma after the last array item in a multi-line array.
mod/accredible/tests/local/mod_accredible_attributemapping_list_test.php: (#28) Unit test mod_accredible_attributemapping_list_test should be declared as final.; (#32) The setUp() method in unit tests must always call to parent::setUp().; (#40) Test method test_validate_attributemapping() is missing a return type; (#49) Short array syntax must be used to define arrays; (#59) Short array syntax must be used to define arrays; (#66) Short array syntax must be used to define arrays; (#75) Test method test_get_text_content() is missing a return type; (#100) Short array syntax must be used to define arrays
mod/accredible/tests/local/mod_accredible_attributemapping_test.php: (#28) Unit test mod_accredible_attributemapping_test should be declared as final.; (#32) The setUp() method in unit tests must always call to parent::setUp().; (#40) Test method test_validate_table() is missing a return type; (#67) Test method test_validate_field() is missing a return type; (#101) Test method test_validate_id() is missing a return type; (#149) Test method test_get_db_object() is missing a return type
mod/accredible/tests/local/mod_accredible_credentials_test.php: (#32) Unit test mod_accredible_credentials_test should be declared as final.; (#36) The setUp() method in unit tests must always call to parent::setUp().; (#61) Short array syntax must be used to define arrays; (#69) Test method test_create_credential() is missing a return type; (#83) Short array syntax must be used to define arrays; (#84) Short array syntax must be used to define arrays; (#86) Short array syntax must be used to define arrays; (#88) There should be a comma after the last array item in a multi-line array.; (#92) Short array syntax must be used to define arrays; (#93) There should be a comma after the last array item in a multi-line array.; (#94) There should be a comma after the last array item in a multi-line array.; (#95) There should be a comma after the last array item in a multi-line array.; (#106) Short array syntax must be used to define arrays; (#143) Test method test_create_credential_legacy() is missing a return type; (#155) Short array syntax must be used to define arrays; (#165) Short array syntax must be used to define arrays; (#168) Short array syntax must be used to define arrays; (#169) Short array syntax must be used to define arrays; (#171) Short array syntax must be used to define arrays; (#173) There should be a comma after the last array item in a multi-line array.; (#180) There should be a comma after the last array item in a multi-line array.; (#181) There should be a comma after the last array item in a multi-line array.; (#231) Test method test_get_credentials() is missing a return type; (#301) Short array syntax must be used to define arrays; (#308) Test method test_check_for_existing_credential() is missing a return type; (#382) Test method test_check_for_existing_certificate() is missing a return type
mod/accredible/tests/local/mod_accredible_evidenceitems_test.php: (#30) Unit test mod_accredible_evidenceitems_test should be declared as final.; (#34) The setUp() method in unit tests must always call to parent::setUp().; (#66) Test method test_post_evidence() is missing a return type; (#77) 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.; (#83) Short array syntax must be used to define arrays; (#149) Test method test_post_essay_answers() is missing a return type; (#179) Short array syntax must be used to define arrays; (#204) Short array syntax must be used to define arrays; (#205) Short array syntax must be used to define arrays; (#208) There should be a comma after the last array item in a multi-line array.; (#209) There should be a comma after the last array item in a multi-line array.; (#228) Test method test_course_duration_evidence() is missing a return type; (#266) Short array syntax must be used to define arrays; (#269) There should be a comma after the last array item in a multi-line array.; (#271) Short array syntax must be used to define arrays; (#272) Short array syntax must be used to define arrays; (#276) There should be a comma after the last array item in a multi-line array.; (#277) There should be a comma after the last array item in a multi-line array.; (#300) Short array syntax must be used to define arrays; (#313) Short array syntax must be used to define arrays; (#324) Short array syntax must be used to define arrays; (#340) Short array syntax must be used to define arrays; (#359) Short array syntax must be used to define arrays; (#377) Short array syntax must be used to define arrays; (#390) Short array syntax must be used to define arrays
mod/accredible/tests/local/mod_accredible_formhelper_test.php: (#28) Unit test mod_accredible_formhelper_test should be declared as final.; (#32) The setUp() method in unit tests must always call to parent::setUp().; (#38) Short array syntax must be used to define arrays; (#41) There should be a comma after the last array item in a multi-line array.; (#53) Test method test_load_grade_item_options() is missing a return type; (#57) Short array syntax must be used to define arrays; (#59) There should be a comma after the last array item in a multi-line array.; (#71) Short array syntax must be used to define arrays; (#86) Test method test_load_course_field_options() is missing a return type; (#106) Test method test_load_course_custom_field_options() is missing a return type; (#113) There should be a comma after the last array item in a multi-line array.; (#119) Short array syntax must be used to define arrays; (#123) There should be a comma after the last array item in a multi-line array.; (#127) Short array syntax must be used to define arrays; (#131) There should be a comma after the last array item in a multi-line array.; (#149) Test method test_load_user_profile_field_options() is missing a return type; (#155) There should be a comma after the last array item in a multi-line array.; (#161) Short array syntax must be used to define arrays; (#163) There should be a comma after the last array item in a multi-line array.; (#167) Short array syntax must be used to define arrays; (#169) There should be a comma after the last array item in a multi-line array.; (#186) Test method test_map_select_options() is missing a return type; (#193) There should be a comma after the last array item in a multi-line array.; (#198) There should be a comma after the last array item in a multi-line array.; (#219) Test method test_get_attributekeys_choices() is missing a return type; (#230) There should be a comma after the last array item in a multi-line array.; (#238) There should be a comma after the last array item in a multi-line array.; (#253) There should be a comma after the last array item in a multi-line array.; (#268) Test method test_attributemapping_default_values() is missing a return type; (#276) There should be a comma after the last array item in a multi-line array.; (#285) There should be a comma after the last array item in a multi-line array.; (#290) There should be a comma after the last array item in a multi-line array.; (#295) There should be a comma after the last array item in a multi-line array.; (#296) There should be a comma after the last array item in a multi-line array.; (#304) There should be a comma after the last array item in a multi-line array.; (#305) There should be a comma after the last array item in a multi-line array.; (#311) There should be a comma after the last array item in a multi-line array.; (#312) There should be a comma after the last array item in a multi-line array.; (#318) There should be a comma after the last array item in a multi-line array.; (#319) There should be a comma after the last array item in a multi-line array.; (#320) There should be a comma after the last array item in a multi-line array.; (#329) Test method test_reindexarray() is missing a return type; (#340) There should be a comma after the last array item in a multi-line array.; (#345) There should be a comma after the last array item in a multi-line array.; (#360) Short array syntax must be used to define arrays; (#363) There should be a comma after the last array item in a multi-line array.; (#380) Short array syntax must be used to define arrays; (#384) There should be a comma after the last array item in a multi-line array.; (#399) 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.; (#420) You must use "/**" style comments for a member variable comment
mod/accredible/tests/local/mod_accredible_groups_test.php: (#33) Unit test mod_accredible_groups_test should be declared as final.; (#37) The setUp() method in unit tests must always call to parent::setUp().; (#66) Test method test_get_groups() is missing a return type; (#89) Short array syntax must be used to define arrays; (#142) Short array syntax must be used to define arrays; (#149) Test method test_get_templates() is missing a return type; (#159) Short array syntax must be used to define arrays; (#160) Short array syntax must be used to define arrays; (#174) Short array syntax must be used to define arrays; (#178) There should be a comma after the last array item in a multi-line array.; (#190) Short array syntax must be used to define arrays; (#218) Short array syntax must be used to define arrays; (#231) Short array syntax must be used to define arrays
mod/accredible/tests/local/mod_accredible_users_test.php: (#32) Unit test mod_accredible_users_test should be declared as final.; (#36) The setUp() method in unit tests must always call to parent::setUp().; (#60) Short array syntax must be used to define arrays; (#69) Test method test_get_users_with_credentials() is missing a return type; (#73) Short array syntax must be used to define arrays; (#74) Short array syntax must be used to define arrays; (#79) Short array syntax must be used to define arrays; (#84) Short array syntax must be used to define arrays; (#90) Short array syntax must be used to define arrays; (#92) Short array syntax must be used to define arrays; (#97) Short array syntax must be used to define arrays; (#141) Short array syntax must be used to define arrays; (#148) Test method test_get_unissued_users() is missing a return type; (#152) Short array syntax must be used to define arrays; (#156) Short array syntax must be used to define arrays; (#161) Short array syntax must be used to define arrays; (#167) Short array syntax must be used to define arrays; (#170) Short array syntax must be used to define arrays; (#171) Short array syntax must be used to define arrays; (#175) Short array syntax must be used to define arrays; (#179) Short array syntax must be used to define arrays; (#186) Short array syntax must be used to define arrays; (#191) Short array syntax must be used to define arrays; (#196) Short array syntax must be used to define arrays; (#204) Test method test_get_user_grades() is missing a return type; (#208) Short array syntax must be used to define arrays; (#212) Short array syntax must be used to define arrays; (#214) There should be a comma after the last array item in a multi-line array.; (#223) Short array syntax must be used to define arrays; (#229) Short array syntax must be used to define arrays; (#235) Short array syntax must be used to define arrays; (#241) Short array syntax must be used to define arrays; (#247) Short array syntax must be used to define arrays; (#255) Short array syntax must be used to define arrays; (#258) Short array syntax must be used to define arrays; (#261) Short array syntax must be used to define arrays; (#262) There should be a comma after the last array item in a multi-line array.; (#267) Short array syntax must be used to define arrays; (#274) Short array syntax must be used to define arrays; (#276) There should be a comma after the last array item in a multi-line array.; (#278) Short array syntax must be used to define arrays; (#283) Short array syntax must be used to define arrays; (#284) There should be a comma after the last array item in a multi-line array.; (#301) Test method test_load_user_grade_as_custom_attributes() is missing a return type; (#308) Short array syntax must be used to define arrays; (#311) Short array syntax must be used to define arrays; (#315) Short array syntax must be used to define arrays; (#316) There should be a comma after the last array item in a multi-line array.; (#319) Short array syntax must be used to define arrays; (#338) Short array syntax must be used to define arrays; (#364) Short array syntax must be used to define arrays; (#370) There should be a comma after the last array item in a multi-line array.; (#381) Short array syntax must be used to define arrays; (#394) Short array syntax must be used to define arrays; (#407) Short array syntax must be used to define arrays; (#411) There should be a comma after the last array item in a multi-line array.
mod/accredible/tests/mod_accredible_lib_test.php: (#32) Unit test mod_accredible_lib_test should be declared as final.; (#38) Test method test_sample() is missing a return type
mod/accredible/tests/mod_accredible_locallib_test.php: (#30) Unit test mod_accredible_locallib_test should be declared as final.; (#34) The setUp() method in unit tests must always call to parent::setUp().; (#65) Test method test_accredible_get_transcript() is missing a return type; (#90) Short array syntax must be used to define arrays; (#95) There should be a comma after the last array item in a multi-line array.; (#119) Short array syntax must be used to define arrays; (#124) There should be a comma after the last array item in a multi-line array.; (#153) Test method test_accredible_check_if_cert_earned() is missing a return type; (#155) Short array syntax must be used to define arrays; (#158) There should be a comma after the last array item in a multi-line array.; (#203) Short array syntax must be used to define arrays; (#210) There should be a comma after the last array item in a multi-line array.; (#214) 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

CSS problems

(0 errors, 0 warnings)

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

PHPDocs style problems

(0 errors, 0 warnings)

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

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

(1 errors, 1 warnings)

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

: Problems running grunt; Task "eslint:amd" failed. Use --force to continue.

shifter problems

(0 errors, 0 warnings)

This section shows problems detected by shifter [More info]

Mustache template problems

(0 errors, 0 warnings)

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

Gherkin .feature problems

(0 errors, 0 warnings)

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

Prechecker results: error

Overview

PHP lint problems

PHP coding style problems

Javascript coding style problems

CSS problems

PHPDocs style problems

Update savepoints problems

Third party library modification problems

grunt changes

shifter problems

Mustache template problems

Gherkin .feature problems