Prechecker results: error


PHP lint problems

(0 errors, 0 warnings)

This section shows php lint problems in the code detected by php -l [More info]


PHP coding style problems

(432 errors, 29 warnings)

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

mod/edusharing/apiClient/src/EduSharing/AppAuthException.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/>.".
(#16) Opening brace should be on the same line as the declaration for class AppAuthException
(#18) Line exceeds maximum limit of 180 characters; contains 190 characters
(#20) Line exceeds 132 characters; contains 144 characters
(#21) There should be a comma after the last array item in a multi-line array.
(#39) Private method name "AppAuthException::getExplanation" must be in lower-case letters only
(#42) Inline control structures are not allowed
mod/edusharing/apiClient/src/EduSharing/CurlHandler.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/>.".
(#14) Opening brace should be on the same line as the declaration for class CurlHandler
(#30) The abstract declaration must precede the visibility declaration
(#30) Public method name "CurlHandler::handleCurlRequest" must be in lower-case letters only
(#30) Variable "curlOptions" must be all lower-case
(#32) Public method name "CurlHandler::setMethod" must be in lower-case letters only
(#37) File must end with a newline character
mod/edusharing/apiClient/src/EduSharing/CurlResult.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/>.".
(#13) Opening brace should be on the same line as the declaration for class CurlResult
mod/edusharing/apiClient/src/EduSharing/DefaultCurlHandler.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) Opening brace should be on the same line as the declaration for class DefaultCurlHandler
(#23) Public method name "DefaultCurlHandler::handleCurlRequest" must be in lower-case letters only
(#23) Variable "curlOptions" must be all lower-case
(#25) Variable "curlOptions" must be all lower-case
(#32) File must end with a newline character
mod/edusharing/apiClient/src/EduSharing/DisplayMode.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/>.".
(#11) Opening brace should be on the same line as the declaration for class DisplayMode
(#16) File must end with a newline character
mod/edusharing/apiClient/src/EduSharing/EduSharingAuthHelper.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/>.".
(#13) Opening brace should be on the same line as the declaration for class EduSharingAuthHelper
(#27) Public method name "EduSharingAuthHelper::getTicketAuthenticationInfo" must be in lower-case letters only
(#36) There should be a comma after the last array item in a multi-line array.
(#61) Public method name "EduSharingAuthHelper::getTicketForUser" must be in lower-case letters only
(#61) Variable "additionalFields" must be all lower-case
(#62) Variable "curlOptions" must be all lower-case
(#68) There should be a comma after the last array item in a multi-line array.
(#70) Variable "additionalFields" must be all lower-case
(#71) Variable "curlOptions" must be all lower-case
(#71) Variable "additionalFields" must be all lower-case
(#73) Variable "curlOptions" must be all lower-case
(#73) Line exceeds 132 characters; contains 146 characters
(#75) Line exceeds maximum limit of 180 characters; contains 231 characters
(#80) The use of function error_log() is forbidden; use debugging() instead
(#83) Variable "gotError" must be all lower-case
(#84) Variable "responseOk" must be all lower-case
(#84) Variable "gotError" must be all lower-case
(#85) Variable "responseOk" must be all lower-case
(#85) Line exceeds 132 characters; contains 140 characters
mod/edusharing/apiClient/src/EduSharing/EduSharingHelper.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/>.".
(#13) Opening brace should be on the same line as the declaration for class EduSharingHelper
(#23) Public method name "EduSharingHelper::generateKeyPair" must be in lower-case letters only
(#26) Variable "privateKey" must be all lower-case
(#27) Variable "publicKeyData" must be all lower-case
(#28) Variable "publicKey" must be all lower-case
(#28) Variable "publicKeyData" must be all lower-case
(#30) Variable "privateKey" must be all lower-case
(#31) There should be a comma after the last array item in a multi-line array.
(#31) Variable "publicKey" must be all lower-case
(#41) Public method name "EduSharingHelper::generateEduAppXMLData" must be in lower-case letters only
(#41) Variable "appId" must be all lower-case
(#41) Variable "publicKey" must be all lower-case
(#41) Variable "publicIP" must be all lower-case
(#41) Line exceeds 132 characters; contains 138 characters
(#45) Variable "appId" must be all lower-case
(#46) Variable "publicKey" must be all lower-case
(#49) Whitespace found at end of line within string
(#50) Variable "publicIP" must be all lower-case
(#52) Whitespace found at end of line within string
mod/edusharing/apiClient/src/EduSharing/EduSharingHelperAbstract.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/>.".
(#11) Opening brace should be on the same line as the declaration for class EduSharingHelperAbstract
(#31) Public method name "EduSharingHelperAbstract::getRESTAuthenticationHeader" must be in lower-case letters only
(#43) Protected method name "EduSharingHelperAbstract::getSignatureHeaders" must be in lower-case letters only
(#43) Variable "signString" must be all lower-case
(#43) Variable "contentType" must be all lower-case
(#43) Line exceeds 132 characters; contains 150 characters
(#45) Variable "toSign" must be all lower-case
(#45) Variable "signString" must be all lower-case
(#46) Variable "toSign" must be all lower-case
(#49) Variable "contentType" must be all lower-case
(#51) Variable "toSign" must be all lower-case
(#63) Variable "toSign" must be all lower-case
(#64) Variable "toSign" must be all lower-case
mod/edusharing/apiClient/src/EduSharing/EduSharingHelperBase.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/>.".
(#13) Opening brace should be on the same line as the declaration for class EduSharingHelperBase
(#14) Member variable "baseUrl" must be all lower-case
(#15) Member variable "privateKey" must be all lower-case
(#16) Member variable "appId" must be all lower-case
(#18) Member variable "curlHandler" must be all lower-case
(#29) Variable "baseUrl" must be all lower-case
(#29) Variable "privateKey" must be all lower-case
(#29) Variable "appId" must be all lower-case
(#30) Variable "appId" must be all lower-case
(#33) Variable "baseUrl" must be all lower-case
(#33) Variable "baseUrl" must be all lower-case
(#34) Variable "baseUrl" must be all lower-case
(#35) Variable "privateKey" must be all lower-case
(#36) Variable "appId" must be all lower-case
(#46) Public method name "EduSharingHelperBase::registerCurlHandler" must be in lower-case letters only
(#57) Public method name "EduSharingHelperBase::handleCurlRequest" must be in lower-case letters only
(#57) Variable "curlOptions" must be all lower-case
(#58) Variable "curlOptions" must be all lower-case
(#67) Public method name "EduSharingHelperBase::setLanguage" must be in lower-case letters only
(#77) Variable "toSign" must be all lower-case
(#78) Variable "privateKeyId" must be all lower-case
(#79) Variable "toSign" must be all lower-case
(#79) Variable "privateKeyId" must be all lower-case
(#89) Public method name "EduSharingHelperBase::verifyCompatibility" must be in lower-case letters only
(#90) Variable "minVersion" must be all lower-case
(#97) There should be a comma after the last array item in a multi-line array.
(#101) Variable "minVersion" must be all lower-case
(#102) Variable "minVersion" must be all lower-case
(#102) The use of function print_r() is forbidden
(#102) Line exceeds maximum limit of 180 characters; contains 216 characters
(#105) The use of function print_r() is forbidden
mod/edusharing/apiClient/src/EduSharing/EduSharingNodeHelper.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) Opening brace should be on the same line as the declaration for class EduSharingNodeHelper
(#47) Public method name "EduSharingNodeHelper::createUsage" must be in lower-case letters only
(#47) Variable "containerId" must be all lower-case
(#47) Variable "resourceId" must be all lower-case
(#47) Variable "nodeId" must be all lower-case
(#47) Variable "nodeVersion" must be all lower-case
(#47) Line exceeds 132 characters; contains 141 characters
(#55) Variable "containerId" must be all lower-case
(#56) Variable "resourceId" must be all lower-case
(#57) Variable "nodeId" must be all lower-case
(#58) Variable "nodeVersion" must be all lower-case
(#61) There should be a comma after the last array item in a multi-line array.
(#65) Variable "nodeVersion" must be all lower-case
(#65) Variable "containerId" must be all lower-case
(#65) Variable "resourceId" must be all lower-case
(#87) Public method name "EduSharingNodeHelper::getUsageIdByParameters" must be in lower-case letters only
(#87) Variable "nodeId" must be all lower-case
(#87) Variable "containerId" must be all lower-case
(#87) Variable "resourceId" must be all lower-case
(#90) Variable "nodeId" must be all lower-case
(#93) There should be a comma after the last array item in a multi-line array.
(#98) Variable "containerId" must be all lower-case
(#98) Variable "resourceId" must be all lower-case
(#98) Line exceeds 132 characters; contains 164 characters
(#105) Line exceeds 132 characters; contains 134 characters
(#128) Public method name "EduSharingNodeHelper::getNodeByUsage" must be in lower-case letters only
(#128) Variable "displayMode" must be all lower-case
(#128) Variable "renderingParams" must be all lower-case
(#128) Variable "userId" must be all lower-case
(#128) Line exceeds 132 characters; contains 156 characters
(#130) Variable "displayMode" must be all lower-case
(#134) Variable "userId" must be all lower-case
(#138) Variable "renderingParams" must be all lower-case
(#140) There should be a comma after the last array item in a multi-line array.
(#150) Line exceeds 132 characters; contains 157 characters
(#152) Line exceeds 132 characters; contains 142 characters
(#170) Public method name "EduSharingNodeHelper::deleteUsage" must be in lower-case letters only
(#170) Variable "nodeId" must be all lower-case
(#170) Variable "usageId" must be all lower-case
(#171) Variable "nodeId" must be all lower-case
(#171) Variable "usageId" must be all lower-case
(#172) Variable "nodeId" must be all lower-case
(#172) Variable "usageId" must be all lower-case
(#172) Line exceeds 132 characters; contains 160 characters
(#176) There should be a comma after the last array item in a multi-line array.
(#194) Private method name "EduSharingNodeHelper::handleURLMapping" must be in lower-case letters only
(#199) Line exceeds maximum limit of 180 characters; contains 204 characters
(#203) Variable "endpointBase" must be all lower-case
(#203) Line exceeds 132 characters; contains 153 characters
(#204) Variable "contentUrl" must be all lower-case
(#204) Variable "endpointBase" must be all lower-case
(#206) Variable "contentUrl" must be all lower-case
(#207) Variable "endpointBase" must be all lower-case
(#207) There should be a comma after the last array item in a multi-line array.
(#209) Variable "contentUrl" must be all lower-case
(#225) Public method name "EduSharingNodeHelper::getRedirectUrl" must be in lower-case letters only
(#225) Variable "userId" must be all lower-case
(#227) Inline comments must end in full-stops, exclamation marks, or question marks
(#228) Variable "userId" must be all lower-case
(#255) Private method name "EduSharingNodeHelper::getUsageSignatureHeaders" must be in lower-case letters only
(#255) Variable "userId" must be all lower-case
(#260) Variable "userId" must be all lower-case
(#261) Variable "userId" must be all lower-case
(#272) Public method name "EduSharingNodeHelper::getPreview" must be in lower-case letters only
(#281) There should be a comma after the last array item in a multi-line array.
mod/edusharing/apiClient/src/EduSharing/EduSharingNodeHelperConfig.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/>.".
(#11) Opening brace should be on the same line as the declaration for class EduSharingNodeHelperConfig
(#12) Member variable "urlHandling" must be all lower-case
(#19) Variable "urlHandling" must be all lower-case
(#20) Variable "urlHandling" must be all lower-case
mod/edusharing/apiClient/src/EduSharing/InvalidAppIdException.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) Opening brace should be on the same line as the declaration for class InvalidAppIdException
(#16) File must end with a newline character
mod/edusharing/apiClient/src/EduSharing/NodeDeletedException.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) Opening brace should be on the same line as the declaration for class NodeDeletedException
(#16) File must end with a newline character
mod/edusharing/apiClient/src/EduSharing/UrlHandling.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) Opening brace should be on the same line as the declaration for class UrlHandling
(#17) Member variable "endpointURL" must be all lower-case
(#25) Variable "endpointURL" must be all lower-case
(#27) Variable "endpointURL" must be all lower-case
(#29) File must end with a newline character
mod/edusharing/apiClient/src/EduSharing/Usage.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/>.".
(#14) Opening brace should be on the same line as the declaration for class Usage
(#15) Member variable "nodeId" must be all lower-case
(#16) Member variable "nodeVersion" must be all lower-case
(#17) Member variable "containerId" must be all lower-case
(#18) Member variable "resourceId" must be all lower-case
(#19) Member variable "usageId" must be all lower-case
(#30) Variable "nodeId" must be all lower-case
(#30) Variable "nodeVersion" must be all lower-case
(#30) Variable "containerId" must be all lower-case
(#30) Variable "resourceId" must be all lower-case
(#30) Variable "usageId" must be all lower-case
(#31) Variable "nodeId" must be all lower-case
(#32) Variable "nodeVersion" must be all lower-case
(#33) Variable "containerId" must be all lower-case
(#34) Variable "resourceId" must be all lower-case
(#35) Variable "usageId" must be all lower-case
mod/edusharing/apiClient/src/EduSharing/UsageDeletedException.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) Opening brace should be on the same line as the declaration for class UsageDeletedException
(#16) File must end with a newline character
mod/edusharing/classes/EduSharingService.php
(#105) Line exceeds 132 characters; contains 166 characters
(#171) Variable "renderingParams" must be all lower-case
(#172) Variable "renderingParams" must be all lower-case
mod/edusharing/classes/MetaDataFrontend.php
(#58) Line exceeds 132 characters; contains 133 characters
(#65) Line exceeds 132 characters; contains 138 characters
(#97) Line exceeds 132 characters; contains 152 characters
(#103) Line exceeds 132 characters; contains 163 characters
mod/edusharing/classes/MetadataLogic.php
(#174) Line exceeds 132 characters; contains 149 characters
mod/edusharing/classes/MoodleCurlHandler.php
(#42) Public method name "MoodleCurlHandler::handleCurlRequest" must be in lower-case letters only
(#42) Variable "curlOptions" must be all lower-case
(#49) Variable "curlOptions" must be all lower-case
mod/edusharing/classes/PluginRegistrationFrontend.php
(#48) Line exceeds 132 characters; contains 170 characters
(#57) Line exceeds 132 characters; contains 134 characters
(#59) Whitespace found at end of line within string
(#61) Whitespace found at end of line within string
(#63) Whitespace found at end of line within string
mod/edusharing/import_metadata.php
(#33) Expected MOODLE_INTERNAL check or config.php inclusion. Change in global state detected.
(#35) Expected login check (require_login, require_course_login, require_admin, admin_externalpage_setup) following config inclusion. None found.
mod/edusharing/metadata.php
(#29) Expected login check (require_login, require_course_login, require_admin, admin_externalpage_setup) following config inclusion. None found.
mod/edusharing/mod_form.php
(#61) Inline comments must end in full-stops, exclamation marks, or question marks
(#99) Line exceeds 132 characters; contains 149 characters
(#142) Line exceeds 132 characters; contains 137 characters
mod/edusharing/preview.php
(#25) Expected MOODLE_INTERNAL check or config.php inclusion. Change in global state detected.
(#46) Variable "curlResult" must be all lower-case
(#52) Variable "curlResult" must be all lower-case
(#53) Variable "curlResult" must be all lower-case
mod/edusharing/settings.php
(#51) Line exceeds 132 characters; contains 156 characters
mod/edusharing/tests/EdusharingServiceTest.php
(#364) Variable "GLOBALS" must be all lower-case
(#430) Variable "GLOBALS" must be all lower-case
(#476) Variable "GLOBALS" must be all lower-case
(#543) Variable "GLOBALS" must be all lower-case
(#615) Variable "GLOBALS" must be all lower-case
mod/edusharing/tests/MetadataLogicTest.php
(#31) Expected MOODLE_INTERNAL check or config.php inclusion. Change in global state detected.
mod/edusharing/tests/PluginRegistrationTest.php
(#29) Expected MOODLE_INTERNAL check or config.php inclusion. Change in global state detected.
mod/edusharing/tests/UtilityFunctionsTest.php
(#198) Variable "GLOBALS" must be all lower-case
(#199) Line exceeds 132 characters; contains 158 characters
(#217) Variable "GLOBALS" must be all lower-case
(#250) Variable "GLOBALS" must be all lower-case
(#277) Variable "GLOBALS" must be all lower-case

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

(30 errors, 0 warnings)

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

mod/edusharing/import_metadata_style.css
(#13) Expected single space before "{" (block-opening-brace-space-before)
(#17) Expected single space before "{" (block-opening-brace-space-before)
(#21) Expected single space before "{" (block-opening-brace-space-before)
(#29) Expected single space before "{" (block-opening-brace-space-before)
(#49) Expected single space before "{" (block-opening-brace-space-before)
(#64) Expected single space before "{" (block-opening-brace-space-before)
(#68) Expected single space before "{" (block-opening-brace-space-before)
(#79) Expected single space before "{" (block-opening-brace-space-before)
(#83) Expected single space before "{" (block-opening-brace-space-before)
(#87) Expected single space before "{" (block-opening-brace-space-before)
(#92) Expected single space before "{" (block-opening-brace-space-before)
(#96) Expected single space before "{" (block-opening-brace-space-before)
(#105) Expected single space before "{" (block-opening-brace-space-before)
(#111) Expected single space before "{" (block-opening-brace-space-before)
(#117) Expected single space before "{" (block-opening-brace-space-before)
(#125) Expected single space before "{" (block-opening-brace-space-before)
(#130) Expected single space before "{" (block-opening-brace-space-before)
(#136) Expected single space before "{" (block-opening-brace-space-before)
(#139) Expected single space before "{" (block-opening-brace-space-before)
(#144) Expected single space before "{" (block-opening-brace-space-before)
(#148) Expected single space before "{" (block-opening-brace-space-before)
(#154) Expected single space before "{" (block-opening-brace-space-before)
(#161) Expected single space before "{" (block-opening-brace-space-before)
(#167) Expected single space before "{" (block-opening-brace-space-before)
(#173) Expected single space before "{" (block-opening-brace-space-before)
(#177) Expected single space before "{" (block-opening-brace-space-before)
(#8) Expected newline after "," (selector-list-comma-newline-after)
(#8) Expected newline after "," (selector-list-comma-newline-after)
(#8) Expected newline after "," (selector-list-comma-newline-after)
(#45) Expected newline after "," (selector-list-comma-newline-after)

PHPDocs style problems

(82 errors, 17 warnings)

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

mod/edusharing/apiClient/src/EduSharing/AppAuthException.php
(#2) Empty line found after PHP open tag
(#17) Constant AppAuthException::KNOWN_ERRORS is not documented
(#7) Package is not specified for class AppAuthException. It is also not specified in file-level phpdocs
mod/edusharing/apiClient/src/EduSharing/CurlHandler.php
(#2) Empty line found after PHP open tag
(#32) Function CurlHandler::setMethod is not documented
(#21) Variable CurlHandler::$method is not documented
(#15) Constant CurlHandler::METHOD_GET is not documented
(#16) Constant CurlHandler::METHOD_POST is not documented
(#17) Constant CurlHandler::METHOD_PUT is not documented
(#18) Constant CurlHandler::METHOD_DELETE is not documented
(#19) Constant CurlHandler::METHODS is not documented
(#5) Package is not specified for class CurlHandler. It is also not specified in file-level phpdocs
mod/edusharing/apiClient/src/EduSharing/CurlResult.php
(#2) Empty line found after PHP open tag
(#14) Variable CurlResult::$content is not documented
(#15) Variable CurlResult::$error is not documented
(#16) Variable CurlResult::$info is not documented
(#5) Package is not specified for class CurlResult. It is also not specified in file-level phpdocs
mod/edusharing/apiClient/src/EduSharing/DefaultCurlHandler.php
(#2) Empty line found after PHP open tag
(#5) Package is not specified for class DefaultCurlHandler. It is also not specified in file-level phpdocs
mod/edusharing/apiClient/src/EduSharing/DisplayMode.php
(#2) Empty line found after PHP open tag
(#12) Constant DisplayMode::INLINE is not documented
(#13) Constant DisplayMode::EMBED is not documented
(#14) Constant DisplayMode::DYNAMIC is not documented
(#15) Constant DisplayMode::PRERENDER is not documented
(#5) Package is not specified for class DisplayMode. It is also not specified in file-level phpdocs
mod/edusharing/apiClient/src/EduSharing/EduSharingAuthHelper.php
(#2) Empty line found after PHP open tag
(#7) Package is not specified for class EduSharingAuthHelper. It is also not specified in file-level phpdocs
mod/edusharing/apiClient/src/EduSharing/EduSharingHelper.php
(#2) Empty line found after PHP open tag
(#35) Phpdocs for function EduSharingHelper::generateEduAppXMLData has incomplete parameters list
(#7) Package is not specified for class EduSharingHelper. It is also not specified in file-level phpdocs
mod/edusharing/apiClient/src/EduSharing/EduSharingHelperAbstract.php
(#2) Empty line found after PHP open tag
(#12) Variable EduSharingHelperAbstract::$base is not documented
(#5) Package is not specified for class EduSharingHelperAbstract. It is also not specified in file-level phpdocs
mod/edusharing/apiClient/src/EduSharing/EduSharingHelperBase.php
(#2) Empty line found after PHP open tag
(#14) Variable EduSharingHelperBase::$baseUrl is not documented
(#15) Variable EduSharingHelperBase::$privateKey is not documented
(#16) Variable EduSharingHelperBase::$appId is not documented
(#17) Variable EduSharingHelperBase::$language is not documented
(#18) Variable EduSharingHelperBase::$curlHandler is not documented
(#20) There is no description in phpdocs for function __construct
(#7) Package is not specified for class EduSharingHelperBase. It is also not specified in file-level phpdocs
mod/edusharing/apiClient/src/EduSharing/EduSharingNodeHelper.php
(#2) Empty line found after PHP open tag
(#18) Function EduSharingNodeHelper::__construct is not documented
(#16) Variable EduSharingNodeHelper::$config is not documented
(#76) Phpdocs for function EduSharingNodeHelper::getUsageIdByParameters has incomplete parameters list
(#191) Phpdocs for function EduSharingNodeHelper::handleURLMapping has incomplete parameters list
(#71) Invalid phpdocs tag @DEPRECATED used
(#8) Package is not specified for class EduSharingNodeHelper. It is also not specified in file-level phpdocs
mod/edusharing/apiClient/src/EduSharing/EduSharingNodeHelperConfig.php
(#2) Empty line found after PHP open tag
(#12) Variable EduSharingNodeHelperConfig::$urlHandling is not documented
(#5) Package is not specified for class EduSharingNodeHelperConfig. It is also not specified in file-level phpdocs
mod/edusharing/apiClient/src/EduSharing/InvalidAppIdException.php
(#2) Empty line found after PHP open tag
(#7) Package is not specified for class InvalidAppIdException. It is also not specified in file-level phpdocs
mod/edusharing/apiClient/src/EduSharing/NodeDeletedException.php
(#2) Empty line found after PHP open tag
(#7) Package is not specified for class NodeDeletedException. It is also not specified in file-level phpdocs
mod/edusharing/apiClient/src/EduSharing/UrlHandling.php
(#2) Empty line found after PHP open tag
(#16) Variable UrlHandling::$enabled is not documented
(#17) Variable UrlHandling::$endpointURL is not documented
(#5) Package is not specified for class UrlHandling. It is also not specified in file-level phpdocs
mod/edusharing/apiClient/src/EduSharing/Usage.php
(#2) Empty line found after PHP open tag
(#15) Variable Usage::$nodeId is not documented
(#16) Variable Usage::$nodeVersion is not documented
(#17) Variable Usage::$containerId is not documented
(#18) Variable Usage::$resourceId is not documented
(#19) Variable Usage::$usageId is not documented
(#5) Package is not specified for class Usage. It is also not specified in file-level phpdocs
mod/edusharing/apiClient/src/EduSharing/UsageDeletedException.php
(#2) Empty line found after PHP open tag
(#7) Package is not specified for class UsageDeletedException. It is also not specified in file-level phpdocs
mod/edusharing/tests/EdusharingServiceTest.php
(#47) Invalid phpdocs tag @backupGlobals used
(#71) Invalid phpdocs tag @backupGlobals used
(#97) Invalid phpdocs tag @backupGlobals used
(#131) Invalid phpdocs tag @backupGlobals used
(#307) Invalid phpdocs tag @backupGlobals used
(#373) Invalid phpdocs tag @backupGlobals used
(#439) Invalid phpdocs tag @backupGlobals used
(#504) Invalid phpdocs tag @backupGlobals used
(#593) Invalid phpdocs tag @backupGlobals used
(#643) Invalid phpdocs tag @backupGlobals used
(#778) Invalid phpdocs tag @backupGlobals used
(#816) Invalid phpdocs tag @backupGlobals used
mod/edusharing/tests/MetadataLogicTest.php
(#48) Invalid phpdocs tag @backupGlobals used
(#101) Invalid phpdocs tag @backupGlobals used
(#137) Invalid phpdocs tag @backupGlobals used
(#174) Invalid phpdocs tag @backupGlobals used
(#209) Invalid phpdocs tag @backupGlobals used
(#246) Invalid phpdocs tag @backupGlobals used
(#284) Invalid phpdocs tag @backupGlobals used
(#322) Invalid phpdocs tag @backupGlobals used
(#361) Invalid phpdocs tag @backupGlobals used
(#398) Invalid phpdocs tag @backupGlobals used
mod/edusharing/tests/UtilityFunctionsTest.php
(#79) Invalid phpdocs tag @backupGlobals used
(#99) Invalid phpdocs tag @backupGlobals used
(#120) Invalid phpdocs tag @backupGlobals used
(#141) Invalid phpdocs tag @backupGlobals used
(#162) Invalid phpdocs tag @backupGlobals used
(#183) Invalid phpdocs tag @backupGlobals used
(#208) Invalid phpdocs tag @backupGlobals used
(#226) Invalid phpdocs tag @backupGlobals used
(#261) Invalid phpdocs tag @backupGlobals used

Update savepoints problems

(1 errors, 0 warnings)

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

mod/edusharing/db/upgrade.php
'return true;' not found

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]