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

(177 errors, 58 warnings)

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

local/cnw_smartcohort/classes/observer.php
(#35) Opening brace should be on the same line as the declaration
(#49) Opening brace should be on the same line as the declaration
(#58) Opening brace should be on the same line as the declaration
local/cnw_smartcohort/classes/privacy/provider.php
(#39) Opening brace should be on the same line as the declaration
local/cnw_smartcohort/classes/task/initialize_filter.php
(#30) "require_once" must be immediately followed by an open parenthesis
(#41) Opening brace should be on the same line as the declaration
(#49) Opening brace should be on the same line as the declaration
local/cnw_smartcohort/classes/task/process_queue.php
(#30) "require_once" must be immediately followed by an open parenthesis
(#41) Opening brace should be on the same line as the declaration
(#49) Opening brace should be on the same line as the declaration
local/cnw_smartcohort/db/upgrade.php
(#36) Opening brace should be on the same line as the declaration
(#96) Inline comments must end in full-stops, exclamation marks, or question marks
local/cnw_smartcohort/db/upgradelib.php
(#31) Opening brace should be on the same line as the declaration
local/cnw_smartcohort/edit.php
(#78) Inline comments must end in full-stops, exclamation marks, or question marks
(#80) Line exceeds 132 characters; contains 167 characters
(#81) Line exceeds 132 characters; contains 162 characters
(#111) Variable "confirmOutput" must be all lower-case
(#112) Variable "confirmOutput" must be all lower-case
(#113) Variable "confirmOutput" must be all lower-case
(#114) Variable "confirmOutput" must be all lower-case
(#115) Variable "confirmOutput" must be all lower-case
(#120) Variable "confirmOutput" must be all lower-case
(#121) Variable "confirmOutput" must be all lower-case
(#122) Variable "confirmOutput" must be all lower-case
(#123) Variable "confirmOutput" must be all lower-case
(#124) Variable "confirmOutput" must be all lower-case
(#124) Line exceeds 132 characters; contains 161 characters
(#125) Variable "confirmOutput" must be all lower-case
(#126) Variable "confirmOutput" must be all lower-case
(#127) Variable "confirmOutput" must be all lower-case
(#128) Variable "confirmOutput" must be all lower-case
local/cnw_smartcohort/edit_form.php
(#41) Opening brace should be on the same line as the declaration
(#47) Line exceeds 132 characters; contains 135 characters
(#49) Inline comments must end in full-stops, exclamation marks, or question marks
(#52) Inline comments must end in full-stops, exclamation marks, or question marks
(#57) Inline comments must end in full-stops, exclamation marks, or question marks
(#73) Inline comments must end in full-stops, exclamation marks, or question marks
(#105) Line exceeds 132 characters; contains 141 characters
(#107) Line exceeds 132 characters; contains 145 characters
(#110) Line exceeds 132 characters; contains 139 characters
(#126) Inline comments must end in full-stops, exclamation marks, or question marks
(#130) Variable "operatorKey" must be all lower-case
(#131) Variable "valueKey" must be all lower-case
(#132) Variable "operatorKey" must be all lower-case
(#133) Variable "valueKey" must be all lower-case
(#141) Opening brace should be on the same line as the declaration
local/cnw_smartcohort/index.php
(#52) Inline comments must end in full-stops, exclamation marks, or question marks
(#52) This comment is 50% valid code; is this commented out code?
(#54) Inline comments must end in full-stops, exclamation marks, or question marks
(#55) Variable "profileFields" must be all lower-case
(#72) Variable "profileFields" must be all lower-case
(#76) Line exceeds 132 characters; contains 160 characters
(#82) Inline comments must end in full-stops, exclamation marks, or question marks
(#84) Line exceeds maximum limit of 180 characters; contains 211 characters
(#85) Line exceeds maximum limit of 180 characters; contains 190 characters
(#86) Line exceeds maximum limit of 180 characters; contains 193 characters
(#88) Inline control structures are not allowed
(#90) Inline comments must end in full-stops, exclamation marks, or question marks
(#92) Variable "rulesString" must be all lower-case
(#94) Variable "rulesString" must be all lower-case
(#99) Variable "rulesString" must be all lower-case
(#99) Variable "profileFields" must be all lower-case
(#101) Variable "rulesString" must be all lower-case
(#101) Variable "profileFields" must be all lower-case
(#103) Variable "rulesString" must be all lower-case
(#103) Line exceeds maximum limit of 180 characters; contains 284 characters
(#107) Variable "rulesString" must be all lower-case
(#109) Inline comments must end in full-stops, exclamation marks, or question marks
(#110) Variable "affectedUsers" must be all lower-case
(#112) Inline comments must end in full-stops, exclamation marks, or question marks
(#118) Variable "rulesString" must be all lower-case
(#119) Line exceeds 132 characters; contains 133 characters
(#120) Variable "affectedUsers" must be all lower-case
(#120) Line exceeds maximum limit of 180 characters; contains 210 characters
(#128) Line exceeds maximum limit of 180 characters; contains 294 characters
local/cnw_smartcohort/lib.php
(#31) Variable "with_deleted" must not contain underscores.
(#32) Opening brace should be on the same line as the declaration
(#35) Variable "with_deleted" must not contain underscores.
(#43) Opening brace should be on the same line as the declaration
(#49) Variable "queryWhere" must be all lower-case
(#50) Variable "queryParams" must be all lower-case
(#71) Variable "queryWhere" must be all lower-case
(#72) Usage of ELSEIF not allowed; use ELSE IF instead
(#73) Variable "queryWhere" must be all lower-case
(#75) Variable "queryWhere" must be all lower-case
(#81) Variable "queryParams" must be all lower-case
(#84) Variable "queryParams" must be all lower-case
(#88) Variable "queryParams" must be all lower-case
(#93) Line exceeds maximum limit of 180 characters; contains 190 characters
(#100) Variable "queryParams" must be all lower-case
(#102) Variable "queryWhere" must be all lower-case
(#103) Variable "queryWhere" must be all lower-case
(#106) Variable "queryParams" must be all lower-case
(#120) Opening brace should be on the same line as the declaration
(#123) Inline comments must end in full-stops, exclamation marks, or question marks
(#126) Inline comments must end in full-stops, exclamation marks, or question marks
(#132) Variable "operatorKey" must be all lower-case
(#133) Variable "operatorKey" must be all lower-case
(#134) Variable "valueKey" must be all lower-case
(#139) Variable "operatorKey" must be all lower-case
(#140) Variable "valueKey" must be all lower-case
(#162) Opening brace should be on the same line as the declaration
(#165) Inline comments must end in full-stops, exclamation marks, or question marks
(#168) Inline comments must end in full-stops, exclamation marks, or question marks
(#171) Inline comments must end in full-stops, exclamation marks, or question marks
(#171) No space found before comment text; expected "// STORE NEW RULES" but found "//STORE NEW RULES"
(#177) Variable "operatorKey" must be all lower-case
(#178) Variable "operatorKey" must be all lower-case
(#179) Variable "valueKey" must be all lower-case
(#184) Variable "operatorKey" must be all lower-case
(#185) Variable "valueKey" must be all lower-case
(#206) Opening brace should be on the same line as the declaration
(#211) Inline comments must end in full-stops, exclamation marks, or question marks
(#212) Variable "shouldRemove" must be all lower-case
(#212) Line exceeds maximum limit of 180 characters; contains 297 characters
(#215) Variable "shouldRemove" must be all lower-case
(#215) Variable "scAdd" must be all lower-case
(#216) Variable "scAdd" must be all lower-case
(#216) Variable "scAdd" must be all lower-case
(#221) Inline comments must end in full-stops, exclamation marks, or question marks
(#237) Opening brace should be on the same line as the declaration
(#240) Variable "affectedUsers" must be all lower-case
(#242) Variable "cohortUsers" must be all lower-case
(#244) Variable "cohortUsers" must be all lower-case
(#247) Variable "affectedUserIds" must be all lower-case
(#248) Variable "affectedUsers" must be all lower-case
(#248) Variable "affectedUser" must be all lower-case
(#249) Variable "affectedUserIds" must be all lower-case
(#249) Variable "affectedUser" must be all lower-case
(#251) Variable "cohortUserIds" must be all lower-case
(#252) Variable "cohortUsers" must be all lower-case
(#252) Variable "cohortUser" must be all lower-case
(#253) Variable "cohortUserIds" must be all lower-case
(#253) Variable "cohortUser" must be all lower-case
(#256) Variable "shouldBeInCohort" must be all lower-case
(#256) Variable "affectedUserIds" must be all lower-case
(#256) Variable "cohortUserIds" must be all lower-case
(#257) Variable "shouldNotBeInCohort" must be all lower-case
(#257) Variable "cohortUserIds" must be all lower-case
(#257) Variable "affectedUserIds" must be all lower-case
(#259) Inline comments must end in full-stops, exclamation marks, or question marks
(#260) Variable "shouldNotBeInCohort" must be all lower-case
(#260) Variable "userId" must be all lower-case
(#261) Variable "scAdds" must be all lower-case
(#261) Variable "userId" must be all lower-case
(#262) Variable "scAdds" must be all lower-case
(#262) Variable "scAdds" must be all lower-case
(#263) Variable "userId" must be all lower-case
(#264) Variable "userId" must be all lower-case
(#268) Inline comments must end in full-stops, exclamation marks, or question marks
(#269) Variable "shouldBeInCohort" must be all lower-case
(#269) Variable "userId" must be all lower-case
(#270) Variable "userId" must be all lower-case
(#271) Variable "userId" must be all lower-case
(#271) Line exceeds 132 characters; contains 143 characters
(#272) Variable "scAdd" must be all lower-case
(#273) Variable "scAdd" must be all lower-case
(#274) Variable "scAdd" must be all lower-case
(#274) Variable "userId" must be all lower-case
(#275) Variable "scAdd" must be all lower-case
(#276) Variable "scAdd" must be all lower-case
(#280) Inline comments must end in full-stops, exclamation marks, or question marks
(#281) Variable "affectedUserIds" must be all lower-case
(#281) Variable "cohortUserIds" must be all lower-case
(#282) Variable "userId" must be all lower-case
(#283) Variable "userId" must be all lower-case
(#283) Line exceeds 132 characters; contains 143 characters
(#284) Variable "scAdd" must be all lower-case
(#285) Variable "scAdd" must be all lower-case
(#286) Variable "scAdd" must be all lower-case
(#286) Variable "userId" must be all lower-case
(#287) Variable "scAdd" must be all lower-case
(#288) Variable "scAdd" must be all lower-case
(#299) Opening brace should be on the same line as the declaration
(#314) Opening brace should be on the same line as the declaration
local/cnw_smartcohort/tests/smartcohort_test.php
(#43) Visibility must be declared on method "arrays_are_similar"
(#44) Opening brace should be on the same line as the declaration
(#45) Inline comments must start with a capital letter, digit or 3-dots sequence
(#45) Inline comments must end in full-stops, exclamation marks, or question marks
(#49) Inline comments must start with a capital letter, digit or 3-dots sequence
(#50) Inline comments must end in full-stops, exclamation marks, or question marks
(#56) Inline comments must start with a capital letter, digit or 3-dots sequence
(#56) Inline comments must end in full-stops, exclamation marks, or question marks
(#67) Variable "getField" must be all lower-case
(#68) Opening brace should be on the same line as the declaration
(#85) Variable "getField" must be all lower-case
(#96) Inline comments must start with a capital letter, digit or 3-dots sequence
(#96) Inline comments must end in full-stops, exclamation marks, or question marks
(#96) This comment is 80% valid code; is this commented out code?
(#96) No space found before comment text; expected "// var_dump($filterid);" but found "//var_dump($filterid);"
(#105) Public method name "smartcohort_test::testCohortLibHasAllFunction" must be in lower-case letters only
(#106) Opening brace should be on the same line as the declaration
(#120) Public method name "smartcohort_test::testCohortTableSchema" must be in lower-case letters only
(#121) Opening brace should be on the same line as the declaration
(#137) Opening brace should be on the same line as the declaration
(#141) Variable "lib_get" must not contain underscores.
(#143) Variable "lib_get" must not contain underscores.
(#148) Opening brace should be on the same line as the declaration
(#172) Functions must not contain multiple empty lines in a row; found 2 empty lines
(#181) Opening brace should be on the same line as the declaration
(#194) Line exceeds 132 characters; contains 170 characters
(#202) Opening brace should be on the same line as the declaration
(#208) Variable "usertoAdd" must be all lower-case
(#209) Variable "usertoAdd" must be all lower-case
(#210) Variable "usertoAdd" must be all lower-case
(#211) Variable "usertoAdd" must be all lower-case
(#212) Variable "usertoAdd" must be all lower-case
(#214) Variable "usertoAdd" must be all lower-case
(#215) Functions must not contain multiple empty lines in a row; found 2 empty lines
(#221) Line exceeds 132 characters; contains 144 characters
(#226) Opening brace should be on the same line as the declaration
(#234) Variable "usertoAdd" must be all lower-case
(#235) Variable "usertoAdd" must be all lower-case
(#236) Variable "usertoAdd" must be all lower-case
(#237) Variable "usertoAdd" must be all lower-case
(#238) Variable "usertoAdd" must be all lower-case
(#240) Variable "usertoAdd" must be all lower-case
(#248) Opening brace should be on the same line as the declaration
(#258) Line exceeds 132 characters; contains 148 characters
(#262) Line exceeds 132 characters; contains 149 characters
(#268) Line exceeds 132 characters; contains 148 characters
(#272) Line exceeds 132 characters; contains 149 characters
(#274) Functions must not contain multiple empty lines in a row; found 2 empty lines
(#279) Opening brace should be on the same line as the declaration
(#289) Line exceeds 132 characters; contains 148 characters
(#293) Line exceeds 132 characters; contains 149 characters
(#299) Line exceeds 132 characters; contains 148 characters
(#303) Line exceeds 132 characters; contains 149 characters
(#305) Functions must not contain multiple empty lines in a row; found 2 empty lines
(#310) Opening brace should be on the same line as the declaration
(#327) Line exceeds 132 characters; contains 148 characters
(#331) Line exceeds 132 characters; contains 150 characters
(#337) Opening brace should be on the same line as the declaration
(#354) Line exceeds 132 characters; contains 148 characters
(#358) Line exceeds 132 characters; contains 150 characters

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

(17 errors, 0 warnings)

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

local/cnw_smartcohort/classes/observer.php
(#29) Class local_cnw_smartcohort_observer is not documented
(#31) There is no description in phpdocs for function user_updated
(#45) There is no description in phpdocs for function user_deleted
(#54) There is no description in phpdocs for function cohort_deleted
local/cnw_smartcohort/classes/privacy/provider.php
(#29) Class provider is not documented
local/cnw_smartcohort/classes/task/initialize_filter.php
(#32) Class initialize_filter is not documented
local/cnw_smartcohort/classes/task/process_queue.php
(#32) Class process_queue is not documented
local/cnw_smartcohort/edit_form.php
(#31) Class filter_edit_form is not documented
(#140) Function filter_edit_form::validation is not documented
local/cnw_smartcohort/lib.php
(#31) Function smartcohort_get_filters is not documented
(#42) Function smartcohort_get_users_by_filter is not documented
(#232) Phpdocs for function smartcohort_run_filter has incomplete parameters list
local/cnw_smartcohort/tests/smartcohort_test.php
(#33) Class smartcohort_test is not documented
(#105) Function smartcohort_test::testCohortLibHasAllFunction is not documented
(#120) Function smartcohort_test::testCohortTableSchema is not documented
(#39) Phpdocs for function smartcohort_test::arrays_are_similar has incomplete parameters list
(#63) Phpdocs for function smartcohort_test::create_filter 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]