Fix lint, tests and styling

client/src/app/+my-account/my-account-import-export/my-account-import-export.component.scss

@@ -1,2 +0,0 @@
-@use '_variables' as *;
-@use '_mixins' as *;

client/src/app/+my-account/my-account-import-export/my-account-import-export.component.ts

@@ -5,8 +5,7 @@ import { first } from 'rxjs'
 
 @Component({
   selector: 'my-account-import-export',
-  templateUrl: './my-account-import-export.component.html',
+  templateUrl: './my-account-import-export.component.html'
-  styleUrls: [ './my-account-import-export.component.scss' ]
 })
 export class MyAccountImportExportComponent implements OnInit, CanComponentDeactivate {
   @ViewChild('accountImport') accountImport: MyAccountImportComponent
@@ -20,11 +19,11 @@ export class MyAccountImportExportComponent implements OnInit, CanComponentDeact
 
   ngOnInit () {
     this.authService.userInformationLoaded
-    .pipe(first())
+      .pipe(first())
-    .subscribe(() => {
+      .subscribe(() => {
-      this.userService.getMyVideoQuotaUsed()
+        this.userService.getMyVideoQuotaUsed()
-        .subscribe(res => this.videoQuotaUsed = res.videoQuotaUsed)
+          .subscribe(res => this.videoQuotaUsed = res.videoQuotaUsed)
-    })
+      })
   }
 
   canDeactivate () {

client/src/app/+my-account/my-account-import-export/my-account-import.component.html

@@ -7,8 +7,7 @@
   <div class="col-12 col-lg-8 col-xl-9">
 
     @if (isImportEnabled()) {
-      <p i18n>
+      <p i18n>You can import an archive created by another PeerTube website.</p>
-        You can import an archive created by another PeerTube website.
 
       <p i18n>
         This is an <strong>import</strong> tool and not a migration tool.

client/src/app/+my-account/my-account.component.ts

@@ -1,5 +1,5 @@
 import { Component, OnInit } from '@angular/core'
-import { AuthUser, ScreenService, ServerService } from '@app/core'
+import { AuthUser, ScreenService } from '@app/core'
 import { TopMenuDropdownParam } from '../shared/shared-main/misc/top-menu-dropdown.component'
 
 @Component({
@@ -11,10 +11,7 @@ export class MyAccountComponent implements OnInit {
   menuEntries: TopMenuDropdownParam[] = []
   user: AuthUser
 
-  constructor (
+  constructor (private screenService: ScreenService) { }
-    private screenService: ScreenService,
-    private server: ServerService
-  ) { }
 
   get isBroadcastMessageDisplayed () {
     return this.screenService.isBroadcastMessageDisplayed

packages/models/src/server/server-config.model.ts

@@ -213,7 +213,7 @@ export interface ServerConfig {
     }
 
     users: {
-      enabled:boolean
+      enabled: boolean
     }
   }
 

packages/server-commands/src/videos/videos-command.ts

@@ -4,8 +4,7 @@ import validator from 'validator'
 import { getAllPrivacies, omit, pick, wait } from '@peertube/peertube-core-utils'
 import {
   HttpStatusCode,
-  HttpStatusCodeType,
+  HttpStatusCodeType, ResultList,
-  ResultList,
   UserVideoRateType,
   Video,
   VideoCreate,
@@ -631,10 +630,12 @@ export class VideosCommand extends AbstractCommand {
     return super.sendResumableChunks<{ video: VideoCreateResult }>(options)
   }
 
-  async buildResumeVideoUpload (options: Parameters<AbstractCommand['buildResumeUpload']>[0]) {
+  async buildResumeVideoUpload (
+    options: Parameters<AbstractCommand['buildResumeUpload']>[0]
+  ): Promise<VideoCreateResult> {
     const result = await super.buildResumeUpload<{ video: VideoCreateResult }>(options)
 
-    return result?.video || undefined
+    return result?.video || result as any
   }
 
   prepareVideoResumableUpload (options: Parameters<AbstractCommand['prepareResumableUpload']>[0]) {

packages/tests/src/api/check-params/user-import.ts

@@ -1,12 +1,12 @@
 /* eslint-disable @typescript-eslint/no-unused-expressions,@typescript-eslint/require-await */
 
+import { HttpStatusCode } from '@peertube/peertube-models'
 import {
   cleanupTests,
   createSingleServer, PeerTubeServer,
   setAccessTokensToServers,
   waitJobs
 } from '@peertube/peertube-server-commands'
-import { HttpStatusCode } from '../../../../models/src/http/http-status-codes.js'
 import { expect } from 'chai'
 
 describe('Test user import API validators', function () {

packages/tests/src/shared/videos.ts

@@ -301,8 +301,8 @@ function checkUploadVideoParam (options: {
     : server.videos.buildResumeVideoUpload({
       token,
       fixture: attributes.fixture,
-      attaches: this.buildUploadAttaches(attributes),
+      attaches: server.videos.buildUploadAttaches(attributes, false),
-      fields: this.buildUploadFields(attributes),
+      fields: server.videos.buildUploadFields(attributes),
       expectedStatus,
       completedExpectedStatus,
       path: '/api/v1/videos/upload-resumable'

server/core/models/video/video-playlist-element.ts

@@ -271,7 +271,8 @@ export class VideoPlaylistElementModel extends Model<Partial<AttributesOnly<Vide
           model: VideoModel.unscoped(),
           required: true
         }
-      ]
+      ],
+      order: getSort('position')
     }
 
     return VideoPlaylistElementModel.findAll(query)