Remove video not found

diff --git a/client/src/app/+page-not-found/page-not-found-routing.module.ts b/client/src/app/+page-not-found/page-not-found-routing.module.ts
index 306d2d54d845c02139d990bc686cef151b7f148b..e3407099da25c5ed4eed753915f3febffd0ccd70 100644 (file)
--- a/client/src/app/+page-not-found/page-not-found-routing.module.ts
+++ b/client/src/app/+page-not-found/page-not-found-routing.module.ts
@@ -1,11 +1,18 @@
 import { NgModule } from '@angular/core'
 import { RouterModule, Routes } from '@angular/router'
 import { PageNotFoundComponent } from './page-not-found.component'
+import { MetaGuard } from '@ngx-meta/core'
 
 const pageNotFoundRoutes: Routes = [
   {
     path: '',
-    component: PageNotFoundComponent
+    component: PageNotFoundComponent,
+    canActivate: [ MetaGuard ],
+    data: {
+      meta: {
+        title: 'Not found'
+      }
+    }
   }
 ]
 

diff --git a/client/src/app/+page-not-found/page-not-found.component.scss b/client/src/app/+page-not-found/page-not-found.component.scss
index 05d45f97fe06074f4ae80c4e69df3e542f9f2ff3..53b6142e15b4eaf5a19f5cf4a9bbb346179f23d3 100644 (file)
--- a/client/src/app/+page-not-found/page-not-found.component.scss
+++ b/client/src/app/+page-not-found/page-not-found.component.scss
@@ -2,7 +2,7 @@ div {
   height: 100%;
   width: 100%;
   text-align: center;
-  margin-top: 50px;
+  margin-top: 150px;
 
   font-size: 32px;
 }
\ No newline at end of file

diff --git a/client/src/app/videos/+video-watch/video-watch.component.html b/client/src/app/videos/+video-watch/video-watch.component.html
index d80b633df15d800359237cc08d2072bf8389e1c6..4c650b121badeaa58c642afe029a2e1a2eb0e429 100644 (file)
--- a/client/src/app/videos/+video-watch/video-watch.component.html
+++ b/client/src/app/videos/+video-watch/video-watch.component.html
@@ -1,10 +1,8 @@
 <div class="row">
   <!-- We need the video container for videojs so we just hide it -->
-  <div [hidden]="videoNotFound" id="video-element-wrapper">
+  <div id="video-element-wrapper">
   </div>
 
-  <div i18n *ngIf="videoNotFound" id="video-not-found">Video not found :'(</div>
-
   <!-- Video information -->
   <div *ngIf="video" class="margin-content video-bottom">
     <div class="video-info">

diff --git a/client/src/app/videos/+video-watch/video-watch.component.ts b/client/src/app/videos/+video-watch/video-watch.component.ts
index 4a67d456e164957e4847ea5b61fd28625a9fb8f2..7ba8d1fa444097b2320fe035e698cdeb20976684 100644 (file)
--- a/client/src/app/videos/+video-watch/video-watch.component.ts
+++ b/client/src/app/videos/+video-watch/video-watch.component.ts
@@ -46,7 +46,6 @@ export class VideoWatchComponent implements OnInit, OnDestroy {
   playerElement: HTMLVideoElement
   userRating: UserVideoRateType = null
   video: VideoDetails = null
-  videoNotFound = false
   descriptionLoading = false
 
   completeDescriptionShown = false
@@ -119,11 +118,6 @@ export class VideoWatchComponent implements OnInit, OnDestroy {
               const startTime = this.route.snapshot.queryParams.start
               this.onVideoFetched(video, startTime)
                   .catch(err => this.handleError(err))
-            },
-
-            error => {
-              this.videoNotFound = true
-              console.error(error)
             }
           )
     })

diff --git a/client/src/assets/player/peertube-videojs-plugin.ts b/client/src/assets/player/peertube-videojs-plugin.ts
index 057bc4b2a9be9de8f8822d9bdc7483d8931661fc..47288c842323b8d2eecfaf06b3fbc80baf5cd36c 100644 (file)
--- a/client/src/assets/player/peertube-videojs-plugin.ts
+++ b/client/src/assets/player/peertube-videojs-plugin.ts
@@ -195,22 +195,19 @@ class PeerTubePlugin extends Plugin {
     this.torrent = webtorrent.add(magnetOrTorrentUrl, torrentOptions, torrent => {
       console.log('Added ' + magnetOrTorrentUrl + '.')
 
-      // Pause the old torrent
       if (oldTorrent) {
+        // Pause the old torrent
         oldTorrent.pause()
         // Pause does not remove actual peers (in particular the webseed peer)
         oldTorrent.removePeer(oldTorrent['ws'])
 
         // We use a fake renderer so we download correct pieces of the next file
-        // This way we'll be able to
         if (options.delay) {
           const fakeVideoElem = document.createElement('video')
           renderVideo(torrent.files[0], fakeVideoElem, { autoplay: false, controls: false }, (err, renderer) => {
             fakeRenderer = renderer
 
-            if (err) {
-              console.error('Cannot render new torrent in fake video element.', err)
-            }
+            if (err) console.error('Cannot render new torrent in fake video element.', err)
 
             // Load the future file at the correct time
             fakeVideoElem.currentTime = this.player.currentTime() + (options.delay / 2000)