[checkmk-commits] [tribe29/checkmk] 2bbf5b: Add form validation to commands

Branch: refs/heads/master Home: https://github.com/tribe29/checkmk Commit: 2bbf5b29418192ae6d670c429bbd287fc467825f https://github.com/tribe29/checkmk/commit/2bbf5b29418192ae6d670c429bbd287fc… Author: Tom Baerwinkel <tom.baerwinkel(a)tribe29.com> Date: 2020-11-20 (Fri, 20 Nov 2020) Changed paths: M cmk/gui/htmllib.py M cmk/gui/plugins/views/commands.py M web/htdocs/themes/facelift/scss/_main.scss Log Message: ----------- Add form validation to commands If a text_field sets required to True the CSS class "required" is set for its label (if present). If a button sets formnovalidate to True validation is bypassed. Align the look of the commands a bit more along the way. Change-Id: I26a33d286d202959bfd73472e341672b5af7b851 Commit: 4f268878263f157f36c5054be731f47596cda604 https://github.com/tribe29/checkmk/commit/4f268878263f157f36c5054be731f4759… Author: Mathias Laurin <mathias.laurin(a)tribe29.com> Date: 2020-11-20 (Fri, 20 Nov 2020) Changed paths: M cmk/base/automations/check_mk.py M cmk/base/checkers/_abstract.py M cmk/base/checkers/_checkers.py M cmk/base/checkers/agent.py M cmk/base/checkers/ipmi.py M cmk/base/checkers/piggyback.py M cmk/base/checkers/programs.py M cmk/base/checkers/snmp.py M cmk/base/checkers/tcp.py M cmk/base/checking.py M cmk/base/discovery.py M cmk/base/inventory.py M cmk/base/modes/check_mk.py M cmk/fetchers/snmp.py M tests/Makefile M tests/unit/cmk/base/checkers/test_agent.py M tests/unit/cmk/base/checkers/test_data_sources.py M tests/unit/cmk/base/checkers/test_host_sections.py M tests/unit/cmk/base/checkers/test_ipmi_management_board_data_source.py M tests/unit/cmk/base/checkers/test_piggy_back_data_source.py M tests/unit/cmk/base/checkers/test_program_data_source.py M tests/unit/cmk/base/checkers/test_snmp_data_source.py M tests/unit/cmk/base/checkers/test_tcp_data_source.py M tests/unit/cmk/base/test_discovery.py Log Message: ----------- Make section selection an arg to parse For the agent, the selection is only really used for parse and therefore needs not be an instance parameter. Also: - consistent naming: * -> section_selection - consistent typing: * -> SectionNameCollection SNMP should be dealt with in another patch. Change-Id: Iacdd2c6080b920d4c8ff97f9398720c403a6b1f5 Compare: https://github.com/tribe29/checkmk/compare/7deea471c7d4...4f268878263f