[checkmk-commits] Fixed a TODO.

Module: check_mk Branch: master Commit: 7317cdfea77d2fc4b8d070616f33ffe167155d2b URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=7317cdfea77d2f… Author: Sven Panne <sp(a)mathias-kettner.de> Date: Mon Jan 7 17:14:37 2019 +0100 Fixed a TODO. Change-Id: Ie116c39d6336daff77223b4bb00ebb0df9d9d40c --- cmk/gui/htmllib.py | 9 +-------- cmk/gui/valuespec.py | 2 +- 2 files changed, 2 insertions(+), 9 deletions(-) diff --git a/cmk/gui/htmllib.py b/cmk/gui/htmllib.py index 2d4996d..8828f34 100644 --- a/cmk/gui/htmllib.py +++ b/cmk/gui/htmllib.py @@ -1134,13 +1134,6 @@ class html(HTMLGenerator): def all_varnames_with_prefix(self, prefix): return self.request.all_varnames_with_prefix(prefix) - # Return all values of a variable that possible occurs more - # than once in the URL. note: self.listvars does contain those - # variable only, if the really occur more than once. - # TODO: Refactor call sites to html.request.* - def list_var(self, varname): - return self.request.list_var(varname) - # Adds a variable to listvars and also set it # TODO: Refactor call sites to html.request.* def add_var(self, varname, value): @@ -1967,7 +1960,7 @@ class html(HTMLGenerator): return self.has_var("filled_in") and ( form_name is None or \ - form_name in self.list_var("filled_in")) + form_name in self.request.list_var("filled_in")) def do_actions(self): return self.var("_do_actions") not in ["", None, _("No")] diff --git a/cmk/gui/valuespec.py b/cmk/gui/valuespec.py index 105aaa0..f822c93 100644 --- a/cmk/gui/valuespec.py +++ b/cmk/gui/valuespec.py @@ -2251,7 +2251,7 @@ class MultiSelect(ListChoice): def from_html_vars(self, varprefix): self.load_elements() value = [] - hv = html.list_var(varprefix) + hv = html.request.list_var(varprefix) for key, _title in self._elements: if key in hv: value.append(key)