diff options
author | Chocobozzz <me@florianbigard.com> | 2019-07-25 17:28:45 +0200 |
---|---|---|
committer | Chocobozzz <me@florianbigard.com> | 2019-07-25 17:28:45 +0200 |
commit | 2b65c4e53511ccb85b17479cb1c62717afb8511c (patch) | |
tree | 1dd9ecd1803023fa66152c4746c6050e57adceaa /server/helpers/custom-validators | |
parent | 552d95b1e69fbbd99f5bc300a127457e1b97b9df (diff) | |
download | PeerTube-2b65c4e53511ccb85b17479cb1c62717afb8511c.tar.gz PeerTube-2b65c4e53511ccb85b17479cb1c62717afb8511c.tar.zst PeerTube-2b65c4e53511ccb85b17479cb1c62717afb8511c.zip |
Fix scheduled privacy and verify email validations
Diffstat (limited to 'server/helpers/custom-validators')
-rw-r--r-- | server/helpers/custom-validators/misc.ts | 5 | ||||
-rw-r--r-- | server/helpers/custom-validators/videos.ts | 8 |
2 files changed, 4 insertions, 9 deletions
diff --git a/server/helpers/custom-validators/misc.ts b/server/helpers/custom-validators/misc.ts index 1b7e00431..0c3df1cd0 100644 --- a/server/helpers/custom-validators/misc.ts +++ b/server/helpers/custom-validators/misc.ts | |||
@@ -1,7 +1,6 @@ | |||
1 | import 'multer' | 1 | import 'multer' |
2 | import * as validator from 'validator' | 2 | import * as validator from 'validator' |
3 | import { sep } from 'path' | 3 | import { sep } from 'path' |
4 | import toBoolean = require('validator/lib/toBoolean') | ||
5 | 4 | ||
6 | function exists (value: any) { | 5 | function exists (value: any) { |
7 | return value !== undefined && value !== null | 6 | return value !== undefined && value !== null |
@@ -52,7 +51,7 @@ function toIntOrNull (value: string) { | |||
52 | if (v === null || v === undefined) return v | 51 | if (v === null || v === undefined) return v |
53 | if (typeof v === 'number') return v | 52 | if (typeof v === 'number') return v |
54 | 53 | ||
55 | return validator.toInt(v) | 54 | return validator.toInt('' + v) |
56 | } | 55 | } |
57 | 56 | ||
58 | function toBooleanOrNull (value: any) { | 57 | function toBooleanOrNull (value: any) { |
@@ -61,7 +60,7 @@ function toBooleanOrNull (value: any) { | |||
61 | if (v === null || v === undefined) return v | 60 | if (v === null || v === undefined) return v |
62 | if (typeof v === 'boolean') return v | 61 | if (typeof v === 'boolean') return v |
63 | 62 | ||
64 | return toBoolean(v) | 63 | return validator.toBoolean('' + v) |
65 | } | 64 | } |
66 | 65 | ||
67 | function toValueOrNull (value: string) { | 66 | function toValueOrNull (value: string) { |
diff --git a/server/helpers/custom-validators/videos.ts b/server/helpers/custom-validators/videos.ts index 9ab1ef234..e92ef9b92 100644 --- a/server/helpers/custom-validators/videos.ts +++ b/server/helpers/custom-validators/videos.ts | |||
@@ -98,15 +98,11 @@ function isVideoImage (files: { [ fieldname: string ]: Express.Multer.File[] } | | |||
98 | } | 98 | } |
99 | 99 | ||
100 | function isVideoPrivacyValid (value: number) { | 100 | function isVideoPrivacyValid (value: number) { |
101 | return validator.isInt(value + '') && VIDEO_PRIVACIES[ value ] !== undefined | 101 | return VIDEO_PRIVACIES[ value ] !== undefined |
102 | } | 102 | } |
103 | 103 | ||
104 | function isScheduleVideoUpdatePrivacyValid (value: number) { | 104 | function isScheduleVideoUpdatePrivacyValid (value: number) { |
105 | return validator.isInt(value + '') && | 105 | return value === VideoPrivacy.UNLISTED || value === VideoPrivacy.PUBLIC |
106 | ( | ||
107 | value === VideoPrivacy.UNLISTED || | ||
108 | value === VideoPrivacy.PUBLIC | ||
109 | ) | ||
110 | } | 106 | } |
111 | 107 | ||
112 | function isVideoOriginallyPublishedAtValid (value: string | null) { | 108 | function isVideoOriginallyPublishedAtValid (value: string | null) { |