Fix #15, Admin Login now works

Too complicated to pull admin data for now, so it just assigns an unkown default name.

Author: peterkos

BrickHack-Mobile/AlertMessage.swift

@@ -23,7 +23,14 @@ class MessageHandler {
     static func showAlertMessage(withTitle title: String, body: String, type: MessageType ) {
 
         let view = MessageView.viewFromNib(layout: .tabView)
-        view.configureTheme(.error)
+
+        // Map our fake type to their type
+        switch type {
+        case .error: view.configureTheme(.error)
+        case .warning: view.configureTheme(.warning)
+        case .info: view.configureTheme(.info)
+        }
+        
         view.button?.isHidden = true
         view.configureContent(title: title,
                               body: body,
@@ -103,4 +110,11 @@ class MessageHandler {
                          body: "Please try again later.",
                          type: .error)
     }
+
+    static func showUnknownUserDataError() {
+        print("ERROR: Could not find questionnaire for user.")
+        showAlertMessage(withTitle: "User name unknown!",
+                         body: "Setting a placeholder name in the meantime...",
+                         type: .info)
+    }
 }

BrickHack-Mobile/Controllers/LoginViewController.swift

@@ -136,23 +136,6 @@ class LoginViewController: UIViewController {
                 print("passed user object")
                 eventsVC.currentUser = self.currentUser
             }
-
-
-            // Check for MainTabBarController (skip through nav controller)
-            // Note: not used!
-            /*
-            if let tabVC = segue.destination.children.first as? MainTabBarController {
-
-                // Check if valid user (on error, user will reauth)
-                guard userID != 0 else {
-                    MessageHandler.showInvalidUserError()
-                    return
-                }
-
-                // Pass data to the tab bar controller, which will handle passing its own children
-                tabVC.userID = userID
-                tabVC.oauthGrant = oauthGrant
-            } */
         }
     }
 
@@ -176,6 +159,7 @@ class LoginViewController: UIViewController {
             SVProgressHUD.show()
         }
 
+        // MARK: User data request
         URLSession.shared.dataTask(with: signedNameRequest) { (data, response, error) in
 
             // MARK: Error checking
@@ -216,27 +200,45 @@ class LoginViewController: UIViewController {
                 return
             }
 
-
             // MARK: Data conversion
 
+            var unknownErrorOccured = false
+
             // Convert server data to our User object
             do {
                 self.currentUser = try JSONDecoder().decode(User.self, from: data)
-            } catch (let error) {
+            } catch {
+
+                print("Got in error")
+
+                // This error might be fixable!
+                // Admin accounts don't have a questionnaire, and if the user was
+                // able to login, then this is almost certainly an admin account.
+                // (Or their questionnaire doesn't exist).
+
+                unknownErrorOccured = true
                 DispatchQueue.main.async {
-                    print("parsing error: \(error)")
-                    MessageHandler.showUserDataParsingError(withText: "Unable to convert JSON")
-                    SVProgressHUD.dismiss()
-                    self.logout()
+                    MessageHandler.showUnknownUserDataError()
                 }
-                return
+
+                // In this case, we just assign some blank data.
+                self.currentUser = User(firstName: "Mystery", lastName: "User", major: "Bachelors of codeRIT")
             }
 
-            // Now that we have the user data, go to the main screen,
-            // passing the data forward!
-            DispatchQueue.main.async {
-                SVProgressHUD.dismiss()
-                self.performSegue(withIdentifier: "authSuccessSegue", sender: self)
+            // Now that we have the user data (or fake admin data),
+            // go to the main screen, and pass the data forward!
+
+            // If error occured, give user like three seconds to read it before moving on
+            if unknownErrorOccured {
+                DispatchQueue.main.asyncAfter(deadline: .now() + .seconds(3)) {
+                    SVProgressHUD.dismiss()
+                    self.performSegue(withIdentifier: "authSuccessSegue", sender: self)
+                }
+            } else {
+                DispatchQueue.main.async {
+                    SVProgressHUD.dismiss()
+                    self.performSegue(withIdentifier: "authSuccessSegue", sender: self)
+                }
             }
 
             return

BrickHack-Mobile/Models/ScheduleParser.swift

@@ -241,7 +241,6 @@ class ScheduleParser {
                 case 0:
 
                     guard rowIndex > 1 || !currentEvent.title.isEmpty else {
-                        print("broken like me right now")
                         break
                     }