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

(225 errors, 44 warnings)

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

mod/onlyofficeeditor/callback.php: (#29) Expected login check (require_login, require_course_login, require_admin, admin_externalpage_setup) following config inclusion. None found.; (#52) The "salt" parameter for function crypt() is missing. Passing this parameter is strongly recommended since PHP 5.6
mod/onlyofficeeditor/classes/crypt.php: (#29) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
mod/onlyofficeeditor/classes/editor.php: (#29) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.; (#85) Line exceeds 132 characters; contains 137 characters; (#117) No space found before comment text; expected "// Top level config object." but found "//Top level config object."; (#121) No space found before comment text; expected "// Document." but found "//Document."; (#135) No space found before comment text; expected "// Editorconfig." but found "//Editorconfig."; (#140) No space found before comment text; expected "// User." but found "//User."; (#146) No space found before comment text; expected "// Customization." but found "//Customization."; (#155) No space found before comment text; expected "// Device type." but found "//Device type."; (#163) No space found before comment text; expected "// Package config object from parts." but found "//Package config object from parts."; (#168) No space found before comment text; expected "// Add token." but found "//Add token."
mod/onlyofficeeditor/classes/event/course_module_instance_list_viewed.php: (#28) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
mod/onlyofficeeditor/classes/event/course_module_viewed.php: (#28) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
mod/onlyofficeeditor/classes/output/renderer.php: (#28) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
mod/onlyofficeeditor/classes/output/summary.php: (#54) No space found before comment text; expected "// Do some stuff." but found "//Do some stuff."
mod/onlyofficeeditor/classes/util.php: (#29) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
mod/onlyofficeeditor/db/upgrade.php: (#25) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.
mod/onlyofficeeditor/dsconfig.php: (#26) Expected login check (require_login, require_course_login, require_admin, admin_externalpage_setup) following config inclusion. None found.
mod/onlyofficeeditor/lib.php: (#80) No space found before comment text; expected "// We need to use context now, so we need to make sure all needed info is already in db." but found "//We need to use context now, so we need to make sure all needed info is already in db."; (#358) The "salt" parameter for function crypt() is missing. Passing this parameter is strongly recommended since PHP 5.6; (#370) Line exceeds 132 characters; contains 139 characters
mod/onlyofficeeditor/mod_form.php: (#79) Inline comments must start with a capital letter, digit or 3-dots sequence
mod/onlyofficeeditor/settings.php: (#33) Line exceeds 132 characters; contains 134 characters
mod/onlyofficeeditor/vendor/autoload.php: (#2) The opening <?php tag must be followed by exactly one newline.; (#3) Inline comments must start with a capital letter, digit or 3-dots sequence; (#3) Inline comments must end in full-stops, exclamation marks, or question marks; (#5) Expected MOODLE_INTERNAL check or config.php inclusion. Change in global state detected.; (#5) "require_once" must be immediately followed by an open parenthesis
mod/onlyofficeeditor/vendor/composer/ClassLoader.php: (#2) The opening <?php tag must be followed by exactly one newline.; (#44) Opening brace should be on the same line as the declaration for class ClassLoader; (#45) Inline comments must end in full-stops, exclamation marks, or question marks; (#46) Member variable "prefixLengthsPsr4" must be all lower-case; (#47) Member variable "prefixDirsPsr4" must be all lower-case; (#48) Member variable "fallbackDirsPsr4" must be all lower-case; (#50) Inline comments must end in full-stops, exclamation marks, or question marks; (#51) Member variable "prefixesPsr0" must be all lower-case; (#52) Member variable "fallbackDirsPsr0" must be all lower-case; (#54) Member variable "useIncludePath" must be all lower-case; (#55) Member variable "classMap" must be all lower-case; (#56) Member variable "classMapAuthoritative" must be all lower-case; (#57) Member variable "missingClasses" must be all lower-case; (#58) Member variable "apcuPrefix" must be all lower-case; (#60) Public method name "ClassLoader::getPrefixes" must be in lower-case letters only; (#61) Opening brace should be on the same line as the declaration; (#69) Public method name "ClassLoader::getPrefixesPsr4" must be in lower-case letters only; (#70) Opening brace should be on the same line as the declaration; (#74) Public method name "ClassLoader::getFallbackDirs" must be in lower-case letters only; (#75) Opening brace should be on the same line as the declaration; (#79) Public method name "ClassLoader::getFallbackDirsPsr4" must be in lower-case letters only; (#80) Opening brace should be on the same line as the declaration; (#84) Public method name "ClassLoader::getClassMap" must be in lower-case letters only; (#85) Opening brace should be on the same line as the declaration; (#92) Public method name "ClassLoader::addClassMap" must be in lower-case letters only; (#92) Variable "classMap" must be all lower-case; (#93) Opening brace should be on the same line as the declaration; (#95) Variable "classMap" must be all lower-case; (#97) Variable "classMap" must be all lower-case; (#110) Opening brace should be on the same line as the declaration; (#156) Public method name "ClassLoader::addPsr4" must be in lower-case letters only; (#157) Opening brace should be on the same line as the declaration; (#171) Usage of ELSEIF not allowed; use ELSE IF instead; (#179) Usage of ELSEIF not allowed; use ELSE IF instead; (#202) Opening brace should be on the same line as the declaration; (#219) Public method name "ClassLoader::setPsr4" must be in lower-case letters only; (#220) Opening brace should be on the same line as the declaration; (#238) Public method name "ClassLoader::setUseIncludePath" must be in lower-case letters only; (#238) Variable "useIncludePath" must be all lower-case; (#239) Opening brace should be on the same line as the declaration; (#240) Variable "useIncludePath" must be all lower-case; (#249) Public method name "ClassLoader::getUseIncludePath" must be in lower-case letters only; (#250) Opening brace should be on the same line as the declaration; (#260) Public method name "ClassLoader::setClassMapAuthoritative" must be in lower-case letters only; (#260) Variable "classMapAuthoritative" must be all lower-case; (#261) Opening brace should be on the same line as the declaration; (#262) Variable "classMapAuthoritative" must be all lower-case; (#270) Public method name "ClassLoader::isClassMapAuthoritative" must be in lower-case letters only; (#271) Opening brace should be on the same line as the declaration; (#280) Public method name "ClassLoader::setApcuPrefix" must be in lower-case letters only; (#280) Variable "apcuPrefix" must be all lower-case; (#281) Opening brace should be on the same line as the declaration; (#282) Variable "apcuPrefix" must be all lower-case; (#282) Line exceeds 132 characters; contains 142 characters; (#290) Public method name "ClassLoader::getApcuPrefix" must be in lower-case letters only; (#291) Opening brace should be on the same line as the declaration; (#301) Opening brace should be on the same line as the declaration; (#309) Opening brace should be on the same line as the declaration; (#319) Public method name "ClassLoader::loadClass" must be in lower-case letters only; (#320) Opening brace should be on the same line as the declaration; (#335) Public method name "ClassLoader::findFile" must be in lower-case letters only; (#336) Opening brace should be on the same line as the declaration; (#337) Inline comments must start with a capital letter, digit or 3-dots sequence; (#337) Inline comments must end in full-stops, exclamation marks, or question marks; (#353) Inline comments must end in full-stops, exclamation marks, or question marks; (#370) Private method name "ClassLoader::findFileWithExtension" must be in lower-case letters only; (#371) Opening brace should be on the same line as the declaration; (#372) Inline comments must end in full-stops, exclamation marks, or question marks; (#373) Variable "logicalPathPsr4" must be all lower-case; (#377) Variable "subPath" must be all lower-case; (#378) Variable "lastPos" must be all lower-case; (#378) Variable "subPath" must be all lower-case; (#379) Variable "subPath" must be all lower-case; (#379) Variable "subPath" must be all lower-case; (#379) Variable "lastPos" must be all lower-case; (#380) Variable "subPath" must be all lower-case; (#382) Variable "pathEnd" must be all lower-case; (#382) Variable "logicalPathPsr4" must be all lower-case; (#382) Variable "lastPos" must be all lower-case; (#384) Variable "pathEnd" must be all lower-case; (#392) Inline comments must end in full-stops, exclamation marks, or question marks; (#394) Variable "logicalPathPsr4" must be all lower-case; (#399) Inline comments must end in full-stops, exclamation marks, or question marks; (#401) Inline comments must start with a capital letter, digit or 3-dots sequence; (#401) Inline comments must end in full-stops, exclamation marks, or question marks; (#402) Variable "logicalPathPsr0" must be all lower-case; (#402) Variable "logicalPathPsr4" must be all lower-case; (#403) Variable "logicalPathPsr4" must be all lower-case; (#405) Inline comments must end in full-stops, exclamation marks, or question marks; (#406) Variable "logicalPathPsr0" must be all lower-case; (#413) Variable "logicalPathPsr0" must be all lower-case; (#421) Inline comments must end in full-stops, exclamation marks, or question marks; (#423) Variable "logicalPathPsr0" must be all lower-case; (#429) Variable "logicalPathPsr0" must be all lower-case; (#442) function name "includeFile" must be lower-case letters only; (#443) Opening brace should be on the same line as the declaration; (#444) "include" must be immediately followed by an open parenthesis
mod/onlyofficeeditor/vendor/composer/autoload_classmap.php: (#2) The opening <?php tag must be followed by exactly one newline.; (#3) Inline comments must start with a capital letter, digit or 3-dots sequence; (#3) Inline comments must end in full-stops, exclamation marks, or question marks; (#5) Expected MOODLE_INTERNAL check or config.php inclusion. Change in global state detected.; (#5) Variable "vendorDir" must be all lower-case; (#6) Variable "baseDir" must be all lower-case; (#6) Variable "vendorDir" must be all lower-case
mod/onlyofficeeditor/vendor/composer/autoload_namespaces.php: (#2) The opening <?php tag must be followed by exactly one newline.; (#3) Inline comments must start with a capital letter, digit or 3-dots sequence; (#3) Inline comments must end in full-stops, exclamation marks, or question marks; (#5) Expected MOODLE_INTERNAL check or config.php inclusion. Change in global state detected.; (#5) Variable "vendorDir" must be all lower-case; (#6) Variable "baseDir" must be all lower-case; (#6) Variable "vendorDir" must be all lower-case
mod/onlyofficeeditor/vendor/composer/autoload_psr4.php: (#2) The opening <?php tag must be followed by exactly one newline.; (#3) Inline comments must start with a capital letter, digit or 3-dots sequence; (#3) Inline comments must end in full-stops, exclamation marks, or question marks; (#5) Expected MOODLE_INTERNAL check or config.php inclusion. Change in global state detected.; (#5) Variable "vendorDir" must be all lower-case; (#6) Variable "baseDir" must be all lower-case; (#6) Variable "vendorDir" must be all lower-case; (#9) Variable "vendorDir" must be all lower-case
mod/onlyofficeeditor/vendor/composer/autoload_real.php: (#2) The opening <?php tag must be followed by exactly one newline.; (#3) Inline comments must start with a capital letter, digit or 3-dots sequence; (#3) Inline comments must end in full-stops, exclamation marks, or question marks; (#6) Opening brace should be on the same line as the declaration for class ComposerAutoloaderInita849e9fe15e5e0d8650886cb38774b7c; (#9) Public method name "ComposerAutoloaderInita849e9fe15e5e0d8650886cb38774b7c::loadClassLoader" must be in lower-case letters only; (#10) Opening brace should be on the same line as the declaration; (#12) "require" must be immediately followed by an open parenthesis; (#16) Public method name "ComposerAutoloaderInita849e9fe15e5e0d8650886cb38774b7c::getLoader" must be in lower-case letters only; (#17) Opening brace should be on the same line as the declaration; (#26) Variable "useStaticLoader" must be all lower-case; (#26) Line exceeds 132 characters; contains 160 characters; (#27) Variable "useStaticLoader" must be all lower-case; (#28) "require_once" must be immediately followed by an open parenthesis; (#32) "require" must be immediately followed by an open parenthesis; (#37) "require" must be immediately followed by an open parenthesis; (#42) Variable "classMap" must be all lower-case; (#42) "require" must be immediately followed by an open parenthesis; (#43) Variable "classMap" must be all lower-case; (#44) Variable "classMap" must be all lower-case
mod/onlyofficeeditor/vendor/composer/autoload_static.php: (#2) The opening <?php tag must be followed by exactly one newline.; (#3) Inline comments must start with a capital letter, digit or 3-dots sequence; (#3) Inline comments must end in full-stops, exclamation marks, or question marks; (#8) Opening brace should be on the same line as the declaration for class ComposerStaticInita849e9fe15e5e0d8650886cb38774b7c; (#9) Member variable "prefixLengthsPsr4" must be all lower-case; (#10) Whitespace found at end of line; (#16) Member variable "prefixDirsPsr4" must be all lower-case; (#17) Whitespace found at end of line; (#23) Public method name "ComposerStaticInita849e9fe15e5e0d8650886cb38774b7c::getInitializer" must be in lower-case letters only; (#24) Opening brace should be on the same line as the declaration; (#26) Variable "prefixLengthsPsr4" must be all lower-case; (#27) Variable "prefixDirsPsr4" must be all lower-case
mod/onlyofficeeditor/vendor/firebase/php-jwt/src/BeforeValidException.php: (#2) Line 1 of the opening comment must start "// This file is part of".; (#2) Line 2 of the opening comment must start "//".; (#2) Line 3 of the opening comment must start "// Moodle is free software: you can redistribute it and/or modify".; (#2) Line 4 of the opening comment must start "// it under the terms of the GNU General Public License as published by".; (#2) Line 5 of the opening comment must start "// the Free Software Foundation, either version 3 of the License, or".; (#2) Line 6 of the opening comment must start "// (at your option) any later version.".; (#2) Line 7 of the opening comment must start "//".; (#3) Line 8 of the opening comment must start "// Moodle is distributed in the hope that it will be useful,".; (#4) Line 9 of the opening comment must start "// but WITHOUT ANY WARRANTY; without even the implied warranty of".; (#4) Line 10 of the opening comment must start "// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the".; (#4) Line 11 of the opening comment must start "// GNU General Public License for more details.".; (#4) Line 12 of the opening comment must start "//".; (#4) Line 13 of the opening comment must start "// You should have received a copy of the GNU General Public License".; (#4) Line 14 of the opening comment must start "// along with Moodle. If not, see <http://www.gnu.org/licenses/>.".; (#5) Opening brace should be on the same line as the declaration for class BeforeValidException
mod/onlyofficeeditor/vendor/firebase/php-jwt/src/ExpiredException.php: (#2) Line 1 of the opening comment must start "// This file is part of".; (#2) Line 2 of the opening comment must start "//".; (#2) Line 3 of the opening comment must start "// Moodle is free software: you can redistribute it and/or modify".; (#2) Line 4 of the opening comment must start "// it under the terms of the GNU General Public License as published by".; (#2) Line 5 of the opening comment must start "// the Free Software Foundation, either version 3 of the License, or".; (#2) Line 6 of the opening comment must start "// (at your option) any later version.".; (#2) Line 7 of the opening comment must start "//".; (#3) Line 8 of the opening comment must start "// Moodle is distributed in the hope that it will be useful,".; (#4) Line 9 of the opening comment must start "// but WITHOUT ANY WARRANTY; without even the implied warranty of".; (#4) Line 10 of the opening comment must start "// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the".; (#4) Line 11 of the opening comment must start "// GNU General Public License for more details.".; (#4) Line 12 of the opening comment must start "//".; (#4) Line 13 of the opening comment must start "// You should have received a copy of the GNU General Public License".; (#4) Line 14 of the opening comment must start "// along with Moodle. If not, see <http://www.gnu.org/licenses/>.".; (#5) Opening brace should be on the same line as the declaration for class ExpiredException
mod/onlyofficeeditor/vendor/firebase/php-jwt/src/JWT.php: (#2) The opening <?php tag must be followed by exactly one newline.; (#23) Opening brace should be on the same line as the declaration for class JWT; (#40) Member variable "supported_algs" must not contain underscores.; (#69) Variable "allowed_algs" must not contain underscores.; (#70) Opening brace should be on the same line as the declaration; (#93) Variable "supported_algs" must not contain underscores.; (#96) Variable "allowed_algs" must not contain underscores.; (#110) Inline comments must end in full-stops, exclamation marks, or question marks; (#156) Variable "keyId" must be all lower-case; (#157) Opening brace should be on the same line as the declaration; (#159) Variable "keyId" must be all lower-case; (#160) Variable "keyId" must be all lower-case; (#168) Variable "signing_input" must not contain underscores.; (#170) Variable "signing_input" must not contain underscores.; (#189) Opening brace should be on the same line as the declaration; (#190) Variable "supported_algs" must not contain underscores.; (#193) Variable "supported_algs" must not contain underscores.; (#222) Opening brace should be on the same line as the declaration; (#223) Variable "supported_algs" must not contain underscores.; (#227) Variable "supported_algs" must not contain underscores.; (#233) Usage of ELSEIF not allowed; use ELSE IF instead; (#236) Inline comments must start with a capital letter, digit or 3-dots sequence; (#267) Public method name "JWT::jsonDecode" must be in lower-case letters only; (#268) Opening brace should be on the same line as the declaration; (#270) Inline doc block comments are not allowed; use "// Comment." instead; (#276) Inline doc block comments are not allowed; use "// Comment." instead; (#280) Variable "max_int_length" must not contain underscores.; (#281) Variable "json_without_bigints" must not contain underscores.; (#281) Variable "max_int_length" must not contain underscores.; (#282) Variable "json_without_bigints" must not contain underscores.; (#287) Usage of ELSEIF not allowed; use ELSE IF instead; (#302) Public method name "JWT::jsonEncode" must be in lower-case letters only; (#303) Opening brace should be on the same line as the declaration; (#307) Usage of ELSEIF not allowed; use ELSE IF instead; (#320) Public method name "JWT::urlsafeB64Decode" must be in lower-case letters only; (#321) Opening brace should be on the same line as the declaration; (#337) Public method name "JWT::urlsafeB64Encode" must be in lower-case letters only; (#338) Opening brace should be on the same line as the declaration; (#349) Private method name "JWT::handleJsonError" must be in lower-case letters only; (#350) Opening brace should be on the same line as the declaration; (#356) Inline comments must end in full-stops, exclamation marks, or question marks; (#356) No space found before comment text; expected "// PHP >= 5.3.3" but found "//PHP >= 5.3.3"; (#372) Private method name "JWT::safeStrlen" must be in lower-case letters only; (#373) Opening brace should be on the same line as the declaration
mod/onlyofficeeditor/vendor/firebase/php-jwt/src/SignatureInvalidException.php: (#2) Line 1 of the opening comment must start "// This file is part of".; (#2) Line 2 of the opening comment must start "//".; (#2) Line 3 of the opening comment must start "// Moodle is free software: you can redistribute it and/or modify".; (#2) Line 4 of the opening comment must start "// it under the terms of the GNU General Public License as published by".; (#2) Line 5 of the opening comment must start "// the Free Software Foundation, either version 3 of the License, or".; (#2) Line 6 of the opening comment must start "// (at your option) any later version.".; (#2) Line 7 of the opening comment must start "//".; (#3) Line 8 of the opening comment must start "// Moodle is distributed in the hope that it will be useful,".; (#4) Line 9 of the opening comment must start "// but WITHOUT ANY WARRANTY; without even the implied warranty of".; (#4) Line 10 of the opening comment must start "// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the".; (#4) Line 11 of the opening comment must start "// GNU General Public License for more details.".; (#4) Line 12 of the opening comment must start "//".; (#4) Line 13 of the opening comment must start "// You should have received a copy of the GNU General Public License".; (#4) Line 14 of the opening comment must start "// along with Moodle. If not, see <http://www.gnu.org/licenses/>.".; (#5) Opening brace should be on the same line as the declaration for class SignatureInvalidException

Javascript coding style problems

(0 errors, 0 warnings)

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

CSS problems

(0 errors, 0 warnings)

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

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]

Missing changes in external functions or backup support for new detected tables or columns

(0 errors, 0 warnings)

This section shows potential problems detected when there are new database structures added in a patch and it is detected that nothing has been changed related with external functions (to be used by WS) or backup and restore. It must be checked that everything is correct and nothing is being missed in those areas. [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]

Prechecker results: error

PHP lint problems

PHP coding style problems

Javascript coding style problems

CSS problems

PHPDocs style problems

Update savepoints problems

Third party library modification problems

Missing changes in external functions or backup support for new detected tables or columns

grunt changes

shifter problems

Mustache template problems

Gherkin .feature problems