diff --git a/src/Squidex/app/features/administration/state/event-consumers.state.spec.ts b/src/Squidex/app/features/administration/state/event-consumers.state.spec.ts index 2a4c32c14..9849635be 100644 --- a/src/Squidex/app/features/administration/state/event-consumers.state.spec.ts +++ b/src/Squidex/app/features/administration/state/event-consumers.state.spec.ts @@ -42,7 +42,7 @@ describe('EventConsumersState', () => { eventConsumersState.load().subscribe(); - expect(eventConsumersState.snapshot.eventConsumers.values).toEqual([eventConsumer1, eventConsumer2]); + expect(eventConsumersState.snapshot.eventConsumers).toEqual([eventConsumer1, eventConsumer2]); expect(eventConsumersState.snapshot.isLoaded).toBeTruthy(); dialogs.verify(x => x.notifyInfo(It.isAnyString()), Times.never()); @@ -87,7 +87,7 @@ describe('EventConsumersState', () => { eventConsumersState.start(eventConsumer2).subscribe(); - const newConsumer2 = eventConsumersState.snapshot.eventConsumers.at(1); + const newConsumer2 = eventConsumersState.snapshot.eventConsumers[1]; expect(newConsumer2).toEqual(updated); }); @@ -100,7 +100,7 @@ describe('EventConsumersState', () => { eventConsumersState.stop(eventConsumer2).subscribe(); - const newConsumer2 = eventConsumersState.snapshot.eventConsumers.at(1); + const newConsumer2 = eventConsumersState.snapshot.eventConsumers[1]; expect(newConsumer2).toEqual(updated); }); @@ -113,7 +113,7 @@ describe('EventConsumersState', () => { eventConsumersState.reset(eventConsumer2).subscribe(); - const newConsumer2 = eventConsumersState.snapshot.eventConsumers.at(1); + const newConsumer2 = eventConsumersState.snapshot.eventConsumers[1]; expect(newConsumer2).toEqual(updated); }); diff --git a/src/Squidex/app/features/administration/state/event-consumers.state.ts b/src/Squidex/app/features/administration/state/event-consumers.state.ts index 4d686eed5..4d5668fa8 100644 --- a/src/Squidex/app/features/administration/state/event-consumers.state.ts +++ b/src/Squidex/app/features/administration/state/event-consumers.state.ts @@ -11,7 +11,6 @@ import { tap } from 'rxjs/operators'; import { DialogService, - ImmutableArray, shareSubscribed, State } from '@app/shared'; @@ -26,7 +25,7 @@ interface Snapshot { isLoaded?: boolean; } -type EventConsumersList = ImmutableArray; +type EventConsumersList = ReadonlyArray; @Injectable() export class EventConsumersState extends State { @@ -40,7 +39,7 @@ export class EventConsumersState extends State { private readonly dialogs: DialogService, private readonly eventConsumersService: EventConsumersService ) { - super({ eventConsumers: ImmutableArray.empty() }); + super({ eventConsumers: [] }); } public load(isReload = false, silent = false): Observable { @@ -49,13 +48,11 @@ export class EventConsumersState extends State { } return this.eventConsumersService.getEventConsumers().pipe( - tap(({ items }) => { + tap(({ items: eventConsumers }) => { if (isReload && !silent) { this.dialogs.notifyInfo('Event Consumers reloaded.'); } - const eventConsumers = ImmutableArray.of(items); - this.next(s => { return { ...s, eventConsumers, isLoaded: true }; }); diff --git a/src/Squidex/app/features/administration/state/users.state.spec.ts b/src/Squidex/app/features/administration/state/users.state.spec.ts index 90fd5321d..3242ab8fd 100644 --- a/src/Squidex/app/features/administration/state/users.state.spec.ts +++ b/src/Squidex/app/features/administration/state/users.state.spec.ts @@ -50,7 +50,7 @@ describe('UsersState', () => { usersState.load().subscribe(); - expect(usersState.snapshot.users.values).toEqual([user1, user2]); + expect(usersState.snapshot.users).toEqual([user1, user2]); expect(usersState.snapshot.usersPager.numberOfItems).toEqual(200); expect(usersState.snapshot.isLoaded).toBeTruthy(); @@ -178,7 +178,7 @@ describe('UsersState', () => { usersState.select(user2.id).subscribe(); usersState.lock(user2).subscribe(); - const user2New = usersState.snapshot.users.at(1); + const user2New = usersState.snapshot.users[1]; expect(user2New).toBe(usersState.snapshot.selectedUser!); }); @@ -192,7 +192,7 @@ describe('UsersState', () => { usersState.select(user2.id).subscribe(); usersState.unlock(user2).subscribe(); - const user2New = usersState.snapshot.users.at(1); + const user2New = usersState.snapshot.users[1]; expect(user2New).toEqual(updated); expect(user2New).toBe(usersState.snapshot.selectedUser!); @@ -209,7 +209,7 @@ describe('UsersState', () => { usersState.select(user2.id).subscribe(); usersState.update(user2, request).subscribe(); - const user2New = usersState.snapshot.users.at(1); + const user2New = usersState.snapshot.users[1]; expect(user2New).toEqual(updated); expect(user2New).toBe(usersState.snapshot.selectedUser!); @@ -223,7 +223,7 @@ describe('UsersState', () => { usersState.create(request).subscribe(); - expect(usersState.snapshot.users.values).toEqual([newUser, user1, user2]); + expect(usersState.snapshot.users).toEqual([newUser, user1, user2]); expect(usersState.snapshot.usersPager.numberOfItems).toBe(201); }); }); diff --git a/src/Squidex/app/features/administration/state/users.state.ts b/src/Squidex/app/features/administration/state/users.state.ts index 8d2122a29..39239bdbd 100644 --- a/src/Squidex/app/features/administration/state/users.state.ts +++ b/src/Squidex/app/features/administration/state/users.state.ts @@ -13,7 +13,6 @@ import '@app/framework/utils/rxjs-extensions'; import { DialogService, - ImmutableArray, Pager, shareSubscribed, State @@ -46,7 +45,7 @@ interface Snapshot { canCreate?: boolean; } -export type UsersList = ImmutableArray; +export type UsersList = ReadonlyArray; export type UsersResult = { total: number, users: UsersList }; @Injectable() @@ -70,7 +69,7 @@ export class UsersState extends State { private readonly dialogs: DialogService, private readonly usersService: UsersService ) { - super({ users: ImmutableArray.empty(), usersPager: new Pager(0) }); + super({ users: [], usersPager: new Pager(0) }); } public select(id: string | null): Observable { @@ -110,14 +109,13 @@ export class UsersState extends State { this.snapshot.usersPager.pageSize, this.snapshot.usersPager.skip, this.snapshot.usersQuery).pipe( - tap(({ total, items, canCreate }) => { + tap(({ total, items: users, canCreate }) => { if (isReload) { this.dialogs.notifyInfo('Users reloaded.'); } this.next(s => { const usersPager = s.usersPager.setCount(total); - const users = ImmutableArray.of(items); let selectedUser = s.selectedUser; @@ -141,7 +139,7 @@ export class UsersState extends State { return this.usersService.postUser(request).pipe( tap(created => { this.next(s => { - const users = s.users.pushFront(created); + const users = [created, ...s.users]; const usersPager = s.usersPager.incrementCount(); return { ...s, users, usersPager }; diff --git a/src/Squidex/app/features/content/pages/content/content-page.component.html b/src/Squidex/app/features/content/pages/content/content-page.component.html index 969dddf9e..2a298404e 100644 --- a/src/Squidex/app/features/content/pages/content/content-page.component.html +++ b/src/Squidex/app/features/content/pages/content/content-page.component.html @@ -112,7 +112,7 @@ [fieldForm]="contentForm.form.get(field.name)" [fieldFormCompare]="contentFormCompare?.form.get(field.name)" [schema]="schema" - [languages]="languages.mutableValues" + [languages]="languages" [(language)]="language"> diff --git a/src/Squidex/app/features/content/pages/content/content-page.component.ts b/src/Squidex/app/features/content/pages/content/content-page.component.ts index 9234a177b..1eace2005 100644 --- a/src/Squidex/app/features/content/pages/content/content-page.component.ts +++ b/src/Squidex/app/features/content/pages/content/content-page.component.ts @@ -25,7 +25,6 @@ import { EditContentForm, fadeAnimation, FieldDto, - ImmutableArray, LanguagesState, MessageBus, ModalModel, @@ -61,7 +60,7 @@ export class ContentPageComponent extends ResourceOwner implements CanComponentD public dropdown = new ModalModel(); public language: AppLanguageDto; - public languages: ImmutableArray; + public languages: ReadonlyArray; public trackByFieldFn: Function; @@ -92,7 +91,7 @@ export class ContentPageComponent extends ResourceOwner implements CanComponentD this.languagesState.languages .subscribe(languages => { this.languages = languages.map(x => x.language); - this.language = this.languages.at(0); + this.language = this.languages[0]; })); this.own( diff --git a/src/Squidex/app/features/content/pages/contents/contents-page.component.html b/src/Squidex/app/features/content/pages/contents/contents-page.component.html index 313bb0044..97fb041ec 100644 --- a/src/Squidex/app/features/content/pages/contents/contents-page.component.html +++ b/src/Squidex/app/features/content/pages/contents/contents-page.component.html @@ -24,7 +24,7 @@
- +