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

(64 errors, 12 warnings)

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

report/allylti/classes/local/hook_callbacks.php
(#1) DocBlock missing a @package tag for file hook_callbacks.php. Expected @package report_allylti
(#26) Missing docblock for class hook_callbacks
(#27) Missing docblock for function before_standard_head_html_generation
report/allylti/classes/local/launch_config.php
(#34) Opening brace must not be followed by a blank line
(#87) No one-line description found in phpdocs for docblock of function get_url
(#90) CASE keyword must be followed by a single space
(#96) Function closing brace must go on the next line following the body; found 1 blank lines before brace
(#101) No one-line description found in phpdocs for docblock of function get_key
(#108) No one-line description found in phpdocs for docblock of function get_secret
(#115) No one-line description found in phpdocs for docblock of function get_launchcontainer
(#119) The closing brace for the class must go on the next line after the body
report/allylti/classes/privacy/provider.php
(#40) The interface names in a "class implements" statement should be ordered alphabetically. Expected: \core_privacy\local\metadata\provider, \core_privacy\local\request\plugin\provider, \core_privacy\local\request\core_userlist_provider; Found: \core_privacy\local\request\core_userlist_provider, \core_privacy\local\metadata\provider, \core_privacy\local\request\plugin\provider
(#40) The first item in a multi-line implements list must be on the line following the implements keyword
(#42) Opening brace must not be followed by a blank line
(#44) Missing docblock for function get_metadata
(#44) There must not be a space before the colon in a return type declaration
(#58) Missing docblock for function get_contexts_for_userid
(#58) There must not be a space before the colon in a return type declaration
(#62) Missing docblock for function export_user_data
(#65) Missing docblock for function delete_data_for_all_users_in_context
(#68) Missing docblock for function delete_data_for_user
(#71) Missing docblock for function get_users_in_context
(#74) Missing docblock for function delete_data_for_users
report/allylti/db/hooks.php
(#1) DocBlock missing a @package tag for file hooks.php. Expected @package report_allylti
(#23) Expected MOODLE_INTERNAL check or config.php inclusion. Change in global state detected.
report/allylti/lang/en/report_allylti.php
(#28) The string key "adminreport" is not in the correct order, it should be before "pluginname"
(#42) The string key "allylti:viewcoursereport" is not in the correct order, it should be before "reportnewwindow"
report/allylti/lib.php
(#17) Missing @copyright tag
(#17) Missing @license tag
(#17) Missing docblock for function report_allylti_extend_navigation_course
(#31) Expected 0 spaces before closing parenthesis; newline found
(#36) Multi-line function call not indented correctly; expected 8 spaces but found 12
(#36) Only one argument is allowed per line in a multi-line function call
(#36) Only one argument is allowed per line in a multi-line function call
(#36) Only one argument is allowed per line in a multi-line function call
(#36) Closing parenthesis of a multi-line function call must be on a line by itself
report/allylti/settings.php
(#31) Logical operator "and" is prohibited; use "&&" instead
(#31) Logical operator "and" is prohibited; use "&&" instead
(#38) Opening parenthesis of a multi-line function call must be the last content on the line
(#38) Only one argument is allowed per line in a multi-line function call
(#39) Multi-line function call not indented correctly; expected 4 spaces but found 8
(#39) Only one argument is allowed per line in a multi-line function call
(#39) Closing parenthesis of a multi-line function call must be on a line by itself
report/allylti/tests/behat/behat_report_allylti.php
(#1) DocBlock missing a @package tag for file behat_report_allylti.php. Expected @package report_allylti
(#40) Incorrect @package tag for class behat_report_allylti. Expected report_allylti, found filter_ally.
(#42) Opening brace must not be followed by a blank line
(#44) Missing docblock for function findnofail
(#56) No one-line description found in phpdocs for docblock of function navigate_ax_report
report/allylti/tests/launch_config_test.php
(#37) Unit test launch_config_test should be declared as final.
(#37) Opening brace must not be followed by a blank line
(#39) The setUp() method in unit tests must always call to parent::setUp().
(#44) Test method test_config_empty_exception() is missing any coverage information, own or at class level
(#49) Test method test_config_no_secret() is missing any coverage information, own or at class level
(#59) Test method test_config_no_key() is missing any coverage information, own or at class level
(#69) Test method test_config_no_url() is missing any coverage information, own or at class level
(#79) Test method test_config_container_override() is missing any coverage information, own or at class level
(#96) Test method test_config() is missing any coverage information, own or at class level
(#112) Test method test_config_course_report_url() is missing any coverage information, own or at class level
report/allylti/tests/privacy_provider_test.php
(#39) Unit test privacy_provider_test should be declared as final.
(#43) Test method test_get_metadata() is missing any coverage information, own or at class level
report/allylti/view.php
(#29) Expected at least 1 space before "."; 0 found
(#29) Expected at least 1 space after "."; 0 found
(#33) Logical operator "and" is prohibited; use "&&" instead
(#33) Logical operator "and" is prohibited; use "&&" instead
(#67) Expected at least 1 space before "."; 0 found
(#67) Expected at least 1 space after "."; 0 found
(#67) Expected at least 1 space before "."; 0 found
(#67) Expected at least 1 space after "."; 0 found
(#70) Expected at least 1 space before "."; 0 found
(#70) Expected at least 1 space after "."; 0 found
(#70) Expected at least 1 space before "."; 0 found
(#70) Expected at least 1 space after "."; 0 found
(#73) Expected at least 1 space before "."; 0 found
(#73) Expected at least 1 space after "."; 0 found
(#73) Expected at least 1 space before "."; 0 found
(#73) Expected at least 1 space after "."; 0 found

Javascript coding style problems

(1 errors, 0 warnings)

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

report/allylti/amd/src/main.js
(#19) @package should be empty. (jsdoc/empty-tags)

CSS problems

(0 errors, 0 warnings)

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


PHPDocs style problems

(1 errors, 0 warnings)

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

report/allylti/classes/local/launch_config.php
(#58) Phpdocs for function launch_config::__construct 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

(1 errors, 1 warnings)

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

Problems running grunt
Task "gherkinlint" 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

(3 errors, 0 warnings)

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

report/allylti/tests/behat/view_report.feature
(#74) Wrong indentation for "Examples", expected indentation level of 4, but got 2
(#75) Wrong indentation for "example", expected indentation level of 6, but got 4
(#76) Wrong indentation for "example", expected indentation level of 6, but got 4