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

(221 errors, 14 warnings)

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

filter/edusharing/classes/FilterLogic.php: (#1) Line 1 of the opening comment must start "// This file is part of".; (#1) Line 2 of the opening comment must start "//".; (#1) Line 3 of the opening comment must start "// Moodle is free software: you can redistribute it and/or modify".; (#1) Line 4 of the opening comment must start "// it under the terms of the GNU General Public License as published by".; (#1) Line 5 of the opening comment must start "// the Free Software Foundation, either version 3 of the License, or".; (#1) Line 6 of the opening comment must start "// (at your option) any later version.".; (#1) Line 7 of the opening comment must start "//".; (#1) Line 8 of the opening comment must start "// Moodle is distributed in the hope that it will be useful,".; (#2) Line 9 of the opening comment must start "// but WITHOUT ANY WARRANTY; without even the implied warranty of".; (#3) Line 10 of the opening comment must start "// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the".; (#3) Line 11 of the opening comment must start "// GNU General Public License for more details.".; (#3) Line 12 of the opening comment must start "//".; (#3) Line 13 of the opening comment must start "// You should have received a copy of the GNU General Public License".; (#3) Line 14 of the opening comment must start "// along with Moodle. If not, see <http://www.gnu.org/licenses/>.".; (#20) Opening brace should be on the same line as the declaration for class FilterLogic; (#57) Public method name "FilterLogic::applyFilter" must be in lower-case letters only; (#59) Variable "edusharing_filter_loaded" must not contain underscores.; (#66) Variable "matchesAtto" must be all lower-case; (#67) Variable "matchesAtto" must be all lower-case; (#70) Variable "edusharing_filter_loaded" must not contain underscores.; (#72) Variable "edusharing_filter_loaded" must not contain underscores.; (#74) Variable "matchesAtto" must be all lower-case; (#96) Private method name "FilterLogic::convertObject" must be in lower-case letters only; (#112) Line exceeds 132 characters; contains 137 characters; (#115) Variable "renderParams" must be all lower-case; (#116) Variable "renderParams" must be all lower-case; (#117) Variable "renderParams" must be all lower-case; (#118) Variable "renderParams" must be all lower-case; (#119) Variable "renderParams" must be all lower-case; (#120) Variable "renderParams" must be all lower-case; (#121) Variable "renderParams" must be all lower-case; (#122) Variable "wrapperAttributes" must be all lower-case; (#123) Variable "wrapperAttributes" must be all lower-case; (#124) Variable "renderParams" must be all lower-case; (#125) Variable "wrapperAttributes" must be all lower-case; (#127) Variable "nodeStyle" must be all lower-case; (#128) Variable "styleAttr" must be all lower-case; (#129) Variable "nodeStyle" must be all lower-case; (#130) Variable "nodeStyle" must be all lower-case; (#131) Variable "renderParams" must be all lower-case; (#131) Variable "renderParams" must be all lower-case; (#131) Line exceeds 132 characters; contains 135 characters; (#134) Variable "wrapperAttributes" must be all lower-case; (#134) Variable "styleAttr" must be all lower-case; (#135) Variable "wrapperAttributes" must be all lower-case; (#147) Private method name "FilterLogic::renderInline" must be in lower-case letters only; (#147) Variable "renderParams" must be all lower-case; (#149) Variable "objectUrl" must be all lower-case; (#150) Variable "objectUrl" must be all lower-case; (#155) Variable "renderParams" must be all lower-case; (#155) Variable "renderParams" must be all lower-case; (#158) Variable "renderParams" must be all lower-case; (#159) Variable "renderParams" must be all lower-case; (#160) Variable "renderParams" must be all lower-case; (#161) Variable "renderParams" must be all lower-case; (#167) Inline comments must start with a capital letter, digit or 3-dots sequence; (#167) Inline comments must end in full-stops, exclamation marks, or question marks
filter/edusharing/classes/FilterUtilities.php: (#1) Line 1 of the opening comment must start "// This file is part of".; (#1) Line 2 of the opening comment must start "//".; (#1) Line 3 of the opening comment must start "// Moodle is free software: you can redistribute it and/or modify".; (#1) Line 4 of the opening comment must start "// it under the terms of the GNU General Public License as published by".; (#1) Line 5 of the opening comment must start "// the Free Software Foundation, either version 3 of the License, or".; (#1) Line 6 of the opening comment must start "// (at your option) any later version.".; (#1) Line 7 of the opening comment must start "//".; (#1) Line 8 of the opening comment must start "// Moodle is distributed in the hope that it will be useful,".; (#2) Line 9 of the opening comment must start "// but WITHOUT ANY WARRANTY; without even the implied warranty of".; (#3) Line 10 of the opening comment must start "// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the".; (#3) Line 11 of the opening comment must start "// GNU General Public License for more details.".; (#3) Line 12 of the opening comment must start "//".; (#3) Line 13 of the opening comment must start "// You should have received a copy of the GNU General Public License".; (#3) Line 14 of the opening comment must start "// along with Moodle. If not, see <http://www.gnu.org/licenses/>.".; (#15) Expected MOODLE_INTERNAL check or config.php inclusion. Change in global state detected.; (#23) Opening brace should be on the same line as the declaration for class FilterUtilities; (#66) Public method name "FilterUtilities::getRedirectUrl" must be in lower-case letters only; (#66) Variable "timeStamp" must be all lower-case; (#66) Variable "appId" must be all lower-case; (#68) Variable "appId" must be all lower-case; (#68) Variable "appId" must be all lower-case; (#69) Variable "resourceId" must be all lower-case; (#70) Variable "childObjectId" must be all lower-case; (#71) Variable "resourceId" must be all lower-case; (#72) Variable "eduSharing" must be all lower-case; (#72) Variable "resourceId" must be all lower-case; (#76) Variable "eduSharing" must be all lower-case; (#77) Variable "redirectUrl" must be all lower-case; (#77) Variable "eduSharing" must be all lower-case; (#78) Variable "timeStamp" must be all lower-case; (#78) Variable "timeStamp" must be all lower-case; (#79) Variable "redirectUrl" must be all lower-case; (#79) Variable "timeStamp" must be all lower-case; (#80) Variable "appId" must be all lower-case; (#80) Variable "timeStamp" must be all lower-case; (#80) Variable "eduSharing" must be all lower-case; (#81) Variable "redirectUrl" must be all lower-case; (#82) Variable "redirectUrl" must be all lower-case; (#84) Variable "redirectUrl" must be all lower-case; (#85) Variable "childObjectId" must be all lower-case; (#86) Variable "redirectUrl" must be all lower-case; (#86) Variable "childObjectId" must be all lower-case; (#88) Variable "redirectUrl" must be all lower-case; (#97) Public method name "FilterUtilities::getHtml" must be in lower-case letters only; (#97) Variable "timeStamp" must be all lower-case; (#101) Variable "internalUrl" must be all lower-case; (#102) Variable "internalUrl" must be all lower-case; (#103) Variable "internalUrl" must be all lower-case; (#108) Variable "timeStamp" must be all lower-case; (#110) Line exceeds 132 characters; contains 145 characters; (#115) Variable "resourceId" must be all lower-case; (#116) Variable "resourceId" must be all lower-case; (#116) Line exceeds maximum limit of 180 characters; contains 181 characters; (#117) Function utf8_decode() is deprecated since PHP 8.2; Use mb_convert_encoding(), UConverter::transcode() or iconv instead; (#117) Line exceeds 132 characters; contains 139 characters; (#123) The use of function error_log() is forbidden; use debugging() instead; (#128) File must end with a newline character
filter/edusharing/db/install.php: (#24) Unexpected MOODLE_INTERNAL check. No side effects or multiple artifacts detected.; (#30) Inline comments must end in full-stops, exclamation marks, or question marks; (#40) Variable "edusharingPos" must be all lower-case; (#42) Variable "edusharingPos" must be all lower-case; (#44) Variable "edusharingPos" must be all lower-case
filter/edusharing/filter.php: (#39) Opening brace should be on the same line as the declaration for class filter_edusharing; (#52) Commas (,) must be followed by white space.
filter/edusharing/inlineHelper.php: (#20) Expected login check (require_login, require_course_login, require_admin, admin_externalpage_setup) following config inclusion. None found.; (#22) Variable "filterUtils" must be all lower-case; (#24) Variable "redirectUrl" must be all lower-case; (#24) Variable "filterUtils" must be all lower-case; (#25) Variable "redirectUrl" must be all lower-case
filter/edusharing/lang/zh/filter_edusharing.php: (#35) File must end with a newline character
filter/edusharing/proxy.php: (#27) Expected login check (require_login, require_course_login, require_admin, admin_externalpage_setup) following config inclusion. None found.; (#29) Variable "filterUtils" must be all lower-case; (#30) Variable "filterUtils" must be all lower-case; (#30) File must end with a newline character
filter/edusharing/settings.php: (#31) Short array syntax must be used to define arrays
filter/edusharing/tests/FilterUtilitiesTest.php: (#1) Line 1 of the opening comment must start "// This file is part of".; (#1) Line 2 of the opening comment must start "//".; (#1) Line 3 of the opening comment must start "// Moodle is free software: you can redistribute it and/or modify".; (#1) Line 4 of the opening comment must start "// it under the terms of the GNU General Public License as published by".; (#1) Line 5 of the opening comment must start "// the Free Software Foundation, either version 3 of the License, or".; (#1) Line 6 of the opening comment must start "// (at your option) any later version.".; (#1) Line 7 of the opening comment must start "//".; (#1) Line 8 of the opening comment must start "// Moodle is distributed in the hope that it will be useful,".; (#2) Line 9 of the opening comment must start "// but WITHOUT ANY WARRANTY; without even the implied warranty of".; (#3) Line 10 of the opening comment must start "// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the".; (#3) Line 11 of the opening comment must start "// GNU General Public License for more details.".; (#3) Line 12 of the opening comment must start "//".; (#3) Line 13 of the opening comment must start "// You should have received a copy of the GNU General Public License".; (#3) Line 14 of the opening comment must start "// along with Moodle. If not, see <http://www.gnu.org/licenses/>.".; (#23) Opening brace should be on the same line as the declaration for class FilterUtilitiesTest; (#34) Public method name "FilterUtilitiesTest::testGetRedirectUrlReturnsProperUrlIfAllGoesWell" must be in lower-case letters only; (#42) Variable "eduReturn" must be all lower-case; (#43) Variable "eduReturn" must be all lower-case; (#44) Variable "eduReturn" must be all lower-case; (#46) Variable "dbMock" must be all lower-case; (#49) Variable "dbMock" must be all lower-case; (#52) Variable "eduReturn" must be all lower-case; (#53) Variable "GLOBALS" must be all lower-case; (#53) Variable "dbMock" must be all lower-case; (#54) Variable "utilsMock" must be all lower-case; (#57) Variable "utilsMock" must be all lower-case; (#60) Variable "utilsMock" must be all lower-case; (#64) Variable "utilsMock" must be all lower-case; (#68) Variable "baseHelper" must be all lower-case; (#69) Variable "nodeConfig" must be all lower-case; (#70) Variable "authHelper" must be all lower-case; (#70) Variable "baseHelper" must be all lower-case; (#71) Variable "nodeHelper" must be all lower-case; (#71) Variable "baseHelper" must be all lower-case; (#71) Variable "nodeConfig" must be all lower-case; (#72) Variable "serviceMock" must be all lower-case; (#73) Variable "authHelper" must be all lower-case; (#73) Variable "nodeHelper" must be all lower-case; (#76) Variable "serviceMock" must be all lower-case; (#79) Variable "serviceMock" must be all lower-case; (#83) Variable "serviceMock" must be all lower-case; (#86) Variable "filterUtils" must be all lower-case; (#86) Variable "serviceMock" must be all lower-case; (#86) Variable "utilsMock" must be all lower-case; (#87) Variable "filterUtils" must be all lower-case; (#95) Line exceeds 132 characters; contains 152 characters; (#109) Public method name "FilterUtilitiesTest::testGetRedirectUrlDoesNotSetChildObjectInUrlIfNoneIsGiven" must be in lower-case letters only; (#119) Variable "eduReturn" must be all lower-case; (#120) Variable "eduReturn" must be all lower-case; (#121) Variable "eduReturn" must be all lower-case; (#123) Variable "dbMock" must be all lower-case; (#126) Variable "dbMock" must be all lower-case; (#129) Variable "eduReturn" must be all lower-case; (#130) Variable "GLOBALS" must be all lower-case; (#130) Variable "dbMock" must be all lower-case; (#131) Variable "utilsMock" must be all lower-case; (#134) Variable "utilsMock" must be all lower-case; (#137) Variable "utilsMock" must be all lower-case; (#141) Variable "utilsMock" must be all lower-case; (#145) Variable "baseHelper" must be all lower-case; (#146) Variable "nodeConfig" must be all lower-case; (#147) Variable "authHelper" must be all lower-case; (#147) Variable "baseHelper" must be all lower-case; (#148) Variable "nodeHelper" must be all lower-case; (#148) Variable "baseHelper" must be all lower-case; (#148) Variable "nodeConfig" must be all lower-case; (#149) Variable "serviceMock" must be all lower-case; (#150) Variable "authHelper" must be all lower-case; (#150) Variable "nodeHelper" must be all lower-case; (#153) Variable "serviceMock" must be all lower-case; (#156) Variable "serviceMock" must be all lower-case; (#160) Variable "serviceMock" must be all lower-case; (#163) Variable "filterUtils" must be all lower-case; (#163) Variable "serviceMock" must be all lower-case; (#163) Variable "utilsMock" must be all lower-case; (#164) Variable "filterUtils" must be all lower-case; (#166) Line exceeds 132 characters; contains 139 characters; (#176) Public method name "FilterUtilitiesTest::testGetHtmlReturnsProperHtmlIfAllGoesWell" must be in lower-case letters only; (#186) Variable "fakeConfig" must be all lower-case; (#187) Variable "fakeConfig" must be all lower-case; (#189) There should be a comma after the last array item in a multi-line array.; (#191) Variable "fakeConfig" must be all lower-case; (#192) Variable "expectedUrlAfterProc" must be all lower-case; (#192) Line exceeds maximum limit of 180 characters; contains 185 characters; (#193) Variable "baseHelper" must be all lower-case; (#194) Variable "nodeConfig" must be all lower-case; (#195) Variable "authHelper" must be all lower-case; (#195) Variable "baseHelper" must be all lower-case; (#196) Variable "nodeHelper" must be all lower-case; (#196) Variable "baseHelper" must be all lower-case; (#196) Variable "nodeConfig" must be all lower-case; (#197) Variable "serviceMock" must be all lower-case; (#198) Variable "authHelper" must be all lower-case; (#198) Variable "nodeHelper" must be all lower-case; (#201) Variable "serviceMock" must be all lower-case; (#205) Variable "serviceMock" must be all lower-case; (#207) Variable "expectedUrlAfterProc" must be all lower-case; (#209) Variable "serviceMock" must be all lower-case; (#212) Variable "serviceMock" must be all lower-case; (#215) Variable "filterUtils" must be all lower-case; (#215) Variable "serviceMock" must be all lower-case; (#216) Variable "filterUtils" must be all lower-case; (#220) Line exceeds maximum limit of 180 characters; contains 187 characters

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, 1 warnings)

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

: Problems running grunt; Task "eslint:amd" failed. Use --force to continue.

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

grunt changes

shifter problems

Mustache template problems

Gherkin .feature problems