Fix home for courses with less scenarios, refactor utf8 decoding (hob…

…byfarm#207)

* fix course overview if courses only have one scenario

* use and improve existing atou function instead of new function

---------

Co-authored-by: Philip Prinz <[email protected]>

src/app/home.component.html

@@ -104,7 +104,10 @@ <h3>Select a scenario</h3>
       </div>
     </ng-container>
 
-    <div class="clr-row" *ngIf="courses.length > 0 && ctx.valid">
+    <div
+      class="clr-row course-container"
+      *ngIf="courses.length > 0 && ctx.valid"
+    >
       <div class="clr-col-12">
         <ng-container *ngFor="let c of courses">
           <h3 class="course-header">{{ c.name | atob }}</h3>

src/app/home.component.scss

@@ -13,3 +13,7 @@
 cds-icon[shape='clock'] {
   color: black;
 }
+
+.course-container {
+  width: 100%;
+}

src/app/services/verification.service.ts

@@ -10,6 +10,7 @@ import { BehaviorSubject, throwError } from 'rxjs';
 import { catchError, tap } from 'rxjs/operators';
 import { HttpErrorResponse } from '@angular/common/http';
 import { VMService } from './vm.service';
+import { atou } from '../unicode';
 
 @Injectable()
 export class VerificationService {
@@ -98,7 +99,7 @@ export class VerificationService {
 
   private publishNewVerificationResults(response: ServerResponse) {
     const newVerificationList = JSON.parse(
-      decodeResponse(response.content),
+      atou(response.content),
     ) as unknown as TaskVerificationResponse[];
     newVerificationList.forEach(
       (newTaskVerification: TaskVerificationResponse) => {
@@ -120,14 +121,3 @@ export class VerificationService {
     this._verifications.next(this._verifications.value);
   }
 }
-
-function decodeResponse(responseContent: string) {
-  const text = atob(responseContent);
-  const length = text.length;
-  const bytes = new Uint8Array(length);
-  for (let i = 0; i < length; i++) {
-    bytes[i] = text.charCodeAt(i);
-  }
-  const decoder = new TextDecoder(); // default is utf-8
-  return decoder.decode(bytes);
-}

src/app/unicode.ts

@@ -1,5 +1,12 @@
 export function atou(b64: string) {
-  return decodeURIComponent(escape(atob(b64)));
+  const text = atob(b64);
+  const length = text.length;
+  const bytes = new Uint8Array(length);
+  for (let i = 0; i < length; i++) {
+    bytes[i] = text.charCodeAt(i);
+  }
+  const decoder = new TextDecoder(); // default is utf-8
+  return decoder.decode(bytes);
 }
 
 export function utoa(data: string) {