Skip to content

Commit

Permalink
Fix stderr checkbox disablement in Metric Explorer.
Browse files Browse the repository at this point in the history
The "Err Bars" and "Err Labels" columns in the "Data" menu in the metric
explorer were incorrectly shown or incorrectly not shown in certian
circumstances.

a) if a chart was 'geared over' from the Usage Tab then the "Data" menu would
always show the "Err Bars" and "Err Labels" checkboxes regardless of whether
they should have been shown.

b) if data were added to the metric explorer from the "Metric Catalog" then
the "Data" menu would never show the checkboxes regardless of whether they
should have been shown.

If data are added to a chart via the "Data Series Definition" dialog then the
checkboxes are correct.

This change fixes it so the checkboxes work correctly.
  • Loading branch information
jpwhite4 committed Dec 12, 2019
1 parent 47f6f18 commit 21b5929
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 1 addition & 1 deletion html/gui/js/modules/Usage.js
Original file line number Diff line number Diff line change
Expand Up @@ -2174,7 +2174,7 @@ Ext.extend(XDMoD.Module.Usage, XDMoD.PortalModule, {
group_by: n.attributes.group_by,
x_axis: false,
log_scale: chartToolbar.getLogScale() == 'y',
has_std_err: 'y',
has_std_err: chartToolbar.getEnableErrors(),
std_err: chartToolbar.getShowErrorBars() == 'y',
std_err_labels: chartToolbar.getShowErrorLabels() == 'y',
value_labels: chartToolbar.getShowAggregateLabels() == 'y' || dt == 'pie',
Expand Down
2 changes: 2 additions & 0 deletions html/gui/js/modules/metric_explorer/MetricExplorer.js
Original file line number Diff line number Diff line change
Expand Up @@ -2829,6 +2829,7 @@ Ext.extend(XDMoD.Module.MetricExplorer, XDMoD.PortalModule, {
singleClickExpand: true,
type: 'metric',
text: realm_metrics[rm].text,
has_std_err: realm_metrics[rm].std_err,
iconCls: 'chart',
category: category,
realm: realm,
Expand Down Expand Up @@ -2861,6 +2862,7 @@ Ext.extend(XDMoD.Module.MetricExplorer, XDMoD.PortalModule, {
group_by: d,
metric: n.attributes.metric,
realm: n.attributes.realm,
has_std_err: n.attributes.has_std_err,
category: n.attributes.category
};
Ext.apply(config, this.defaultDatasetConfig);
Expand Down

0 comments on commit 21b5929

Please sign in to comment.