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

(31 errors, 144 warnings)

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

mod/scormremote/backup/moodle2/backup_scormremote_stepslib.php
(#43) Short array syntax must be used to define arrays
(#43) Short array syntax must be used to define arrays
(#47) Short array syntax must be used to define arrays
(#50) Inline comments must end in full-stops, exclamation marks, or question marks
(#51) Inline comments must end in full-stops, exclamation marks, or question marks
mod/scormremote/backup/moodle2/restore_scormremote_activity_task.class.php
(#56) Short array syntax must be used to define arrays
(#58) Short array syntax must be used to define arrays
(#68) Short array syntax must be used to define arrays
(#84) Short array syntax must be used to define arrays
(#101) Short array syntax must be used to define arrays
mod/scormremote/backup/moodle2/restore_scormremote_stepslib.php
(#41) Short array syntax must be used to define arrays
mod/scormremote/classes/client.php
(#37) Short array syntax must be used to define arrays
(#38) Short array syntax must be used to define arrays
(#42) Short array syntax must be used to define arrays
(#47) Short array syntax must be used to define arrays
mod/scormremote/classes/client_domain.php
(#38) Short array syntax must be used to define arrays
(#39) Short array syntax must be used to define arrays
(#41) There should be a comma after the last array item in a multi-line array.
(#43) Short array syntax must be used to define arrays
(#86) Short array syntax must be used to define arrays
mod/scormremote/classes/course_tier.php
(#37) Short array syntax must be used to define arrays
(#38) Short array syntax must be used to define arrays
(#42) 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.
(#57) Short array syntax must be used to define arrays
(#68) Short array syntax must be used to define arrays
mod/scormremote/classes/event/new_seat_allocated.php
(#75) Short array syntax must be used to define arrays
mod/scormremote/classes/event/remote_view_error.php
(#75) Short array syntax must be used to define arrays
mod/scormremote/classes/event/remote_viewed.php
(#78) Short array syntax must be used to define arrays
mod/scormremote/classes/form/client.php
(#16) Boilerplate comment must be followed by a single blank line or end of file
(#37) Short array syntax must be used to define arrays
(#60) Short array syntax must be used to define arrays
(#64) Short array syntax must be used to define arrays
(#71) Short array syntax must be used to define arrays
(#83) Short array syntax must be used to define arrays
(#92) Short array syntax must be used to define arrays
(#119) Short array syntax must be used to define arrays
(#138) Short array syntax must be used to define arrays
mod/scormremote/classes/form/tier.php
(#16) Boilerplate comment must be followed by a single blank line or end of file
(#34) Short array syntax must be used to define arrays
(#57) Short array syntax must be used to define arrays
(#61) Short array syntax must be used to define arrays
mod/scormremote/classes/form/wrapper.php
(#16) Boilerplate comment must be followed by a single blank line or end of file
(#74) Short array syntax must be used to define arrays
(#81) Short array syntax must be used to define arrays
(#85) There should be a comma after the last array item in a multi-line array.
mod/scormremote/classes/packagefile.php
(#57) There should be a comma after the last array item in a multi-line array.
mod/scormremote/classes/privacy/provider.php
(#28) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
(#115) There should be a comma after the last array item in a multi-line array.
mod/scormremote/classes/subscription.php
(#37) Short array syntax must be used to define arrays
(#38) Short array syntax must be used to define arrays
(#42) 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.
(#128) Short array syntax must be used to define arrays
(#139) Short array syntax must be used to define arrays
mod/scormremote/classes/tier.php
(#37) Short array syntax must be used to define arrays
(#38) Short array syntax must be used to define arrays
(#42) 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.
(#46) Short array syntax must be used to define arrays
(#48) There should be a comma after the last array item in a multi-line array.
mod/scormremote/classes/utils.php
(#78) Short array syntax must be used to define arrays
mod/scormremote/clients.php
(#34) Import statements must not begin with a leading backslash
(#35) Import statements must not begin with a leading backslash
(#36) Import statements must not begin with a leading backslash
(#39) Import statements must not begin with a leading backslash
(#125) Short array syntax must be used to define arrays
(#132) Short array syntax must be used to define arrays
(#190) Short array syntax must be used to define arrays
mod/scormremote/db/access.php
(#36) There should be a comma after the last array item in a multi-line array.
(#38) There should be a comma after the last array item in a multi-line array.
mod/scormremote/db/upgrade.php
(#213) Inline comments must end in full-stops, exclamation marks, or question marks
(#224) Inline comments must end in full-stops, exclamation marks, or question marks
mod/scormremote/error.php
(#34) Short array syntax must be used to define arrays
mod/scormremote/index.php
(#32) Short array syntax must be used to define arrays
(#37) Short array syntax must be used to define arrays
(#52) Short array syntax must be used to define arrays
(#60) Short array syntax must be used to define arrays
(#61) Short array syntax must be used to define arrays
(#63) Short array syntax must be used to define arrays
(#64) Short array syntax must be used to define arrays
(#66) Short array syntax must be used to define arrays
(#67) Short array syntax must be used to define arrays
(#73) Short array syntax must be used to define arrays
(#75) Short array syntax must be used to define arrays
(#78) Short array syntax must be used to define arrays
(#83) Short array syntax must be used to define arrays
(#85) Short array syntax must be used to define arrays
mod/scormremote/lang/en/scormremote.php
(#30) The string key "modulename" is not in the correct order, it should be before "pluginname"
(#33) The string key "client" is not in the correct order, it should be before "pluginadministration"
(#38) The string key "chooseclient" is not in the correct order, it should be before "expiry"
(#41) The string key "optionalsettings" is not in the correct order, it should be before "searchclient"
(#42) The string key "chooseclientdesc" is not in the correct order, it should be before "optionalsettings"
(#47) The string key "manage_clients" is not in the correct order, it should be before "manage_domains_desc"
(#48) The string key "manage_clientdetails" is not in the correct order, it should be before "manage_clients"
(#50) The string key "manage_clientdeletesuccess" is not in the correct order, it should be before "manage_clientupdateheader"
(#51) The string key "manage_clientcreateheader" is not in the correct order, it should be before "manage_clientdeletesuccess"
(#52) The string key "manage_clientadd" is not in the correct order, it should be before "manage_clientcreateheader"
(#55) The string key "manage_clientdomain" is not in the correct order, it should be before "manage_clientname"
(#60) The string key "manage_tieradd" is not in the correct order, it should be before "manage_tiersdescription"
(#67) The string key "manage_tierscmexplaination" is not in the correct order, it should be before "manage_tierseats"
(#68) The string key "filename" is not in the correct order, it should be before "manage_tierscmexplaination"
(#71) The string key "filenameother_error" is not in the correct order, it should be before "filenameother_help"
(#77) The string key "wrappedpackagefile" is not in the correct order, it should be before "wrapper"
(#78) The string key "error_clientnotfound" is not in the correct order, it should be before "wrappedpackagefile"
(#79) The string key "error_clientnamelength" is not in the correct order, it should be before "error_clientnotfound"
(#81) The string key "error_clientdomainnotvalid" is not in the correct order, it should be before "error_clientnamenotvalid"
(#82) The string key "error_clientdomainnotunique" is not in the correct order, it should be before "error_clientdomainnotvalid"
(#87) The string key "error_tiernamelength" is not in the correct order, it should be before "error_tiernotfound"
(#89) The string key "error_imsmanifestmissing" is not in the correct order, it should be before "error_tierseatnumber"
(#92) The string key "event_remoteviewed_name" is not in the correct order, it should be before "event_seatallocated_name"
(#93) The string key "event_nomanualenrolmentinstance" is not in the correct order, it should be before "event_remoteviewed_name"
(#94) The string key "event_missingmanualenrolment" is not in the correct order, it should be before "event_nomanualenrolmentinstance"
(#98) The string key "event_scormviewed" is not in the correct order, it should be before "event_seatlimitreached"
(#102) The string key "event_clientidrequired" is not in the correct order, it should be before "event_unknownpair"
(#105) The string key "scormremote:deleteclient" is not in the correct order, it should be before "scormremote:manageclient"
(#107) The string key "scormremote:managetier" is not in the correct order, it should be before "scormremote:viewclient"
(#108) The string key "scormremote:deletetier" is not in the correct order, it should be before "scormremote:managetier"
(#110) The string key "scormremote:downloadwrapper" is not in the correct order, it should be before "scormremote:viewtier"
(#111) The string key "errorpage_badrequesttitle" is not in the correct order, it should be before "scormremote:downloadwrapper"
(#112) The string key "errorpage_badrequestmessage" is not in the correct order, it should be before "errorpage_badrequesttitle"
(#114) The string key "errorpage_expiredmessage" is not in the correct order, it should be before "errorpage_expiredtitle"
(#116) The string key "errorpage_nomanualenrolmentinstancemessage" is not in the correct order, it should be before "errorpage_nomanualenrolmentinstancetitle"
(#118) The string key "errorpage_unauthorizedmessage" is not in the correct order, it should be before "errorpage_unauthorizedtitle"
(#119) The string key "errorpage_clientidrequiredtitle" is not in the correct order, it should be before "errorpage_unauthorizedmessage"
(#120) The string key "errorpage_clientidrequiredmessage" is not in the correct order, it should be before "errorpage_clientidrequiredtitle"
(#122) The string key "errorpage_subrequiredmessage" is not in the correct order, it should be before "errorpage_subrequiredtitle"
(#123) The string key "errorpage_sublimittitle" is not in the correct order, it should be before "errorpage_subrequiredmessage"
(#124) The string key "errorpage_sublimitmessage" is not in the correct order, it should be before "errorpage_sublimittitle"
(#139) The string key "privacy:metadata:mod_scormremote:scormremote" is not in the correct order, it should be before "settings:validationtype:referrerandclient"
(#140) The string key "privacy:metadata:mod_scormremote:course" is not in the correct order, it should be before "privacy:metadata:mod_scormremote:scormremote"
(#144) The string key "privacy:metadata:mod_scormremote:intro" is not in the correct order, it should be before "privacy:metadata:mod_scormremote:sha1hash"
(#147) The string key "privacy:metadata:mod_scormremote:name" is duplicated
(#148) The string key "privacy:metadata:mod_scormremote:primarydomain" is not in the correct order, it should be before "privacy:metadata:mod_scormremote:scormremote_clients"
(#150) The string key "privacy:metadata:mod_scormremote:clientid" is not in the correct order, it should be before "privacy:metadata:mod_scormremote:scormremote_client_domains"
(#153) The string key "privacy:metadata:mod_scormremote:name" is duplicated
mod/scormremote/lib.php
(#65) Short array syntax must be used to define arrays
(#131) Short array syntax must be used to define arrays
(#136) Short array syntax must be used to define arrays
(#155) Short array syntax must be used to define arrays
(#193) There should be a comma after the last array item in a multi-line array.
(#216) There should be a comma after the last array item in a multi-line array.
(#245) There should be a comma after the last array item in a multi-line array.
(#263) There should be a comma after the last array item in a multi-line array.
(#289) 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.
(#332) There should be a comma after the last array item in a multi-line array.
(#355) There should be a comma after the last array item in a multi-line array.
(#378) There should be a comma after the last array item in a multi-line array.
mod/scormremote/mod_form.php
(#70) Short array syntax must be used to define arrays
(#71) Short array syntax must be used to define arrays
(#111) Short array syntax must be used to define arrays
(#144) Short array syntax must be used to define arrays
mod/scormremote/submit_completion.php
(#26) Import statements must not begin with a leading backslash
(#27) Import statements must not begin with a leading backslash
(#48) Short array syntax must be used to define arrays
mod/scormremote/tabs.php
(#40) Short array syntax must be used to define arrays
(#41) Short array syntax must be used to define arrays
(#42) Short array syntax must be used to define arrays
(#43) Short array syntax must be used to define arrays
mod/scormremote/tests/client_test.php
(#28) Unit test client_test should be declared as final.
(#41) Test method test_validation_of_client() is missing a return type
(#72) Data provider method "client_validation_provider" will need to be converted to static in future.
(#99) There should be a comma after the last array item in a multi-line array.
mod/scormremote/tests/generator/lib.php
(#35) Implicitly marking a parameter as nullable is deprecated since PHP 8.4. Update the type to be explicitly nullable instead. Found implicitly nullable parameter: $options.
mod/scormremote/tests/requirejs_test.php
(#27) Unit test requirejs_test should be declared as final.
(#39) Test method test_build_files() is missing any coverage information, own or at class level
mod/scormremote/tiers.php
(#125) Short array syntax must be used to define arrays
mod/scormremote/view.php
(#37) Short array syntax must be used to define arrays
(#38) Short array syntax must be used to define arrays
(#40) Short array syntax must be used to define arrays
(#41) Short array syntax must be used to define arrays
(#49) Short array syntax must be used to define arrays
mod/scormremote/wrapper.php
(#40) Short array syntax must be used to define arrays
(#41) Short array syntax must be used to define arrays

Javascript coding style problems

(0 errors, 86 warnings)

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

mod/scormremote/amd/src/layer2.js
(#12) There should be no space after '{'. (@babel/object-curly-spacing)
(#12) There should be no space before '}'. (@babel/object-curly-spacing)
(#13) There should be no space after '{'. (@babel/object-curly-spacing)
(#13) There should be no space before '}'. (@babel/object-curly-spacing)
(#17) Comments should not begin with a lowercase character. (capitalized-comments)
(#87) A function with a name starting with an uppercase letter should only be used as a constructor. (@babel/new-cap)
(#91) There should be no space after this paren. (space-in-parens)
(#91) There should be no space before this paren. (space-in-parens)
(#91) Comments should not begin with a lowercase character. (capitalized-comments)
(#144) A function with a name starting with an uppercase letter should only be used as a constructor. (@babel/new-cap)
(#146) Closing curly brace does not appear on the same line as the subsequent block. (brace-style)
(#162) Requires a space after '{'. (block-spacing)
(#162) Statement inside of curly braces should be on next line. (brace-style)
(#162) Requires a space before '}'. (block-spacing)
(#162) Closing curly brace should be on the same line as opening curly brace or on the line after the previous block. (brace-style)
(#168) Closing curly brace does not appear on the same line as the subsequent block. (brace-style)
(#170) Comments should not begin with a lowercase character. (capitalized-comments)
(#173) A function with a name starting with an uppercase letter should only be used as a constructor. (@babel/new-cap)
(#180) 'result' declared on line 171 column 13 is used outside of binding context. (block-scoped-var)
(#194) Closing curly brace does not appear on the same line as the subsequent block. (brace-style)
(#195) A function with a name starting with an uppercase letter should only be used as a constructor. (@babel/new-cap)
(#196) A function with a name starting with an uppercase letter should only be used as a constructor. (@babel/new-cap)
(#196) Comments should not begin with a lowercase character. (capitalized-comments)
(#198) Closing curly brace does not appear on the same line as the subsequent block. (brace-style)
(#199) A function with a name starting with an uppercase letter should only be used as a constructor. (@babel/new-cap)
(#202) A function with a name starting with an uppercase letter should only be used as a constructor. (@babel/new-cap)
(#204) Comments should not begin with a lowercase character. (capitalized-comments)
(#229) Closing curly brace does not appear on the same line as the subsequent block. (brace-style)
(#230) A function with a name starting with an uppercase letter should only be used as a constructor. (@babel/new-cap)
(#231) 'err' declared on line 237 column 17 is used outside of binding context. (block-scoped-var)
(#231) A function with a name starting with an uppercase letter should only be used as a constructor. (@babel/new-cap)
(#231) Comments should not begin with a lowercase character. (capitalized-comments)
(#232) 'err' declared on line 237 column 17 is used outside of binding context. (block-scoped-var)
(#233) Closing curly brace does not appear on the same line as the subsequent block. (brace-style)
(#237) 'err' is already defined. (no-redeclare)
(#237) 'err' declared on line 231 column 13 is used outside of binding context. (block-scoped-var)
(#237) A function with a name starting with an uppercase letter should only be used as a constructor. (@babel/new-cap)
(#238) 'err' declared on line 231 column 13 is used outside of binding context. (block-scoped-var)
(#238) 'err' declared on line 231 column 13 is used outside of binding context. (block-scoped-var)
(#256) Closing curly brace does not appear on the same line as the subsequent block. (brace-style)
(#257) A function with a name starting with an uppercase letter should only be used as a constructor. (@babel/new-cap)
(#258) 'err' declared on line 264 column 17 is used outside of binding context. (block-scoped-var)
(#258) A function with a name starting with an uppercase letter should only be used as a constructor. (@babel/new-cap)
(#258) Comments should not begin with a lowercase character. (capitalized-comments)
(#259) 'err' declared on line 264 column 17 is used outside of binding context. (block-scoped-var)
(#260) Closing curly brace does not appear on the same line as the subsequent block. (brace-style)
(#264) 'err' is already defined. (no-redeclare)
(#264) 'err' declared on line 258 column 13 is used outside of binding context. (block-scoped-var)
(#264) A function with a name starting with an uppercase letter should only be used as a constructor. (@babel/new-cap)
(#265) 'err' declared on line 258 column 13 is used outside of binding context. (block-scoped-var)
(#282) Comments should not begin with a lowercase character. (capitalized-comments)
(#282) Expected space or tab after '//' in comment. (spaced-comment)
(#283) Expected space or tab after '//' in comment. (spaced-comment)
(#338) There should be no space after '{'. (@babel/object-curly-spacing)
(#338) There should be no space before '}'. (@babel/object-curly-spacing)
(#348) Comments should not begin with a lowercase character. (capitalized-comments)
(#351) Comments should not begin with a lowercase character. (capitalized-comments)
(#471) ["function"] is better written in dot notation. (dot-notation)
(#472) ["params"] is better written in dot notation. (dot-notation)
(#509) Operator '+' must be spaced. (space-infix-ops)
(#509) Operator '+' must be spaced. (space-infix-ops)
(#514) Operator '+' must be spaced. (space-infix-ops)
(#514) Operator '+' must be spaced. (space-infix-ops)
(#533) A function with a name starting with an uppercase letter should only be used as a constructor. (@babel/new-cap)
(#547) A function with a name starting with an uppercase letter should only be used as a constructor. (@babel/new-cap)
(#553) A function with a name starting with an uppercase letter should only be used as a constructor. (@babel/new-cap)
(#559) A function with a name starting with an uppercase letter should only be used as a constructor. (@babel/new-cap)
(#562) A function with a name starting with an uppercase letter should only be used as a constructor. (@babel/new-cap)
(#566) A function with a name starting with an uppercase letter should only be used as a constructor. (@babel/new-cap)
(#581) A function with a name starting with an uppercase letter should only be used as a constructor. (@babel/new-cap)
(#600) A function with a name starting with an uppercase letter should only be used as a constructor. (@babel/new-cap)
(#614) A function with a name starting with an uppercase letter should only be used as a constructor. (@babel/new-cap)
(#616) There should be no space after '{'. (@babel/object-curly-spacing)
(#616) Unquoted reserved word 'function' used as key. (quote-props)
(#616) There should be no space before '}'. (@babel/object-curly-spacing)
(#630) Unnecessary escape character: \d. (no-useless-escape)
mod/scormremote/amd/src/layer3.js
(#20) Expected space or tab after '//' in comment. (spaced-comment)
(#22) Expected space or tab after '//' in comment. (spaced-comment)
(#30) Expected space or tab after '//' in comment. (spaced-comment)
(#97) ["function"] is better written in dot notation. (dot-notation)
(#98) ["arguments"] is better written in dot notation. (dot-notation)
(#181) Multiple spaces found before '='. (no-multi-spaces)
(#181) Multiple spaces found before 'new'. (no-multi-spaces)
(#219) Multiple spaces found before '='. (no-multi-spaces)
(#219) Multiple spaces found before 'new'. (no-multi-spaces)
(#265) Unquoted reserved word 'function' used as key. (quote-props)

CSS problems

(0 errors, 0 warnings)

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


PHPDocs style problems

(0 errors, 0 warnings)

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


Update savepoints problems

(0 errors, 0 warnings)

This section shows problems detected with the handling of upgrade savepoints [More info]


Third party library modification problems

(0 errors, 0 warnings)

This section shows problems detected with the modification of third party libraries [More info]


grunt changes

(1 errors, 0 warnings)

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

Problems running grunt

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]

mod/scormremote/templates/secondlayer.mustache
(#1) HTML Validation info, line 6: Trailing slash on void elements has no effect and interacts badly with unquoted attribute values. (/title> <meta charset="UTF-8" /> <sc)
mod/scormremote/templates/thirdlayer.mustache
(#1) HTML Validation info, line 6: Trailing slash on void elements has no effect and interacts badly with unquoted attribute values. (/title> <meta charset="UTF-8" /> <sc)

Gherkin .feature problems

(0 errors, 0 warnings)

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