From 0dc647775881eb1378b213a530996cd096de24ea Mon Sep 17 00:00:00 2001
From: Chocobozzz <me@florianbigard.com>
Date: Mon, 8 Apr 2019 15:47:44 +0200
Subject: [PATCH] Add delete/manual approval instance followers in client

---
 .../edit-custom-config.component.html         | 27 ++++++++
 .../edit-custom-config.component.ts           |  8 ++-
 .../followers-list.component.html             | 20 ++++--
 .../followers-list.component.scss             |  8 ++-
 .../followers-list.component.ts               | 64 +++++++++++++++++--
 .../+admin/follows/shared/follow.service.ts   | 30 +++++++++
 .../my-account-ownership.component.html       | 13 ++--
 7 files changed, 149 insertions(+), 21 deletions(-)

diff --git a/client/src/app/+admin/config/edit-custom-config/edit-custom-config.component.html b/client/src/app/+admin/config/edit-custom-config/edit-custom-config.component.html
index 00a0d98f8..637484622 100644
--- a/client/src/app/+admin/config/edit-custom-config/edit-custom-config.component.html
+++ b/client/src/app/+admin/config/edit-custom-config/edit-custom-config.component.html
@@ -84,6 +84,7 @@
           </div>
         </ng-container>
 
+
         <div i18n class="inner-form-title">Signup</div>
 
         <ng-container formGroupName="signup">
@@ -111,6 +112,7 @@
           </div>
         </ng-container>
 
+
         <div i18n class="inner-form-title">Users</div>
 
         <ng-container formGroupName="user">
@@ -139,6 +141,7 @@
           </div>
         </ng-container>
 
+
         <div i18n class="inner-form-title">Import</div>
 
         <ng-container formGroupName="import">
@@ -161,6 +164,7 @@
           </ng-container>
         </ng-container>
 
+
         <div i18n class="inner-form-title">Auto-blacklist</div>
 
         <ng-container formGroupName="autoBlacklist">
@@ -178,6 +182,29 @@
           </ng-container>
         </ng-container>
 
+
+        <div i18n class="inner-form-title">Instance followers</div>
+
+        <ng-container formGroupName="followers">
+          <ng-container formGroupName="instance">
+
+            <div class="form-group">
+              <my-peertube-checkbox
+                inputName="followersInstanceEnabled" formControlName="enabled"
+                i18n-labelText labelText="Other instances can follow your instance"
+              ></my-peertube-checkbox>
+            </div>
+
+            <div class="form-group">
+              <my-peertube-checkbox
+                inputName="followersInstanceManualApproval" formControlName="manualApproval"
+                i18n-labelText labelText="Manually approve new instance follower"
+              ></my-peertube-checkbox>
+            </div>
+          </ng-container>
+        </ng-container>
+
+
         <div i18n class="inner-form-title">Administrator</div>
 
         <div class="form-group" formGroupName="admin">
diff --git a/client/src/app/+admin/config/edit-custom-config/edit-custom-config.component.ts b/client/src/app/+admin/config/edit-custom-config/edit-custom-config.component.ts
index d8eb55da7..e64750713 100644
--- a/client/src/app/+admin/config/edit-custom-config/edit-custom-config.component.ts
+++ b/client/src/app/+admin/config/edit-custom-config/edit-custom-config.component.ts
@@ -5,7 +5,7 @@ import { CustomConfigValidatorsService, FormReactive, UserValidatorsService } fr
 import { Notifier } from '@app/core'
 import { CustomConfig } from '../../../../../../shared/models/server/custom-config.model'
 import { I18n } from '@ngx-translate/i18n-polyfill'
-import { BuildFormDefaultValues, FormValidatorService } from '@app/shared/forms/form-validators/form-validator.service'
+import { FormValidatorService } from '@app/shared/forms/form-validators/form-validator.service'
 
 @Component({
   selector: 'my-edit-custom-config',
@@ -124,6 +124,12 @@ export class EditCustomConfigComponent extends FormReactive implements OnInit {
             enabled: null
           }
         }
+      },
+      followers: {
+        instance: {
+          enabled: null,
+          manualApproval: null
+        }
       }
     }
 
diff --git a/client/src/app/+admin/follows/followers-list/followers-list.component.html b/client/src/app/+admin/follows/followers-list/followers-list.component.html
index fc022bdb4..da0ba95e3 100644
--- a/client/src/app/+admin/follows/followers-list/followers-list.component.html
+++ b/client/src/app/+admin/follows/followers-list/followers-list.component.html
@@ -14,25 +14,33 @@
   <ng-template pTemplate="header">
     <tr>
       <th i18n style="width: 60px">ID</th>
-      <th i18n>Score</th>
-      <th i18n>Name</th>
-      <th i18n>Host</th>
+      <th i18n>Follower handle</th>
       <th i18n>State</th>
+      <th i18n>Score</th>
       <th i18n pSortableColumn="createdAt">Created <p-sortIcon field="createdAt"></p-sortIcon></th>
+      <th></th>
     </tr>
   </ng-template>
 
   <ng-template pTemplate="body" let-follow>
     <tr>
       <td>{{ follow.id }}</td>
-      <td>{{ follow.score }}</td>
-      <td>{{ follow.follower.name }}</td>
-      <td>{{ follow.follower.host }}</td>
+      <td>{{ follow.follower.name + '@' + follow.follower.host }}</td>
 
       <td *ngIf="follow.state === 'accepted'" i18n>Accepted</td>
       <td *ngIf="follow.state === 'pending'" i18n>Pending</td>
 
+      <td>{{ follow.score }}</td>
       <td>{{ follow.createdAt }}</td>
+
+      <td class="action-cell">
+        <ng-container *ngIf="follow.state === 'pending'">
+          <my-button i18n-label label="Accept" icon="tick" (click)="acceptFollower(follow)"></my-button>
+          <my-button i18n-label label="Refuse" icon="cross" (click)="rejectFollower(follow)"></my-button>
+        </ng-container>
+
+        <my-delete-button *ngIf="follow.state === 'accepted'" (click)="deleteFollower(follow)"></my-delete-button>
+      </td>
     </tr>
   </ng-template>
 </p-table>
diff --git a/client/src/app/+admin/follows/followers-list/followers-list.component.scss b/client/src/app/+admin/follows/followers-list/followers-list.component.scss
index a6f0656b8..964b3f99b 100644
--- a/client/src/app/+admin/follows/followers-list/followers-list.component.scss
+++ b/client/src/app/+admin/follows/followers-list/followers-list.component.scss
@@ -7,4 +7,10 @@
   input {
     @include peertube-input-text(250px);
   }
-}
\ No newline at end of file
+}
+
+.action-cell {
+  my-button:first-child {
+    margin-right: 10px;
+  }
+}
diff --git a/client/src/app/+admin/follows/followers-list/followers-list.component.ts b/client/src/app/+admin/follows/followers-list/followers-list.component.ts
index 9a8848bfb..b78cdf656 100644
--- a/client/src/app/+admin/follows/followers-list/followers-list.component.ts
+++ b/client/src/app/+admin/follows/followers-list/followers-list.component.ts
@@ -1,6 +1,5 @@
 import { Component, OnInit } from '@angular/core'
-
-import { Notifier } from '@app/core'
+import { ConfirmService, Notifier } from '@app/core'
 import { SortMeta } from 'primeng/primeng'
 import { ActorFollow } from '../../../../../../shared/models/actors/follow.model'
 import { RestPagination, RestTable } from '../../../shared'
@@ -20,9 +19,10 @@ export class FollowersListComponent extends RestTable implements OnInit {
   pagination: RestPagination = { count: this.rowsPerPage, start: 0 }
 
   constructor (
+    private confirmService: ConfirmService,
     private notifier: Notifier,
-    private followService: FollowService,
-    private i18n: I18n
+    private i18n: I18n,
+    private followService: FollowService
   ) {
     super()
   }
@@ -31,6 +31,62 @@ export class FollowersListComponent extends RestTable implements OnInit {
     this.initialize()
   }
 
+  acceptFollower (follow: ActorFollow) {
+    follow.state = 'accepted'
+
+    this.followService.acceptFollower(follow)
+      .subscribe(
+        () => {
+          const handle = follow.follower.name + '@' + follow.follower.host
+          this.notifier.success(this.i18n('{{handle}} accepted in instance followers', { handle }))
+        },
+
+        err => {
+          follow.state = 'pending'
+          this.notifier.error(err.message)
+        }
+      )
+  }
+
+  async rejectFollower (follow: ActorFollow) {
+    const message = this.i18n('Do you really want to reject this follower?')
+    const res = await this.confirmService.confirm(message, this.i18n('Reject'))
+    if (res === false) return
+
+    this.followService.rejectFollower(follow)
+        .subscribe(
+          () => {
+            const handle = follow.follower.name + '@' + follow.follower.host
+            this.notifier.success(this.i18n('{{handle}} rejected from instance followers', { handle }))
+
+            this.loadData()
+          },
+
+          err => {
+            follow.state = 'pending'
+            this.notifier.error(err.message)
+          }
+        )
+  }
+
+  async deleteFollower (follow: ActorFollow) {
+    const message = this.i18n('Do you really want to delete this follower?')
+    const res = await this.confirmService.confirm(message, this.i18n('Delete'))
+    if (res === false) return
+
+    this.followService.removeFollower(follow)
+        .subscribe(
+          () => {
+            const handle = follow.follower.name + '@' + follow.follower.host
+            this.notifier.success(this.i18n('{{handle}} removed from instance followers', { handle }))
+
+            this.loadData()
+          },
+
+          err => this.notifier.error(err.message)
+        )
+  }
+
   protected loadData () {
     this.followService.getFollowers(this.pagination, this.sort, this.search)
                       .subscribe(
diff --git a/client/src/app/+admin/follows/shared/follow.service.ts b/client/src/app/+admin/follows/shared/follow.service.ts
index a2904179e..c2b8ef006 100644
--- a/client/src/app/+admin/follows/shared/follow.service.ts
+++ b/client/src/app/+admin/follows/shared/follow.service.ts
@@ -63,4 +63,34 @@ export class FollowService {
                  catchError(res => this.restExtractor.handleError(res))
                )
   }
+
+  acceptFollower (follow: ActorFollow) {
+    const handle = follow.follower.name + '@' + follow.follower.host
+
+    return this.authHttp.post(`${FollowService.BASE_APPLICATION_URL}/followers/${handle}/accept`, {})
+               .pipe(
+                 map(this.restExtractor.extractDataBool),
+                 catchError(res => this.restExtractor.handleError(res))
+               )
+  }
+
+  rejectFollower (follow: ActorFollow) {
+    const handle = follow.follower.name + '@' + follow.follower.host
+
+    return this.authHttp.post(`${FollowService.BASE_APPLICATION_URL}/followers/${handle}/reject`, {})
+               .pipe(
+                 map(this.restExtractor.extractDataBool),
+                 catchError(res => this.restExtractor.handleError(res))
+               )
+  }
+
+  removeFollower (follow: ActorFollow) {
+    const handle = follow.follower.name + '@' + follow.follower.host
+
+    return this.authHttp.delete(`${FollowService.BASE_APPLICATION_URL}/followers/${handle}`)
+               .pipe(
+                 map(this.restExtractor.extractDataBool),
+                 catchError(res => this.restExtractor.handleError(res))
+               )
+  }
 }
diff --git a/client/src/app/+my-account/my-account-ownership/my-account-ownership.component.html b/client/src/app/+my-account/my-account-ownership/my-account-ownership.component.html
index 5709e9f54..c5fd3ccb9 100644
--- a/client/src/app/+my-account/my-account-ownership/my-account-ownership.component.html
+++ b/client/src/app/+my-account/my-account-ownership/my-account-ownership.component.html
@@ -30,8 +30,7 @@
         </a>
       </td>
       <td>
-        <a [href]="videoChangeOwnership.video.url" i18n-title title="Go to the video" target="_blank"
-           rel="noopener noreferrer">
+        <a [href]="videoChangeOwnership.video.url" i18n-title title="Go to the video" target="_blank" rel="noopener noreferrer">
           {{ videoChangeOwnership.video.name }}
         </a>
       </td>
@@ -39,16 +38,12 @@
       <td i18n>{{ videoChangeOwnership.status }}</td>
       <td class="action-cell">
         <ng-container *ngIf="videoChangeOwnership.status === 'WAITING'">
-          <my-button i18n label="Accept"
-                     icon="tick"
-                     (click)="openAcceptModal(videoChangeOwnership)"></my-button>
-          <my-button i18n label="Refuse"
-                     icon="cross"
-                     (click)="refuse(videoChangeOwnership)">Refuse</my-button>
+          <my-button i18n-label label="Accept" icon="tick" (click)="openAcceptModal(videoChangeOwnership)"></my-button>
+          <my-button i18n-label label="Refuse" icon="cross" (click)="refuse(videoChangeOwnership)"></my-button>
         </ng-container>
       </td>
     </tr>
   </ng-template>
 </p-table>
 
-<my-account-accept-ownership #myAccountAcceptOwnershipComponent (accepted)="accepted()"></my-account-accept-ownership>
\ No newline at end of file
+<my-account-accept-ownership #myAccountAcceptOwnershipComponent (accepted)="accepted()"></my-account-accept-ownership>
-- 
2.25.1