aboutsummaryrefslogtreecommitdiff
path: root/web/app/view/dialog/ReportConfigController.js
diff options
context:
space:
mode:
authortsmgeek <tsmgeek@gmail.com>2017-08-16 00:19:51 +0100
committertsmgeek <tsmgeek@gmail.com>2017-08-16 00:28:00 +0100
commiteeb55ae6800aa5f051a182760d3564523ff03d85 (patch)
treecf180265d27552526cccc41663015cc8b7300a45 /web/app/view/dialog/ReportConfigController.js
parent9a536a3ef3b8972c64de2de76a340115ab036d3f (diff)
downloadtrackermap-web-eeb55ae6800aa5f051a182760d3564523ff03d85.tar.gz
trackermap-web-eeb55ae6800aa5f051a182760d3564523ff03d85.tar.bz2
trackermap-web-eeb55ae6800aa5f051a182760d3564523ff03d85.zip
- switch to using switch statements instead of ifthenelse
- optimize excess getView() calls
Diffstat (limited to 'web/app/view/dialog/ReportConfigController.js')
-rw-r--r--web/app/view/dialog/ReportConfigController.js21
1 files changed, 11 insertions, 10 deletions
diff --git a/web/app/view/dialog/ReportConfigController.js b/web/app/view/dialog/ReportConfigController.js
index 7233562b..2634b455 100644
--- a/web/app/view/dialog/ReportConfigController.js
+++ b/web/app/view/dialog/ReportConfigController.js
@@ -27,22 +27,23 @@ Ext.define('Traccar.view.dialog.ReportConfigController', {
onSaveClick: function (button) {
var eventType;
- this.getView().callingPanel.deviceId = this.lookupReference('deviceField').getValue();
- this.getView().callingPanel.groupId = this.lookupReference('groupField').getValue();
+ var callingPanel = this.getView().callingPanel;
+ callingPanel.deviceId = this.lookupReference('deviceField').getValue();
+ callingPanel.groupId = this.lookupReference('groupField').getValue();
eventType = this.lookupReference('eventTypeField').getValue();
if (eventType.indexOf(Traccar.store.ReportEventTypes.allEvents) > -1) {
eventType = [Traccar.store.ReportEventTypes.allEvents];
} else if (eventType.length === this.lookupReference('eventTypeField').getStore().getCount() - 1) {
eventType = [Traccar.store.ReportEventTypes.allEvents];
}
- this.getView().callingPanel.eventType = eventType;
- this.getView().callingPanel.chartType = this.lookupReference('chartTypeField').getValue();
- this.getView().callingPanel.showMarkers = this.lookupReference('showMarkersField').getValue();
- this.getView().callingPanel.fromDate = this.lookupReference('fromDateField').getValue();
- this.getView().callingPanel.fromTime = this.lookupReference('fromTimeField').getValue();
- this.getView().callingPanel.toDate = this.lookupReference('toDateField').getValue();
- this.getView().callingPanel.toTime = this.lookupReference('toTimeField').getValue();
- this.getView().callingPanel.updateButtons();
+ callingPanel.eventType = eventType;
+ callingPanel.chartType = this.lookupReference('chartTypeField').getValue();
+ callingPanel.showMarkers = this.lookupReference('showMarkersField').getValue();
+ callingPanel.fromDate = this.lookupReference('fromDateField').getValue();
+ callingPanel.fromTime = this.lookupReference('fromTimeField').getValue();
+ callingPanel.toDate = this.lookupReference('toDateField').getValue();
+ callingPanel.toTime = this.lookupReference('toTimeField').getValue();
+ callingPanel.updateButtons();
button.up('window').close();
}
});