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

(10 errors, 24 warnings)

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

course/format/topcoll/js/tc_colourpopup.php
(#34) Expected MOODLE_INTERNAL check or config.php inclusion. Change in global state detected.
(#50) Member variable "_hiddenLabel" must be all lower-case
(#57) Public method name "MoodleQuickForm_tccolourpopup::setHiddenLabel" must be in lower-case letters only
(#57) Variable "hiddenLabel" must be all lower-case
(#58) Variable "hiddenLabel" must be all lower-case
(#61) Public method name "MoodleQuickForm_tccolourpopup::toHtml" must be in lower-case letters only
(#92) Public method name "MoodleQuickForm_tccolourpopup::_generateId" must be in lower-case letters only
(#106) Public method name "MoodleQuickForm_tccolourpopup::setHelpButton" must be in lower-case letters only
(#115) Public method name "MoodleQuickForm_tccolourpopup::getHelpButton" must be in lower-case letters only
(#126) Public method name "MoodleQuickForm_tccolourpopup::getElementTemplateType" must be in lower-case letters only
course/format/topcoll/lang/en/format_topcoll.php
(#82) This comment is 50% valid code; is this commented out code?
(#83) This comment is 50% valid code; is this commented out code?
(#84) This comment is 50% valid code; is this commented out code?
(#85) This comment is 50% valid code; is this commented out code?
(#86) This comment is 50% valid code; is this commented out code?
(#87) This comment is 50% valid code; is this commented out code?
(#90) This comment is 50% valid code; is this commented out code?
(#91) This comment is 50% valid code; is this commented out code?
(#92) This comment is 50% valid code; is this commented out code?
(#93) This comment is 50% valid code; is this commented out code?
(#94) This comment is 50% valid code; is this commented out code?
(#95) This comment is 50% valid code; is this commented out code?
(#96) This comment is 50% valid code; is this commented out code?
(#97) This comment is 50% valid code; is this commented out code?
course/format/topcoll/lang/en_ar/format_topcoll.php
(#54) This comment is 50% valid code; is this commented out code?
(#55) This comment is 50% valid code; is this commented out code?
(#56) This comment is 50% valid code; is this commented out code?
(#59) This comment is 50% valid code; is this commented out code?
(#60) This comment is 50% valid code; is this commented out code?
(#61) This comment is 50% valid code; is this commented out code?
(#62) This comment is 50% valid code; is this commented out code?
course/format/topcoll/renderer.php
(#916) Line exceeds 132 characters; contains 133 characters
course/format/topcoll/settopcollpref.php
(#32) Expected login check (require_login, require_course_login, admin_externalpage_setup) following config inclusion. None found.
course/format/topcoll/version.php
(#39) This comment is 45% valid code; is this commented out code?

Javascript coding style problems

(17 errors, 115 warnings)

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

course/format/topcoll/format.js
(#46) Identifier 'get_config' is not in camel case. (camelcase)
(#48) Identifier 'container_node' is not in camel case. (camelcase)
(#48) Extra space after key 'container_node'. (key-spacing)
(#49) Extra space after key 'container_class'. (key-spacing)
(#49) Identifier 'container_class' is not in camel case. (camelcase)
(#50) Identifier 'section_node' is not in camel case. (camelcase)
(#50) Extra space after key 'section_node'. (key-spacing)
(#51) Identifier 'section_class' is not in camel case. (camelcase)
(#51) Extra space after key 'section_class'. (key-spacing)
(#55) Unexpected @return tag; function has no return statement. (valid-jsdoc)
(#63) Identifier 'swap_sections' is not in camel case. (camelcase)
(#65) Extra space after key 'COURSECONTENT'. (key-spacing)
(#66) Extra space after key 'SECTIONADDMENUS'. (key-spacing)
(#74) Unexpected @return tag; function has no return statement. (valid-jsdoc)
(#74) Expected JSDoc for 'sectionlist' but found 'response'. (valid-jsdoc)
(#74) Expected JSDoc for 'response' but found 'sectionfrom'. (valid-jsdoc)
(#74) Expected JSDoc for 'sectionfrom' but found 'sectionto'. (valid-jsdoc)
(#83) Identifier 'process_sections' is not in camel case. (camelcase)
(#85) Extra space after key 'SECTIONNAME'. (key-spacing)
(#88) Extra space after key 'LEFTCONTENT'. (key-spacing)
(#89) Extra space after key 'SECTIONLEFTSIDE'. (key-spacing)
course/format/topcoll/js/tc_colourpopup.js
(#34) Identifier 'init_tccolour_popup' is not in camel case. (camelcase)
(#35) Missing space before opening brace. (space-before-blocks)
(#40) Extra space after key 'box'. (key-spacing)
(#41) Extra space after key 'input'. (key-spacing)
(#42) Extra space after key 'swatch'. (key-spacing)
(#43) Extra space after key 'image'. (key-spacing)
(#44) Extra space after key 'preview'. (key-spacing)
(#45) Extra space after key 'current'. (key-spacing)
(#46) Extra space after key 'eventClick'. (key-spacing)
(#47) Extra space after key 'eventFocus'. (key-spacing)
(#48) Extra space after key 'eventMouseEnter'. (key-spacing)
(#49) Extra space after key 'eventMouseLeave'. (key-spacing)
(#50) Extra space after key 'eventMouseMove'. (key-spacing)
(#51) Extra space after key 'width'. (key-spacing)
(#52) Extra space after key 'height'. (key-spacing)
(#53) Extra space after key 'factor'. (key-spacing)
(#57) Extra space after key 'init'. (key-spacing)
(#64) Line 64 exceeds the maximum line length of 132. (max-len)
(#66) Line 66 exceeds the maximum line length of 132. (max-len)
(#68) Expected space or tab after '//' in comment. (spaced-comment)
(#70) Unary word operator 'typeof' must be followed by whitespace. (space-unary-ops)
(#71) 'e' is defined but never used. (no-unused-vars)
(#71) Missing space before opening brace. (space-before-blocks)
(#81) A space is required after ','. (comma-spacing)
(#81) A space is required after ','. (comma-spacing)
(#86) 'e' is defined but never used. (no-unused-vars)
(#86) Missing space before opening brace. (space-before-blocks)
(#89) Extra space after key 'showColours'. (key-spacing)
(#89) 'e' is defined but never used. (no-unused-vars)
(#89) Missing space before opening brace. (space-before-blocks)
(#93) Extra space after key 'setColour'. (key-spacing)
(#93) 'e' is defined but never used. (no-unused-vars)
(#93) Missing space before opening brace. (space-before-blocks)
(#97) Extra space after key 'startFollow'. (key-spacing)
(#97) 'e' is defined but never used. (no-unused-vars)
(#100) Missing space before opening brace. (space-before-blocks)
(#102) Expected space or tab after '//' in comment. (spaced-comment)
(#106) Missing JSDoc for parameter 'e'. (valid-jsdoc)
(#109) Extra space after key 'endFollow'. (key-spacing)
(#109) 'e' is defined but never used. (no-unused-vars)
(#112) A space is required after ','. (comma-spacing)
(#115) Missing JSDoc for parameter 'e'. (valid-jsdoc)
(#118) Extra space after key 'pickColour'. (key-spacing)
(#126) Missing JSDoc for parameter 'e'. (valid-jsdoc)
(#126) Missing JSDoc @returns for function. (valid-jsdoc)
(#129) Extra space after key 'determineColour'. (key-spacing)
(#136) A space is required after ','. (comma-spacing)
(#136) A space is required after ','. (comma-spacing)
(#139) There should be no space after '['. (array-bracket-spacing)
(#139) Multiple spaces found before '0'. (no-multi-spaces)
(#139) Multiple spaces found before '1'. (no-multi-spaces)
(#139) Multiple spaces found before '0'. (no-multi-spaces)
(#140) There should be no space after '['. (array-bracket-spacing)
(#140) Multiple spaces found before '0'. (no-multi-spaces)
(#140) Multiple spaces found before '0'. (no-multi-spaces)
(#141) There should be no space after '['. (array-bracket-spacing)
(#141) Multiple spaces found before '0'. (no-multi-spaces)
(#141) Multiple spaces found before '0'. (no-multi-spaces)
(#141) Multiple spaces found before '1'. (no-multi-spaces)
(#142) There should be no space after '['. (array-bracket-spacing)
(#142) Multiple spaces found before '0'. (no-multi-spaces)
(#142) Multiple spaces found before '0'. (no-multi-spaces)
(#143) There should be no space after '['. (array-bracket-spacing)
(#143) Multiple spaces found before '1'. (no-multi-spaces)
(#143) Multiple spaces found before '0'. (no-multi-spaces)
(#143) Multiple spaces found before '0'. (no-multi-spaces)
(#144) There should be no space after '['. (array-bracket-spacing)
(#144) Multiple spaces found before '0'. (no-multi-spaces)
(#144) Multiple spaces found before '0'. (no-multi-spaces)
(#177) Missing JSDoc for parameter 'rgb'. (valid-jsdoc)
(#177) Missing JSDoc @returns for function. (valid-jsdoc)
(#180) Identifier 'convert_rgb_to_hex' is not in camel case. (camelcase)
(#180) Extra space after key 'convert_rgb_to_hex'. (key-spacing)
course/format/topcoll/module.js
(#35) Identifier 'format_topcoll' is not in camel case. (camelcase)
(#180) Identifier 'toggle_topic' is not in camel case. (camelcase)
(#228) A space is required after ','. (comma-spacing)
(#228) A space is required after ','. (comma-spacing)
(#229) A space is required after ','. (comma-spacing)
(#229) A space is required after ','. (comma-spacing)
(#236) A space is required after ','. (comma-spacing)
(#240) A space is required after ','. (comma-spacing)
(#248) Identifier 'save_toggles' is not in camel case. (camelcase)
(#256) Identifier 'is_old_preference' is not in camel case. (camelcase)
(#268) Identifier 'convert_to_new_preference' is not in camel case. (camelcase)
(#273) 'logbintext' is defined but never used. (no-unused-vars)
(#286) Unexpected use of '<<'. (no-bitwise)
(#291) Missing JSDoc for parameter 'togglenum'. (valid-jsdoc)
(#291) Missing JSDoc for parameter 'state'. (valid-jsdoc)
(#296) Identifier 'set_toggle_state' is not in camel case. (camelcase)
(#303) Unexpected use of '|='. (no-bitwise)
(#307) Unexpected use of '&='. (no-bitwise)
(#307) Unexpected use of '~'. (no-bitwise)
(#311) A space is required after ','. (comma-spacing)
(#316) Identifier 'get_required_digits' is not in camel case. (camelcase)
(#321) Identifier 'get_toggle_pos' is not in camel case. (camelcase)
(#326) Identifier 'get_min_digit' is not in camel case. (camelcase)
(#331) Identifier 'get_max_digit' is not in camel case. (camelcase)
(#336) Identifier 'get_toggle_flag' is not in camel case. (camelcase)
(#363) Identifier 'decode_character_to_value' is not in camel case. (camelcase)
(#368) Identifier 'encode_value_to_character' is not in camel case. (camelcase)
(#373) Missing JSDoc parameter type for 'String'. (valid-jsdoc)
(#373) Expected JSDoc for 'name' but found 'String'. (valid-jsdoc)
(#373) Missing JSDoc for parameter 'value'. (valid-jsdoc)
(#373) Duplicate JSDoc parameter 'String'. (valid-jsdoc)
(#373) Missing JSDoc parameter type for 'String'. (valid-jsdoc)
(#384) Identifier 'set_user_preference' is not in camel case. (camelcase)
(#385) 'YUI' is not defined. (no-undef)
(#385) A function with a name starting with an uppercase letter should only be used as a constructor. (new-cap)
(#386) Line 386 exceeds the maximum line length of 132. (max-len)
(#395) Line 395 exceeds the maximum line length of 132. (max-len)
(#395) Unexpected console statement. (no-console)

CSS problems

(21 errors, 3 warnings)

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

course/format/topcoll/styles.css
(#491) Expected single space before "{" (block-opening-brace-space-before)
(#527) Expected single space before "{" (block-opening-brace-space-before)
(#562) Expected single space before "{" (block-opening-brace-space-before)
(#603) Expected single space before "{" (block-opening-brace-space-before)
(#616) Expected single space before "{" (block-opening-brace-space-before)
(#503) Expected single space after ":" with a single-line declaration (declaration-colon-space-after)
(#516) Expected single space after ":" with a single-line declaration (declaration-colon-space-after)
(#664) Expected single space after ":" with a single-line declaration (declaration-colon-space-after)
(#74) Expected line length to be no more than 132 characters (max-line-length)
(#468) Expected line length to be no more than 132 characters (max-line-length)
(#475) Expected line length to be no more than 132 characters (max-line-length)
(#725) Expected single space after ">" (selector-combinator-space-after)
(#725) Expected single space before ">" (selector-combinator-space-before)
(#149) Expected newline after "," (selector-list-comma-newline-after)
(#495) Expected newline after "," (selector-list-comma-newline-after)
(#496) Expected newline after "," (selector-list-comma-newline-after)
(#502) Expected newline after "," (selector-list-comma-newline-after)
(#668) Expected newline after "," (selector-list-comma-newline-after)
(#672) Expected newline after "," (selector-list-comma-newline-after)
(#709) Expected newline after "," (selector-list-comma-newline-after)
(#713) Expected newline after "," (selector-list-comma-newline-after)
(#769) Expected newline after "," (selector-list-comma-newline-after)
(#769) Expected newline after "," (selector-list-comma-newline-after)
(#388) Unexpected whitespace before "," (selector-list-comma-space-before)

PHPDocs style problems

(103 errors, 22 warnings)

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

course/format/topcoll/backup/moodle2/restore_format_topcoll_plugin.class.php
(#100) Function restore_format_topcoll_plugin::after_execute_structure is not documented
(#37) No one-line description found in phpdocs for class restore_format_topcoll_plugin
(#60) Phpdocs for function restore_format_topcoll_plugin::process_topcoll has incomplete parameters list
(#37) Class restore_format_topcoll_plugin does not have @copyright tag
(#37) Class restore_format_topcoll_plugin does not have @license tag
(#27) Not recommended phpdocs tag @version used
(#25) Package course/format is not valid
course/format/topcoll/classes/observer.php
(#38) Class format_topcoll_observer does not have @copyright tag
(#38) Class format_topcoll_observer does not have @license tag
(#28) Not recommended phpdocs tag @version used
(#25) Package course/format is not valid
course/format/topcoll/classes/toolbox.php
(#39) Class toolbox is not documented
(#43) Function toolbox::__construct is not documented
(#46) Function toolbox::get_instance is not documented
(#41) Variable toolbox::$instance is not documented
(#27) Not recommended phpdocs tag @version used
(#25) Package course/format is not valid
course/format/topcoll/config.php
(#27) Not recommended phpdocs tag @version used
(#25) Package course/format is not valid
course/format/topcoll/db/access.php
(#27) Not recommended phpdocs tag @version used
(#25) Package course/format is not valid
course/format/topcoll/db/events.php
(#28) Not recommended phpdocs tag @version used
(#25) Package course/format is not valid
course/format/topcoll/db/upgrade.php
(#40) Function xmldb_format_topcoll_upgrade is not documented
(#27) Not recommended phpdocs tag @version used
(#25) Package course/format is not valid
course/format/topcoll/format.php
(#27) Not recommended phpdocs tag @version used
(#25) Package course/format is not valid
course/format/topcoll/js/tc_colourpopup.php
(#52) Function MoodleQuickForm_tccolourpopup::__construct is not documented
(#57) Function MoodleQuickForm_tccolourpopup::setHiddenLabel is not documented
(#61) Function MoodleQuickForm_tccolourpopup::toHtml is not documented
(#49) Variable MoodleQuickForm_tccolourpopup::$_helpbutton is not documented
(#50) Variable MoodleQuickForm_tccolourpopup::$_hiddenLabel is not documented
(#103) Phpdocs for function MoodleQuickForm_tccolourpopup::setHelpButton has incomplete parameters list
(#36) Class MoodleQuickForm_tccolourpopup does not have @copyright tag
(#36) Class MoodleQuickForm_tccolourpopup does not have @license tag
(#27) Not recommended phpdocs tag @version used
(#40) Not recommended phpdocs tag @access used
(#25) Package course/format is not valid
course/format/topcoll/lang/en/format_topcoll.php
(#27) Not recommended phpdocs tag @version used
(#25) Package course/format is not valid
course/format/topcoll/lang/en_ar/format_topcoll.php
(#27) Not recommended phpdocs tag @version used
(#25) Package course/format is not valid
course/format/topcoll/lang/en_us/format_topcoll.php
(#27) Not recommended phpdocs tag @version used
(#25) Package course/format is not valid
course/format/topcoll/lib.php
(#38) Class format_topcoll is not documented
(#179) Function format_topcoll::get_section_dates is not documented
(#304) Function format_topcoll::section_format_options is not documented
(#1306) Function format_topcoll::get_context is not documented
(#40) Variable format_topcoll::$coursedisplay is not documented
(#41) Variable format_topcoll::$settings is not documented
(#65) Phpdocs for function format_topcoll::get_settings has incomplete parameters list
(#27) Not recommended phpdocs tag @version used
(#25) Package course/format is not valid
course/format/topcoll/renderer.php
(#40) Class format_topcoll_renderer is not documented
(#542) Function format_topcoll_renderer::section_summary_container is not documented
(#1083) Function format_topcoll_renderer::set_portable is not documented
(#1098) Function format_topcoll_renderer::set_user_preference is not documented
(#1151) Function format_topcoll_renderer::get_row_class is not documented
(#1159) Function format_topcoll_renderer::get_column_class is not documented
(#1173) Function format_topcoll_renderer::get_format_responsive is not documented
(#42) Variable format_topcoll_renderer::$tccolumnwidth is not documented
(#43) Variable format_topcoll_renderer::$tccolumnpadding is not documented
(#44) Variable format_topcoll_renderer::$mobiletheme is not documented
(#45) Variable format_topcoll_renderer::$tablettheme is not documented
(#46) Variable format_topcoll_renderer::$courseformat is not documented
(#47) Variable format_topcoll_renderer::$tcsettings is not documented
(#48) Variable format_topcoll_renderer::$defaulttogglepersistence is not documented
(#49) Variable format_topcoll_renderer::$defaultuserpreference is not documented
(#50) Variable format_topcoll_renderer::$togglelib is not documented
(#51) Variable format_topcoll_renderer::$currentsection is not documented
(#52) Variable format_topcoll_renderer::$isoldtogglepreference is not documented
(#53) Variable format_topcoll_renderer::$userisediting is not documented
(#54) Variable format_topcoll_renderer::$tctoggleiconsize is not documented
(#55) Variable format_topcoll_renderer::$formatresponsive is not documented
(#56) Variable format_topcoll_renderer::$rtl is not documented
(#57) Variable format_topcoll_renderer::$bsnewgrid is not documented
(#62) Phpdocs for function format_topcoll_renderer::__construct has incomplete parameters list
(#27) Not recommended phpdocs tag @version used
(#25) Package course/format is not valid
course/format/topcoll/settings.php
(#27) Not recommended phpdocs tag @version used
(#25) Package course/format is not valid
course/format/topcoll/settopcollpref.php
(#25) Not recommended phpdocs tag @version used
(#23) Package course/format is not valid
course/format/topcoll/tests/courseformatlib_test.php
(#36) Variable format_topcoll_courseformatlib_testcase::$course is not documented
(#37) Variable format_topcoll_courseformatlib_testcase::$courseformat is not documented
(#30) Class format_topcoll_courseformatlib_testcase does not have @copyright tag
(#30) Class format_topcoll_courseformatlib_testcase does not have @license tag
(#32) Invalid phpdocs tag @group used
(#22) Not recommended phpdocs tag @version used
(#20) Package course/format is not valid
course/format/topcoll/tests/courseformatrenderer_test.php
(#36) Variable format_topcoll_courseformatrenderer_testcase::$outputus is not documented
(#37) Variable format_topcoll_courseformatrenderer_testcase::$course is not documented
(#38) Variable format_topcoll_courseformatrenderer_testcase::$courseformat is not documented
(#30) Class format_topcoll_courseformatrenderer_testcase does not have @copyright tag
(#30) Class format_topcoll_courseformatrenderer_testcase does not have @license tag
(#32) Invalid phpdocs tag @group used
(#22) Not recommended phpdocs tag @version used
(#20) Package course/format is not valid
course/format/topcoll/tests/courseformattoolbox_test.php
(#30) Class format_topcoll_courseformattoolbox_testcase does not have @copyright tag
(#30) Class format_topcoll_courseformattoolbox_testcase does not have @license tag
(#32) Invalid phpdocs tag @group used
(#22) Not recommended phpdocs tag @version used
(#20) Package course/format is not valid
course/format/topcoll/togglelib.php
(#36) Class topcoll_togglelib is not documented
(#49) Variable topcoll_togglelib::$toggles is not documented
(#42) Constant topcoll_togglelib::TOGGLE_6 is not documented
(#43) Constant topcoll_togglelib::TOGGLE_5 is not documented
(#44) Constant topcoll_togglelib::TOGGLE_4 is not documented
(#45) Constant topcoll_togglelib::TOGGLE_3 is not documented
(#46) Constant topcoll_togglelib::TOGGLE_2 is not documented
(#47) Constant topcoll_togglelib::TOGGLE_1 is not documented
(#51) Phpdocs for function topcoll_togglelib::set_toggles has incomplete parameters list
(#69) Phpdocs for function topcoll_togglelib::get_toggle_state has incomplete parameters list
(#80) Phpdocs for function topcoll_togglelib::set_toggle_state has incomplete parameters list
(#97) Phpdocs for function topcoll_togglelib::is_old_preference has incomplete parameters list
(#113) Phpdocs for function topcoll_togglelib::get_required_digits has incomplete parameters list
(#138) Phpdocs for function topcoll_togglelib::get_toggle_pos has incomplete parameters list
(#147) Phpdocs for function topcoll_togglelib::get_toggle_flag has incomplete parameters list
(#178) Phpdocs for function topcoll_togglelib::decode_character_to_value has incomplete parameters list
(#187) Phpdocs for function topcoll_togglelib::encode_value_to_character has incomplete parameters list
(#27) Not recommended phpdocs tag @version used
(#25) Package course/format is not valid
course/format/topcoll/version.php
(#27) Not recommended phpdocs tag @version used
(#25) Package course/format is not valid

Update savepoints problems

(2 errors, 0 warnings)

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

course/format/topcoll/db/upgrade.php
Detected more 'if' blocks (1) than 'savepoint' calls (0)
version 2114052000 is missing corresponding savepoint call

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]

Task "stylelint:css" failed. Use --force to continue.
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]