Use model instead of attributes to store notifications options

2 files changed, 7 insertions, 30 deletions

diff --git a/web/app/view/Notifications.js b/web/app/view/Notifications.js
index 795cf32..419d961 100644
--- a/web/app/view/Notifications.js
+++ b/web/app/view/Notifications.js
@@ -24,7 +24,7 @@ Ext.define('Traccar.view.Notifications', {
     ],
 
     controller: 'notificationsController',
-    store: 'AllNotifications',
+    store: 'Notifications',
 
     selModel: {
         selType: 'cellmodel'
@@ -48,14 +48,14 @@ Ext.define('Traccar.view.Notifications', {
             }
         }, {
             text: Strings.notificationWeb,
-            dataIndex: 'attributes.web',
+            dataIndex: 'web',
             xtype: 'checkcolumn',
             listeners: {
                 checkChange: 'onCheckChange'
             }
         }, {
             text: Strings.notificationMail,
-            dataIndex: 'attributes.mail',
+            dataIndex: 'mail',
             xtype: 'checkcolumn',
             listeners: {
                 checkChange: 'onCheckChange'
diff --git a/web/app/view/NotificationsController.js b/web/app/view/NotificationsController.js
index b920c18..4c83b14 100644
--- a/web/app/view/NotificationsController.js
+++ b/web/app/view/NotificationsController.js
@@ -24,42 +24,19 @@ Ext.define('Traccar.view.NotificationsController', {
     ],
 
     init: function () {
-        this.userId = this.getView().user.getId();
         this.getView().getStore().load({
-            scope: this,
-            callback: function (records, operation, success) {
-                Ext.create('Traccar.store.Notifications').load({
-                    params: {
-                        userId: this.userId
-                    },
-                    scope: this,
-                    callback: function (records, operation, success) {
-                        if (success) {
-                            this.getView().getStore().loadData(records);
-                        }
-                    }
-                });
+            params: {
+                userId: this.getView().user.getId()
             }
         });
     },
 
     onCheckChange: function (column, rowIndex, checked, eOpts) {
-        var record, attributes = {};
-        record = this.getView().getStore().getAt(rowIndex);
-        if (record.get('attributes.web')) {
-            attributes.web = 'true';
-        }
-        if (record.get('attributes.mail')) {
-            attributes.mail = 'true';
-        }
+        var record = this.getView().getStore().getAt(rowIndex);
         Ext.Ajax.request({
             scope: this,
             url: 'api/users/notifications',
-            jsonData: {
-                userId: this.userId,
-                type: record.get('type'),
-                attributes: attributes
-            },
+            jsonData: record.data,
             callback: function (options, success, response) {
                 if (!success) {
                     Traccar.app.showError(response);