From 0146f3516ec458d7eaba3ede2addb23005bd4a28 Mon Sep 17 00:00:00 2001 From: Chocobozzz Date: Mon, 4 Jul 2022 11:31:22 +0200 Subject: Fix comments/download attributes on import --- .../video-import-torrent.component.ts | 16 ++++------ .../video-import-url.component.ts | 34 ++++++---------------- .../+video-edit/video-update.component.html | 2 +- .../+videos/+video-edit/video-update.component.ts | 22 +++++++------- 4 files changed, 27 insertions(+), 47 deletions(-) (limited to 'client/src/app/+videos/+video-edit') diff --git a/client/src/app/+videos/+video-edit/video-add-components/video-import-torrent.component.ts b/client/src/app/+videos/+video-edit/video-add-components/video-import-torrent.component.ts index c369ba2b7..da4996902 100644 --- a/client/src/app/+videos/+video-edit/video-add-components/video-import-torrent.component.ts +++ b/client/src/app/+videos/+video-edit/video-add-components/video-import-torrent.component.ts @@ -1,3 +1,4 @@ +import { switchMap } from 'rxjs' import { AfterViewInit, Component, ElementRef, EventEmitter, OnInit, Output, ViewChild } from '@angular/core' import { Router } from '@angular/router' import { AuthService, CanComponentDeactivate, HooksService, Notifier, ServerService } from '@app/core' @@ -87,21 +88,16 @@ export class VideoImportTorrentComponent extends VideoSend implements OnInit, Af this.loadingBar.useRef().start() this.videoImportService.importVideoTorrent(torrentfile || this.magnetUri, videoUpdate) + .pipe(switchMap(({ video }) => this.videoService.getVideo({ videoId: video.uuid }))) .subscribe({ - next: res => { + next: video => { this.loadingBar.useRef().complete() - this.firstStepDone.emit(res.video.name) + this.firstStepDone.emit(video.name) this.isImportingVideo = false this.hasImportedVideo = true - this.video = new VideoEdit(Object.assign(res.video, { - commentsEnabled: videoUpdate.commentsEnabled, - downloadEnabled: videoUpdate.downloadEnabled, - privacy: { id: this.firstStepPrivacyId }, - support: null, - thumbnailUrl: null, - previewUrl: null - })) + this.video = new VideoEdit(video) + this.video.patch({ privacy: this.firstStepPrivacyId }) hydrateFormFromVideo(this.form, this.video, false) }, diff --git a/client/src/app/+videos/+video-edit/video-add-components/video-import-url.component.ts b/client/src/app/+videos/+video-edit/video-add-components/video-import-url.component.ts index 4c74eda84..971a2a070 100644 --- a/client/src/app/+videos/+video-edit/video-add-components/video-import-url.component.ts +++ b/client/src/app/+videos/+video-edit/video-add-components/video-import-url.component.ts @@ -1,8 +1,9 @@ +import { forkJoin } from 'rxjs' import { map, switchMap } from 'rxjs/operators' import { AfterViewInit, Component, EventEmitter, OnInit, Output } from '@angular/core' import { Router } from '@angular/router' import { AuthService, CanComponentDeactivate, HooksService, Notifier, ServerService } from '@app/core' -import { getAbsoluteAPIUrl, scrollToTop } from '@app/helpers' +import { scrollToTop } from '@app/helpers' import { FormValidatorService } from '@app/shared/shared-forms' import { VideoCaptionService, VideoEdit, VideoImportService, VideoService } from '@app/shared/shared-main' import { LoadingBarService } from '@ngx-loading-bar/core' @@ -76,12 +77,11 @@ export class VideoImportUrlComponent extends VideoSend implements OnInit, AfterV this.videoImportService .importVideoUrl(this.targetUrl, videoUpdate) .pipe( - switchMap(res => { - return this.videoCaptionService - .listCaptions(res.video.uuid) - .pipe( - map(result => ({ video: res.video, videoCaptions: result.data })) - ) + switchMap(previous => { + return forkJoin([ + this.videoCaptionService.listCaptions(previous.video.uuid), + this.videoService.getVideo({ videoId: previous.video.uuid }) + ]).pipe(map(([ videoCaptionsResult, video ]) => ({ videoCaptions: videoCaptionsResult.data, video }))) }) ) .subscribe({ @@ -91,24 +91,8 @@ export class VideoImportUrlComponent extends VideoSend implements OnInit, AfterV this.isImportingVideo = false this.hasImportedVideo = true - const absoluteAPIUrl = getAbsoluteAPIUrl() - - const thumbnailUrl = video.thumbnailPath - ? absoluteAPIUrl + video.thumbnailPath - : null - - const previewUrl = video.previewPath - ? absoluteAPIUrl + video.previewPath - : null - - this.video = new VideoEdit(Object.assign(video, { - commentsEnabled: videoUpdate.commentsEnabled, - downloadEnabled: videoUpdate.downloadEnabled, - privacy: { id: this.firstStepPrivacyId }, - support: null, - thumbnailUrl, - previewUrl - })) + this.video = new VideoEdit(video) + this.video.patch({ privacy: this.firstStepPrivacyId }) this.videoCaptions = videoCaptions diff --git a/client/src/app/+videos/+video-edit/video-update.component.html b/client/src/app/+videos/+video-edit/video-update.component.html index ffd125695..a33ac3db4 100644 --- a/client/src/app/+videos/+video-edit/video-update.component.html +++ b/client/src/app/+videos/+video-edit/video-update.component.html @@ -1,7 +1,7 @@
Update - {{ video?.name }} + {{ videoDetails?.name }}
diff --git a/client/src/app/+videos/+video-edit/video-update.component.ts b/client/src/app/+videos/+video-edit/video-update.component.ts index 43e8ba3e5..13e786a8e 100644 --- a/client/src/app/+videos/+video-edit/video-update.component.ts +++ b/client/src/app/+videos/+video-edit/video-update.component.ts @@ -18,7 +18,7 @@ import { VideoSource } from '@shared/models/videos/video-source' templateUrl: './video-update.component.html' }) export class VideoUpdateComponent extends FormReactive implements OnInit { - video: VideoEdit + videoEdit: VideoEdit videoDetails: VideoDetails videoSource: VideoSource userVideoChannels: SelectChannelItem[] = [] @@ -50,19 +50,19 @@ export class VideoUpdateComponent extends FormReactive implements OnInit { const { videoData } = this.route.snapshot.data const { video, videoChannels, videoCaptions, videoSource, liveVideo } = videoData - this.video = new VideoEdit(video) this.videoDetails = video + this.videoEdit = new VideoEdit(this.videoDetails) this.userVideoChannels = videoChannels this.videoCaptions = videoCaptions this.videoSource = videoSource this.liveVideo = liveVideo - this.forbidScheduledPublication = this.video.privacy !== VideoPrivacy.PRIVATE + this.forbidScheduledPublication = this.videoEdit.privacy !== VideoPrivacy.PRIVATE } onFormBuilt () { - hydrateFormFromVideo(this.form, this.video, true) + hydrateFormFromVideo(this.form, this.videoEdit, true) if (this.liveVideo) { this.form.patchValue({ @@ -115,16 +115,16 @@ export class VideoUpdateComponent extends FormReactive implements OnInit { return } - this.video.patch(this.form.value) + this.videoEdit.patch(this.form.value) this.loadingBar.useRef().start() this.isUpdatingVideo = true // Update the video - this.videoService.updateVideo(this.video) + this.videoService.updateVideo(this.videoEdit) .pipe( // Then update captions - switchMap(() => this.videoCaptionService.updateCaptions(this.video.id, this.videoCaptions)), + switchMap(() => this.videoCaptionService.updateCaptions(this.videoEdit.id, this.videoCaptions)), switchMap(() => { if (!this.liveVideo) return of(undefined) @@ -140,7 +140,7 @@ export class VideoUpdateComponent extends FormReactive implements OnInit { .some(key => this.liveVideo[key] !== liveVideoUpdate[key]) if (!liveChanged) return of(undefined) - return this.liveVideoService.updateLive(this.video.id, liveVideoUpdate) + return this.liveVideoService.updateLive(this.videoEdit.id, liveVideoUpdate) }) ) .subscribe({ @@ -149,7 +149,7 @@ export class VideoUpdateComponent extends FormReactive implements OnInit { this.isUpdatingVideo = false this.loadingBar.useRef().complete() this.notifier.success($localize`Video updated.`) - this.router.navigateByUrl(Video.buildWatchUrl(this.video)) + this.router.navigateByUrl(Video.buildWatchUrl(this.videoEdit)) }, error: err => { @@ -162,10 +162,10 @@ export class VideoUpdateComponent extends FormReactive implements OnInit { } hydratePluginFieldsFromVideo () { - if (!this.video.pluginData) return + if (!this.videoEdit.pluginData) return this.form.patchValue({ - pluginData: this.video.pluginData + pluginData: this.videoEdit.pluginData }) } -- cgit v1.2.3