Prechecker results: error


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

(21 errors, 15 warnings)

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

plagiarism/strike/classes/configplagiarism.php
(#59) The static declaration must come after the visibility declaration
plagiarism/strike/classes/forms/setup.php
(#126) This comment is 50% valid code; is this commented out code?
plagiarism/strike/classes/privacy/provider.php
(#67) Method name "_get_metadata" should not be prefixed with an underscore to indicate visibility
(#115) Method name "_get_contexts_for_userid" should not be prefixed with an underscore to indicate visibility
(#182) Method name "_export_plagiarism_user_data" should not be prefixed with an underscore to indicate visibility
(#233) Method name "_export_plagiarism_strike_data_for_user" should not be prefixed with an underscore to indicate visibility
(#251) Method name "_delete_plagiarism_for_context" should not be prefixed with an underscore to indicate visibility
(#282) Method name "_delete_plagiarism_for_user" should not be prefixed with an underscore to indicate visibility
plagiarism/strike/classes/strikeapitest.php
(#69) Inline comments must end in full-stops, exclamation marks, or question marks
(#69) No space found before comment text; expected "// self::setup_api_debugging($api);" but found "//self::setup_api_debugging($api);"
(#142) The static declaration must come after the visibility declaration
plagiarism/strike/classes/strikefile.php
(#202) The static declaration must come after the visibility declaration
plagiarism/strike/classes/strikeplagiarism.php
(#74) The static declaration must come after the visibility declaration
(#287) The static declaration must come after the visibility declaration
(#318) The static declaration must come after the visibility declaration
(#330) The static declaration must come after the visibility declaration
(#475) The static declaration must come after the visibility declaration
(#725) The static declaration must come after the visibility declaration
plagiarism/strike/classes/strikeremove.php
(#98) The static declaration must come after the visibility declaration
plagiarism/strike/classes/strikeremovedocument.php
(#98) The static declaration must come after the visibility declaration
plagiarism/strike/classes/strikereport.php
(#87) The static declaration must come after the visibility declaration
plagiarism/strike/classes/tables/strikefiles.php
(#305) Inline comments must start with a capital letter, digit or 3-dots sequence
(#305) This comment is 65% valid code; is this commented out code?
(#305) No space found before comment text; expected "// $url = \plagiarism_strike\strikeplagiarism::get_report_url($row);" but found "//$url = \plagiarism_strike\strikeplagiarism::get_report_url($row);"
plagiarism/strike/lib.php
(#230) Empty IF statement detected
(#233) No space found before comment text; expected "// $ranking .= '<a href="'.$results['reporturl'].'" target="_blank">';" but found "//$ranking .= '<a href="'.$results['reporturl'].'" target="_blank">';"
(#234) No space found before comment text; expected "// $ranking .= plagiarism_strike\strikeplagiarism::get_image('report', 'report');" but found "//$ranking .= plagiarism_strike\strikeplagiarism::get_image('report', 'report');"
(#235) Inline comments must end in full-stops, exclamation marks, or question marks
(#235) No space found before comment text; expected "// $ranking .= '</a>';" but found "//$ranking .= '</a>';"
(#906) This comment is 50% valid code; is this commented out code?
plagiarism/strike/report2.php
(#42) Inline comments must start with a capital letter, digit or 3-dots sequence
(#42) This comment is 63% valid code; is this commented out code?
(#42) No space found before comment text; expected "// $sr = \plagiarism_strike\strikereport::get_by_guid($sf->guid);" but found "//$sr = \plagiarism_strike\strikereport::get_by_guid($sf->guid);"
(#73) Empty CATCH statement detected
(#74) Inline comments must end in full-stops, exclamation marks, or question marks
(#74) No space found before comment text; expected "// TODO show error" but found "//TODO show error"

Javascript coding style problems

(0 errors, 7 warnings)

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

plagiarism/strike/amd/src/apitest.js
(#8) Unexpected space before function parentheses. (space-before-function-paren)
(#9) Missing JSDoc comment. (require-jsdoc)
(#9) Identifier 'test_api_status' is not in camel case. (camelcase)
(#20) Unexpected space before function parentheses. (space-before-function-paren)
(#35) Unexpected space before function parentheses. (space-before-function-paren)
(#41) Unexpected space before function parentheses. (space-before-function-paren)
(#43) Unexpected space before function parentheses. (space-before-function-paren)

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]

plagiarism/strike/classes/strikeplagiarism.php
(#56) Phpdocs for function strikeplagiarism::use_strike 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, 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]