Increase video attributes length

Number of tags: 3 -> 5
Tag length: 10 -> 30
Video name: 50 -> 120
Video channel name: 50 -> 120

diff --git a/client/src/app/shared/forms/form-validators/video.ts b/client/src/app/shared/forms/form-validators/video.ts
index 213ab15db2caa82d0a70c2ab1d7b9d5eef73b979..286a1117981d664d3d5309e90f46c31aeca6e64b 100644 (file)
--- a/client/src/app/shared/forms/form-validators/video.ts
+++ b/client/src/app/shared/forms/form-validators/video.ts
@@ -1,11 +1,11 @@
 import { Validators } from '@angular/forms'
 
 export const VIDEO_NAME = {
-  VALIDATORS: [ Validators.required, Validators.minLength(3), Validators.maxLength(50) ],
+  VALIDATORS: [ Validators.required, Validators.minLength(3), Validators.maxLength(120) ],
   MESSAGES: {
     'required': 'Video name is required.',
     'minlength': 'Video name must be at least 3 characters long.',
-    'maxlength': 'Video name cannot be more than 50 characters long.'
+    'maxlength': 'Video name cannot be more than 120 characters long.'
   }
 }
 
@@ -45,10 +45,10 @@ export const VIDEO_DESCRIPTION = {
 }
 
 export const VIDEO_TAGS = {
-  VALIDATORS: [ Validators.minLength(2), Validators.maxLength(10) ],
+  VALIDATORS: [ Validators.minLength(2), Validators.maxLength(30) ],
   MESSAGES: {
     'minlength': 'A tag should be more than 2 characters long.',
-    'maxlength': 'A tag should be less than 10 characters long.'
+    'maxlength': 'A tag should be less than 30 characters long.'
   }
 }
 

diff --git a/client/src/app/videos/+video-edit/video-add.component.html b/client/src/app/videos/+video-edit/video-add.component.html
index 7946c0879d7294887809921729b1e4b8b535cacb..136c82a7c2c689b36b948ddbc85e76a00a5c25b6 100644 (file)
--- a/client/src/app/videos/+video-edit/video-add.component.html
+++ b/client/src/app/videos/+video-edit/video-add.component.html
@@ -77,7 +77,7 @@
         <label class="label-tags">Tags</label> <span class="little-information">(press enter to add the tag)</span>
         <tag-input
           [ngModel]="tags" [validators]="tagValidators" [errorMessages]="tagValidatorsMessages"
-          formControlName="tags" maxItems="3" modelAsStrings="true"
+          formControlName="tags" maxItems="5" modelAsStrings="true"
         ></tag-input>
       </div>
 

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 7f4faf21bb0e0cec179ce4c741e0b31db37fa8f4..4ade5a671e2782b11fc12200056278b70628b22b 100644 (file)
--- a/client/src/app/videos/+video-edit/video-update.component.html
+++ b/client/src/app/videos/+video-edit/video-update.component.html
@@ -65,7 +65,7 @@
       <label for="tags" class="label-tags">Tags</label> <span class="little-information">(press enter to add the tag)</span>
       <tag-input
         [ngModel]="tags" [validators]="tagValidators" [errorMessages]="tagValidatorsMessages"
-        formControlName="tags" maxItems="3" modelAsStrings="true"
+        formControlName="tags" maxItems="5" modelAsStrings="true"
       ></tag-input>
     </div>
 

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 30390ac058ef23836a9a2f420e6c0bdaa407d383..c4ae228962a2fd7d7d504babfaedcaed8f30054b 100644 (file)
--- a/client/src/app/videos/+video-edit/video-update.component.ts
+++ b/client/src/app/videos/+video-edit/video-update.component.ts
@@ -131,3 +131,4 @@ export class VideoUpdateComponent extends FormReactive implements OnInit {
     this.form.patchValue(this.video.toJSON())
   }
 }
+

diff --git a/server/initializers/constants.ts b/server/initializers/constants.ts
index 71012f3a3d932ad9e407d323d4fbaee4b6ca6e96..1581a31951d3a7a6028291a2b90f0fb8ee63f0f1 100644 (file)
--- a/server/initializers/constants.ts
+++ b/server/initializers/constants.ts
@@ -118,17 +118,17 @@ const CONSTRAINTS_FIELDS = {
     REASON: { min: 2, max: 300 } // Length
   },
   VIDEO_CHANNELS: {
-    NAME: { min: 3, max: 50 }, // Length
+    NAME: { min: 3, max: 120 }, // Length
     DESCRIPTION: { min: 3, max: 250 } // Length
   },
   VIDEOS: {
-    NAME: { min: 3, max: 50 }, // Length
+    NAME: { min: 3, max: 120 }, // Length
     DESCRIPTION: { min: 3, max: 250 }, // Length
     EXTNAME: [ '.mp4', '.ogv', '.webm' ],
     INFO_HASH: { min: 40, max: 40 }, // Length, info hash is 20 bytes length but we represent it in hexadecimal so 20 * 2
     DURATION: { min: 1, max: 7200 }, // Number
-    TAGS: { min: 0, max: 3 }, // Number of total tags
-    TAG: { min: 2, max: 10 }, // Length
+    TAGS: { min: 0, max: 5 }, // Number of total tags
+    TAG: { min: 2, max: 30 }, // Length
     THUMBNAIL: { min: 2, max: 30 },
     THUMBNAIL_DATA: { min: 0, max: 20000 }, // Bytes
     VIEWS: { min: 0 },

diff --git a/server/tests/api/check-params/video-channels.ts b/server/tests/api/check-params/video-channels.ts
index b46b90c146df0312ca142cd59ca0e9307ce1c69f..08d1af52e57ac581b291ad92ad4a3878dcf37fc1 100644 (file)
--- a/server/tests/api/check-params/video-channels.ts
+++ b/server/tests/api/check-params/video-channels.ts
@@ -119,6 +119,9 @@ describe('Test videos API validator', function () {
     it('Should fail with a long name', async function () {
       const fields = {
         name: 'hello tooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo' +
+              'oooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo' +
+              'oooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo' +
+              'oooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo' +
               'oooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo long',
         description: 'super description'
       }
@@ -178,6 +181,9 @@ describe('Test videos API validator', function () {
     it('Should fail with a long name', async function () {
       const fields = {
         name: 'hello tooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo' +
+        'oooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo' +
+        'oooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo' +
+        'oooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo' +
         'oooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo long',
         description: 'super description'
       }

diff --git a/server/tests/api/check-params/videos.ts b/server/tests/api/check-params/videos.ts
index 167b45b61ef6a519a59e6c2eac69fa3b63f9410c..765b9c16e023ac3b59c899bca26752901f3ed565 100644 (file)
--- a/server/tests/api/check-params/videos.ts
+++ b/server/tests/api/check-params/videos.ts
@@ -124,7 +124,9 @@ describe('Test videos API validator', function () {
 
     it('Should fail with a long name', async function () {
       const fields = {
-        name: 'My very very very very very very very very very very very very very very very very long name',
+        name: 'My very very very very very very very very very very very very very very very very very  ' +
+              'very very very very very very very very very very very very very very very very long long' +
+              'very very very very very very very very very very very very very very very very long name',
         category: 5,
         licence: 1,
         language: 6,
@@ -358,7 +360,7 @@ describe('Test videos API validator', function () {
         language: 6,
         nsfw: false,
         description: 'my super description',
-        tags: [ 'tag1', 'tag2', 'tag3', 'tag4' ],
+        tags: [ 'tag1', 'tag2', 'tag3', 'tag4', 'tag5', 'tag6' ],
         channelId
       }
       const attaches = {
@@ -392,7 +394,7 @@ describe('Test videos API validator', function () {
         language: 6,
         nsfw: false,
         description: 'my super description',
-        tags: [ 'my_super_tag_too_long', 'tag1' ],
+        tags: [ 'my_super_tag_too_long_long_long_long_long_long', 'tag1' ],
         channelId
       }
       const attaches = {
@@ -543,7 +545,9 @@ describe('Test videos API validator', function () {
 
     it('Should fail with a long name', async function () {
       const fields = {
-        name: 'My very very very very very very very very very very very very very very very very long name',
+        name: 'My very very very very very very very very very very very very very very very very very  ' +
+        'very very very very very very very very very very very very very very very very long long' +
+        'very very very very very very very very very very very very very very very very long name',
         category: 5,
         licence: 2,
         language: 6,
@@ -629,7 +633,7 @@ describe('Test videos API validator', function () {
         language: 6,
         nsfw: false,
         description: 'my super description',
-        tags: [ 'tag1', 'tag2', 'tag3', 'tag4' ]
+        tags: [ 'tag1', 'tag2', 'tag3', 'tag4', 'tag5', 'tag6' ]
       }
       await makePutBodyRequest({ url: server.url, path: path + videoId, token: server.accessToken, fields })
     })
@@ -655,7 +659,7 @@ describe('Test videos API validator', function () {
         language: 6,
         nsfw: false,
         description: 'my super description',
-        tags: [ 'my_super_tag_too_long', 'tag1' ]
+        tags: [ 'my_super_tag_too_long_long_long_long', 'tag1' ]
       }
       await makePutBodyRequest({ url: server.url, path: path + videoId, token: server.accessToken, fields })
     })