Skip to content

Commit 2704e97

Browse files
committed
refactor: naming convention in subscriptionSet class
1 parent 419bdb5 commit 2704e97

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

src/entities/SubscriptionSet.ts

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ export class SubscriptionSet implements SubscribeCapable {
77
private options: SubscriptionOptions;
88
private pubnub: any;
99
private eventEmitter: any;
10-
private _subscriptions: Subscription[];
10+
private subscriptionList: Subscription[];
1111

1212
constructor({
1313
channels = [],
@@ -27,7 +27,7 @@ export class SubscriptionSet implements SubscribeCapable {
2727
this.options = subscriptionOptions;
2828
this.eventEmitter = eventEmitter;
2929
this.pubnub = pubnub;
30-
this._subscriptions = [
30+
this.subscriptionList = [
3131
new Subscription({
3232
channels: this.channelNames,
3333
channelGroups: this.groupNames,
@@ -60,7 +60,7 @@ export class SubscriptionSet implements SubscribeCapable {
6060
}
6161

6262
addSubscription(subscription: Subscription) {
63-
this._subscriptions.push(subscription);
63+
this.subscriptionList.push(subscription);
6464
this.channelNames = [...this.channelNames, ...subscription.channels];
6565
this.groupNames = [...this.groupNames, ...subscription.channelGroups];
6666
}
@@ -70,11 +70,11 @@ export class SubscriptionSet implements SubscribeCapable {
7070
const groupsToRemove = subscription.channelGroups;
7171
this.channelNames = this.channelNames.filter((c) => !channelsToRemove.includes(c));
7272
this.groupNames = this.groupNames.filter((cg) => !groupsToRemove.includes(cg));
73-
this._subscriptions = this._subscriptions.filter((s) => s !== subscription);
73+
this.subscriptionList = this.subscriptionList.filter((s) => s !== subscription);
7474
}
7575

7676
addSubscriptionSet(subscriptionSet: SubscriptionSet) {
77-
this._subscriptions = [...this._subscriptions, ...subscriptionSet.subscriptions];
77+
this.subscriptionList = [...this.subscriptionList, ...subscriptionSet.subscriptions];
7878
this.channelNames = [...this.channelNames, ...subscriptionSet.channels];
7979
this.groupNames = [...this.groupNames, ...subscriptionSet.channelGroups];
8080
}
@@ -84,7 +84,7 @@ export class SubscriptionSet implements SubscribeCapable {
8484
const groupsToRemove = subscriptionSet.channelGroups;
8585
this.channelNames = this.channelNames.filter((c) => !channelsToRemove.includes(c));
8686
this.groupNames = this.groupNames.filter((cg) => !groupsToRemove.includes(cg));
87-
this._subscriptions = this._subscriptions.filter((s) => !subscriptionSet.subscriptions.includes(s));
87+
this.subscriptionList = this.subscriptionList.filter((s) => !subscriptionSet.subscriptions.includes(s));
8888
}
8989

9090
get channels() {
@@ -94,6 +94,6 @@ export class SubscriptionSet implements SubscribeCapable {
9494
return this.groupNames.slice(0);
9595
}
9696
get subscriptions() {
97-
return this._subscriptions.slice(0);
97+
return this.subscriptionList.slice(0);
9898
}
9999
}

0 commit comments

Comments
 (0)