From cb6a426c7db35a85712f6e80e705651f00d5ff51 Mon Sep 17 00:00:00 2001
From: "Shijun Sun (c)" <sshijun@vmware.com>
Date: Tue, 30 Jul 2019 17:27:40 +0800
Subject: [PATCH] Optimization for tag-retention UI

Signed-off-by: Shijun Sun (c) <sshijun@vmware.com>
---
 src/portal/lib/src/cron-schedule/index.ts     |  2 +
 src/portal/lib/src/index.ts                   |  1 +
 .../app/project/tag-retention/retention.ts    |  2 +-
 .../tag-retention.component.html              | 28 ++++----
 .../tag-retention.component.scss              | 18 ++++-
 .../tag-retention/tag-retention.component.ts  | 70 +++++++++++++++++--
 src/portal/src/i18n/lang/en-us-lang.json      |  3 +-
 src/portal/src/i18n/lang/es-es-lang.json      |  3 +-
 src/portal/src/i18n/lang/fr-fr-lang.json      |  3 +-
 src/portal/src/i18n/lang/pt-br-lang.json      |  3 +-
 src/portal/src/i18n/lang/zh-cn-lang.json      |  3 +-
 11 files changed, 109 insertions(+), 27 deletions(-)

diff --git a/src/portal/lib/src/cron-schedule/index.ts b/src/portal/lib/src/cron-schedule/index.ts
index dfcdf2056..014a84eda 100644
--- a/src/portal/lib/src/cron-schedule/index.ts
+++ b/src/portal/lib/src/cron-schedule/index.ts
@@ -3,6 +3,8 @@ import { Type } from "@angular/core";
 import { CronScheduleComponent } from "./cron-schedule.component";
 import { CronTooltipComponent } from "./cron-tooltip/cron-tooltip.component";
 
+export * from "./cron-schedule.component";
+export * from './cron-tooltip/cron-tooltip.component';
 export const CRON_SCHEDULE_DIRECTIVES: Type<any>[] = [
     CronScheduleComponent,
     CronTooltipComponent
diff --git a/src/portal/lib/src/index.ts b/src/portal/lib/src/index.ts
index 745e73fa0..c37e890a6 100644
--- a/src/portal/lib/src/index.ts
+++ b/src/portal/lib/src/index.ts
@@ -27,4 +27,5 @@ export * from "./gridview/index";
 export * from "./repository-gridview/index";
 export * from "./operation/index";
 export * from "./_animations/index";
+export * from "./cron-schedule/index";
 
diff --git a/src/portal/src/app/project/tag-retention/retention.ts b/src/portal/src/app/project/tag-retention/retention.ts
index d536632bd..d8460c86b 100644
--- a/src/portal/src/app/project/tag-retention/retention.ts
+++ b/src/portal/src/app/project/tag-retention/retention.ts
@@ -32,7 +32,7 @@ export class Retention {
         this.trigger = {
             kind: "Schedule",
             settings: {
-                cron: "@daily",
+                cron: "0 0 0 * * *",
             }
         };
     }
diff --git a/src/portal/src/app/project/tag-retention/tag-retention.component.html b/src/portal/src/app/project/tag-retention/tag-retention.component.html
index 92622ba74..dd84bf30f 100644
--- a/src/portal/src/app/project/tag-retention/tag-retention.component.html
+++ b/src/portal/src/app/project/tag-retention/tag-retention.component.html
@@ -1,9 +1,9 @@
 <div class="clr-row pt-1 fw8">
-    <div class="clr-col-2">
-        <label class="fw9-l">{{'TAG_RETENTION.RETENTION_RULES' | translate}}</label><span
-            class="badge badge-3 clr-offset-1">{{retention?.rules?.length ? retention?.rules?.length : 0}}</span>
+    <div class="clr-col">
+        <label class="label-left font-size-54">{{'TAG_RETENTION.RETENTION_RULES' | translate}}</label><span
+            class="badge badge-3 ml-5">{{retention?.rules?.length ? retention?.rules?.length : 0}}</span>
+        <span *ngIf="loadingRule" class="spinner spinner-inline ml-2">Loading...</span>
     </div>
-
 </div>
 <div class="clr-row pt-1">
     <div class="clr-col">
@@ -45,14 +45,9 @@
                 </div>
             </li>
         </ul>
-        <div class="clr-row pt-1">
-            <div class="clr-col-5 color-97">
-                <div>
-                    <span>{{'TAG_RETENTION.ADD_RULE_HELP_1' | translate}}</span>
-                </div>
-                <div>
-                    <span>{{'TAG_RETENTION.ADD_RULE_HELP_2' | translate}}</span>
-                </div>
+        <div class="v-center clr-row pt-1">
+            <div class="clr-col-5 color-97 font-size-54">
+                {{'TAG_RETENTION.ADD_RULE_HELP_1' | translate}}
             </div>
             <div class="clr-col">
                 <button class="btn btn-link" (click)="openAddRule()">{{'TAG_RETENTION.ADD_RULE' | translate}}</button>
@@ -60,8 +55,11 @@
         </div>
     </div>
 </div>
+<div class="cron-selection">
+    <cron-selection #cronScheduleComponent [labelCurrent]="label" [labelEdit]='label' [originCron]='originCron()' (inputvalue)="updateCron($event)"></cron-selection>
+</div>
 <div class="clr-row pt-1">
-    <div class="clr-col-2 pt-2"><label class="fw9-l">{{'TAG_RETENTION.RETENTION_RUNS' | translate}}</label></div>
+    <div class="clr-col-2 pt-2"><label class="label-left font-size-54">{{'TAG_RETENTION.RETENTION_RUNS' | translate}}</label></div>
     <div class="clr-col-10">
         <clr-dg-action-bar>
                 <button [disabled]="!(retention?.rules?.length > 0)" class="btn btn-outline"
@@ -71,7 +69,9 @@
                 <button [disabled]="!(retention?.rules?.length > 0)" class="btn btn-outline"
                         (click)="whatIfRun()">{{'TAG_RETENTION.WHAT_IF_RUN' | translate}}</button>
                 <button [disabled]="!(selectedItem && (selectedItem.status ==='InProgress' || selectedItem.status ==='Running'))"
-                        class="btn btn-outline" (click)="abortRun()">{{'TAG_RETENTION.ABORT' | translate}}</button>
+                        class="btn btn-outline" (click)="abortRun()">
+                    <clr-icon shape="stop"></clr-icon>
+                    <span class="ml-5">{{'TAG_RETENTION.ABORT' | translate}}</span></button>
                 <button [disabled]="!retentionId" class="btn btn-outline"
                         (click)="refreshList()">
                     <clr-icon shape="refresh"></clr-icon>
diff --git a/src/portal/src/app/project/tag-retention/tag-retention.component.scss b/src/portal/src/app/project/tag-retention/tag-retention.component.scss
index 4973c5c87..abab3d670 100644
--- a/src/portal/src/app/project/tag-retention/tag-retention.component.scss
+++ b/src/portal/src/app/project/tag-retention/tag-retention.component.scss
@@ -11,8 +11,7 @@
   overflow: hidden;
   white-space: nowrap;
   text-overflow: ellipsis;
-  font-size: 0.6rem;
-  font-weight: 500;
+  font-size: .541667rem;
 }
 
 .ml-5 {
@@ -43,4 +42,19 @@
   left: 0;
   opacity: 0;
   z-index: 999;
+}
+.cron-selection {
+  margin-top: 20px;
+  display: flex;
+  align-items: center;
+}
+.label-left {
+  color: #000;
+}
+.v-center {
+  height: 48px;
+  line-height: 48px;
+}
+.font-size-54 {
+  font-size: .541667rem;
 }
\ No newline at end of file
diff --git a/src/portal/src/app/project/tag-retention/tag-retention.component.ts b/src/portal/src/app/project/tag-retention/tag-retention.component.ts
index 4f56cfba2..2a69f9096 100644
--- a/src/portal/src/app/project/tag-retention/tag-retention.component.ts
+++ b/src/portal/src/app/project/tag-retention/tag-retention.component.ts
@@ -19,12 +19,20 @@ import { TagRetentionService } from "./tag-retention.service";
 import { Retention, Rule } from "./retention";
 import { Project } from "../project";
 import { clone, ErrorHandler } from "@harbor/ui";
+import { OriginCron } from "@harbor/ui";
+import { CronScheduleComponent } from "@harbor/ui";
 
 const MIN = 60000;
 const SEC = 1000;
 const MIN_STR = "min";
 const SEC_STR = "sec";
-
+const SCHEDULE_TYPE = {
+    NONE: "None",
+    DAILY: "Daily",
+    WEEKLY: "Weekly",
+    HOURLY: "Hourly",
+    CUSTOM: "Custom"
+};
 @Component({
     selector: 'tag-retention',
     templateUrl: './tag-retention.component.html',
@@ -48,7 +56,6 @@ export class TagRetentionComponent implements OnInit {
         }
     };
     projectId: number;
-    addRuleOpened: boolean = false;
     isRetentionRunOpened: boolean = false;
     isAbortedOpened: boolean = false;
     selectedItem: any;
@@ -61,15 +68,36 @@ export class TagRetentionComponent implements OnInit {
     historyList = [];
     loadingExecutions: boolean = false;
     loadingHistories: boolean = false;
-    dryRun: boolean = false;
+    label: string = 'TAG_RETENTION.TRIGGER';
+    loadingRule: boolean = false;
+    @ViewChild('cronScheduleComponent')
+    cronScheduleComponent: CronScheduleComponent;
     @ViewChild('addRule') addRuleComponent: AddRuleComponent;
-
     constructor(
         private route: ActivatedRoute,
         private tagRetentionService: TagRetentionService,
         private errorHandler: ErrorHandler,
     ) {
     }
+    originCron(): OriginCron {
+        let originCron: OriginCron = {
+            type: SCHEDULE_TYPE.DAILY,
+            cron: "0 0 0 * * *"
+        };
+        originCron.cron = this.retention.trigger.settings.cron;
+        if (originCron.cron === "") {
+            originCron.type = SCHEDULE_TYPE.NONE;
+        } else if (originCron.cron === "0 0 * * * *") {
+            originCron.type = SCHEDULE_TYPE.HOURLY;
+        } else if (originCron.cron === "0 0 0 * * *") {
+            originCron.type = SCHEDULE_TYPE.DAILY;
+        } else if (originCron.cron === "0 0 0 * * 0") {
+            originCron.type = SCHEDULE_TYPE.WEEKLY;
+        } else {
+            originCron.type = SCHEDULE_TYPE.CUSTOM;
+        }
+        return originCron;
+    }
 
     ngOnInit() {
         this.projectId = +this.route.snapshot.parent.params['id'];
@@ -88,7 +116,27 @@ export class TagRetentionComponent implements OnInit {
         this.getMetadata();
         this.refreshList();
     }
-
+    updateCron(cron: string) {
+        let retention: Retention = clone(this.retention);
+        retention.trigger.settings.cron = cron;
+        if (!this.retentionId) {
+            this.tagRetentionService.createRetention(retention).subscribe(
+                response => {
+                    this.cronScheduleComponent.isEditMode = false;
+                    this.refreshAfterCreatRetention();
+                }, error => {
+                    this.errorHandler.error(error);
+                });
+        } else {
+            this.tagRetentionService.updateRetention(this.retentionId, retention).subscribe(
+                response => {
+                    this.cronScheduleComponent.isEditMode = false;
+                    this.retention = retention;
+                }, error => {
+                    this.errorHandler.error(error);
+                });
+        }
+    }
     getMetadata() {
         this.tagRetentionService.getRetentionMetadata().subscribe(
             response => {
@@ -103,8 +151,10 @@ export class TagRetentionComponent implements OnInit {
             this.tagRetentionService.getRetention(this.retentionId).subscribe(
                 response => {
                     this.retention = response;
+                    this.loadingRule = false;
                 }, error => {
                     this.errorHandler.error(error);
+                    this.loadingRule = false;
                 });
         }
     }
@@ -122,10 +172,13 @@ export class TagRetentionComponent implements OnInit {
         let retention: Retention = clone(this.retention);
         retention.rules.splice(index, 1);
         this.ruleIndex = -1;
+        this.loadingRule = true;
         this.tagRetentionService.updateRetention(this.retentionId, retention).subscribe(
             response => {
+                this.loadingRule = false;
                 this.retention = retention;
             }, error => {
+                this.loadingRule = false;
                 this.errorHandler.error(error);
             });
     }
@@ -241,11 +294,13 @@ export class TagRetentionComponent implements OnInit {
                 this.retentionId = response.metadata.retention_id;
                 this.getRetention();
             }, error => {
+                this.loadingRule = false;
                 this.errorHandler.error(error);
             });
     }
 
     clickAdd(rule) {
+        this.loadingRule = true;
         if (this.addRuleComponent.isAdd) {
             let retention: Retention = clone(this.retention);
             retention.rules.push(rule);
@@ -255,12 +310,15 @@ export class TagRetentionComponent implements OnInit {
                         this.refreshAfterCreatRetention();
                     }, error => {
                         this.errorHandler.error(error);
+                        this.loadingRule = false;
                     });
             } else {
                 this.tagRetentionService.updateRetention(this.retentionId, retention).subscribe(
                     response => {
+                        this.loadingRule = false;
                         this.retention = retention;
                     }, error => {
+                        this.loadingRule = false;
                         this.errorHandler.error(error);
                     });
             }
@@ -270,8 +328,10 @@ export class TagRetentionComponent implements OnInit {
             this.tagRetentionService.updateRetention(this.retentionId, retention).subscribe(
                 response => {
                     this.retention = retention;
+                    this.loadingRule = false;
                 }, error => {
                     this.errorHandler.error(error);
+                    this.loadingRule = false;
                 });
         }
     }
diff --git a/src/portal/src/i18n/lang/en-us-lang.json b/src/portal/src/i18n/lang/en-us-lang.json
index 177a3b381..9845ed1bf 100644
--- a/src/portal/src/i18n/lang/en-us-lang.json
+++ b/src/portal/src/i18n/lang/en-us-lang.json
@@ -1161,7 +1161,8 @@
         "WITHOUT": " without ",
         "LOWER_LABELS": " labels",
         "WITH_CONDITION": " with",
-        "LOWER_TAGS": " tags"
+        "LOWER_TAGS": " tags",
+        "TRIGGER": "Schedule"
     }
 
 }
diff --git a/src/portal/src/i18n/lang/es-es-lang.json b/src/portal/src/i18n/lang/es-es-lang.json
index 770913a63..cadf2986b 100644
--- a/src/portal/src/i18n/lang/es-es-lang.json
+++ b/src/portal/src/i18n/lang/es-es-lang.json
@@ -1159,7 +1159,8 @@
         "WITHOUT": " without ",
         "LOWER_LABELS": " labels",
         "WITH_CONDITION": " with",
-        "LOWER_TAGS": " tags"
+        "LOWER_TAGS": " tags",
+        "TRIGGER": "Schedule"
     }
 
 }
diff --git a/src/portal/src/i18n/lang/fr-fr-lang.json b/src/portal/src/i18n/lang/fr-fr-lang.json
index 69747b272..a5d578505 100644
--- a/src/portal/src/i18n/lang/fr-fr-lang.json
+++ b/src/portal/src/i18n/lang/fr-fr-lang.json
@@ -1131,7 +1131,8 @@
         "WITHOUT": " without ",
         "LOWER_LABELS": " labels",
         "WITH_CONDITION": " with",
-        "LOWER_TAGS": " tags"
+        "LOWER_TAGS": " tags",
+        "TRIGGER": "Schedule"
     }
 
 }
diff --git a/src/portal/src/i18n/lang/pt-br-lang.json b/src/portal/src/i18n/lang/pt-br-lang.json
index 3a10b2745..72f05c897 100644
--- a/src/portal/src/i18n/lang/pt-br-lang.json
+++ b/src/portal/src/i18n/lang/pt-br-lang.json
@@ -1156,7 +1156,8 @@
         "WITHOUT": " without ",
         "LOWER_LABELS": " labels",
         "WITH_CONDITION": " with",
-        "LOWER_TAGS": " tags"
+        "LOWER_TAGS": " tags",
+        "TRIGGER": "Schedule"
     }
 
     
diff --git a/src/portal/src/i18n/lang/zh-cn-lang.json b/src/portal/src/i18n/lang/zh-cn-lang.json
index 48a5b3d18..cd6f6865d 100644
--- a/src/portal/src/i18n/lang/zh-cn-lang.json
+++ b/src/portal/src/i18n/lang/zh-cn-lang.json
@@ -1157,7 +1157,8 @@
         "WITHOUT": "没有",
         "LOWER_LABELS": "标签",
         "WITH_CONDITION": "基于条件",
-        "LOWER_TAGS": "tags"
+        "LOWER_TAGS": "tags",
+        "TRIGGER": "定时执行"
     }
 
 }