Rename downloadingEnabled property to downloadEnabled

pull/1217/head
Lucas Declercq 2018-10-08 14:45:22 +02:00
parent 4ffdcfc63b
commit 7f2cfe3a79
35 changed files with 64 additions and 64 deletions

View File

@ -81,7 +81,7 @@ export class VideoImportService {
nsfw: video.nsfw,
waitTranscoding: video.waitTranscoding,
commentsEnabled: video.commentsEnabled,
downloadingEnabled: video.downloadingEnabled,
downloadEnabled: video.downloadEnabled,
thumbnailfile: video.thumbnailfile,
previewfile: video.previewfile,
scheduleUpdate

View File

@ -12,7 +12,7 @@ export class VideoDetails extends Video implements VideoDetailsServerModel {
files: VideoFile[]
account: Account
commentsEnabled: boolean
downloadingEnabled: boolean
downloadEnabled: boolean
waitTranscoding: boolean
state: VideoConstant<VideoState>
@ -30,7 +30,7 @@ export class VideoDetails extends Video implements VideoDetailsServerModel {
this.tags = hash.tags
this.support = hash.support
this.commentsEnabled = hash.commentsEnabled
this.downloadingEnabled = hash.downloadingEnabled
this.downloadEnabled = hash.downloadEnabled
this.buildLikeAndDislikePercents()
}

View File

@ -14,7 +14,7 @@ export class VideoEdit implements VideoUpdate {
tags: string[]
nsfw: boolean
commentsEnabled: boolean
downloadingEnabled: boolean
downloadEnabled: boolean
waitTranscoding: boolean
channelId: number
privacy: VideoPrivacy
@ -27,7 +27,7 @@ export class VideoEdit implements VideoUpdate {
id?: number
scheduleUpdate?: VideoScheduleUpdate
constructor (video?: Video & { tags: string[], commentsEnabled: boolean, downloadingEnabled: boolean, support: string, thumbnailUrl: string, previewUrl: string }) {
constructor (video?: Video & { tags: string[], commentsEnabled: boolean, downloadEnabled: boolean, support: string, thumbnailUrl: string, previewUrl: string }) {
if (video) {
this.id = video.id
this.uuid = video.uuid
@ -39,7 +39,7 @@ export class VideoEdit implements VideoUpdate {
this.tags = video.tags
this.nsfw = video.nsfw
this.commentsEnabled = video.commentsEnabled
this.downloadingEnabled = video.downloadingEnabled
this.downloadEnabled = video.downloadEnabled
this.waitTranscoding = video.waitTranscoding
this.channelId = video.channel.id
this.privacy = video.privacy.id
@ -82,7 +82,7 @@ export class VideoEdit implements VideoUpdate {
tags: this.tags,
nsfw: this.nsfw,
commentsEnabled: this.commentsEnabled,
downloadingEnabled: this.downloadingEnabled,
downloadEnabled: this.downloadEnabled,
waitTranscoding: this.waitTranscoding,
channelId: this.channelId,
privacy: this.privacy

View File

@ -95,7 +95,7 @@ export class VideoService implements VideosProvider {
nsfw: video.nsfw,
waitTranscoding: video.waitTranscoding,
commentsEnabled: video.commentsEnabled,
downloadingEnabled: video.downloadingEnabled,
downloadEnabled: video.downloadEnabled,
thumbnailfile: video.thumbnailfile,
previewfile: video.previewfile,
scheduleUpdate

View File

@ -126,7 +126,7 @@
></my-peertube-checkbox>
<my-peertube-checkbox
inputName="downloadingEnabled" formControlName="downloadingEnabled"
inputName="downloadEnabled" formControlName="downloadEnabled"
i18n-labelText labelText="Disable downloading"
></my-peertube-checkbox>

View File

@ -80,7 +80,7 @@ export class VideoEditComponent implements OnInit, OnDestroy {
const defaultValues = {
nsfw: 'false',
commentsEnabled: 'true',
downloadingEnabled: 'true',
downloadEnabled: 'true',
waitTranscoding: 'true',
tags: []
}
@ -90,7 +90,7 @@ export class VideoEditComponent implements OnInit, OnDestroy {
channelId: this.videoValidatorsService.VIDEO_CHANNEL,
nsfw: null,
commentsEnabled: null,
downloadingEnabled: null,
downloadEnabled: null,
waitTranscoding: null,
category: this.videoValidatorsService.VIDEO_CATEGORY,
licence: this.videoValidatorsService.VIDEO_LICENCE,

View File

@ -77,7 +77,7 @@ export class VideoImportTorrentComponent extends VideoSend implements OnInit, Ca
privacy: this.firstStepPrivacyId,
waitTranscoding: false,
commentsEnabled: true,
downloadingEnabled: true,
downloadEnabled: true,
channelId: this.firstStepChannelId
}
@ -92,7 +92,7 @@ export class VideoImportTorrentComponent extends VideoSend implements OnInit, Ca
this.video = new VideoEdit(Object.assign(res.video, {
commentsEnabled: videoUpdate.commentsEnabled,
downloadingEnabled: videoUpdate.downloadingEnabled,
downloadEnabled: videoUpdate.downloadEnabled,
support: null,
thumbnailUrl: null,
previewUrl: null

View File

@ -69,7 +69,7 @@ export class VideoImportUrlComponent extends VideoSend implements OnInit, CanCom
privacy: this.firstStepPrivacyId,
waitTranscoding: false,
commentsEnabled: true,
downloadingEnabled: true,
downloadEnabled: true,
channelId: this.firstStepChannelId
}
@ -84,7 +84,7 @@ export class VideoImportUrlComponent extends VideoSend implements OnInit, CanCom
this.video = new VideoEdit(Object.assign(res.video, {
commentsEnabled: videoUpdate.commentsEnabled,
downloadingEnabled: videoUpdate.downloadingEnabled,
downloadEnabled: videoUpdate.downloadEnabled,
support: null,
thumbnailUrl: null,
previewUrl: null

View File

@ -159,7 +159,7 @@ export class VideoUploadComponent extends VideoSend implements OnInit, OnDestroy
const nsfw = false
const waitTranscoding = true
const commentsEnabled = true
const downloadingEnabled = true
const downloadEnabled = true
const channelId = this.firstStepChannelId.toString()
const formData = new FormData()
@ -168,7 +168,7 @@ export class VideoUploadComponent extends VideoSend implements OnInit, OnDestroy
formData.append('privacy', VideoPrivacy.PRIVATE.toString())
formData.append('nsfw', '' + nsfw)
formData.append('commentsEnabled', '' + commentsEnabled)
formData.append('downloadingEnabled', '' + downloadingEnabled)
formData.append('downloadEnabled', '' + downloadEnabled)
formData.append('waitTranscoding', '' + waitTranscoding)
formData.append('channelId', '' + channelId)
formData.append('videofile', videofile)

View File

@ -314,7 +314,7 @@ export class VideoWatchComponent implements OnInit, OnDestroy {
}
isVideoDownloadable () {
return this.video && this.video.downloadingEnabled
return this.video && this.video.downloadEnabled
}
isVideoToImport () {

View File

@ -171,7 +171,7 @@ function buildVideo (channelId: number, body: VideoImportCreate, importData: You
licence: body.licence || importData.licence,
language: body.language || undefined,
commentsEnabled: body.commentsEnabled || true,
downloadingEnabled: body.downloadingEnabled || true,
downloadEnabled: body.downloadEnabled || true,
waitTranscoding: body.waitTranscoding || false,
state: VideoState.TO_IMPORT,
nsfw: body.nsfw || importData.nsfw || false,

View File

@ -179,7 +179,7 @@ async function addVideo (req: express.Request, res: express.Response) {
licence: videoInfo.licence,
language: videoInfo.language,
commentsEnabled: videoInfo.commentsEnabled || false,
downloadingEnabled: videoInfo.downloadingEnabled || true,
downloadEnabled: videoInfo.downloadEnabled || true,
waitTranscoding: videoInfo.waitTranscoding || false,
state: CONFIG.TRANSCODING.ENABLED ? VideoState.TO_TRANSCODE : VideoState.PUBLISHED,
nsfw: videoInfo.nsfw || false,
@ -323,7 +323,7 @@ async function updateVideo (req: express.Request, res: express.Response) {
if (videoInfoToUpdate.support !== undefined) videoInstance.set('support', videoInfoToUpdate.support)
if (videoInfoToUpdate.description !== undefined) videoInstance.set('description', videoInfoToUpdate.description)
if (videoInfoToUpdate.commentsEnabled !== undefined) videoInstance.set('commentsEnabled', videoInfoToUpdate.commentsEnabled)
if (videoInfoToUpdate.downloadingEnabled !== undefined) videoInstance.set('downloadingEnabled', videoInfoToUpdate.downloadingEnabled)
if (videoInfoToUpdate.downloadEnabled !== undefined) videoInstance.set('downloadEnabled', videoInfoToUpdate.downloadEnabled)
if (videoInfoToUpdate.privacy !== undefined) {
const newPrivacy = parseInt(videoInfoToUpdate.privacy.toString(), 10)
videoInstance.set('privacy', newPrivacy)

View File

@ -28,7 +28,7 @@ function activityPubContextify <T> (data: T) {
size: 'schema:Number',
fps: 'schema:Number',
commentsEnabled: 'schema:Boolean',
downloadingEnabled: 'schema:Boolean',
downloadEnabled: 'schema:Boolean',
waitTranscoding: 'schema:Boolean',
expires: 'schema:expires',
support: 'schema:Text',

View File

@ -118,7 +118,7 @@ const videoKeysToKeep = [
'channel-name',
'support',
'commentsEnabled',
'downloadingEnabled'
'downloadEnabled'
]
class VideoAuditView extends EntityAuditView {
constructor (private video: VideoDetails) {

View File

@ -56,7 +56,7 @@ function sanitizeAndCheckVideoTorrentObject (video: any) {
// Default attributes
if (!isVideoStateValid(video.state)) video.state = VideoState.PUBLISHED
if (!isBooleanValid(video.waitTranscoding)) video.waitTranscoding = false
if (!isBooleanValid(video.downloadingEnabled)) video.downloadingEnabled = true
if (!isBooleanValid(video.downloadEnabled)) video.downloadEnabled = true
return isActivityPubUrlValid(video.id) &&
isVideoNameValid(video.name) &&
@ -68,7 +68,7 @@ function sanitizeAndCheckVideoTorrentObject (video: any) {
isVideoViewsValid(video.views) &&
isBooleanValid(video.sensitive) &&
isBooleanValid(video.commentsEnabled) &&
isBooleanValid(video.downloadingEnabled) &&
isBooleanValid(video.downloadEnabled) &&
isDateValid(video.published) &&
isDateValid(video.updated) &&
(!video.content || isRemoteVideoContentValid(video.mediaType, video.content)) &&

View File

@ -11,10 +11,10 @@ async function up (utils: {
allowNull: false,
defaultValue: true
} as Migration.Boolean
await utils.queryInterface.addColumn('video', 'downloadingEnabled', data)
await utils.queryInterface.addColumn('video', 'downloadEnabled', data)
data.defaultValue = null
return utils.queryInterface.changeColumn('video', 'downloadingEnabled', data)
return utils.queryInterface.changeColumn('video', 'downloadEnabled', data)
}
function down (options) {

View File

@ -230,7 +230,7 @@ async function updateVideoFromAP (options: {
options.video.set('support', videoData.support)
options.video.set('nsfw', videoData.nsfw)
options.video.set('commentsEnabled', videoData.commentsEnabled)
options.video.set('downloadingEnabled', videoData.downloadingEnabled)
options.video.set('downloadEnabled', videoData.downloadEnabled)
options.video.set('waitTranscoding', videoData.waitTranscoding)
options.video.set('state', videoData.state)
options.video.set('duration', videoData.duration)
@ -442,7 +442,7 @@ async function videoActivityObjectToDBAttributes (
support,
nsfw: videoObject.sensitive,
commentsEnabled: videoObject.commentsEnabled,
downloadingEnabled: videoObject.downloadingEnabled,
downloadEnabled: videoObject.downloadEnabled,
waitTranscoding: videoObject.waitTranscoding,
state: videoObject.state,
channelId: videoChannel.id,

View File

@ -349,7 +349,7 @@ function getCommonVideoAttributes () {
.optional()
.toBoolean()
.custom(isBooleanValid).withMessage('Should have comments enabled boolean'),
body('downloadingEnabled')
body('downloadEnabled')
.optional()
.toBoolean()
.custom(isBooleanValid).withMessage('Should have downloading enabled boolean'),

View File

@ -128,7 +128,7 @@ function videoModelToFormattedDetailsJSON (video: VideoModel): VideoDetails {
account: video.VideoChannel.Account.toFormattedJSON(),
tags,
commentsEnabled: video.commentsEnabled,
downloadingEnabled: video.downloadingEnabled,
downloadEnabled: video.downloadEnabled,
waitTranscoding: video.waitTranscoding,
state: {
id: video.state,
@ -260,7 +260,7 @@ function videoModelToActivityPubObject (video: VideoModel): VideoTorrentObject {
waitTranscoding: video.waitTranscoding,
state: video.state,
commentsEnabled: video.commentsEnabled,
downloadingEnabled: video.downloadingEnabled,
downloadEnabled: video.downloadEnabled,
published: video.publishedAt.toISOString(),
updated: video.updatedAt.toISOString(),
mediaType: 'text/markdown',

View File

@ -600,7 +600,7 @@ export class VideoModel extends Model<VideoModel> {
@AllowNull(false)
@Column
downloadingEnabled: boolean
downloadEnabled: boolean
@AllowNull(false)
@Column

View File

@ -84,7 +84,7 @@ describe('Test video imports API validator', function () {
language: 'pt',
nsfw: false,
commentsEnabled: true,
downloadingEnabled: true,
downloadEnabled: true,
waitTranscoding: true,
description: 'my super description',
support: 'my super support text',

View File

@ -175,7 +175,7 @@ describe('Test videos API validator', function () {
language: 'pt',
nsfw: false,
commentsEnabled: true,
downloadingEnabled: true,
downloadEnabled: true,
waitTranscoding: true,
description: 'my super description',
support: 'my super support text',
@ -420,7 +420,7 @@ describe('Test videos API validator', function () {
language: 'pt',
nsfw: false,
commentsEnabled: false,
downloadingEnabled: false,
downloadEnabled: false,
description: 'my super description',
privacy: VideoPrivacy.PUBLIC,
tags: [ 'tag1', 'tag2' ]

View File

@ -305,7 +305,7 @@ describe('Test follows', function () {
},
isLocal,
commentsEnabled: true,
downloadingEnabled: true,
downloadEnabled: true,
duration: 5,
tags: [ 'tag1', 'tag2', 'tag3' ],
privacy: VideoPrivacy.PUBLIC,

View File

@ -70,7 +70,7 @@ describe('Test handle downs', function () {
tags: [ 'tag1p1', 'tag2p1' ],
privacy: VideoPrivacy.PUBLIC,
commentsEnabled: true,
downloadingEnabled: true,
downloadEnabled: true,
channel: {
name: 'root_channel',
displayName: 'Main root channel',

View File

@ -127,7 +127,7 @@ describe('Test multiple servers', function () {
tags: [ 'tag1p1', 'tag2p1' ],
privacy: VideoPrivacy.PUBLIC,
commentsEnabled: true,
downloadingEnabled: true,
downloadEnabled: true,
channel: {
displayName: 'my channel',
name: 'super_channel_name',
@ -199,7 +199,7 @@ describe('Test multiple servers', function () {
},
isLocal,
commentsEnabled: true,
downloadingEnabled: true,
downloadEnabled: true,
duration: 5,
tags: [ 'tag1p2', 'tag2p2', 'tag3p2' ],
privacy: VideoPrivacy.PUBLIC,
@ -308,7 +308,7 @@ describe('Test multiple servers', function () {
isLocal,
duration: 5,
commentsEnabled: true,
downloadingEnabled: true,
downloadEnabled: true,
tags: [ 'tag1p3' ],
privacy: VideoPrivacy.PUBLIC,
channel: {
@ -340,7 +340,7 @@ describe('Test multiple servers', function () {
host: 'localhost:9003'
},
commentsEnabled: true,
downloadingEnabled: true,
downloadEnabled: true,
isLocal,
duration: 5,
tags: [ 'tag2p3', 'tag3p3', 'tag4p3' ],
@ -658,7 +658,7 @@ describe('Test multiple servers', function () {
isLocal,
duration: 5,
commentsEnabled: true,
downloadingEnabled: true,
downloadEnabled: true,
tags: [ 'tag_up_1', 'tag_up_2' ],
privacy: VideoPrivacy.PUBLIC,
channel: {
@ -980,7 +980,7 @@ describe('Test multiple servers', function () {
isLocal,
duration: 5,
commentsEnabled: false,
downloadingEnabled: false,
downloadEnabled: false,
tags: [ ],
privacy: VideoPrivacy.PUBLIC,
channel: {

View File

@ -55,7 +55,7 @@ describe('Test a single server', function () {
tags: [ 'tag1', 'tag2', 'tag3' ],
privacy: VideoPrivacy.PUBLIC,
commentsEnabled: true,
downloadingEnabled: true,
downloadEnabled: true,
channel: {
displayName: 'Main root channel',
name: 'root_channel',
@ -88,7 +88,7 @@ describe('Test a single server', function () {
privacy: VideoPrivacy.PUBLIC,
duration: 5,
commentsEnabled: false,
downloadingEnabled: false,
downloadEnabled: false,
channel: {
name: 'root_channel',
displayName: 'Main root channel',
@ -358,7 +358,7 @@ describe('Test a single server', function () {
nsfw: false,
description: 'my super description updated',
commentsEnabled: false,
downloadingEnabled: false,
downloadEnabled: false,
tags: [ 'tagup1', 'tagup2' ]
}
await updateVideo(server.url, server.accessToken, videoId, attributes)

View File

@ -27,7 +27,7 @@ type VideoAttributes = {
language?: string
nsfw?: boolean
commentsEnabled?: boolean
downloadingEnabled?: boolean
downloadEnabled?: boolean
waitTranscoding?: boolean
description?: string
tags?: string[]
@ -311,7 +311,7 @@ async function uploadVideo (url: string, accessToken: string, videoAttributesArg
tags: [ 'tag' ],
privacy: VideoPrivacy.PUBLIC,
commentsEnabled: true,
downloadingEnabled: true,
downloadEnabled: true,
fixture: 'video_short.webm'
}, videoAttributesArg)
@ -322,7 +322,7 @@ async function uploadVideo (url: string, accessToken: string, videoAttributesArg
.field('name', attributes.name)
.field('nsfw', JSON.stringify(attributes.nsfw))
.field('commentsEnabled', JSON.stringify(attributes.commentsEnabled))
.field('downloadingEnabled', JSON.stringify(attributes.downloadingEnabled))
.field('downloadEnabled', JSON.stringify(attributes.downloadEnabled))
.field('waitTranscoding', JSON.stringify(attributes.waitTranscoding))
.field('privacy', attributes.privacy.toString())
.field('channelId', attributes.channelId)
@ -373,7 +373,7 @@ function updateVideo (url: string, accessToken: string, id: number | string, att
if (attributes.language) body['language'] = attributes.language
if (attributes.nsfw !== undefined) body['nsfw'] = JSON.stringify(attributes.nsfw)
if (attributes.commentsEnabled !== undefined) body['commentsEnabled'] = JSON.stringify(attributes.commentsEnabled)
if (attributes.downloadingEnabled !== undefined) body['downloadingEnabled'] = JSON.stringify(attributes.downloadingEnabled)
if (attributes.downloadEnabled !== undefined) body['downloadEnabled'] = JSON.stringify(attributes.downloadEnabled)
if (attributes.description) body['description'] = attributes.description
if (attributes.tags) body['tags'] = attributes.tags
if (attributes.privacy) body['privacy'] = attributes.privacy
@ -439,7 +439,7 @@ async function completeVideoCheck (
language: string
nsfw: boolean
commentsEnabled: boolean
downloadingEnabled: boolean
downloadEnabled: boolean
description: string
publishedAt?: string
support: string
@ -514,7 +514,7 @@ async function completeVideoCheck (
expect(dateIsValid(videoDetails.channel.createdAt.toString())).to.be.true
expect(dateIsValid(videoDetails.channel.updatedAt.toString())).to.be.true
expect(videoDetails.commentsEnabled).to.equal(attributes.commentsEnabled)
expect(videoDetails.downloadingEnabled).to.equal(attributes.downloadingEnabled)
expect(videoDetails.downloadEnabled).to.equal(attributes.downloadEnabled)
for (const attributeFile of attributes.files) {
const file = videoDetails.files.find(f => f.resolution.id === attributeFile.resolution)

View File

@ -212,7 +212,7 @@ async function uploadVideoOnPeerTube (videoInfo: any, videoPath: string, cwd: st
nsfw: isNSFW(videoInfo),
waitTranscoding: true,
commentsEnabled: true,
downloadingEnabled: true,
downloadEnabled: true,
description: videoInfo.description || undefined,
support: undefined,
tags,

View File

@ -30,7 +30,7 @@ if (!program['tags']) program['tags'] = []
if (!program['nsfw']) program['nsfw'] = false
if (!program['privacy']) program['privacy'] = VideoPrivacy.PUBLIC
if (!program['commentsEnabled']) program['commentsEnabled'] = false
if (!program['downloadingEnabled']) program['downloadingEnabled'] = true
if (!program['downloadEnabled']) program['downloadEnabled'] = true
getSettings()
.then(settings => {
@ -117,7 +117,7 @@ async function run () {
description: program['videoDescription'],
tags: program['tags'],
commentsEnabled: program['commentsEnabled'],
downloadingEnabled: program['downloadingEnabled'],
downloadEnabled: program['downloadEnabled'],
fixture: program['file'],
thumbnailfile: program['thumbnail'],
previewfile: program['preview'],

View File

@ -21,7 +21,7 @@ export interface VideoTorrentObject {
views: number
sensitive: boolean
commentsEnabled: boolean,
downloadingEnabled: boolean,
downloadEnabled: boolean,
waitTranscoding: boolean
state: VideoState
published: string

View File

@ -13,7 +13,7 @@ export interface VideoCreate {
name: string
tags?: string[]
commentsEnabled?: boolean
downloadingEnabled?: boolean
downloadEnabled?: boolean
privacy: VideoPrivacy
scheduleUpdate?: VideoScheduleUpdate
}

View File

@ -11,7 +11,7 @@ export interface VideoUpdate {
privacy?: VideoPrivacy
tags?: string[]
commentsEnabled?: boolean
downloadingEnabled?: boolean
downloadEnabled?: boolean
nsfw?: boolean
waitTranscoding?: boolean
channelId?: number

View File

@ -82,7 +82,7 @@ export interface VideoDetails extends Video {
files: VideoFile[]
account: Account
commentsEnabled: boolean
downloadingEnabled: boolean
downloadEnabled: boolean
// Not optional in details (unlike in Video)
waitTranscoding: boolean

View File

@ -536,7 +536,7 @@ paths:
- $ref: "videos.yaml#/parameters/name"
- $ref: "videos.yaml#/parameters/tags"
- $ref: "videos.yaml#/parameters/commentsEnabled"
- $ref: "videos.yaml#/parameters/downloadingEnabled"
- $ref: "videos.yaml#/parameters/downloadEnabled"
- $ref: "videos.yaml#/parameters/privacy"
- $ref: "videos.yaml#/parameters/scheduleUpdate"
responses:
@ -633,7 +633,7 @@ paths:
- $ref: "videos.yaml#/parameters/name"
- $ref: "videos.yaml#/parameters/tags"
- $ref: "videos.yaml#/parameters/commentsEnabled"
- $ref: "videos.yaml#/parameters/downloadingEnabled"
- $ref: "videos.yaml#/parameters/downloadEnabled"
- $ref: "videos.yaml#/parameters/privacy"
- $ref: "videos.yaml#/parameters/scheduleUpdate"
responses:

View File

@ -65,8 +65,8 @@ parameters:
in: formData
type: boolean
description: 'Enable or disable comments for this video'
downloadingEnabled:
name: downloadingEnabled
downloadEnabled:
name: downloadEnabled
in: formData
type: boolean
description: 'Enable or disable downloading for this video'