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

(0 errors, 5 warnings)

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

auth/saml2/autologin.php
(#25) Expected login check (require_login, require_course_login, require_admin, admin_externalpage_setup) following config inclusion. None found.
(#34) Function print_error() has been deprecated; use throw new moodle_exception() (using lang strings only if meant to be shown to final user)() instead
auth/saml2/classes/privacy/provider.php
(#50) Method name "_get_reason" should not be prefixed with an underscore to indicate visibility
auth/saml2/tests/fixtures/mockidp/slo.php
(#33) Expected login check (require_login, require_course_login, require_admin, admin_externalpage_setup) following config inclusion. None found.
auth/saml2/tests/fixtures/mockidp/sso.php
(#35) Expected login check (require_login, require_course_login, require_admin, admin_externalpage_setup) following config inclusion. None found.

Javascript coding style problems

(0 errors, 1 warnings)

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

auth/saml2/sp/resources/post.js
(#5) Unexpected space before function parentheses. (space-before-function-paren)

CSS problems

(0 errors, 0 warnings)

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


PHPDocs style problems

(188 errors, 1 warnings)

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

auth/saml2/auth.php
(#110) Function auth_plugin_saml2::saml2_authproc_filters_hook is not documented
(#129) Function auth_plugin_saml2::check_filters_priority is not documented
(#141) Function auth_plugin_saml2::get_saml2_directory is not documented
(#150) Function auth_plugin_saml2::get_file is not documented
(#154) Function auth_plugin_saml2::get_file_sp_metadata_file is not documented
(#1071) Function auth_plugin_saml2::can_be_manually_set is not documented
(#158) There is no description in phpdocs for function get_file_idp_metadata_file
(#584) Phpdocs for function auth_plugin_saml2::saml_login_complete has incomplete parameters list
(#822) Phpdocs for function auth_plugin_saml2::simplify_attr has incomplete parameters list
(#846) Phpdocs for function auth_plugin_saml2::update_user_profile_fields has incomplete parameters list
(#928) Phpdocs for function auth_plugin_saml2::is_email_taken has incomplete parameters list
(#1019) Phpdocs for function auth_plugin_saml2::user_login has incomplete parameters list
(#1125) Phpdocs for function auth_plugin_saml2::execute_callback has incomplete parameters list
(#1152) Phpdocs for function auth_saml2_after_logout_from_sp has incomplete parameters list
auth/saml2/classes/admin/saml2_settings.php
(#35) Constant saml2_settings::OPTION_DUAL_LOGIN_NO is not documented
(#37) Constant saml2_settings::OPTION_DUAL_LOGIN_YES is not documented
(#39) Constant saml2_settings::OPTION_DUAL_LOGIN_PASSIVE is not documented
(#41) Constant saml2_settings::OPTION_MULTI_IDP_DISPLAY_DROPDOWN is not documented
(#43) Constant saml2_settings::OPTION_MULTI_IDP_DISPLAY_BUTTONS is not documented
(#45) Constant saml2_settings::OPTION_FLAGGED_LOGIN_MESSAGE is not documented
(#47) Constant saml2_settings::OPTION_FLAGGED_LOGIN_REDIRECT is not documented
(#49) Constant saml2_settings::OPTION_AUTO_LOGIN_NO is not documented
(#51) Constant saml2_settings::OPTION_AUTO_LOGIN_SESSION is not documented
(#53) Constant saml2_settings::OPTION_AUTO_LOGIN_COOKIE is not documented
(#17) No one-line description found in phpdocs for file
(#28) No one-line description found in phpdocs for class saml2_settings
auth/saml2/classes/admin/setting_button.php
(#57) Phpdocs for function setting_button::output_html has incomplete parameters list
auth/saml2/classes/admin/setting_idpmetadata.php
(#46) Function setting_idpmetadata::__construct is not documented
(#106) Function setting_idpmetadata::process_idp_metadata is not documented
(#121) Function setting_idpmetadata::process_idp_xml is not documented
(#171) Function setting_idpmetadata::remove_old_idps is not documented
(#249) Function setting_idpmetadata::save_idp_metadata_xml is not documented
(#17) No one-line description found in phpdocs for file
(#82) There is no description in phpdocs for function process_all_idps_metadata
(#181) There is no description in phpdocs for function get_idps_data
(#211) There is no description in phpdocs for function get_idp_xml_path
(#240) There is no description in phpdocs for function find_all_idp_sso_descriptors
(#182) Phpdocs for function setting_idpmetadata::get_idps_data has incomplete parameters list
(#212) Phpdocs for function setting_idpmetadata::get_idp_xml_path has incomplete parameters list
auth/saml2/classes/admin/setting_idpmetadata_exception.php
(#30) Class setting_idpmetadata_exception is not documented
(#17) No one-line description found in phpdocs for file
auth/saml2/classes/admin/setting_textonly.php
(#38) Phpdocs for function setting_textonly::output_html has incomplete parameters list
auth/saml2/classes/event/cert_regenerated.php
(#17) File-level phpdocs block does not have @copyright tag
auth/saml2/classes/form/selectidp_buttons.php
(#72) Function selectidp_buttons::get_idpbutton is not documented
auth/saml2/classes/idp_data.php
(#30) Package is not specified for class idp_data. It is also not specified in file-level phpdocs
(#30) Class idp_data is not documented
(#53) Function idp_data::get_rawxml is not documented
(#57) Function idp_data::set_rawxml is not documented
(#31) Variable idp_data::$idpname is not documented
(#33) Variable idp_data::$idpurl is not documented
(#35) Variable idp_data::$idpicon is not documented
(#37) Variable idp_data::$rawxml is not documented
(#42) Phpdocs for function idp_data::__construct has incomplete parameters list
auth/saml2/classes/idp_parser.php
(#29) Class idp_parser is not documented
(#79) Function idp_parser::parse_xml is not documented
(#85) Function idp_parser::parse_urls is not documented
(#44) Phpdocs for function idp_parser::parse has incomplete parameters list
(#59) Phpdocs for function idp_parser::check_xml has incomplete parameters list
auth/saml2/classes/metadata_fetcher.php
(#54) There is no description in phpdocs for function fetch
(#101) There is no description in phpdocs for function get_curlinfo
(#108) There is no description in phpdocs for function get_curlerrorno
(#115) There is no description in phpdocs for function get_curlerror
(#55) Phpdocs for function metadata_fetcher::fetch has incomplete parameters list
auth/saml2/classes/metadata_parser.php
(#47) There is no description in phpdocs for function parse
(#73) There is no description in phpdocs for function get_entityid
(#80) There is no description in phpdocs for function get_idpdefaultname
auth/saml2/classes/metadata_writer.php
(#55) There is no description in phpdocs for function write
auth/saml2/classes/redis_store.php
(#51) Function redis_store::__construct is not documented
(#62) There is no description in phpdocs for function set
(#72) There is no description in phpdocs for function get
(#86) There is no description in phpdocs for function delete
(#126) There is no description in phpdocs for function make_key
(#135) There is no description in phpdocs for function get_set_options
auth/saml2/classes/ssl_algorithms.php
(#30) Class ssl_algorithms is not documented
(#34) Function ssl_algorithms::get_default_saml_signature_algorithm is not documented
(#42) Function ssl_algorithms::get_valid_saml_signature_algorithms is not documented
(#54) Function ssl_algorithms::convert_signature_algorithm_to_digest_alg_format is not documented
(#17) No one-line description found in phpdocs for file
auth/saml2/classes/task/metadata_refresh.php
(#69) Function metadata_refresh::get_name is not documented
(#73) Function metadata_refresh::execute is not documented
(#106) There is no description in phpdocs for function set_fetcher
(#113) There is no description in phpdocs for function set_parser
(#120) There is no description in phpdocs for function set_writer
(#127) There is no description in phpdocs for function set_idpmetadata
auth/saml2/classes/test/mock_settings.php
(#42) Function mock_settings::add is not documented
(#17) No one-line description found in phpdocs for file
(#32) No one-line description found in phpdocs for class mock_settings
auth/saml2/lib.php
(#17) No one-line description found in phpdocs for file
auth/saml2/locallib.php
(#480) Phpdocs for function auth_saml2_process_regenerate_form has incomplete parameters list
(#518) Phpdocs for function auth_saml2_admin_nav has incomplete parameters list
auth/saml2/tests/behat/behat_auth_saml2.php
(#119) Function behat_auth_saml2::apply_defaults is not documented
(#149) Function behat_auth_saml2::initialise_saml2 is not documented
(#326) Function behat_auth_saml2::visit_saml2_login_page is not documented
(#330) Function behat_auth_saml2::reset_saml2_session is not documented
(#335) Function behat_auth_saml2::reset_moodle_session is not documented
(#340) Function behat_auth_saml2::execute is not documented
(#17) No one-line description found in phpdocs for file
(#35) No one-line description found in phpdocs for class behat_auth_saml2
(#43) There is no description in phpdocs for function theAuthenticationPluginIsEnabledAuth_saml
(#66) There is no description in phpdocs for function iGoToTheLoginPageAuth_saml
(#83) There is no description in phpdocs for function iGoToTheLoginPageWithAuth_saml
(#90) There is no description in phpdocs for function iAmAnAdministratorAuth_saml
(#97) There is no description in phpdocs for function iGoToTheSamlsettingsPageAuth_saml
(#105) There is no description in phpdocs for function iChangeTheSettingToAuth_saml
(#112) There is no description in phpdocs for function theSettingShouldBeAuth_saml
(#154) There is no description in phpdocs for function theSamlsettingIsSetToAuth_saml
(#43) Phpdocs for function behat_auth_saml2::theAuthenticationPluginIsEnabledAuth_saml has incomplete parameters list
(#66) Phpdocs for function behat_auth_saml2::iGoToTheLoginPageAuth_saml has incomplete parameters list
(#83) Phpdocs for function behat_auth_saml2::iGoToTheLoginPageWithAuth_saml has incomplete parameters list
(#105) Phpdocs for function behat_auth_saml2::iChangeTheSettingToAuth_saml has incomplete parameters list
(#112) Phpdocs for function behat_auth_saml2::theSettingShouldBeAuth_saml has incomplete parameters list
(#154) Phpdocs for function behat_auth_saml2::theSamlsettingIsSetToAuth_saml has incomplete parameters list
(#245) Phpdocs for function behat_auth_saml2::the_mock_saml_idp_allows_login_with_the_following_attributes has incomplete parameters list
(#306) Phpdocs for function behat_auth_saml2::the_cookie_is_set_to has incomplete parameters list
(#316) Phpdocs for function behat_auth_saml2::the_cookie_is_removed has incomplete parameters list
(#22) Invalid phpdocs tag @codingStandardsIgnoreFile used
(#40) Invalid phpdocs tag @SuppressWarnings(public) used
auth/saml2/tests/fixtures/mockidp/slo.php
(#17) No one-line description found in phpdocs for file
auth/saml2/tests/fixtures/mockidp/sso.php
(#17) No one-line description found in phpdocs for file
auth/saml2/tests/phpunit/autoloader_test.php
(#35) Function auth_saml2_autoloader_test::test_it_loads_classes is not documented
(#17) No one-line description found in phpdocs for file
(#28) No one-line description found in phpdocs for class auth_saml2_autoloader_test
auth/saml2/tests/phpunit/form_regenerate_test.php
(#35) Function auth_saml2_form_regenerate_testcase::test_regenerate_certificate_form is not documented
(#17) File-level phpdocs block does not have @copyright tag
auth/saml2/tests/phpunit/locallib_test.php
(#60) Function auth_saml2_locallib_testcase::test_auth_saml2_sp_metadata is not documented
(#529) Function auth_saml2_locallib_testcase::test_is_configured_works_with_multi_idp_in_one_xml is not documented
(#561) Function auth_saml2_locallib_testcase::test_get_email_from_attributes is not documented
(#582) Function auth_saml2_locallib_testcase::test_is_email_taken is not documented
(#279) Phpdocs for function auth_saml2_locallib_testcase::test_check_whitelisted_ip_redirect has incomplete parameters list
(#325) Phpdocs for function auth_saml2_locallib_testcase::test_update_custom_user_profile_fields has incomplete parameters list
(#384) Phpdocs for function auth_saml2_locallib_testcase::test_missing_user_custom_profile_fields has incomplete parameters list
(#432) Phpdocs for function auth_saml2_locallib_testcase::test_invalid_map_user_profile_fields has incomplete parameters list
(#668) Phpdocs for function auth_saml2_locallib_testcase::test_is_access_allowed_for_member_not_configured has incomplete parameters list
(#704) Phpdocs for function auth_saml2_locallib_testcase::test_is_access_allowed_for_member_blocked_empty has incomplete parameters list
(#737) Phpdocs for function auth_saml2_locallib_testcase::test_is_access_allowed_for_member_allowed_empty has incomplete parameters list
(#769) Phpdocs for function auth_saml2_locallib_testcase::test_is_access_allowed_for_member_allowed_and_blocked has incomplete parameters list
(#802) Phpdocs for function auth_saml2_locallib_testcase::test_is_access_allowed_for_member_allowed_and_blocked_with_allowed_priority has incomplete parameters list
auth/saml2/tests/phpunit/metadata_fetcher_test.php
(#38) Function auth_saml2_metadata_fetcher_testcase::test_fetch_metadata_404 is not documented
(#51) Function auth_saml2_metadata_fetcher_testcase::test_fetch_metadata_success is not documented
(#61) Function auth_saml2_metadata_fetcher_testcase::test_fetch_metadata_curlerrorno is not documented
(#90) Function auth_saml2_metadata_fetcher_testcase::test_fetch_metadata_nohttpstatus is not documented
(#117) Function auth_saml2_metadata_fetcher_testcase::test_fetch_metadata_override_ssl_options is not documented
auth/saml2/tests/phpunit/metadata_parser_test.php
(#38) Function auth_saml2_metadata_parser_testcase::test_parse_metadata is not documented
(#48) There is no description in phpdocs for function test_parse_metadata_fail
auth/saml2/tests/phpunit/metadata_refresh_test.php
(#39) Function auth_saml2_metadata_refresh_testcase::setUp is not documented
(#43) Function auth_saml2_metadata_refresh_testcase::test_metadata_refresh_disabled is not documented
(#53) Function auth_saml2_metadata_refresh_testcase::test_metadata_refresh_idpmetadata_non_url is not documented
(#67) Function auth_saml2_metadata_refresh_testcase::test_metadata_refresh_idpmetadata_notconfigured is not documented
(#122) Function auth_saml2_metadata_refresh_testcase::test_metadata_refresh_parse_no_entityid is not documented
(#145) Function auth_saml2_metadata_refresh_testcase::test_metadata_refresh_parse_no_idpname is not documented
(#77) There is no description in phpdocs for function test_metadata_refresh_fetch_fails
(#98) There is no description in phpdocs for function test_metadata_refresh_parse_fails
(#171) There is no description in phpdocs for function test_metadata_refresh_write_fails
auth/saml2/tests/phpunit/metadata_writer_test.php
(#38) Function auth_saml2_metadata_writer_testcase::test_write_default_path is not documented
(#61) Function auth_saml2_metadata_writer_testcase::test_write_non_dataroot_path is not documented
(#76) Function auth_saml2_metadata_writer_testcase::test_write_trailing_slash is not documented
(#50) There is no description in phpdocs for function test_write_empty_filename
auth/saml2/tests/phpunit/redis_store_test.php
(#43) Function auth_saml2_redis_store_testcase::setUp is not documented
(#54) Function auth_saml2_redis_store_testcase::tearDown is not documented
(#58) Function auth_saml2_redis_store_testcase::test_set_with_expire is not documented
(#69) Function auth_saml2_redis_store_testcase::test_set_no_expire is not documented
(#79) Function auth_saml2_redis_store_testcase::test_get_key_exists is not documented
(#88) Function auth_saml2_redis_store_testcase::test_get_key_not_exists is not documented
(#94) Function auth_saml2_redis_store_testcase::test_delete is not documented
(#104) Function auth_saml2_redis_store_testcase::test_delete_key_not_exists is not documented
(#109) Function auth_saml2_redis_store_testcase::test_bootstrap_redis is not documented
auth/saml2/tests/phpunit/saml2_sitedata_test.php
(#28) Class auth_saml2_sitedata_test is not documented
(#29) Function auth_saml2_sitedata_test::test_it_creates_the_directory_if_it_does_not_exist is not documented
(#45) Function auth_saml2_sitedata_test::test_it_emits_an_event_when_saml_certificate_regenerated is not documented
(#17) No one-line description found in phpdocs for file
auth/saml2/tests/phpunit/setting_idpmetadata_test.php
(#31) Class setting_idpmetadata_test is not documented
(#35) Function setting_idpmetadata_test::setUp is not documented
(#40) Function setting_idpmetadata_test::get_test_metadata_url is not documented
(#47) Function setting_idpmetadata_test::test_it_validates_the_xml is not documented
(#54) Function setting_idpmetadata_test::test_it_saves_all_idp_information is not documented
(#82) Function setting_idpmetadata_test::test_it_saves_all_idps_information_from_single_xml is not documented
(#114) Function setting_idpmetadata_test::test_it_allows_empty_values is not documented
(#120) Function setting_idpmetadata_test::test_it_gets_idp_data_for_xml is not documented
(#127) Function setting_idpmetadata_test::test_it_gets_idp_data_for_two_urls is not documented
(#135) Function setting_idpmetadata_test::test_it_returns_error_if_metadata_url_is_not_valid is not documented
(#17) No one-line description found in phpdocs for file
(#149) There is no description in phpdocs for function validate_idp_data_array
(#162) Not recommended phpdocs tag @static used
auth/saml2/tests/phpunit/ssl_algorithm_test.php
(#28) Class auth_saml2_ssl_algorithms_test is not documented
(#29) Function auth_saml2_ssl_algorithms_test::test_default_saml_signature_algorithm_is_valid_saml_signature_algorithm is not documented
(#34) Function auth_saml2_ssl_algorithms_test::test_sha256_is_valid_saml_signature_algorithm is not documented
(#39) Function auth_saml2_ssl_algorithms_test::test_sha256_is_matching_digest_algorithm_for_default_saml_algorithm is not documented
(#44) Function auth_saml2_ssl_algorithms_test::test_sha256_is_matching_digest_algorithm_for_garbage_algorithm is not documented
(#17) No one-line description found in phpdocs for file

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

(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]