Simlified ModalManager ctor. Added TODOs to Identity/Users/Index.cshtml

src/Volo.Abp.Identity.Web/Pages/Identity/Users/Index.cshtml

@@ -10,10 +10,10 @@
 @section scripts {
     <script type="text/javascript" src="~/modules/identity/helpers/jquery.js"></script>
     <script type="text/javascript" src="~/modules/identity/helpers/datatables_helper.js"></script>
+    <script type="text/javascript" src="~/modules/identity/helpers/datatables/datatables.record-actions.js"></script>
     <script type="text/javascript" src="~/modules/identity/helpers/ResourceLoader.js"></script>
     <script type="text/javascript" src="~/modules/identity/helpers/ModalManager.js"></script>
     <script type="text/javascript" src="~/modules/identity/views/users/index.js"></script>
-    <script type="text/javascript" src="~/modules/identity/helpers/datatables/datatables.record-actions.js"></script>
 }
 
 <abp-card id="IdentityUsersWrapper">

src/Volo.Abp.Identity.Web/wwwroot/modules/identity/helpers/ModalManager.js

@@ -31,6 +31,12 @@ $.validator.defaults.ignore = ''; //TODO: Would be better if we can apply only f
 
         return function (options) {
 
+            if (typeof options === 'string') {
+                options = {
+                    viewUrl: options
+                };
+            }
+
             var _options = options;
 
             var _$modalContainer = null;

src/Volo.Abp.Identity.Web/wwwroot/modules/identity/views/users/index.js

@@ -3,13 +3,8 @@
     var l = abp.localization.getResource('AbpIdentity');
     var _identityUserAppService = volo.abp.identity.identityUser;
 
-    var _editModal = new abp.ModalManager({
+    var _editModal = new abp.ModalManager(abp.appPath + 'Identity/Users/EditModal');
-        viewUrl: abp.appPath + 'Identity/Users/EditModal'
+    var _createModal = new abp.ModalManager(abp.appPath + 'Identity/Users/CreateModal');
-    });
-
-    var _createModal = new abp.ModalManager({
-        viewUrl: abp.appPath + 'Identity/Users/CreateModal'
-    });
 
     var app = new Vue({
         el: '#IdentityUsersWrapper',
@@ -30,42 +25,44 @@
             ajax: abp.libs.datatables.createAjax(_identityUserAppService.getList),
             columnDefs: [
                 {
+                    //TODO: Can we eleminate targets, data, orderable, autoWidth, defaultContent fields or make these values default
                     targets: 0,
                     data: null,
                     orderable: false,
                     autoWidth: false,
                     defaultContent: '',
                     rowAction: {
-                        text: '<i class="fa fa-cog"></i> ' + l('Actions') + ' <span class="caret"></span>',
+                        text: '<i class="fa fa-cog"></i> ' + l('Actions') + ' <span class="caret"></span>', //TODO: Add icon option and set text as only l('Actions')
                         items:
-                        [
+                            [
-                            {
+                                {
-                                text: l('Edit'),
+                                    //TODO: Allow to add icon
-                                visible: function () {
+                                    text: l('Edit'),
-                                    return true;
+                                    visible: function () { //TODO: Allow visible to be a boolean for simple cases
-                                },
+                                        return true;
-                                action: function (data) {
+                                    },
-                                    _editModal.open({
+                                    action: function (data) {
-                                        id: data.record.id
+                                        _editModal.open({
-                                    });
+                                            id: data.record.id
-                                }
+                                        });
-                            },
+                                    }
-                            {
-                                text: l('Delete'),
-                                visible: function () {
-                                    return true;
                                 },
-                                action: function (data) {
+                                {
-                                    if (confirm(l('UserDeletionConfirmationMessage', data.record.userName))) {
+                                    text: l('Delete'),
-                                        _identityUserAppService
+                                    visible: function () {
-                                            .delete(data.record.id)
+                                        return true;
-                                            .then(function () {
+                                    },
-                                                _dataTable.ajax.reload();
+                                    action: function (data) {
-                                            });
+                                        if (confirm(l('UserDeletionConfirmationMessage', data.record.userName))) {
+                                            _identityUserAppService
+                                                .delete(data.record.id)
+                                                .then(function () {
+                                                    _dataTable.ajax.reload();
+                                                });
+                                        }
                                     }
                                 }
-                            }
+                            ]
-                        ]
                     }
                 },
                 {
@@ -81,7 +78,7 @@
                     data: "phoneNumber"
                 }
             ]
-        }); 
+        });
 
         _createModal.onResult(function () {
             _dataTable.ajax.reload();