Skip to content

Commit d16657c

Browse files
committed
refactored
1 parent ca1acbd commit d16657c

File tree

2 files changed

+20
-24
lines changed
  • SimpleTaskSystemSPA_AngularJs_EntityFramework/SimpleTaskSystem.Web

2 files changed

+20
-24
lines changed

SimpleTaskSystemSPA_AngularJs_EntityFramework/SimpleTaskSystem.Web/Abp/Framework/scripts/libs/angular/abp.ng.js

Lines changed: 16 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -10,49 +10,44 @@
1010

1111
abpModule.config([
1212
'$httpProvider', function ($httpProvider) {
13-
$httpProvider.interceptors.push(function () {
13+
$httpProvider.interceptors.push(function ($q) {
1414
return {
15-
'request': function (config) {
16-
return config;
17-
},
18-
1915
'response': function (response) {
20-
if (!response.config || !response.config.abp) {
16+
if (!response.config || !response.config.abp || !response.data) {
2117
return response;
2218
}
2319

24-
//var data = response.data;
25-
if (!response.data) { //Needless check?
26-
return response;
27-
}
20+
var defer = $q.defer();
2821

2922
if (response.data.targetUrl) { //TODO: Check if it works and does not prevent return value!
3023
location.href = data.targetUrl;
3124
}
3225

3326
if (response.data.success === true) {
3427
response.data = response.data.result;
28+
defer.resolve(response);
3529
} else { //data.success === false
3630
if (response.data.error) {
37-
//abp.log.error(response.data.error.details);
3831
abp.message.error(response.data.error.message);
39-
throw response.data.error.message;
32+
} else {
33+
response.data.error = {
34+
message: 'Ajax request is not succeed!',
35+
details: 'Error detail is not sent by server.'
36+
};
4037
}
4138

39+
abp.log.error(response.data.error.message + ' | ' + response.data.error.details);
40+
41+
response.data = response.data.error;
42+
defer.reject(response);
43+
4244
if (response.data.unAuthorizedRequest && !response.data.targetUrl) {
4345
location.reload();
4446
}
4547
}
4648

47-
return response;
48-
},
49-
50-
//'responseError': function (rejection) {
51-
// alert(1);
52-
// console.log('asd: ' + rejection);
53-
54-
// return $q.reject(rejection);
55-
//}
49+
return defer.promise;
50+
}
5651
};
5752
});
5853
}
@@ -61,7 +56,6 @@
6156
abpModule.factory('services.tasksystem.task', [
6257
'$http', function ($http) {
6358
return new function () {
64-
//Working on this code!
6559
this.getTasks = function (input) {
6660
return $http({
6761
url: '/api/services/tasksystem/task/GetTasks',

SimpleTaskSystemSPA_AngularJs_EntityFramework/SimpleTaskSystem.Web/App/Main/views/task/list.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,11 @@
1919
taskService.getTasks({
2020
state: $scope.selectedTaskState > 0 ? $scope.selectedTaskState : null
2121
}).success(function (data) {
22+
console.log(arguments);
2223
vm.tasks = data.tasks;
24+
}).error(function (a) {
25+
console.log('error!');
26+
console.log(arguments);
2327
});
2428
};
2529

@@ -46,7 +50,5 @@
4650
vm.getTaskCountText = function () {
4751
return abp.utils.formatString(vm.localize('Xtasks'), vm.tasks.length);
4852
};
49-
50-
vm.refreshTasks();
5153
}]);
5254
})();

0 commit comments

Comments
 (0)