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
(4 errors, 50 warnings)
This section shows the coding style problems detected in the code by phpcs [More info]
- admin/tool/opensesame/classes/local/data/opensesame_course.php
- (#92) Short array syntax must be used to define arrays
- admin/tool/opensesame/classes/local/opensesame_handler.php
- (#128) Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable parameter: $api.
- (#199) Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable parameter: $api.
- (#366) Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable parameter: $draftitemid.
- (#366) Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable parameter: $launchurl.
- (#430) Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable parameter: $launchurl.
- (#430) Declaring an optional parameter with a non-nullable type and a null default value before a required parameter is deprecated since PHP 8.4 Parameter $launchurl is optional, while parameter $courseid is required. The $launchurl parameter is implicitly treated as a required parameter.
- (#430) Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable parameter: $draftitemid.
- (#430) Declaring an optional parameter with a non-nullable type and a null default value before a required parameter is deprecated since PHP 8.4 Parameter $draftitemid is optional, while parameter $mod is required. The $draftitemid parameter is implicitly treated as a required parameter.
- (#430) Line exceeds 132 characters; contains 138 characters
- (#431) Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable parameter: $updt.
- (#431) Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable parameter: $instance.
- (#431) Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable parameter: $cm.
- admin/tool/opensesame/classes/privacy/provider.php
- (#42) There must not be a space before the colon in a return type declaration
- admin/tool/opensesame/lang/en/tool_opensesame.php
- (#34) The string key "autoconfigureconfirmation" is not in the correct order, it should be before "autoconfiguredesc"
- (#35) The string key "autoconfigsuccess" is not in the correct order, it should be before "autoconfigureconfirmation"
- (#36) The string key "apiauthtoken" is not in the correct order, it should be before "autoconfigsuccess"
- (#39) The string key "apitokenempty" is not in the correct order, it should be before "apiurlempty"
- (#53) The string key "onlyaiccurl" is not in the correct order, it should be before "onlylocal"
- (#55) The string key "opensesamecat" is not in the correct order, it should be before "opensesamesync"
- (#58) The string key "clientid_missing" is not in the correct order, it should be before "pluginname"
- (#60) The string key "authurl_missing" is not in the correct order, it should be before "clientsecret_missing"
- (#65) The string key "optableth:category" is not in the correct order, it should be before "optableth:title"
- (#67) The string key "optableth:course" is not in the correct order, it should be before "optableth:status"
- (#68) The string key "optableth:actions" is not in the correct order, it should be before "optableth:course"
- (#69) The string key "opsecoursestatuspage" is not in the correct order, it should be before "optableth:actions"
- (#70) The string key "opcategory" is not in the correct order, it should be before "opsecoursestatuspage"
- (#72) The string key "apicall_pagesize" is not in the correct order, it should be before "opcategory_desc"
- (#78) The string key "adhocblocked" is not in the correct order, it should be before "resumeadhoc"
- (#80) The string key "queued" is not in the correct order, it should be before "retrieved"
- (#81) The string key "created" is not in the correct order, it should be before "queued"
- (#84) The string key "archived" is not in the correct order, it should be before "scormimported"
- (#86) The string key "queued_info" is not in the correct order, it should be before "retrieved_info"
- (#87) The string key "created_info" is not in the correct order, it should be before "queued_info"
- (#90) The string key "resynccourses" is not in the correct order, it should be before "scormimported_info"
- (#92) The string key "resyncallqueued" is not in the correct order, it should be before "resyncqueued"
- (#93) The string key "archived_info" is not in the correct order, it should be before "resyncallqueued"
- (#105) The string key "namesupdated" is not in the correct order, it should be before "updatenames"
- (#106) The string key "guid" is not in the correct order, it should be before "namesupdated"
- (#107) The string key "coursename" is not in the correct order, it should be before "guid"
- (#108) The string key "courseid" is not in the correct order, it should be before "coursename"
- (#110) The string key "activity_name" is not in the correct order, it should be before "prefix"
- admin/tool/opensesame/tests/course_generation_test.php
- (#55) Unit test course_generation_test should be declared as final.
- (#63) The setUp() method in unit tests must always call to parent::setUp().
- (#76) Test method test_process_single_os_course() is missing any coverage information, own or at class level
- (#76) Test method test_process_single_os_course() is missing a return type
- admin/tool/opensesame/tests/course_queing_test.php
- (#54) Unit test course_queing_test should be declared as final.
- (#64) The setUp() method in unit tests must always call to parent::setUp().
- (#74) Test method test_retrieve_and_process_queue_courses() is missing any coverage information, own or at class level
- (#74) Test method test_retrieve_and_process_queue_courses() is missing a return type
- admin/tool/opensesame/tests/queries_test.php
- (#52) Unit test queries_test should be declared as final.
- (#59) The setUp() method in unit tests must always call to parent::setUp().
- (#66) Test method test_queries_funtions() is missing any coverage information, own or at class level
- (#66) Test method test_queries_funtions() is missing a return type
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
(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
(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, 0 warnings)
This section shows problems detected in mustache templates [More info]
- admin/tool/opensesame/templates/opensesame_courses_table.mustache
- (#1) HTML Validation info, line 10: The “button” role is unnecessary for element “button”. (sesame+)) <button role="button" data-toggle="popover" data-trigger="hover" data-placement="bottom" title="((+statusinfo%2C+tool_opensesame+))" data-html=true data-content="((+statusinfo_body%2C+tool_opensesame+))" style="border: none; background: none;"><i cla)
Gherkin .feature problems
(0 errors, 0 warnings)
This section shows problems detected in behat .feature files [More info]