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

(334 errors, 542 warnings)

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

admin/tool/trigger/classes/edit_form.php
(#61) Short array syntax must be used to define arrays
(#68) There should be a comma after the last array item in a multi-line array.
(#98) Short array syntax must be used to define arrays
(#98) Short array syntax must be used to define arrays
(#107) Short array syntax must be used to define arrays
(#107) Short array syntax must be used to define arrays
(#116) Short array syntax must be used to define arrays
(#116) Short array syntax must be used to define arrays
admin/tool/trigger/classes/event_processor.php
(#141) Short array syntax must be used to define arrays
(#201) Missing docblock for function process_realtime_workflow
(#295) Short array syntax must be used to define arrays
(#300) Short array syntax must be used to define arrays
(#307) There should be a comma after the last array item in a multi-line array.
(#491) There should be a comma after the last array item in a multi-line array.
(#532) There should be a comma after the last array item in a multi-line array.
(#562) There should be a comma after the last array item in a multi-line array.
(#585) There should be a comma after the last array item in a multi-line array.
(#622) There should be a comma after the last array item in a multi-line array.
(#663) There should be a comma after the last array item in a multi-line array.
(#784) Inline comments must end in full-stops, exclamation marks, or question marks
(#812) Short array syntax must be used to define arrays
(#817) Inline comments must end in full-stops, exclamation marks, or question marks
(#818) Line exceeds 132 characters; contains 141 characters
(#825) There should be a comma after the last array item in a multi-line array.
admin/tool/trigger/classes/helper/datafield_manager.php
(#39) Missing member variable doc comment
admin/tool/trigger/classes/helper/processor_helper.php
(#28) Missing docblock for trait processor_helper
(#40) Short array syntax must be used to define arrays
(#44) Short array syntax must be used to define arrays
admin/tool/trigger/classes/import_form.php
(#61) Short array syntax must be used to define arrays
(#76) Short array syntax must be used to define arrays
admin/tool/trigger/classes/json/json_export.php
(#135) Expected 0 spaces after "print_json_data"; 1 found
admin/tool/trigger/classes/learn_process.php
(#40) Short array syntax must be used to define arrays
(#67) Short array syntax must be used to define arrays
(#164) Short array syntax must be used to define arrays
(#183) Expected 0 spaces after "process"; 1 found
(#191) Short array syntax must be used to define arrays
(#195) Short array syntax must be used to define arrays
(#200) Short array syntax must be used to define arrays
(#242) Short array syntax must be used to define arrays
(#255) Short array syntax must be used to define arrays
(#259) Short array syntax must be used to define arrays
(#264) Short array syntax must be used to define arrays
(#266) 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
(#273) There should be a comma after the last array item in a multi-line array.
(#275) Short array syntax must be used to define arrays
(#277) There should be a comma after the last array item in a multi-line array.
admin/tool/trigger/classes/output/manageworkflows/renderable.php
(#62) Short array syntax must be used to define arrays
(#71) There should be a comma after the last array item in a multi-line array.
(#73) Short array syntax must be used to define arrays
(#104) Short array syntax must be used to define arrays
(#144) Missing docblock for function col_triggerhistory
(#145) Short array syntax must be used to define arrays
(#160) Short array syntax must be used to define arrays
(#162) Short array syntax must be used to define arrays
(#166) Short array syntax must be used to define arrays
(#168) Short array syntax must be used to define arrays
(#170) Short array syntax must be used to define arrays
(#173) Short array syntax must be used to define arrays
(#175) Short array syntax must be used to define arrays
(#178) Short array syntax must be used to define arrays
(#183) Missing docblock for function col_active
admin/tool/trigger/classes/output/manageworkflows/renderer.php
(#78) Short array syntax must be used to define arrays
(#134) There should be a comma after the last array item in a multi-line array.
admin/tool/trigger/classes/output/workflowhistory/filter_form.php
(#42) Missing docblock for function definition
(#52) Line exceeds 132 characters; contains 136 characters
(#53) Line exceeds 132 characters; contains 140 characters
(#54) Line exceeds 132 characters; contains 136 characters
(#55) Line exceeds 132 characters; contains 138 characters
(#56) Line exceeds 132 characters; contains 142 characters
admin/tool/trigger/classes/output/workflowhistory/renderer.php
(#59) There should be a comma after the last array item in a multi-line array.
(#85) Line exceeds 132 characters; contains 145 characters
(#149) Missing docblock for function step_actions_button
(#226) Missing docblock for function run_actions_button
(#228) Short array syntax must be used to define arrays
admin/tool/trigger/classes/output/workflowhistory/run.php
(#34) Missing docblock for class runhistory_renderable
(#62) Short array syntax must be used to define arrays
(#69) 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
(#78) There should be a comma after the last array item in a multi-line array.
(#92) Missing docblock for function col_step
(#97) Missing docblock for function col_id
(#101) Missing docblock for function col_name
(#105) Missing docblock for function col_type
(#109) Missing docblock for function col_executed
(#114) Missing docblock for function col_prevstep
(#122) Missing docblock for function col_actions
admin/tool/trigger/classes/output/workflowhistory/workflow.php
(#31) Missing docblock for class workflowhistory_renderable
(#104) Missing docblock for function col_id
(#108) Missing docblock for function col_number
(#112) Missing docblock for function col_eventid
(#116) Missing docblock for function col_username
(#120) Missing docblock for function col_description
(#129) Missing docblock for function col_time
(#134) Missing docblock for function col_runstatus
(#176) Missing docblock for function col_actions
admin/tool/trigger/classes/privacy/provider.php
(#30) Import statements must not begin with a leading backslash
(#31) Import statements must not begin with a leading backslash
(#61) There must not be a space before the colon in a return type declaration
(#144) There must not be a space before the colon in a return type declaration
admin/tool/trigger/classes/steps/actions/assign_cohort_action_step.php
(#30) Missing docblock for class assign_cohort_action_step
(#49) Short array syntax must be used to define arrays
(#69) Missing docblock for function init
(#81) No one-line description found in phpdocs for docblock of function execute
(#87) Short array syntax must be used to define arrays
admin/tool/trigger/classes/steps/actions/email_action_step.php
(#64) Short array syntax must be used to define arrays
(#75) There should be a comma after the last array item in a multi-line array.
(#115) No one-line description found in phpdocs for docblock of function execute
(#128) Short array syntax must be used to define arrays
(#172) Short array syntax must be used to define arrays
(#194) Short array syntax must be used to define arrays
(#202) There should be a comma after the last array item in a multi-line array.
admin/tool/trigger/classes/steps/actions/http_post_action_step.php
(#48) Missing member variable doc comment
(#49) Missing member variable doc comment
(#50) Missing member variable doc comment
(#51) Missing member variable doc comment
(#52) Missing member variable doc comment
(#53) Missing member variable doc comment
(#54) Missing member variable doc comment
(#61) Short array syntax must be used to define arrays
(#67) Missing docblock for function init
(#130) No one-line description found in phpdocs for docblock of function execute
(#176) Short array syntax must be used to define arrays
(#186) Short array syntax must be used to define arrays
(#200) Short array syntax must be used to define arrays
(#206) Short array syntax must be used to define arrays
(#213) Short array syntax must be used to define arrays
(#213) Short array syntax must be used to define arrays
(#219) Short array syntax must be used to define arrays
admin/tool/trigger/classes/steps/actions/logdump_action_step.php
(#34) Short array syntax must be used to define arrays
(#38) Missing docblock for function form_definition_extra
(#45) Missing docblock for function get_step_desc
(#49) Missing docblock for function get_step_name
(#53) Missing docblock for function execute
admin/tool/trigger/classes/steps/actions/role_assign_action_step.php
(#27) Missing docblock for class role_assign_action_step
(#74) Invalid docblock tag "@inheritdoc".
(#76) No one-line description found in phpdocs for docblock of function init
(#83) Invalid docblock tag "@inheritdoc".
(#85) No one-line description found in phpdocs for docblock of function execute
admin/tool/trigger/classes/steps/actions/role_unassign_action_step.php
(#28) Missing docblock for class role_unassign_action_step
(#75) Invalid docblock tag "@inheritdoc".
(#77) No one-line description found in phpdocs for docblock of function init
(#84) Invalid docblock tag "@inheritdoc".
(#86) No one-line description found in phpdocs for docblock of function execute
admin/tool/trigger/classes/steps/actions/roles_unassign_action_step.php
(#43) Short array syntax must be used to define arrays
(#51) Missing member variable doc comment
(#53) Missing docblock for function init
(#91) No one-line description found in phpdocs for docblock of function execute
(#96) Short array syntax must be used to define arrays
(#104) Short array syntax must be used to define arrays
admin/tool/trigger/classes/steps/actions/webservice_action_step.php
(#55) Missing docblock for function init
admin/tool/trigger/classes/steps/base/base_form.php
(#95) Short array syntax must be used to define arrays
(#97) Short array syntax must be used to define arrays
(#113) Short array syntax must be used to define arrays
(#116) Short array syntax must be used to define arrays
(#120) Short array syntax must be used to define arrays
(#135) Short array syntax must be used to define arrays
(#137) There should be a comma after the last array item in a multi-line array.
(#164) Short array syntax must be used to define arrays
(#179) Short array syntax must be used to define arrays
(#199) Short array syntax must be used to define arrays
(#206) Short array syntax must be used to define arrays
admin/tool/trigger/classes/steps/base/base_step.php
(#36) Missing member variable doc comment
(#38) Missing member variable doc comment
(#38) Short array syntax must be used to define arrays
(#40) Missing docblock for function __construct
admin/tool/trigger/classes/steps/debounce/debounce_form.php
(#40) Missing docblock for function definition
admin/tool/trigger/classes/steps/debounce/debounce_step.php
(#195) Line exceeds 132 characters; contains 138 characters
(#261) Expected 0 spaces after "form_definition_extra"; 1 found
admin/tool/trigger/classes/steps/filters/numcompare_filter_step.php
(#40) Missing docblock for constant numcompare_filter_step::OPERATOR_EQUAL
(#41) Missing docblock for constant numcompare_filter_step::OPERATOR_NOTEQUAL
(#42) Missing docblock for constant numcompare_filter_step::OPERATOR_LT
(#43) Missing docblock for constant numcompare_filter_step::OPERATOR_LTE
(#44) Missing docblock for constant numcompare_filter_step::OPERATOR_GTE
(#45) Missing docblock for constant numcompare_filter_step::OPERATOR_GT
(#47) Missing member variable doc comment
(#48) Missing member variable doc comment
(#49) Missing member variable doc comment
(#97) Missing docblock for function get_field_value
(#160) There should be a comma after the last array item in a multi-line array.
admin/tool/trigger/classes/steps/filters/stringcompare_filter_step.php
(#212) There should be a comma after the last array item in a multi-line array.
(#221) There should be a comma after the last array item in a multi-line array.
(#233) Short array syntax must be used to define arrays
(#233) Short array syntax must be used to define arrays
admin/tool/trigger/classes/steps/lookups/cohort_lookup_step.php
(#59) Short array syntax must be used to define arrays
(#60) There should be a comma after the last array item in a multi-line array.
(#63) Missing docblock for function init
admin/tool/trigger/classes/steps/lookups/course_cat_lookup_step.php
(#64) There should be a comma after the last array item in a multi-line array.
(#67) Missing docblock for function init
admin/tool/trigger/classes/steps/lookups/course_lookup_step.php
(#50) Short array syntax must be used to define arrays
(#85) Missing docblock for function init
admin/tool/trigger/classes/steps/lookups/roles_lookup_step.php
(#50) Short array syntax must be used to define arrays
(#52) Missing docblock for function init
(#72) Short array syntax must be used to define arrays
admin/tool/trigger/classes/steps/lookups/user_lookup_step.php
(#55) Short array syntax must be used to define arrays
(#110) Missing docblock for function init
admin/tool/trigger/classes/task/cleanup.php
(#67) There should be a comma after the last array item in a multi-line array.
(#77) There should be a comma after the last array item in a multi-line array.
admin/tool/trigger/classes/task/process_workflows.php
(#94) Short array syntax must be used to define arrays
(#119) Missing docblock for function process_queue
(#133) There should be a comma after the last array item in a multi-line array.
(#149) Missing docblock for function process_item
(#172) Line exceeds 132 characters; contains 145 characters
admin/tool/trigger/classes/workflow.php
(#123) Short array syntax must be used to define arrays
(#133) Short array syntax must be used to define arrays
admin/tool/trigger/classes/workflow_manager.php
(#39) Short array syntax must be used to define arrays
(#48) Short array syntax must be used to define arrays
(#177) Line exceeds 132 characters; contains 133 characters
(#226) Short array syntax must be used to define arrays
(#227) Short array syntax must be used to define arrays
(#253) Short array syntax must be used to define arrays
admin/tool/trigger/classes/workflow_process.php
(#44) Short array syntax must be used to define arrays
(#50) There should be a comma after the last array item in a multi-line array.
(#78) There should be a comma after the last array item in a multi-line array.
(#85) There should be a comma after the last array item in a multi-line array.
(#274) Short array syntax must be used to define arrays
admin/tool/trigger/cli/export_fields.php
(#31) Short array syntax must be used to define arrays
(#31) Short array syntax must be used to define arrays
(#61) Short array syntax must be used to define arrays
admin/tool/trigger/cli/import_csv_records.php
(#35) Short array syntax must be used to define arrays
(#37) There should be a comma after the last array item in a multi-line array.
(#39) Short array syntax must be used to define arrays
(#40) There should be a comma after the last array item in a multi-line array.
(#76) Short array syntax must be used to define arrays
(#77) Short array syntax must be used to define arrays
admin/tool/trigger/db/access.php
(#29) Short array syntax must be used to define arrays
(#31) Short array syntax must be used to define arrays
(#35) Short array syntax must be used to define arrays
(#36) There should be a comma after the last array item in a multi-line array.
(#39) Short array syntax must be used to define arrays
(#43) Short array syntax must be used to define arrays
(#44) There should be a comma after the last array item in a multi-line array.
(#47) Short array syntax must be used to define arrays
(#51) Short array syntax must be used to define arrays
(#52) There should be a comma after the last array item in a multi-line array.
admin/tool/trigger/db/caches.php
(#28) Short array syntax must be used to define arrays
(#29) Short array syntax must be used to define arrays
(#34) There should be a comma after the last array item in a multi-line array.
(#35) There should be a comma after the last array item in a multi-line array.
admin/tool/trigger/db/events.php
(#28) Short array syntax must be used to define arrays
(#29) Short array syntax must be used to define arrays
admin/tool/trigger/db/messages.php
(#16) Missing @copyright tag
(#16) Boilerplate comment must be followed by a single blank line or end of file
(#26) Short array syntax must be used to define arrays
(#27) Short array syntax must be used to define arrays
admin/tool/trigger/db/services.php
(#28) Short array syntax must be used to define arrays
(#29) Short array syntax must be used to define arrays
(#36) There should be a comma after the last array item in a multi-line array.
(#38) Short array syntax must be used to define arrays
(#45) There should be a comma after the last array item in a multi-line array.
(#47) 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.
admin/tool/trigger/db/tasks.php
(#20) Incorrect @package tag for file tasks.php. Expected tool_trigger, found tool_monitor.
(#28) Short array syntax must be used to define arrays
(#29) Short array syntax must be used to define arrays
(#36) There should be a comma after the last array item in a multi-line array.
(#38) Short array syntax must be used to define arrays
(#45) There should be a comma after the last array item in a multi-line array.
(#47) 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.
(#56) Short array syntax must be used to define arrays
(#63) There should be a comma after the last array item in a multi-line array.
(#64) There should be a comma after the last array item in a multi-line array.
admin/tool/trigger/db/upgrade.php
(#52) Short array syntax must be used to define arrays
(#89) Short array syntax must be used to define arrays
(#122) Short array syntax must be used to define arrays
(#123) Short array syntax must be used to define arrays
(#123) Short array syntax must be used to define arrays
(#125) Short array syntax must be used to define arrays
(#126) Short array syntax must be used to define arrays
(#176) Short array syntax must be used to define arrays
(#220) Short array syntax must be used to define arrays
(#221) Short array syntax must be used to define arrays
(#221) Short array syntax must be used to define arrays
(#250) Short array syntax must be used to define arrays
(#251) Short array syntax must be used to define arrays
(#251) Short array syntax must be used to define arrays
(#252) Short array syntax must be used to define arrays
(#252) Short array syntax must be used to define arrays
(#255) Short array syntax must be used to define arrays
admin/tool/trigger/edit.php
(#34) Short array syntax must be used to define arrays
(#55) Short array syntax must be used to define arrays
(#63) Short array syntax must be used to define arrays
admin/tool/trigger/externallib.php
(#43) Short array syntax must be used to define arrays
(#78) Inline comments must end in full-stops, exclamation marks, or question marks
(#83) There should be a comma after the last array item in a multi-line array.
(#96) Short array syntax must be used to define arrays
(#110) Short array syntax must be used to define arrays
(#112) There should be a comma after the last array item in a multi-line array.
(#139) Short array syntax must be used to define arrays
(#146) Short array syntax must be used to define arrays
(#171) Short array syntax must be used to define arrays
(#172) There should be a comma after the last array item in a multi-line array.
(#195) Short array syntax must be used to define arrays
(#224) Short array syntax must be used to define arrays
admin/tool/trigger/history.php
(#28) Short array syntax must be used to define arrays
(#191) Duplicate array key found. The value will be overwritten. The string array key "id" was first seen on line 190
(#203) Duplicate array key found. The value will be overwritten. The string array key "id" was first seen on line 202
admin/tool/trigger/lang/en/tool_trigger.php
(#31) The string key "action" is not in the correct order, it should be before "pluginname_help"
(#39) The string key "assigncohortactionstepdesc" is not in the correct order, it should be before "assigncohortactionstepname"
(#50) The string key "badstepclass" is not in the correct order, it should be before "badsteptype"
(#71) The string key "downloadrundetails" is not in the correct order, it should be before "downloadworkflow"
(#76) The string key "duplicatedworkflowname" is not in the correct order, it should be before "duplicaterule"
(#85) The string key "emailcontent" is not in the correct order, it should be before "emailto_help"
(#87) The string key "emailactionstepname" is not in the correct order, it should be before "emailcontent_help"
(#88) The string key "emailactionstepdesc" is not in the correct order, it should be before "emailactionstepname"
(#97) The string key "erroreditstep" is not in the correct order, it should be before "eventtomonitor_help"
(#113) The string key "filterlabelrunstatus" is not in the correct order, it should be before "filterpassed"
(#125) The string key "httpostactionheaders" is not in the correct order, it should be before "httpostmethod_help"
(#130) The string key "httppostactionstepdesc" is not in the correct order, it should be before "httppostactionstepname"
(#134) The string key "inputprefixrole" is not in the correct order, it should be before "inputprefixuser"
(#154) The string key "movestepdown" is not in the correct order, it should be before "movestepup"
(#164) The string key "privacy:metadata:events" is not in the correct order, it should be before "privacy:path:events"
(#175) The string key "privacy:metadata:learnevents" is not in the correct order, it should be before "privacy:path:learnevents"
(#209) The string key "roleassignactionstepdesc" is not in the correct order, it should be before "roleassignactionstepname"
(#211) The string key "roleunassignactionstepdesc" is not in the correct order, it should be before "roleunassignactionstepname"
(#218) The string key "stepmodalbutton" is not in the correct order, it should be before "sttepidembed"
(#225) The string key "step_filter_fail_desc" is not in the correct order, it should be before "steptype_help"
(#230) The string key "step_lookup_course:privacy:categorydata_desc" is not in the correct order, it should be before "step_lookup_course:privacy:coursedata_desc"
(#233) The string key "step_lookup_course_courseidfield" is not in the correct order, it should be before "step_lookup_course_name"
(#234) The string key "courseidfield" is not in the correct order, it should be before "step_lookup_course_courseidfield"
(#238) The string key "step_lookup_category_categoryidfield" is not in the correct order, it should be before "step_lookup_category_name"
(#239) The string key "categoryidfield" is not in the correct order, it should be before "step_lookup_category_categoryidfield"
(#244) The string key "step_lookup_user_name" is not in the correct order, it should be before "step_lookup_user_nodeleted"
(#247) The string key "step_lookup_roles:privacy:userdata_desc" is not in the correct order, it should be before "step_lookup_user_useridfield"
(#251) The string key "step_action_email:privacy:desc" is not in the correct order, it should be before "step_lookup_roles_name"
(#260) The string key "step_action_role_assign_roleidfield" is not in the correct order, it should be before "useridfield_help"
(#262) The string key "rerunerrors" is not in the correct order, it should be before "step_action_role_unassign_roleidfield"
(#273) The string key "contextidfield" is not in the correct order, it should be before "stepnumber"
(#284) The string key "trigger:exportrundetails" is not in the correct order, it should be before "triggernumberembed"
(#287) The string key "timeexecuted" is not in the correct order, it should be before "trigger:manageworkflows"
(#297) The string key "webserviceactionparams" is not in the correct order, it should be before "webserviceactionusername_help"
(#300) The string key "webserviceactionstepdesc" is not in the correct order, it should be before "webserviceactionstepname"
(#306) The string key "workflowcopysuccess" is not in the correct order, it should be before "workflowrealtime_help"
(#315) The string key "workflowfile" is not in the correct order, it should be before "workflowimported"
(#320) The string key "viewtriggerrun" is not in the correct order, it should be before "workflowviewhistory"
(#321) The string key "viewdetailedrun" is not in the correct order, it should be before "viewtriggerrun"
(#324) The string key "roleunassignactionstepname" is duplicated
(#325) The string key "roleunassignactionstepdesc" is duplicated
(#326) The string key "rolesunassignactionstepname" is not in the correct order, it should be before "viewstepinfo"
(#327) The string key "rolesunassignactionstepdesc" is not in the correct order, it should be before "rolesunassignactionstepname"
admin/tool/trigger/lib.php
(#75) Short array syntax must be used to define arrays
(#82) There should be a comma after the last array item in a multi-line array.
(#85) Short array syntax must be used to define arrays
admin/tool/trigger/manage.php
(#32) Short array syntax must be used to define arrays
(#40) Short array syntax must be used to define arrays
admin/tool/trigger/manageworkflow.php
(#92) There should be a comma after the last array item in a multi-line array.
admin/tool/trigger/stepdetails.php
(#27) Short array syntax must be used to define arrays
admin/tool/trigger/tests/course_cat_lookup_step_test.php
(#28) Unit test course_cat_lookup_step_test should be declared as final.
(#28) Missing docblock for class course_cat_lookup_step_test
(#32) Content missing for @var tag in member variable comment
(#38) Content missing for @var tag in member variable comment
(#44) Content missing for @var tag in member variable comment
(#51) There must be a single space between the colon and type in a return type declaration
(#72) Test method test_get_fields() is missing any coverage information, own or at class level
(#72) Test method test_get_fields() is missing a return type
(#97) Test method test_execute_basic() is missing any coverage information, own or at class level
(#97) Test method test_execute_basic() is missing a return type
(#101) There should be a comma after the last array item in a multi-line array.
(#117) Test method test_execute_nosuchfield() is missing any coverage information, own or at class level
(#117) Test method test_execute_nosuchfield() is missing a return type
(#121) There should be a comma after the last array item in a multi-line array.
(#132) Test method test_execute_no_such_category() is missing any coverage information, own or at class level
(#132) Test method test_execute_no_such_category() is missing a return type
(#140) There should be a comma after the last array item in a multi-line array.
(#152) Data provider method "hardcoded_category_id_data_provider" will need to be converted to static in future.
(#152) Data provider method "hardcoded_category_id_data_provider" must return an array, a Generator or an Iterable.
(#183) Test method test_execute_category_id() is missing any coverage information, own or at class level
(#183) Test method test_execute_category_id() is missing a return type
(#187) There should be a comma after the last array item in a multi-line array.
(#212) Test method test_execute_category_id_integer() is missing any coverage information, own or at class level
(#212) Test method test_execute_category_id_integer() is missing a return type
(#216) There should be a comma after the last array item in a multi-line array.
(#232) Test method test_execute_category_id_string() is missing any coverage information, own or at class level
(#232) Test method test_execute_category_id_string() is missing a return type
(#236) There should be a comma after the last array item in a multi-line array.
admin/tool/trigger/tests/course_lookup_step_test.php
(#28) Unit test course_lookup_step_test should be declared as final.
(#28) Missing docblock for class course_lookup_step_test
(#32) Content missing for @var tag in member variable comment
(#38) Content missing for @var tag in member variable comment
(#52) There must be a single space between the colon and type in a return type declaration
(#64) There should be a comma after the last array item in a multi-line array.
(#65) There should be a comma after the last array item in a multi-line array.
(#76) Test method test_execute_basic() is missing any coverage information, own or at class level
(#76) Test method test_execute_basic() is missing a return type
(#80) There should be a comma after the last array item in a multi-line array.
(#96) Test method test_execute_nosuchfield() is missing any coverage information, own or at class level
(#96) Test method test_execute_nosuchfield() is missing a return type
(#100) There should be a comma after the last array item in a multi-line array.
(#111) Test method test_execute_nosuchcourse() is missing any coverage information, own or at class level
(#111) Test method test_execute_nosuchcourse() is missing a return type
(#117) There should be a comma after the last array item in a multi-line array.
(#129) Data provider method "hardcoded_course_id_data_provider" will need to be converted to static in future.
(#129) Data provider method "hardcoded_course_id_data_provider" must return an array, a Generator or an Iterable.
(#160) Test method test_execute_course_id() is missing any coverage information, own or at class level
(#160) Test method test_execute_course_id() is missing a return type
(#164) There should be a comma after the last array item in a multi-line array.
(#189) Test method test_execute_course_id_integer() is missing any coverage information, own or at class level
(#189) Test method test_execute_course_id_integer() is missing a return type
(#193) There should be a comma after the last array item in a multi-line array.
(#209) Test method test_execute_course_id_string() is missing any coverage information, own or at class level
(#209) Test method test_execute_course_id_string() is missing a return type
(#213) There should be a comma after the last array item in a multi-line array.
admin/tool/trigger/tests/datafield_manager_test.php
(#33) Unit test datafield_manager_test should be declared as final.
(#33) Missing docblock for class datafield_manager_test
(#47) Test method test_get_datafields() is missing any coverage information, own or at class level
(#47) Test method test_get_datafields() is missing a return type
(#100) Test method test_render_datafields() is missing any coverage information, own or at class level
(#100) Test method test_render_datafields() is missing a return type
(#145) Test method test_render_datafields_transformcallback() is missing any coverage information, own or at class level
(#145) Test method test_render_datafields_transformcallback() is missing a return type
(#148) There should be a comma after the last array item in a multi-line array.
admin/tool/trigger/tests/debounce_step_test.php
(#33) Unit test debounce_step_test should be declared as final.
(#33) Missing docblock for class debounce_step_test
(#43) Missing @var tag in member variable comment
(#56) There must be a single space between the colon and type in a return type declaration
(#71) Short array syntax must be used to define arrays
(#93) Missing docblock for function get_mock_queue_item in testcase
(#103) There should be a comma after the last array item in a multi-line array.
(#109) Test method test_event_queue() is missing any coverage information, own or at class level
(#109) Test method test_event_queue() is missing a return type
(#123) Inline comments must end in full-stops, exclamation marks, or question marks
(#139) Inline comments must end in full-stops, exclamation marks, or question marks
(#156) Test method test_event_cancellation() is missing any coverage information, own or at class level
(#156) Test method test_event_cancellation() is missing a return type
(#159) Inline comments must end in full-stops, exclamation marks, or question marks
(#192) Inline comments must end in full-stops, exclamation marks, or question marks
(#219) Inline comments must end in full-stops, exclamation marks, or question marks
admin/tool/trigger/tests/email_action_step_test.php
(#33) Unit test email_action_step_test should be declared as final.
(#33) Missing docblock for class email_action_step_test
(#46) There must be a single space between the colon and type in a return type declaration
(#54) Test method test_execute_basic() is missing any coverage information, own or at class level
(#54) Test method test_execute_basic() is missing a return type
(#59) There should be a comma after the last array item in a multi-line array.
(#91) Test method test_execute_external_email_address() is missing any coverage information, own or at class level
(#91) Test method test_execute_external_email_address() is missing a return type
(#96) There should be a comma after the last array item in a multi-line array.
(#122) Test method test_execute_with_datafields() is missing any coverage information, own or at class level
(#122) Test method test_execute_with_datafields() is missing a return type
(#134) There should be a comma after the last array item in a multi-line array.
admin/tool/trigger/tests/event_processor_test.php
(#41) Unit test event_processor_test should be declared as final.
(#41) Missing docblock for class event_processor_test
(#45) Content missing for @var tag in member variable comment
(#51) Content missing for @var tag in member variable comment
(#57) Content missing for @var tag in member variable comment
(#62) Missing docblock for function setup in testcase
(#62) There must be a single space between the colon and type in a return type declaration
(#68) Short array syntax must be used to define arrays
(#87) The tearDown() method in unit tests must always call to parent::tearDown().
(#87) There must be a single space between the colon and type in a return type declaration
(#103) Test method test_is_event_ignored() is missing any coverage information, own or at class level
(#103) Test method test_is_event_ignored() is missing a return type
(#119) Test method test_is_event_ignored_false() is missing any coverage information, own or at class level
(#119) Test method test_is_event_ignored_false() is missing a return type
(#135) Test method test_prepare_event() is missing any coverage information, own or at class level
(#135) Test method test_prepare_event() is missing a return type
(#151) Test method test_process_event_add_event_to_db() is missing any coverage information, own or at class level
(#151) Test method test_process_event_add_event_to_db() is missing a return type
(#167) Test method test_process_realtime_workflow() is missing any coverage information, own or at class level
(#167) Test method test_process_realtime_workflow() is missing a return type
(#190) Test method test_process_realtime_workflow_save_to_queue_if_failed() is missing any coverage information, own or at class level
(#190) Test method test_process_realtime_workflow_save_to_queue_if_failed() is missing a return type
(#203) There should be a comma after the last array item in a multi-line array.
(#223) Test method test_record_workflow_trigger() is missing any coverage information, own or at class level
(#223) Test method test_record_workflow_trigger() is missing a return type
(#263) There should be a comma after the last array item in a multi-line array.
(#280) Test method test_record_step_trigger() is missing any coverage information, own or at class level
(#280) Test method test_record_step_trigger() is missing a return type
(#302) 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.
(#313) There should be a comma after the last array item in a multi-line array.
(#354) There should be a comma after the last array item in a multi-line array.
(#364) There should be a comma after the last array item in a multi-line array.
(#365) There should be a comma after the last array item in a multi-line array.
(#377) Test method test_execute_current_step() is missing any coverage information, own or at class level
(#377) Test method test_execute_current_step() is missing a return type
(#389) 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.
(#442) Test method test_execute_historic_step() is missing any coverage information, own or at class level
(#442) Test method test_execute_historic_step() is missing a return type
(#454) There should be a comma after the last array item in a multi-line array.
(#464) There should be a comma after the last array item in a multi-line array.
(#465) There should be a comma after the last array item in a multi-line array.
(#500) Test method test_execute_next_step_current() is missing any coverage information, own or at class level
(#500) Test method test_execute_next_step_current() is missing a return type
(#512) There should be a comma after the last array item in a multi-line array.
(#522) There should be a comma after the last array item in a multi-line array.
(#523) There should be a comma after the last array item in a multi-line array.
(#557) Test method test_execute_next_step_historic() is missing any coverage information, own or at class level
(#557) Test method test_execute_next_step_historic() is missing a return type
(#569) There should be a comma after the last array item in a multi-line array.
(#579) There should be a comma after the last array item in a multi-line array.
(#580) There should be a comma after the last array item in a multi-line array.
(#609) Test method test_execute_step_and_continue_current() is missing any coverage information, own or at class level
(#609) Test method test_execute_step_and_continue_current() is missing a return type
(#621) There should be a comma after the last array item in a multi-line array.
(#633) There should be a comma after the last array item in a multi-line array.
(#634) There should be a comma after the last array item in a multi-line array.
(#700) There should be a comma after the last array item in a multi-line array.
(#712) There should be a comma after the last array item in a multi-line array.
(#724) There should be a comma after the last array item in a multi-line array.
(#736) There should be a comma after the last array item in a multi-line array.
(#737) There should be a comma after the last array item in a multi-line array.
(#768) Test method test_execute_step_and_continue_historic() is missing any coverage information, own or at class level
(#768) Test method test_execute_step_and_continue_historic() is missing a return type
(#780) There should be a comma after the last array item in a multi-line array.
(#790) There should be a comma after the last array item in a multi-line array.
(#791) There should be a comma after the last array item in a multi-line array.
(#848) There should be a comma after the last array item in a multi-line array.
(#858) There should be a comma after the last array item in a multi-line array.
(#868) There should be a comma after the last array item in a multi-line array.
(#878) There should be a comma after the last array item in a multi-line array.
(#879) There should be a comma after the last array item in a multi-line array.
(#907) Test method test_execute_workflow_from_event_current() is missing any coverage information, own or at class level
(#907) Test method test_execute_workflow_from_event_current() is missing a return type
(#919) There should be a comma after the last array item in a multi-line array.
(#929) There should be a comma after the last array item in a multi-line array.
(#930) There should be a comma after the last array item in a multi-line array.
(#970) Test method test_execute_workflow_from_event_historic() is missing any coverage information, own or at class level
(#970) Test method test_execute_workflow_from_event_historic() is missing a return type
(#982) There should be a comma after the last array item in a multi-line array.
(#992) There should be a comma after the last array item in a multi-line array.
(#993) There should be a comma after the last array item in a multi-line array.
(#1021) Test method test_rerun_all_error_runs() is missing any coverage information, own or at class level
(#1021) Test method test_rerun_all_error_runs() is missing a return type
(#1036) There should be a comma after the last array item in a multi-line array.
(#1046) There should be a comma after the last array item in a multi-line array.
(#1047) There should be a comma after the last array item in a multi-line array.
(#1082) There should be a comma after the last array item in a multi-line array.
(#1118) There should be a comma after the last array item in a multi-line array.
(#1162) Test method test_cleanup_history() is missing any coverage information, own or at class level
(#1162) Test method test_cleanup_history() is missing a return type
admin/tool/trigger/tests/fail_filter_step_test.php
(#28) Unit test fail_filter_step_test should be declared as final.
(#28) Missing docblock for class fail_filter_step_test
(#29) Test method test_execute() is missing any coverage information, own or at class level
(#29) Test method test_execute() is missing a return type
admin/tool/trigger/tests/fixtures/user_event_fixture.php
(#31) Missing docblock for trait tool_trigger_user_event_fixture
(#32) Missing member variable doc comment
(#33) Missing member variable doc comment
(#34) Missing member variable doc comment
(#35) Missing member variable doc comment
(#50) There should be a comma after the last array item in a multi-line array.
(#68) There should be a comma after the last array item in a multi-line array.
admin/tool/trigger/tests/http_post_action_step_test.php
(#28) Unit test http_post_action_step_test should be declared as final.
(#28) Missing docblock for class http_post_action_step_test
(#32) Content missing for @var tag in member variable comment
(#38) Content missing for @var tag in member variable comment
(#44) Content missing for @var tag in member variable comment
(#48) Missing docblock for function setup in testcase
(#48) There must be a single space between the colon and type in a return type declaration
(#60) There should be a comma after the last array item in a multi-line array.
(#61) There should be a comma after the last array item in a multi-line array.
(#68) Missing docblock for function make_mock_http_handler in testcase
(#83) Test method test_supported_http_methods() is missing any coverage information, own or at class level
(#83) Test method test_supported_http_methods() is missing a return type
(#98) Test method test_if_httpmethod_is_not_set_post_method_set_as_default() is missing any coverage information, own or at class level
(#98) Test method test_if_httpmethod_is_not_set_post_method_set_as_default() is missing a return type
(#103) There should be a comma after the last array item in a multi-line array.
(#119) Data provider method "http_methods_data_provider" will need to be converted to static in future.
(#119) Data provider method "http_methods_data_provider" will need to be converted to static in future.
(#135) Test method test_execute_200() is missing any coverage information, own or at class level
(#135) Test method test_execute_200() is missing a return type
(#141) There should be a comma after the last array item in a multi-line array.
(#164) Test method test_execute_404() is missing any coverage information, own or at class level
(#164) Test method test_execute_404() is missing a return type
(#171) There should be a comma after the last array item in a multi-line array.
(#192) Test method test_execute_with_datafields() is missing any coverage information, own or at class level
(#192) Test method test_execute_with_datafields() is missing a return type
(#197) There should be a comma after the last array item in a multi-line array.
(#213) There should be a comma after the last array item in a multi-line array.
(#250) Test method test_execute_with_datafields_json() is missing any coverage information, own or at class level
(#250) Test method test_execute_with_datafields_json() is missing a return type
(#255) There should be a comma after the last array item in a multi-line array.
(#271) There should be a comma after the last array item in a multi-line array.
admin/tool/trigger/tests/json_export_test.php
(#34) Unit test json_export_test should be declared as final.
(#36) Missing docblock for function setup in testcase
(#36) There must be a single space between the colon and type in a return type declaration
(#43) Test method test_set_filename() is missing any coverage information, own or at class level
(#43) Test method test_set_filename() is missing a return type
(#48) Short array syntax must be used to define arrays
(#49) Short array syntax must be used to define arrays
(#59) Short array syntax must be used to define arrays
(#66) There should be a comma after the last array item in a multi-line array.
(#67) There should be a comma after the last array item in a multi-line array.
admin/tool/trigger/tests/learn_process_test.php
(#35) Unit test learn_process_test should be declared as final.
(#35) Missing docblock for class learn_process_test
(#37) Missing docblock for function setup in testcase
(#37) There must be a single space between the colon and type in a return type declaration
(#79) Short array syntax must be used to define arrays
(#99) There should be a comma after the last array item in a multi-line array.
(#108) Test method test_get_learnt_events() is missing any coverage information, own or at class level
(#108) Test method test_get_learnt_events() is missing a return type
(#117) Short array syntax must be used to define arrays
(#119) Short array syntax must be used to define arrays
(#132) Test method test_get_learnt_records() is missing any coverage information, own or at class level
(#132) Test method test_get_learnt_records() is missing a return type
(#135) Short array syntax must be used to define arrays
(#140) Short array syntax must be used to define arrays
(#142) Short array syntax must be used to define arrays
(#164) Test method test_convert_record_type() is missing any coverage information, own or at class level
(#164) Test method test_convert_record_type() is missing a return type
(#182) Test method test_merge_records() is missing any coverage information, own or at class level
(#182) Test method test_merge_records() is missing a return type
(#187) Short array syntax must be used to define arrays
(#204) Test method test_merge_db_record() is missing any coverage information, own or at class level
(#204) Test method test_merge_db_record() is missing a return type
(#241) Test method test_store_json_fields_insert() is missing any coverage information, own or at class level
(#241) Test method test_store_json_fields_insert() is missing a return type
(#254) Short array syntax must be used to define arrays
(#264) Test method test_store_json_fields_update() is missing any coverage information, own or at class level
(#264) Test method test_store_json_fields_update() is missing a return type
(#283) Short array syntax must be used to define arrays
(#293) Test method test_get_event_fields_with_type() is missing any coverage information, own or at class level
(#293) Test method test_get_event_fields_with_type() is missing a return type
(#312) Short array syntax must be used to define arrays
(#314) Short array syntax must be used to define arrays
(#319) Short array syntax must be used to define arrays
(#324) Short array syntax must be used to define arrays
(#329) Short array syntax must be used to define arrays
(#334) Short array syntax must be used to define arrays
(#339) Short array syntax must be used to define arrays
(#344) Short array syntax must be used to define arrays
(#349) Short array syntax must be used to define arrays
(#354) Short array syntax must be used to define arrays
(#359) Short array syntax must be used to define arrays
(#364) Short array syntax must be used to define arrays
(#369) Short array syntax must be used to define arrays
(#374) Short array syntax must be used to define arrays
(#379) Short array syntax must be used to define arrays
(#384) Short array syntax must be used to define arrays
(#389) Short array syntax must be used to define arrays
(#394) Short array syntax must be used to define arrays
(#399) Short array syntax must be used to define arrays
(#404) Short array syntax must be used to define arrays
(#409) Short array syntax must be used to define arrays
(#414) Short array syntax must be used to define arrays
(#416) There should be a comma after the last array item in a multi-line array.
(#419) Short array syntax must be used to define arrays
(#421) There should be a comma after the last array item in a multi-line array.
(#432) Test method test_get_event_fields_events() is missing any coverage information, own or at class level
(#432) Test method test_get_event_fields_events() is missing a return type
(#457) Test method test_get_event_fields_json() is missing any coverage information, own or at class level
(#457) Test method test_get_event_fields_json() is missing a return type
(#482) Test method test_process_fixtures() is missing any coverage information, own or at class level
(#482) Test method test_process_fixtures() is missing a return type
admin/tool/trigger/tests/logdump_action_step_test.php
(#28) Unit test logdump_action_step_test should be declared as final.
(#28) Missing docblock for class logdump_action_step_test
(#29) Test method test_execute() is missing any coverage information, own or at class level
(#29) Test method test_execute() is missing a return type
(#43) There should be a comma after the last array item in a multi-line array.
(#44) There should be a comma after the last array item in a multi-line array.
(#49) There should be a comma after the last array item in a multi-line array.
admin/tool/trigger/tests/numcompare_filter_step_test.php
(#35) Unit test numcompare_filter_step_test should be declared as final.
(#35) Missing docblock for class numcompare_filter_step_test
(#47) There must be a single space between the colon and type in a return type declaration
(#61) Short array syntax must be used to define arrays
(#75) Test method test_all_operators() is missing any coverage information, own or at class level
(#75) Test method test_all_operators() is missing a return type
(#80) There should be a comma after the last array item in a multi-line array.
(#98) Data provider method "operator_permutations" will need to be converted to static in future.
(#98) Data provider method "operator_permutations" must return an array, a Generator or an Iterable.
(#117) There should be a comma after the last array item in a multi-line array.
(#121) Test method test_datafield() is missing any coverage information, own or at class level
(#121) Test method test_datafield() is missing a return type
(#128) There should be a comma after the last array item in a multi-line array.
(#137) Test method test_nosuch_datafield() is missing any coverage information, own or at class level
(#137) Test method test_nosuch_datafield() is missing a return type
(#142) There should be a comma after the last array item in a multi-line array.
(#150) Test method test_not_numeric() is missing any coverage information, own or at class level
(#150) Test method test_not_numeric() is missing a return type
(#155) There should be a comma after the last array item in a multi-line array.
admin/tool/trigger/tests/privacy_test.php
(#39) Unit test privacy_test should be declared as final.
(#44) The setUp() method in unit tests must always call to parent::setUp().
(#44) There must be a single space between the colon and type in a return type declaration
(#51) Test method test_get_users_in_context() is missing any coverage information, own or at class level
(#51) Test method test_get_users_in_context() is missing a return type
(#102) Test method test_delete_data_for_users() is missing any coverage information, own or at class level
(#102) Test method test_delete_data_for_users() is missing a return type
admin/tool/trigger/tests/process_workflows_test.php
(#35) Unit test process_workflows_test should be declared as final.
(#35) Missing docblock for class process_workflows_test
(#39) Content missing for @var tag in member variable comment
(#45) Content missing for @var tag in member variable comment
(#51) Content missing for @var tag in member variable comment
(#57) Content missing for @var tag in member variable comment
(#63) Missing docblock for function setup in testcase
(#63) There must be a single space between the colon and type in a return type declaration
(#110) Test method test_process_workflow() is missing any coverage information, own or at class level
(#110) Test method test_process_workflow() is missing a return type
(#130) There should be a comma after the last array item in a multi-line array.
(#142) There should be a comma after the last array item in a multi-line array.
(#143) There should be a comma after the last array item in a multi-line array.
(#147) Inline comments must end in full-stops, exclamation marks, or question marks
admin/tool/trigger/tests/processor_helper_test.php
(#33) Unit test processor_helper_test should be declared as final.
(#33) Missing docblock for class processor_helper_test
(#37) Content missing for @var tag in member variable comment
(#44) There must be a single space between the colon and type in a return type declaration
(#56) Test method test_restore_event() is missing any coverage information, own or at class level
(#56) Test method test_restore_event() is missing a return type
(#97) Short array syntax must be used to define arrays
(#104) Inline comments must end in full-stops, exclamation marks, or question marks
(#111) Test method test_execute_step() is missing any coverage information, own or at class level
(#111) Test method test_execute_step() is missing a return type
(#119) Short array syntax must be used to define arrays
(#138) Test method test_get_workflow_steps() is missing any coverage information, own or at class level
(#138) Test method test_get_workflow_steps() is missing a return type
(#156) Test method test_update_workflow_record() is missing any coverage information, own or at class level
(#156) Test method test_update_workflow_record() is missing a return type
(#189) Test method test_insert_queue_records() is missing any coverage information, own or at class level
(#189) Test method test_insert_queue_records() is missing a return type
(#210) Test method test_get_event_record() is missing any coverage information, own or at class level
(#210) Test method test_get_event_record() is missing a return type
(#216) Short array syntax must be used to define arrays
admin/tool/trigger/tests/role_assign_action_step_test.php
(#27) Unit test role_assign_action_step_test should be declared as final.
(#31) Content missing for @var tag in member variable comment
(#37) Content missing for @var tag in member variable comment
(#43) Content missing for @var tag in member variable comment
(#49) Content missing for @var tag in member variable comment
(#55) Content missing for @var tag in member variable comment
(#62) There must be a single space between the colon and type in a return type declaration
(#81) There should be a comma after the last array item in a multi-line array.
(#92) Test method test_get_fields() is missing any coverage information, own or at class level
(#92) Test method test_get_fields() is missing a return type
(#103) Test method test_execute_basic() is missing any coverage information, own or at class level
(#103) Test method test_execute_basic() is missing a return type
(#124) Test method test_execute_placeholder() is missing any coverage information, own or at class level
(#124) Test method test_execute_placeholder() is missing a return type
(#145) Test method test_execute_nosuchfield() is missing any coverage information, own or at class level
(#145) Test method test_execute_nosuchfield() is missing a return type
admin/tool/trigger/tests/role_unassign_action_step_test.php
(#27) Unit test role_unassign_action_step_test should be declared as final.
(#31) Content missing for @var tag in member variable comment
(#37) Content missing for @var tag in member variable comment
(#43) Content missing for @var tag in member variable comment
(#49) Content missing for @var tag in member variable comment
(#55) Content missing for @var tag in member variable comment
(#62) There must be a single space between the colon and type in a return type declaration
(#81) There should be a comma after the last array item in a multi-line array.
(#94) Test method test_get_fields() is missing any coverage information, own or at class level
(#94) Test method test_get_fields() is missing a return type
(#105) Test method test_execute_basic() is missing any coverage information, own or at class level
(#105) Test method test_execute_basic() is missing a return type
(#127) Test method test_execute_placeholder() is missing any coverage information, own or at class level
(#127) Test method test_execute_placeholder() is missing a return type
(#148) Test method test_execute_nosuchfield() is missing any coverage information, own or at class level
(#148) Test method test_execute_nosuchfield() is missing a return type
admin/tool/trigger/tests/steps_form_test.php
(#30) Testcase tool_trigger_steps_form_testcase should be declared as abstract.
(#30) PHPUnit testcase name "tool_trigger_steps_form_testcase" does not match file name "steps_form_test"
(#30) PHPUnit class "tool_trigger_steps_form_testcase" does not have any namespace. It is recommended to add it to the "tool_trigger" namespace, using more levels if needed, in order to match the code being tested
(#30) Missing docblock for class tool_trigger_steps_form_testcase
(#32) The setUp() method in unit tests must always call to parent::setUp().
(#32) There must be a single space between the colon and type in a return type declaration
(#37) The tearDown() method in unit tests must always call to parent::tearDown().
(#37) There must be a single space between the colon and type in a return type declaration
(#48) Short array syntax must be used to define arrays
(#68) There should be a comma after the last array item in a multi-line array.
(#77) Test method test_base_form() is missing any coverage information, own or at class level
(#77) Test method test_base_form() is missing a return type
(#80) There should be a comma after the last array item in a multi-line array.
(#103) Test method test_step_form() is missing any coverage information, own or at class level
(#103) Test method test_step_form() is missing a return type
(#114) There should be a comma after the last array item in a multi-line array.
(#127) Test method test_get_trigger_fields() is missing any coverage information, own or at class level
(#127) Test method test_get_trigger_fields() is missing a return type
(#150) Short array syntax must be used to define arrays
(#154) Short array syntax must be used to define arrays
(#156) Short array syntax must be used to define arrays
(#158) Short array syntax must be used to define arrays
(#163) Short array syntax must be used to define arrays
(#168) Short array syntax must be used to define arrays
(#173) Short array syntax must be used to define arrays
(#178) Short array syntax must be used to define arrays
(#183) Short array syntax must be used to define arrays
(#188) Short array syntax must be used to define arrays
(#193) Short array syntax must be used to define arrays
(#198) Short array syntax must be used to define arrays
(#203) Short array syntax must be used to define arrays
(#208) Short array syntax must be used to define arrays
(#213) Short array syntax must be used to define arrays
(#218) Short array syntax must be used to define arrays
(#223) Short array syntax must be used to define arrays
(#228) Short array syntax must be used to define arrays
(#233) Short array syntax must be used to define arrays
(#238) Short array syntax must be used to define arrays
(#243) Short array syntax must be used to define arrays
(#248) Short array syntax must be used to define arrays
(#253) Short array syntax must be used to define arrays
(#258) Short array syntax must be used to define arrays
(#263) Short array syntax must be used to define arrays
(#268) Short array syntax must be used to define arrays
(#270) There should be a comma after the last array item in a multi-line array.
(#273) Short array syntax must be used to define arrays
(#275) 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
(#278) There should be a comma after the last array item in a multi-line array.
admin/tool/trigger/tests/stringcompare_filter_step_test.php
(#36) Unit test stringcompare_filter_step_test should be declared as final.
(#36) Missing docblock for class stringcompare_filter_step_test
(#44) There must be a single space between the colon and type in a return type declaration
(#58) Test method test_all_operators() is missing any coverage information, own or at class level
(#58) Test method test_all_operators() is missing a return type
(#63) There should be a comma after the last array item in a multi-line array.
(#90) Data provider method "operator_permutations" will need to be converted to static in future.
(#90) Data provider method "operator_permutations" must return an array, a Generator or an Iterable.
(#105) There should be a comma after the last array item in a multi-line array.
(#112) Test method test_datafield() is missing any coverage information, own or at class level
(#112) Test method test_datafield() is missing a return type
(#120) There should be a comma after the last array item in a multi-line array.
(#129) Test method test_invalidregex() is missing any coverage information, own or at class level
(#129) Test method test_invalidregex() is missing a return type
(#135) There should be a comma after the last array item in a multi-line array.
admin/tool/trigger/tests/tool_trigger_testcase.php
(#37) Missing docblock for class tool_trigger_testcase
(#57) There should be a comma after the last array item in a multi-line array.
(#75) Inline comments must end in full-stops, exclamation marks, or question marks
admin/tool/trigger/tests/user_lookup_step_test.php
(#33) Unit test user_lookup_step_test should be declared as final.
(#33) Missing docblock for class user_lookup_step_test
(#40) There must be a single space between the colon and type in a return type declaration
(#49) Test method test_execute_basic() is missing any coverage information, own or at class level
(#49) Test method test_execute_basic() is missing a return type
(#54) There should be a comma after the last array item in a multi-line array.
(#68) Test method test_execute_basic_with_custom_profile_fields() is missing any coverage information, own or at class level
(#68) Test method test_execute_basic_with_custom_profile_fields() is missing a return type
(#82) There should be a comma after the last array item in a multi-line array.
(#100) Test method test_execute_relateduser() is missing any coverage information, own or at class level
(#100) Test method test_execute_relateduser() is missing a return type
(#105) There should be a comma after the last array item in a multi-line array.
(#119) Test method test_execute_nosuchfield() is missing any coverage information, own or at class level
(#119) Test method test_execute_nosuchfield() is missing a return type
(#124) There should be a comma after the last array item in a multi-line array.
(#135) Test method test_execute_deleted_user() is missing any coverage information, own or at class level
(#135) Test method test_execute_deleted_user() is missing a return type
(#145) There should be a comma after the last array item in a multi-line array.
(#158) There should be a comma after the last array item in a multi-line array.
admin/tool/trigger/tests/webservice_action_step_test.php
(#32) Unit test webservice_action_step_test should be declared as final.
(#47) Test method test_with_valid_call_to_enrol_user() is missing any coverage information, own or at class level
(#47) Test method test_with_valid_call_to_enrol_user() is missing a return type
(#89) Test method test_with_invalid_username() is missing any coverage information, own or at class level
(#89) Test method test_with_invalid_username() is missing a return type
(#105) Test method test_with_non_existent_function() is missing any coverage information, own or at class level
(#105) Test method test_with_non_existent_function() is missing a return type
(#115) Inline comments must end in full-stops, exclamation marks, or question marks
(#125) Test method test_with_invalid_function_parameters() is missing any coverage information, own or at class level
(#125) Test method test_with_invalid_function_parameters() is missing a return type
admin/tool/trigger/tests/workflow_manager_test.php
(#34) Unit test workflow_manager_test should be declared as final.
(#39) Test method test_get_step_class_names() is missing any coverage information, own or at class level
(#39) Test method test_get_step_class_names() is missing a return type
(#53) Test method test_lookup_step_names() is missing any coverage information, own or at class level
(#53) Test method test_lookup_step_names() is missing a return type
(#55) Short array syntax must be used to define arrays
(#70) Test method test_get_steps_by_type() is missing any coverage information, own or at class level
(#70) Test method test_get_steps_by_type() is missing a return type
(#88) Test method test_validate_step_class_good() is missing any coverage information, own or at class level
(#88) Test method test_validate_step_class_good() is missing a return type
(#100) Test method test_validate_step_class_bad() is missing any coverage information, own or at class level
(#100) Test method test_validate_step_class_bad() is missing a return type
(#113) Test method test_get_workflow_data_with_steps() is missing any coverage information, own or at class level
(#113) Test method test_get_workflow_data_with_steps() is missing a return type
(#136) Short array syntax must be used to define arrays
(#146) Test method test_cleanup() is missing any coverage information, own or at class level
(#146) Test method test_cleanup() is missing a return type
admin/tool/trigger/tests/workflow_process_test.php
(#26) Unit test workflow_process_test should be declared as final.
(#31) Test method test_processform() is missing any coverage information, own or at class level
(#31) Test method test_processform() is missing a return type
(#52) Short array syntax must be used to define arrays
(#53) Short array syntax must be used to define arrays
(#63) Test method test_processjson() is missing any coverage information, own or at class level
(#63) Test method test_processjson() is missing a return type
(#90) Test method test_processjson_multiple_steps() is missing any coverage information, own or at class level
(#90) Test method test_processjson_multiple_steps() is missing a return type
(#131) Test method test_import_prep() is missing any coverage information, own or at class level
(#131) Test method test_import_prep() is missing a return type
(#131) Expected 0 spaces after "test_import_prep"; 1 found
(#144) Short array syntax must be used to define arrays
(#145) Short array syntax must be used to define arrays
(#155) There should be a comma after the last array item in a multi-line array.
(#157) 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.
(#165) There should be a comma after the last array item in a multi-line array.

Javascript coding style problems

(0 errors, 33 warnings)

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

admin/tool/trigger/amd/src/import_workflow.js
(#27) A space is required after ','. (comma-spacing)
(#29) Unexpected space before function parentheses. (space-before-function-paren)
(#77) Multiple spaces found before '// We're lazy ...'. (no-multi-spaces)
(#111) Expected catch,fail,always() or return (promise/catch-or-return)
(#111) Each then() should return a value or throw (promise/always-return)
admin/tool/trigger/amd/src/step_select.js
(#27) A space is required after ','. (comma-spacing)
(#29) Unexpected space before function parentheses. (space-before-function-paren)
(#99) Each then() should return a value or throw (promise/always-return)
(#144) ["stepdesc"] is better written in dot notation. (dot-notation)
(#145) ["typedesc"] is better written in dot notation. (dot-notation)
(#151) ["stepclass"] is better written in dot notation. (dot-notation)
(#176) ["type"] is better written in dot notation. (dot-notation)
(#176) ["stepclass"] is better written in dot notation. (dot-notation)
(#192) Extra space after key 'text'. (key-spacing)
(#196) Unexpected space before function parentheses. (space-before-function-paren)
(#199) Extra space after key 'text'. (key-spacing)
(#212) There should be no space after '{'. (@babel/object-curly-spacing)
(#212) There should be no space before '}'. (@babel/object-curly-spacing)
(#259) Extra space after key 'steptype'. (key-spacing)
(#260) Extra space after key 'stepclass'. (key-spacing)
(#295) Missing space before opening brace. (space-before-blocks)
(#315) Missing space before opening brace. (space-before-blocks)
(#319) Missing space before opening brace. (space-before-blocks)
(#323) Missing space before opening brace. (space-before-blocks)
(#325) Missing space before opening brace. (space-before-blocks)
(#329) Missing space before opening brace. (space-before-blocks)
(#342) Missing space before opening brace. (space-before-blocks)
(#368) There should be no space before ','. (comma-spacing)
(#402) ["type"] is better written in dot notation. (dot-notation)
(#403) ["stepclass"] is better written in dot notation. (dot-notation)
(#429) Missing space before opening brace. (space-before-blocks)
(#449) Expected catch,fail,always() or return (promise/catch-or-return)
(#449) Each then() should return a value or throw (promise/always-return)

CSS problems

(0 errors, 0 warnings)

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


PHPDocs style problems

(59 errors, 0 warnings)

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

admin/tool/trigger/classes/event_processor.php
(#356) Phpdocs for function event_processor::record_failed_step has incomplete parameters list
(#419) Phpdocs for function event_processor::execute_current_step has incomplete parameters list
(#461) Phpdocs for function event_processor::execute_next_step_historic has incomplete parameters list
(#542) Phpdocs for function event_processor::execute_next_step_current has incomplete parameters list
(#597) Phpdocs for function event_processor::execute_step_and_continue_historic has incomplete parameters list
(#642) Phpdocs for function event_processor::execute_step_and_continue_current has incomplete parameters list
admin/tool/trigger/classes/output/workflowhistory/renderer.php
(#269) Phpdocs for function renderer::rerun_all_historic_button has incomplete parameters list
(#281) Phpdocs for function renderer::rerun_all_current_button has incomplete parameters list
admin/tool/trigger/classes/steps/actions/assign_cohort_action_step.php
(#75) Phpdocs for function assign_cohort_action_step::execute has incomplete parameters list
(#90) Phpdocs for function assign_cohort_action_step::form_definition_extra has incomplete parameters list
admin/tool/trigger/classes/steps/actions/email_action_step.php
(#109) Phpdocs for function email_action_step::execute has incomplete parameters list
(#175) Phpdocs for function email_action_step::form_definition_extra has incomplete parameters list
(#212) Phpdocs for function email_action_step::add_privacy_metadata has incomplete parameters list
admin/tool/trigger/classes/steps/actions/http_post_action_step.php
(#124) Phpdocs for function http_post_action_step::execute has incomplete parameters list
(#179) Phpdocs for function http_post_action_step::form_definition_extra has incomplete parameters list
(#226) Phpdocs for function http_post_action_step::add_privacy_metadata has incomplete parameters list
admin/tool/trigger/classes/steps/actions/role_assign_action_step.php
(#82) Phpdocs for function role_assign_action_step::execute has incomplete parameters list
(#131) Phpdocs for function role_assign_action_step::form_definition_extra has incomplete parameters list
admin/tool/trigger/classes/steps/actions/role_unassign_action_step.php
(#83) Phpdocs for function role_unassign_action_step::execute has incomplete parameters list
(#128) Phpdocs for function role_unassign_action_step::form_definition_extra has incomplete parameters list
admin/tool/trigger/classes/steps/actions/roles_unassign_action_step.php
(#85) Phpdocs for function roles_unassign_action_step::execute has incomplete parameters list
(#107) Phpdocs for function roles_unassign_action_step::form_definition_extra has incomplete parameters list
admin/tool/trigger/classes/steps/actions/webservice_action_step.php
(#123) Phpdocs for function webservice_action_step::execute has incomplete parameters list
(#183) Phpdocs for function webservice_action_step::form_definition_extra has incomplete parameters list
(#208) Phpdocs for function webservice_action_step::add_privacy_metadata has incomplete parameters list
(#229) Phpdocs for function webservice_action_step::form_validation has incomplete parameters list
(#296) Phpdocs for function webservice_action_step::transform_form_data has incomplete parameters list
admin/tool/trigger/classes/steps/base/base_form.php
(#51) Phpdocs for function base_form::__construct has incomplete parameters list
(#106) Phpdocs for function base_form::get_trigger_fields has incomplete parameters list
(#237) Phpdocs for function base_form::validation has incomplete parameters list
admin/tool/trigger/classes/steps/debounce/debounce_step.php
(#79) Phpdocs for function debounce_step::execute has incomplete parameters list
(#241) Phpdocs for function debounce_step::make_form has incomplete parameters list
(#257) Phpdocs for function debounce_step::form_definition_extra has incomplete parameters list
admin/tool/trigger/classes/steps/filters/fail_filter_step.php
(#39) Phpdocs for function fail_filter_step::execute has incomplete parameters list
(#63) Phpdocs for function fail_filter_step::form_definition_extra has incomplete parameters list
admin/tool/trigger/classes/steps/filters/numcompare_filter_step.php
(#62) Phpdocs for function numcompare_filter_step::execute has incomplete parameters list
(#143) Phpdocs for function numcompare_filter_step::form_definition_extra has incomplete parameters list
admin/tool/trigger/classes/steps/filters/stringcompare_filter_step.php
(#127) Phpdocs for function stringcompare_filter_step::execute has incomplete parameters list
(#200) Phpdocs for function stringcompare_filter_step::form_definition_extra has incomplete parameters list
admin/tool/trigger/classes/steps/lookups/cohort_lookup_step.php
(#68) Phpdocs for function cohort_lookup_step::execute has incomplete parameters list
(#92) Phpdocs for function cohort_lookup_step::form_definition_extra has incomplete parameters list
admin/tool/trigger/classes/steps/lookups/course_cat_lookup_step.php
(#72) Phpdocs for function course_cat_lookup_step::execute has incomplete parameters list
(#113) Phpdocs for function course_cat_lookup_step::form_definition_extra has incomplete parameters list
admin/tool/trigger/classes/steps/lookups/course_lookup_step.php
(#90) Phpdocs for function course_lookup_step::execute has incomplete parameters list
(#131) Phpdocs for function course_lookup_step::form_definition_extra has incomplete parameters list
admin/tool/trigger/classes/steps/lookups/roles_lookup_step.php
(#57) Phpdocs for function roles_lookup_step::execute has incomplete parameters list
(#84) Phpdocs for function roles_lookup_step::form_definition_extra has incomplete parameters list
admin/tool/trigger/classes/steps/lookups/user_lookup_step.php
(#122) Phpdocs for function user_lookup_step::execute has incomplete parameters list
(#168) Phpdocs for function user_lookup_step::form_definition_extra has incomplete parameters list
admin/tool/trigger/classes/task/process_workflows.php
(#87) Phpdocs for function process_workflows::create_trigger_queue has incomplete parameters list
admin/tool/trigger/classes/workflow_manager.php
(#152) Phpdocs for function workflow_manager::export_workflow_and_run_history has incomplete parameters list
admin/tool/trigger/externallib.php
(#49) Phpdocs for function tool_trigger_external::step_by_type has incomplete parameters list
(#120) Phpdocs for function tool_trigger_external::validate_form has incomplete parameters list
(#180) Phpdocs for function tool_trigger_external::process_import_form has incomplete parameters list
admin/tool/trigger/lib.php
(#32) Phpdocs for function tool_trigger_output_fragment_new_base_form has incomplete parameters list
(#53) Phpdocs for function tool_trigger_output_fragment_new_step_form has incomplete parameters list
admin/tool/trigger/tests/course_cat_lookup_step_test.php
(#178) Phpdocs for function course_cat_lookup_step_test::test_execute_category_id has incomplete parameters list
admin/tool/trigger/tests/course_lookup_step_test.php
(#155) Phpdocs for function course_lookup_step_test::test_execute_course_id has incomplete parameters list
admin/tool/trigger/tests/tool_trigger_testcase.php
(#42) Phpdocs for function tool_trigger_testcase::create_workflow 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]