diff options
author | Chocobozzz <me@florianbigard.com> | 2021-10-18 11:44:54 +0200 |
---|---|---|
committer | Chocobozzz <me@florianbigard.com> | 2021-10-18 11:44:54 +0200 |
commit | 2cc276f92fefeff88fcc74217e1174a3ddb3f074 (patch) | |
tree | 1f87381575de9500d96739cf85321e3e5edb0bf7 /client/src/app/shared | |
parent | 27c3c9456dcf436946635c3c85d4a7653b5cccdb (diff) | |
download | PeerTube-2cc276f92fefeff88fcc74217e1174a3ddb3f074.tar.gz PeerTube-2cc276f92fefeff88fcc74217e1174a3ddb3f074.tar.zst PeerTube-2cc276f92fefeff88fcc74217e1174a3ddb3f074.zip |
Fix find in bulk
Diffstat (limited to 'client/src/app/shared')
3 files changed, 8 insertions, 7 deletions
diff --git a/client/src/app/shared/shared-search/find-in-bulk.service.ts b/client/src/app/shared/shared-search/find-in-bulk.service.ts index 6d77941d3..30cddc154 100644 --- a/client/src/app/shared/shared-search/find-in-bulk.service.ts +++ b/client/src/app/shared/shared-search/find-in-bulk.service.ts | |||
@@ -1,6 +1,6 @@ | |||
1 | import * as debug from 'debug' | 1 | import * as debug from 'debug' |
2 | import { Observable, Subject } from 'rxjs' | 2 | import { Observable, Subject } from 'rxjs' |
3 | import { first, map } from 'rxjs/operators' | 3 | import { filter, first, map } from 'rxjs/operators' |
4 | import { Injectable } from '@angular/core' | 4 | import { Injectable } from '@angular/core' |
5 | import { buildBulkObservable } from '@app/helpers' | 5 | import { buildBulkObservable } from '@app/helpers' |
6 | import { ResultList } from '@shared/models/common' | 6 | import { ResultList } from '@shared/models/common' |
@@ -12,7 +12,7 @@ const logger = debug('peertube:search:FindInBulkService') | |||
12 | 12 | ||
13 | type BulkObservables <P extends number | string, R> = { | 13 | type BulkObservables <P extends number | string, R> = { |
14 | notifier: Subject<P> | 14 | notifier: Subject<P> |
15 | result: Observable<R> | 15 | result: Observable<{ params: P[], response: R }> |
16 | } | 16 | } |
17 | 17 | ||
18 | @Injectable() | 18 | @Injectable() |
@@ -70,8 +70,9 @@ export class FindInBulkService { | |||
70 | return new Observable<R>(obs => { | 70 | return new Observable<R>(obs => { |
71 | observableObject.result | 71 | observableObject.result |
72 | .pipe( | 72 | .pipe( |
73 | filter(result => result.params.includes(param)), | ||
73 | first(), | 74 | first(), |
74 | map(({ data }) => data), | 75 | map(result => result.response.data), |
75 | map(data => data.find(finder)) | 76 | map(data => data.find(finder)) |
76 | ) | 77 | ) |
77 | .subscribe(result => { | 78 | .subscribe(result => { |
@@ -105,8 +106,8 @@ export class FindInBulkService { | |||
105 | return this.searchService.searchVideoPlaylists({ uuids }) | 106 | return this.searchService.searchVideoPlaylists({ uuids }) |
106 | } | 107 | } |
107 | 108 | ||
108 | private buildBulkObservableObject <T extends number | string, R> (bulkGet: (params: T[]) => Observable<R>) { | 109 | private buildBulkObservableObject <P extends number | string, R> (bulkGet: (params: P[]) => Observable<R>) { |
109 | const notifier = new Subject<T>() | 110 | const notifier = new Subject<P>() |
110 | 111 | ||
111 | return { | 112 | return { |
112 | notifier, | 113 | notifier, |
diff --git a/client/src/app/shared/shared-user-subscription/user-subscription.service.ts b/client/src/app/shared/shared-user-subscription/user-subscription.service.ts index 8d2de8173..f289fb6cf 100644 --- a/client/src/app/shared/shared-user-subscription/user-subscription.service.ts +++ b/client/src/app/shared/shared-user-subscription/user-subscription.service.ts | |||
@@ -37,7 +37,7 @@ export class UserSubscriptionService { | |||
37 | time: 500, | 37 | time: 500, |
38 | notifierObservable: this.existsSubject, | 38 | notifierObservable: this.existsSubject, |
39 | bulkGet: this.doSubscriptionsExist.bind(this) | 39 | bulkGet: this.doSubscriptionsExist.bind(this) |
40 | }), | 40 | }).pipe(map(r => r.response)), |
41 | 41 | ||
42 | this.myAccountSubscriptionCacheSubject | 42 | this.myAccountSubscriptionCacheSubject |
43 | ) | 43 | ) |
diff --git a/client/src/app/shared/shared-video-playlist/video-playlist.service.ts b/client/src/app/shared/shared-video-playlist/video-playlist.service.ts index 02632c9eb..fc291329a 100644 --- a/client/src/app/shared/shared-video-playlist/video-playlist.service.ts +++ b/client/src/app/shared/shared-video-playlist/video-playlist.service.ts | |||
@@ -54,7 +54,7 @@ export class VideoPlaylistService { | |||
54 | time: 500, | 54 | time: 500, |
55 | bulkGet: this.doVideosExistInPlaylist.bind(this), | 55 | bulkGet: this.doVideosExistInPlaylist.bind(this), |
56 | notifierObservable: this.videoExistsInPlaylistNotifier | 56 | notifierObservable: this.videoExistsInPlaylistNotifier |
57 | }), | 57 | }).pipe(map(({ response }) => response)), |
58 | 58 | ||
59 | this.videoExistsInPlaylistCacheSubject | 59 | this.videoExistsInPlaylistCacheSubject |
60 | ) | 60 | ) |