#454: Added simple copy button in task template list

frontend/src/app/template/template-builder/components/tasks-list/task-templates-list.component.html

@@ -47,6 +47,9 @@ <h2>Tasks</h2>
       <button mat-icon-button aria-label="Edit task." matTooltip="Edit task." (click)="editTask(task)">
         <mat-icon>edit</mat-icon>
       </button>
+      <button mat-icon-button aria-label="Remove task." matTooltip="Remove task." (click)="copyTask(task)">
+        <mat-icon>content_copy</mat-icon>
+      </button>
       <button mat-icon-button aria-label="Remove task." matTooltip="Remove task." (click)="removeTask(task)">
         <mat-icon>delete</mat-icon>
       </button>

frontend/src/app/template/template-builder/components/tasks-list/task-templates-list.component.scss

@@ -7,7 +7,7 @@
 }
 
 .mat-column-actions{
-  width: 20%;
+  width: 25%;
 }
 
 .mat-mdc-row:hover {

frontend/src/app/template/template-builder/components/tasks-list/task-templates-list.component.ts

@@ -103,6 +103,23 @@ export class TaskTemplatesListComponent extends AbstractTemplateBuilderComponent
     this.selection.toggle(task);
   }
 
+  public copyTask(task: ApiTaskTemplate){
+    const copy = JSON.parse(JSON.stringify(task))
+    copy.id = undefined;
+    const temp = {
+      name: "<COPY-TEMPLATE>",
+      description: "---Automatically generated template whose elements get copied. If this is seen, there was a programmer's error somewhere---",
+      taskTypes: [],
+      taskGroups: [],
+      tasks: [copy],
+      teams: [],
+      teamGroups: [],
+      judges: [],
+      id: "---COPY_TEMPLATE_NO_ID---"
+    } as ApiEvaluationTemplate;
+    this.builderService.importFrom(temp, "(Copy)")
+  }
+
   public tasksLength() {
     return this.builderService.getTemplate().tasks.length;
   }

frontend/src/app/template/template-builder/template-builder.service.ts

@@ -262,7 +262,7 @@ export class TemplateBuilderService {
     return result;
   }
 
-  importFrom(from: ApiEvaluationTemplate) {
+  importFrom(from: ApiEvaluationTemplate, nameCollisionSuffix = "(Imported)") {
     /* Import check is currently on name, may switch to completely UUID based matching */
 
     const alteredTypes: Map<string,ApiTaskType> = new Map<string, ApiTaskType>()
@@ -274,7 +274,7 @@ export class TemplateBuilderService {
         this.getTemplate().taskTypes.push(it)
       }else{
         const legacyName = it.name
-        it.name = `${it.name} (Imported)`
+        it.name = `${it.name} ${nameCollisionSuffix}`
         alteredTypes.set(legacyName, it)
         this.getTemplate().taskTypes.push(it)
       }
@@ -288,7 +288,7 @@ export class TemplateBuilderService {
         this.getTemplate().taskGroups.push(it)
       }else{
         const legacyName = it.name
-        it.name = `${it.name} (Imported)`
+        it.name = `${it.name} ${nameCollisionSuffix}`
         alteredGroups.set(legacyName, it)
         this.getTemplate().taskGroups.push(it)
       }
@@ -304,7 +304,7 @@ export class TemplateBuilderService {
       if(!this.getTemplate().tasks.map(that => that.name).includes(it.name)){
         this.getTemplate().tasks.push(it)
       }else{
-        it.name = `${it.name} (Imported)`
+        it.name = `${it.name} ${nameCollisionSuffix}`
         this.getTemplate().tasks.push(it)
       }
     })