From 61c8026bd01ceeb2631b995739d4c964043e4d64 Mon Sep 17 00:00:00 2001 From: Aleksei Bezrodnov Date: Mon, 12 Jun 2023 21:19:35 +0200 Subject: [PATCH] fix sleeve university job (#604) --- src/PersonObjects/Sleeve/Sleeve.ts | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/src/PersonObjects/Sleeve/Sleeve.ts b/src/PersonObjects/Sleeve/Sleeve.ts index dfc8ab3fd..f15deb873 100644 --- a/src/PersonObjects/Sleeve/Sleeve.ts +++ b/src/PersonObjects/Sleeve/Sleeve.ts @@ -214,23 +214,25 @@ export class Sleeve extends Person implements SleevePerson { // Set experience/money gains based on class let classType: ClassType | undefined; + // TODO: why lower case??? It's not effecient, not typesafe and in general a bad idea switch (className.toLowerCase()) { - case "study computer science": + case "study computer science": // deprecated, leave it here for backwards compatibility + case ClassType.computerScience.toLowerCase(): classType = UniversityClassType.computerScience; break; - case "data structures": + case ClassType.dataStructures.toLowerCase(): classType = UniversityClassType.dataStructures; break; - case "networks": + case ClassType.networks.toLowerCase(): classType = UniversityClassType.networks; break; - case "algorithms": + case ClassType.algorithms.toLowerCase(): classType = UniversityClassType.algorithms; break; - case "management": + case ClassType.management.toLowerCase(): classType = UniversityClassType.management; break; - case "leadership": + case ClassType.leadership.toLowerCase(): classType = UniversityClassType.leadership; break; }