Skip to content

Commit d7b859c

Browse files
KaseyPowersrobmcguinness
authored andcommitted
refactor(api-core): pass config object to core classes
BREAKING CHANGE: Options to core classes are being passed in as config object instead of parameters.
1 parent a48a072 commit d7b859c

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

52 files changed

+758
-187
lines changed

packages/api-angular/src/api.js

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,13 @@ function factory($http, $q, avApiOptions) {
99
throw new Error('[options] cannot be null or undefined');
1010
}
1111
options = angular.merge({}, avApiOptions, options);
12-
super($http, $q, options);
12+
13+
super({
14+
http: $http,
15+
promise: $q,
16+
merge: angular.merge,
17+
config: options,
18+
});
1319
}
1420
}
1521
return AvApi;

packages/api-angular/src/logs.js

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,9 @@ import angular from 'angular';
22
import { AvLogMessages } from '@availity/api-core';
33

44
export default ($http, $q, avApiOptions) =>
5-
new AvLogMessages($http, $q, angular.copy(avApiOptions));
5+
new AvLogMessages({
6+
http: $http,
7+
promise: $q,
8+
merge: angular.merge,
9+
config: angular.copy(avApiOptions),
10+
});

packages/api-angular/src/navigation.js

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,9 @@ import angular from 'angular';
33
import { AvNavigation } from '@availity/api-core';
44

55
export default ($http, $q, avApiOptions) =>
6-
new AvNavigation($http, $q, angular.copy(avApiOptions));
6+
new AvNavigation({
7+
http: $http,
8+
promise: $q,
9+
merge: angular.merge,
10+
config: angular.copy(avApiOptions),
11+
});

packages/api-angular/src/notification.js

Lines changed: 7 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -2,13 +2,10 @@ import angular from 'angular';
22

33
import { AvNotification } from '@availity/api-core';
44

5-
function factory($http, $q, avApiOptions) {
6-
class AvNotificationApi extends AvNotification {
7-
constructor() {
8-
const theseOptions = angular.copy(avApiOptions);
9-
super($http, $q, theseOptions);
10-
}
11-
}
12-
return new AvNotificationApi();
13-
}
14-
export default factory;
5+
export default ($http, $q, avApiOptions) =>
6+
new AvNotification({
7+
http: $http,
8+
promise: $q,
9+
merge: angular.merge,
10+
config: angular.copy(avApiOptions),
11+
});

packages/api-angular/src/organizations.js

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,10 @@ import angular from 'angular';
33
import { AvOrganizations } from '@availity/api-core';
44

55
export default ($http, $q, avUsersApi, avApiOptions) =>
6-
new AvOrganizations($http, $q, avUsersApi, angular.copy(avApiOptions));
6+
new AvOrganizations({
7+
http: $http,
8+
promise: $q,
9+
merge: angular.merge,
10+
avUsers: avUsersApi,
11+
config: angular.copy(avApiOptions),
12+
});

packages/api-angular/src/pdfs.js

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,9 @@ import angular from 'angular';
33
import { AvPdf } from '@availity/api-core';
44

55
export default ($http, $q, avApiOptions) =>
6-
new AvPdf($http, $q, angular.copy(avApiOptions));
6+
new AvPdf({
7+
http: $http,
8+
promise: $q,
9+
merge: angular.merge,
10+
config: angular.copy(avApiOptions),
11+
});

packages/api-angular/src/permissions.js

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,9 @@ import angular from 'angular';
33
import { AvPermissions } from '@availity/api-core';
44

55
export default ($http, $q, avApiOptions) =>
6-
new AvPermissions($http, $q, angular.copy(avApiOptions));
6+
new AvPermissions({
7+
http: $http,
8+
promise: $q,
9+
merge: angular.merge,
10+
config: angular.copy(avApiOptions),
11+
});

packages/api-angular/src/providers.js

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,9 @@ import angular from 'angular';
33
import { AvProviders } from '@availity/api-core';
44

55
export default ($http, $q, avApiOptions) =>
6-
new AvProviders($http, $q, angular.copy(avApiOptions));
6+
new AvProviders({
7+
http: $http,
8+
promise: $q,
9+
merge: angular.merge,
10+
config: angular.copy(avApiOptions),
11+
});

packages/api-angular/src/proxy.js

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,12 @@ function factory($http, $q, avApiOptions) {
66
class AvProxyApi extends AvProxy {
77
constructor(options) {
88
options = angular.merge({}, avApiOptions, options);
9-
super($http, $q, options);
9+
super({
10+
http: $http,
11+
promise: $q,
12+
merge: angular.merge,
13+
config: options,
14+
});
1015
}
1116
}
1217
return AvProxyApi;

packages/api-angular/src/regions.js

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,10 @@ import angular from 'angular';
33
import { AvRegions } from '@availity/api-core';
44

55
export default ($http, $q, avUsersApi, avApiOptions) =>
6-
new AvRegions($http, $q, avUsersApi, angular.copy(avApiOptions));
6+
new AvRegions({
7+
http: $http,
8+
promise: $q,
9+
merge: angular.merge,
10+
avUsers: avUsersApi,
11+
config: angular.copy(avApiOptions),
12+
});

0 commit comments

Comments
 (0)