From 8734e48aef6271b42c6fe6f6521991e4a079705f Mon Sep 17 00:00:00 2001
From: Sean <sean.f.t.engelhardt@gmail.com>
Date: Wed, 26 May 2021 15:49:19 +0200
Subject: [PATCH] Respect candidate enumeration per assignment

Uses candidate enum per assignment as given in the form.
Changes enumeration in drag-list and poll-meta-info
---
 .../components/sorting-list/sorting-list.component.scss  | 5 +++--
 .../assignment-detail/assignment-detail.component.html   | 2 +-
 .../assignment-detail/assignment-detail.component.ts     | 4 ++++
 .../assignment-poll-meta-info.component.html             | 5 ++++-
 .../assignment-poll-meta-info.component.ts               | 9 +++++++++
 5 files changed, 21 insertions(+), 4 deletions(-)

diff --git a/client/src/app/shared/components/sorting-list/sorting-list.component.scss b/client/src/app/shared/components/sorting-list/sorting-list.component.scss
index faef8a76f..fe3b57021 100644
--- a/client/src/app/shared/components/sorting-list/sorting-list.component.scss
+++ b/client/src/app/shared/components/sorting-list/sorting-list.component.scss
@@ -22,13 +22,14 @@
     .count {
         display: table-cell;
         vertical-align: middle;
-        min-width: 2em;
-        padding-left: 1.25em;
+        min-width: 1.5em;
+        padding-left: 0.75em;
     }
 
     .content {
         display: table-cell;
         vertical-align: middle;
         width: 100%;
+        padding-left: 0.75em;
     }
 }
diff --git a/client/src/app/site/assignments/components/assignment-detail/assignment-detail.component.html b/client/src/app/site/assignments/components/assignment-detail/assignment-detail.component.html
index 364d0d4d4..4cbe263aa 100644
--- a/client/src/app/site/assignments/components/assignment-detail/assignment-detail.component.html
+++ b/client/src/app/site/assignments/components/assignment-detail/assignment-detail.component.html
@@ -149,7 +149,7 @@
                     <os-sorting-list
                         [input]="assignment.assignment_related_users"
                         [live]="true"
-                        [count]="true"
+                        [count]="enumerateCandidates"
                         [enable]="hasPerms('manage')"
                         (sortEvent)="onSortingChange($event)"
                     >
diff --git a/client/src/app/site/assignments/components/assignment-detail/assignment-detail.component.ts b/client/src/app/site/assignments/components/assignment-detail/assignment-detail.component.ts
index 8552bfa91..41b28b9a7 100644
--- a/client/src/app/site/assignments/components/assignment-detail/assignment-detail.component.ts
+++ b/client/src/app/site/assignments/components/assignment-detail/assignment-detail.component.ts
@@ -143,6 +143,10 @@ export class AssignmentDetailComponent extends BaseViewComponentDirective implem
         return this.agendaObserver.getValue().length > 0;
     }
 
+    public get enumerateCandidates(): boolean {
+        return this.assignment?.number_poll_candidates || false;
+    }
+
     /**
      * Hold the subscription to the navigation.
      * This cannot go into the subscription-list, since it should
diff --git a/client/src/app/site/assignments/modules/assignment-poll/components/assignment-poll-meta-info/assignment-poll-meta-info.component.html b/client/src/app/site/assignments/modules/assignment-poll/components/assignment-poll-meta-info/assignment-poll-meta-info.component.html
index 4a5544d9e..f4a6e10d1 100644
--- a/client/src/app/site/assignments/modules/assignment-poll/components/assignment-poll-meta-info/assignment-poll-meta-info.component.html
+++ b/client/src/app/site/assignments/modules/assignment-poll/components/assignment-poll-meta-info/assignment-poll-meta-info.component.html
@@ -3,9 +3,12 @@
     <small class="meta-info-left subtitle" *ngIf="poll.options?.length && showCandidates">
         <div>
             {{ 'Candidates' | translate }}:
-            <ol>
+            <ol *ngIf="enumerateCandidates">
                 <li *ngFor="let option of poll.options">{{ getOptionTitle(option) | translate }}</li>
             </ol>
+            <ul *ngIf="!enumerateCandidates">
+                <li *ngFor="let option of poll.options">{{ getOptionTitle(option) | translate }}</li>
+            </ul>
         </div>
         <div *ngIf="hasGlobalOption">
             {{ 'Options' | translate }}:
diff --git a/client/src/app/site/assignments/modules/assignment-poll/components/assignment-poll-meta-info/assignment-poll-meta-info.component.ts b/client/src/app/site/assignments/modules/assignment-poll/components/assignment-poll-meta-info/assignment-poll-meta-info.component.ts
index e2fa47906..89999e345 100644
--- a/client/src/app/site/assignments/modules/assignment-poll/components/assignment-poll-meta-info/assignment-poll-meta-info.component.ts
+++ b/client/src/app/site/assignments/modules/assignment-poll/components/assignment-poll-meta-info/assignment-poll-meta-info.component.ts
@@ -1,5 +1,6 @@
 import { Component, Input } from '@angular/core';
 
+import { ViewAssignment } from 'app/site/assignments/models/view-assignment';
 import { ViewAssignmentOption } from 'app/site/assignments/models/view-assignment-option';
 import { ViewAssignmentPoll } from 'app/site/assignments/models/view-assignment-poll';
 import { UnknownUserLabel } from 'app/site/assignments/modules/assignment-poll/services/assignment-poll.service';
@@ -20,6 +21,14 @@ export class AssignmentPollMetaInfoComponent {
     @Input()
     public showCandidates = true;
 
+    private get assignment(): ViewAssignment {
+        return this.poll.assignment;
+    }
+
+    public get enumerateCandidates(): boolean {
+        return this.assignment?.number_poll_candidates || false;
+    }
+
     public get hasGlobalOption(): boolean {
         return this.poll.hasGlobalOption;
     }