Checkmk git commits October 2014

checkmk-commits@lists.checkmk.com

6 participants
310 discussions

Check_MK Git: check_mk: visuals: make popup code for adding generic

by Mathias Kettner

Module: check_mk Branch: master Commit: 418e0dac2c795969bd1a6df921ecc92df7acfa93 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=418e0dac2c7959… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Tue Oct 7 19:35:28 2014 +0200 visuals: make popup code for adding generic --- web/htdocs/dashboard.py | 29 ++++++++------- web/htdocs/htmllib.py | 6 ++-- web/htdocs/js/checkmk.js | 80 +++++++++++++++++++----------------------- web/htdocs/views.css | 29 ++++++--------- web/htdocs/visuals.py | 47 ++++++++++++++++++++++--- web/plugins/pages/shipped.py | 6 ++-- 6 files changed, 112 insertions(+), 85 deletions(-) Diff: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commitdiff;h=418e0dac2c…

9 years, 11 months

Check_MK Git: check_mk: Better styling of context/filter editor

by Mathias Kettner

Module: check_mk Branch: master Commit: d7756dfcf4297e835f4b1348805a0bbfb21ec4a7 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=d7756dfcf4297e… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Tue Oct 7 18:36:52 2014 +0200 Better styling of context/filter editor --- web/htdocs/valuespec.py | 35 +++++++++++++++++++++++++++++------ web/htdocs/views.css | 24 ++++++++++++++++++++++++ web/htdocs/visuals.py | 2 +- 3 files changed, 54 insertions(+), 7 deletions(-) diff --git a/web/htdocs/valuespec.py b/web/htdocs/valuespec.py index cd6d6fb..95eea18 100644 --- a/web/htdocs/valuespec.py +++ b/web/htdocs/valuespec.py @@ -908,6 +908,7 @@ class ListOfMultiple(ValueSpec): self._choice_dict = dict(choices) self._size = kwargs.get("size") self._add_label = kwargs.get("add_label", _("Add element")) + self._delete_style = kwargs.get("delete_style", "default") # or "filter" def del_button(self, varprefix, ident): js = "vs_listofmultiple_del('%s', '%s')" % (varprefix, ident) @@ -921,6 +922,12 @@ class ListOfMultiple(ValueSpec): # here is *not* an option since this might not work in case of # a wrong user input. + # Special styling for filters + if self._delete_style == "filter": + extra_css=" filter" + else: + extra_css="" + # In the 'complain' phase, where the user already saved the # form but the validation failed, we must not display the # original 'value' but take the value from the HTML variables. @@ -933,15 +940,29 @@ class ListOfMultiple(ValueSpec): id = '%s_active' % varprefix, add_var = True) # Actual table of currently existing entries - html.write('<table class="valuespec_listof" id="%s_table">' % varprefix) + html.write('<table class="valuespec_listof%s" id="%s_table">' % (extra_css, varprefix)) + + def render_content(): + html.write('<td class="vlof_content%s">' % extra_css) + vs.render_input(prefix, value.get(ident)) + html.write("</td>") + + def render_del(): + html.write('<td class="vlof_buttons%s">' % extra_css) + self.del_button(varprefix, ident) + html.write('</td>') + for ident, vs in self._choices: cls = ident not in value and 'unused' or '' prefix = varprefix + '_' + ident - html.write('<tr id="%s_row" class="%s"><td class=vlof_buttons>' % (prefix, cls)) - self.del_button(varprefix, ident) - html.write("</td><td class=vlof_content>") - vs.render_input(prefix, value.get(ident)) - html.write("</td></tr>") + html.write('<tr id="%s_row" class="%s">' % (prefix, cls)) + if self._delete_style == "filter": + render_content() + render_del() + else: + render_del() + render_content() + html.write("</tr>") html.write("</table>") html.write("<br>") @@ -949,6 +970,8 @@ class ListOfMultiple(ValueSpec): attrs = {} if self._size != None: attrs["style"] = "width: %dex" % self._size + if self._delete_style == "filter": + attrs["class"] = "vlof_filter" html.select(varprefix + '_choice', choosable, attrs=attrs) html.javascript('vs_listofmultiple_init(\'%s\');' % varprefix) html.jsbutton(varprefix + '_add', self._add_label, "vs_listofmultiple_add('%s')" % varprefix) diff --git a/web/htdocs/views.css b/web/htdocs/views.css index d3d0a67..f238ccd 100644 --- a/web/htdocs/views.css +++ b/web/htdocs/views.css @@ -124,6 +124,30 @@ div.floatfilter .content .filtertime select { width: 136px; } +/* Improving styling of floatfilters that are contained in a ListOf() */ +table.valuespec_listof.filter { + width: 300px; + border-spacing: 0px; +} +select.vlof_filter { + width: 312px; +} +table.valuespec_listof td.vlof_content.filter { + width: 1px; + border-style: none; +} +table.valuespec_listof td.vlof_buttons.filter { + width: 1px; + border-style: none; +} +table.valuespec_listof td.vlof_buttons.filter img { + position: relative; + top: -18px; + left: -28px; + width: 16px; + height: 16px; +} + table.filtertime td { vertical-align: middle; } diff --git a/web/htdocs/visuals.py b/web/htdocs/visuals.py index fed8282..693c527 100644 --- a/web/htdocs/visuals.py +++ b/web/htdocs/visuals.py @@ -943,7 +943,7 @@ class VisualFilterList(ListOfMultiple): kwargs.setdefault('title', _('Filters')) kwargs.setdefault('add_label', _('Add filter')) - kwargs.setdefault("size", 50) + kwargs["delete_style"] = "filter" ListOfMultiple.__init__(self, fspecs, **kwargs)

9 years, 11 months

Check_MK Git: check_mk: This filter just worked in the positive sense, not in the negative or

by Mathias Kettner

Module: check_mk Branch: master Commit: 8daa6355a40a996ce425b857ad5fadb5d4ff4af2 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=8daa6355a40a99… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Tue Oct 7 17:57:41 2014 +0200 This filter just worked in the positive sense, not in the negative or in the ignore sense. This has been fixed. --- .werks/1379 | 10 ++++++++++ ChangeLog | 1 + web/plugins/visuals/inventory.py | 8 +++++++- 3 files changed, 18 insertions(+), 1 deletion(-) diff --git a/.werks/1379 b/.werks/1379 new file mode 100644 index 0000000..74e6964 --- /dev/null +++ b/.werks/1379 @@ -0,0 +1,10 @@ +Title: Fixed filter "Host has inventory data" +Level: 1 +Component: inv +Compatible: compat +Version: 1.2.5i6 +Date: 1412697314 +Class: fix + +This filter just worked in the positive sense, not in the negative or +in the ignore sense. This has been fixed. diff --git a/ChangeLog b/ChangeLog index 554de9d..5bc5c41 100644 --- a/ChangeLog +++ b/ChangeLog @@ -164,6 +164,7 @@ NOTE: Please refer to the migration notes! * 0653 script to extract hw/sw inventory data in csv format... * 0655 FIX: win_cpuinfo and mk_inventory.ps1 agent: unit of CPU speed fixed, fixes for long output lines in agent + * 1379 FIX: Fixed filter "Host has inventory data"... check: * 1384 oracle_jobs: new WATO rules, changed service name to SID.OWNER.NAME... diff --git a/web/plugins/visuals/inventory.py b/web/plugins/visuals/inventory.py index b7701c4..6d96135 100644 --- a/web/plugins/visuals/inventory.py +++ b/web/plugins/visuals/inventory.py @@ -121,7 +121,13 @@ class FilterHasInventory(FilterTristate): return "" # No Livestatus filtering right now def filter_table(self, rows): - return [ row for row in rows if row["host_inventory"] ] + tri = self.tristate_value() + if tri == -1: + return rows + elif tri == 1: + return [ row for row in rows if row["host_inventory"] ] + else: # not + return [ row for row in rows if not row["host_inventory"] ] declare_filter(801, FilterHasInventory())

9 years, 11 months

Check_MK Git: check_mk: Renamed several filter URI variables

by Mathias Kettner

Module: check_mk Branch: master Commit: eacb614142b8b8aa89d943a57707c937f4d0ba0d URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=eacb614142b8b8… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Tue Oct 7 17:58:59 2014 +0200 Renamed several filter URI variables These are variables that conflict with other filters. In most cases these are pairs of regex and exact match filters. The regex filters have been renamed in a way that there is no conflict anymore. --- web/htdocs/views.py | 22 ++++++++++++++-------- web/htdocs/visuals.py | 29 ++++++++++++++++++++--------- web/plugins/visuals/filters.py | 12 ++++++------ 3 files changed, 40 insertions(+), 23 deletions(-) Diff: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commitdiff;h=eacb614142…

9 years, 11 months

Check_MK Git: check_mk: htmllib: new functions for stashing current variables

by Mathias Kettner

Module: check_mk Branch: master Commit: 0f143221e711abec4d1e8a222aab33c0e4ac871e URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=0f143221e711ab… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Tue Oct 7 17:54:40 2014 +0200 htmllib: new functions for stashing current variables --- web/htdocs/htmllib.py | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/web/htdocs/htmllib.py b/web/htdocs/htmllib.py index 3de2f9f..4b7dacd 100644 --- a/web/htdocs/htmllib.py +++ b/web/htdocs/htmllib.py @@ -58,6 +58,7 @@ class html: self.link_target = None self.form_name = None self.form_vars = [] + self.var_stash = [] self.context_buttons_open = False self.mobile = False self.buffering = True @@ -987,6 +988,12 @@ class html: self.vars = dict([(k,v) for (k,v) in self.vars.iteritems() if not k.startswith(prefix)]) self.listvars = dict([(k,v) for (k,v) in self.listvars.iteritems() if not k.startswith(prefix)]) + def stash_vars(self): + self.var_stash.append(self.vars.copy()) + + def unstash_vars(self): + self.vars = self.var_stash.pop() + def javascript(self, code): self.write("<script language=\"javascript\">\n%s\n</script>\n" % code)

9 years, 11 months

Check_MK Git: check_mk: remove old outdated comment

by Mathias Kettner

Module: check_mk Branch: master Commit: 03f3b096f95ff5829f9c233faec581fea67eb8a1 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=03f3b096f95ff5… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Tue Oct 7 15:13:33 2014 +0200 remove old outdated comment --- web/plugins/visuals/filters.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/web/plugins/visuals/filters.py b/web/plugins/visuals/filters.py index 2babf82..e953e0d 100644 --- a/web/plugins/visuals/filters.py +++ b/web/plugins/visuals/filters.py @@ -299,10 +299,7 @@ class FilterServiceState(Filter): for var, text in [(self.prefix + "st0", _("OK")), (self.prefix + "st1", _("WARN")), \ (self.prefix + "st2", _("CRIT")), (self.prefix + "st3", _("UNKNOWN")), (self.prefix + "stp", _("PEND"))]: - #if html.mobile: - #text = text[:1] html.checkbox(var, True, label=text) - # html.write(" %s " % text) html.end_checkbox_group() def filter(self, infoname):

9 years, 11 months

Check_MK Git: check_mk: Fixed editing dashlet details

by Mathias Kettner

Module: check_mk Branch: master Commit: c24c7b10d5908acb7a20088ccbe5e10aff1f5653 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=c24c7b10d5908a… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Tue Oct 7 16:38:02 2014 +0200 Fixed editing dashlet details --- web/plugins/dashboard/dashlets.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/web/plugins/dashboard/dashlets.py b/web/plugins/dashboard/dashlets.py index 49e9167..ea44fad 100644 --- a/web/plugins/dashboard/dashlets.py +++ b/web/plugins/dashboard/dashlets.py @@ -536,7 +536,7 @@ def dashlet_view_render_input(dashlet): views.load_views() if html.var("id"): # only needed in case of editing an existing view - views.transform_view_to_valuespec(dashlet) + views.transform_view_to_valuespec_value(dashlet) return views.render_view_config(dashlet) def dashlet_view_handle_input(ident, dashlet): @@ -544,7 +544,7 @@ def dashlet_view_handle_input(ident, dashlet): dashlet.setdefault('title', _('View')) import views # FIXME: HACK, clean this up somehow views.load_views() - return views.create_view_config(dashlet, dashlet) + return views.create_view_from_valuespec(dashlet, dashlet) dashlet_types["view"] = { "title" : _("View"),

9 years, 11 months

Check_MK Git: check_mk: Avoid exception on Dictionary without elements

by Mathias Kettner

Module: check_mk Branch: master Commit: 50307c43cf54c5da94b47f7caa2d3305d0351c6f URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=50307c43cf54c5… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Tue Oct 7 16:38:47 2014 +0200 Avoid exception on Dictionary without elements --- web/htdocs/valuespec.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/web/htdocs/valuespec.py b/web/htdocs/valuespec.py index 5edd1ad..cd6d6fb 100644 --- a/web/htdocs/valuespec.py +++ b/web/htdocs/valuespec.py @@ -2558,10 +2558,12 @@ class Dictionary(ValueSpec): return value def _get_elements(self): - if type(self._elements) == list: + if type(self._elements) == type(lambda: None): + return self._elements() + elif type(self._elements) == list: return self._elements else: - return self._elements() + return [] def render_input(self, varprefix, value): value = self.migrate(value)

9 years, 11 months

Check_MK Git: check_mk: Extract function for filters from view to visuals

by Mathias Kettner

Module: check_mk Branch: master Commit: f5548c7a69f4a7fbb6c11cc4d8fa5e05de1c3b10 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=f5548c7a69f4a7… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Tue Oct 7 16:39:17 2014 +0200 Extract function for filters from view to visuals --- web/htdocs/views.py | 20 ++------------------ web/htdocs/visuals.py | 34 ++++++++++++++++++++++++++++++---- 2 files changed, 32 insertions(+), 22 deletions(-) diff --git a/web/htdocs/views.py b/web/htdocs/views.py index eec6912..92e7f87 100644 --- a/web/htdocs/views.py +++ b/web/htdocs/views.py @@ -536,7 +536,6 @@ def transform_view_to_valuespec_value(view): view['visibility'].append(key) view['grouping'] = { "grouping" : view.get('group_painters', []) } - view['filters'] = { "context" : view.get('context', {}) } view['sorting'] = { "sorters" : view.get('sorters', {}) } columns = [] @@ -559,8 +558,6 @@ def transform_view_to_valuespec_value(view): columns.append((pname, viewname, '')) - - def transform_valuespec_value_to_view(view): for ident, attrs in view.items(): # Transform some valuespec specific options to legacy view @@ -572,6 +569,7 @@ def transform_valuespec_value_to_view(view): view[option] = True del attrs['options'] view.update(attrs) + del view["view"] elif ident == 'sorting': view.update(attrs) @@ -871,21 +869,7 @@ def show_view(view, show_heading = False, show_buttons = True, # in context. In this case, take them from the context type definition. show_filters = visuals.show_filters(view, datasource['infos']) - # Populate the HTML vars with missing context vars. The context vars set - # in single context are enforced (can not be overwritten by URL). The normal - # filter vars in "multiple" context are not enforced. - for key in visuals.get_single_info_keys(view): - if key in view['context']: - html.set_var(key, view['context'][key]) - - # Now apply the multiple context filters - for info_key in datasource['infos']: - for fname, fval in view['context'].items(): - if type(fval) == dict: # this is a real filter - for varname, val in fval.items(): - # shown filters are set, if form is fresh and variable not supplied in URL - if only_count or (html.var("filled_in") != "filter" and not html.has_var(varname)): - html.set_var(varname, val) + visuals.add_context_to_html_vars(view, datasource["infos"], only_count) # Af any painter, sorter or filter needs the information about the host's # inventory, then we load it and attach it as column "host_inventory" diff --git a/web/htdocs/visuals.py b/web/htdocs/visuals.py index d7c674e..8ff8d70 100644 --- a/web/htdocs/visuals.py +++ b/web/htdocs/visuals.py @@ -467,7 +467,6 @@ def process_context_specs(context_specs): ident = 'context_' + info_key attrs = spec.from_html_vars(ident) - html.debug(attrs) spec.validate_value(attrs, ident) context.update(attrs) return context @@ -477,6 +476,9 @@ def render_context_specs(visual, context_specs): for info_key, spec in context_specs: forms.section(spec.title()) ident = 'context_' + info_key + # Trick: the field "context" contains a dictionary with + # all filter settings, from which the value spec will automatically + # extract those that it needs. value = visual.get('context', {}) spec.render_input(ident, value) @@ -623,8 +625,7 @@ def page_edit_visual(what, all_visuals, custom_field_handler = None, save_and_go = None for nr, (title, pagename, icon) in enumerate(sub_pages): if html.var("save%d" % nr): - save_and_go = html.makeuri_contextless([(visual_types[what]['ident_attr'], visualname)], - filename = pagename + '.py') + save_and_go = pagename if save_and_go or html.var("save") or html.var("try") or html.var("search"): try: @@ -656,7 +657,8 @@ def page_edit_visual(what, all_visuals, custom_field_handler = None, if html.var("save") or save_and_go: if save_and_go: - back = save_and_go + back = html.makeuri_contextless([(visual_types[what]['ident_attr'], visual['name'])], + filename = save_and_go + '.py') else: back = html.var('back') if not back: @@ -869,6 +871,30 @@ def show_filters(visual, info_keys): return list(set(show_filters)) # remove duplicates + +def add_context_to_html_vars(visual, only_infos=None, only_count=False): + if only_infos == None: + only_infos = infos.keys() # all datasources! + + # Populate the HTML vars with missing context vars. The context vars set + # in single context are enforced (can not be overwritten by URL). The normal + # filter vars in "multiple" context are not enforced. + for key in get_single_info_keys(visual): + if key in visual['context']: + html.set_var(key, visual['context'][key]) + + # Now apply the multiple context filters + for info_key in only_infos: + for fname, fval in visual['context'].items(): + if type(fval) == dict: # this is a real filter + for varname, val in fval.items(): + # shown filters are set, if form is fresh and variable not supplied in URL + if only_count or (html.var("filled_in") != "filter" and not html.has_var(varname)): + html.set_var(varname, val) + + + + #. # .--ValueSpecs----------------------------------------------------------. # | __ __ _ ____ |

9 years, 11 months

Check_MK Git: check_mk: views: make transform function symmetric

by Mathias Kettner

Module: check_mk Branch: master Commit: 2b217065c944577b2b118b89b31770aa8ef586f2 URL: http://git.mathias-kettner.de/git/?p=check_mk.git;a=commit;h=2b217065c94457… Author: Mathias Kettner <mk(a)mathias-kettner.de> Date: Tue Oct 7 13:20:23 2014 +0200 views: make transform function symmetric --- web/htdocs/views.py | 116 +++++++++++++++++++++++++++++++++++---------------- 1 file changed, 79 insertions(+), 37 deletions(-) diff --git a/web/htdocs/views.py b/web/htdocs/views.py index 20cc908..eec6912 100644 --- a/web/htdocs/views.py +++ b/web/htdocs/views.py @@ -97,6 +97,9 @@ def transform_old_views(): ds_name = view['datasource'] datasource = multisite_datasources[ds_name] + if "context" not in view: # legacy views did not have this explicitly + view.setdefault("user_sortable", True) + # This tries to map the datasource and additional settings of the # views to get the correct view context if 'single_infos' not in view: @@ -295,8 +298,8 @@ def page_edit_view(): visuals.page_edit_visual('views', multisite_views, custom_field_handler = render_view_config, - load_handler = transform_view_to_valuespec, - create_handler = create_view_config, + load_handler = transform_view_to_valuespec_value, + create_handler = create_view_from_valuespec, info_handler = get_view_infos, try_handler = lambda view: show_view(view, False, False) ) @@ -509,71 +512,74 @@ def render_view_config(view, general_properties=True): view['datasource'] = ds_name for ident, vs in view_editor_specs(ds_name, general_properties): - vs.render_input(ident, view) - -# Is used to change the view structure to be compatible to the valuespec -# This needs to perform the inverted steps of the create_view() function -# FIXME: One day we should rewrite this to make no transform needed anymore -def transform_view_to_valuespec(view): - view['options'] = [] + vs.render_input(ident, view[ident]) + +# Is used to change the view structure to be compatible to +# the valuespec This needs to perform the inverted steps of the +# transform_valuespec_value_to_view() function. FIXME: One day we should +# rewrite this to make no transform needed anymore +def transform_view_to_valuespec_value(view): + view["view"] = {} # Several global variables are put into a sub-dict + # Only copy our known keys. Reporting element, etc. might have their own keys as well + for key in [ "datasource", "browser_reload", "layout", "num_columns", "column_headers" ]: + if key in view: + view["view"][key] = view[key] + + view["view"]['options'] = [] for key, title in view_editor_options(): if view.get(key): - view['options'].append(key) + view['view']['options'].append(key) view['visibility'] = [] for key in [ 'hidden', 'hidebutton', 'public' ]: if view.get(key): view['visibility'].append(key) - view['grouping'] = view['group_painters'] - view['filters'] = view['context'] + view['grouping'] = { "grouping" : view.get('group_painters', []) } + view['filters'] = { "context" : view.get('context', {}) } + view['sorting'] = { "sorters" : view.get('sorters', {}) } - view['columns'] = [] - for entry in view['painters']: + columns = [] + view['columns'] = { "columns" : columns } + for entry in view.get('painters', []): if len(entry) == 5: pname, viewname, tooltip, join_index, col_title = entry - view['columns'].append((pname, join_index, viewname, tooltip, col_title)) + columns.append((pname, join_index, viewname, tooltip, col_title)) elif len(entry) == 4: pname, viewname, tooltip, join_index = entry - view['columns'].append((pname, join_index, viewname, tooltip, '')) + columns.append((pname, join_index, viewname, tooltip, '')) elif len(entry) == 3: pname, viewname, tooltip = entry - view['columns'].append((pname, viewname, tooltip)) + columns.append((pname, viewname, tooltip)) else: pname, viewname = entry - view['columns'].append((pname, viewname, '')) + columns.append((pname, viewname, '')) + -# Extract properties of view from HTML variables and construct -# view object, to be used for saving or displaying -# -# old_view is the old view dict which might be loaded from storage. -# view is the new dict object to be updated. -def create_view_config(old_view, view): - ds_name = old_view.get('datasource', html.var('datasource')) - datasource = multisite_datasources[ds_name] - for ident, vs in view_editor_specs(ds_name): - attrs = vs.from_html_vars(ident) - vs.validate_value(attrs, ident) +def transform_valuespec_value_to_view(view): + for ident, attrs in view.items(): # Transform some valuespec specific options to legacy view # format. We do not want to change the view data structure # at the moment. if ident == 'view': - for option in attrs['options']: - view[option] = True - del attrs['options'] - + if "options" in attrs: + for option in attrs['options']: + view[option] = True + del attrs['options'] view.update(attrs) elif ident == 'sorting': view.update(attrs) + del view["sorting"] elif ident == 'grouping': view['group_painters'] = attrs['grouping'] + del view["grouping"] elif ident == 'columns': painters = [] @@ -593,7 +599,25 @@ def create_view_config(old_view, view): else: painters.append((pname, viewname, tooltip)) view['painters'] = painters + del view["columns"] + +# Extract properties of view from HTML variables and construct +# view object, to be used for saving or displaying +# +# old_view is the old view dict which might be loaded from storage. +# view is the new dict object to be updated. +def create_view_from_valuespec(old_view, view): + ds_name = old_view.get('datasource', html.var('datasource')) + datasource = multisite_datasources[ds_name] + vs_value = {} + for ident, vs in view_editor_specs(ds_name): + attrs = vs.from_html_vars(ident) + vs.validate_value(attrs, ident) + vs_value[ident] = attrs + + transform_valuespec_value_to_view(vs_value) + view.update(vs_value) return view #. @@ -704,6 +728,13 @@ def page_view(): # (e.g. from the services table while we are in a hosts view) # If join_columns is False, we only return the "normal" columns. def get_needed_columns(view, painters): + # Make sure that the information about the available views is present. If + # called via the reporting, than this might not be the case + try: + available_views + except: + load_views() + columns = [] for entry in painters: painter = entry[0] @@ -818,8 +849,13 @@ def show_view(view, show_heading = False, show_buttons = True, display_options = prepare_display_options() # User can override the layout settings via HTML variables (buttons) - # which are safed persistently. This is known as "view options" - vo = view_options(view["name"]) + # which are safed persistently. This is known as "view options". Note: a few + # can be anonymous (e.g. when embedded into a report). In that case there + # are no display options. + if "name" in view: + vo = view_options(view["name"]) + else: + vo = {} num_columns = vo.get("num_columns", view.get("num_columns", 1)) browser_reload = vo.get("refresh", view.get("browser_reload", None)) @@ -994,9 +1030,15 @@ def show_view(view, show_heading = False, show_buttons = True, html.set_browser_reload(browser_reload) # The layout of the view: it can be overridden by several specifying - # an output format (like json or python). + # an output format (like json or python). Note: the layout is not + # always needed. In case of an embedded view in the reporting this + # field is simply missing, because the rendering is done by the + # report itself. if html.output_format == "html": - layout = multisite_layouts[view["layout"]] + if "layout" in view: + layout = multisite_layouts[view["layout"]] + else: + layout = None else: layout = multisite_layouts.get(html.output_format) if not layout:

9 years, 11 months

← Newer
1
...
22
23
24
25
26
27
28
...
31
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

Checkmk git commits October 2014