Fix Add Client UI Crash & Tracker Parsing Bug

Author: RudyB

Deluge Remote.xcodeproj/project.pbxproj

@@ -724,7 +724,7 @@
 					"@executable_path/Frameworks",
 				);
 				LIBRARY_SEARCH_PATHS = "";
-				MARKETING_VERSION = 1.5.5;
+				MARKETING_VERSION = 1.5.6;
 				OTHER_LDFLAGS = "-ObjC";
 				PRODUCT_BUNDLE_IDENTIFIER = "io.rudybermudez.Deluge-Remote";
 				PRODUCT_NAME = "$(TARGET_NAME)";
@@ -756,7 +756,7 @@
 					"@executable_path/Frameworks",
 				);
 				LIBRARY_SEARCH_PATHS = "";
-				MARKETING_VERSION = 1.5.5;
+				MARKETING_VERSION = 1.5.6;
 				OTHER_LDFLAGS = "-ObjC";
 				PRODUCT_BUNDLE_IDENTIFIER = "io.rudybermudez.Deluge-Remote";
 				PRODUCT_NAME = "$(TARGET_NAME)";

Deluge Remote/Resources/Base.lproj/Main.storyboard

@@ -453,7 +453,13 @@
                     </navigationItem>
                     <simulatedToolbarMetrics key="simulatedBottomBarMetrics"/>
                     <connections>
+                        <outlet property="hostnameTextField" destination="SIf-P6-25J" id="zmL-AD-veC"/>
+                        <outlet property="networkSecurityControl" destination="CxY-rK-tfD" id="fEF-3p-3Pa"/>
                         <outlet property="nicknameTextField" destination="gR7-yB-iAd" id="VKL-b6-bSU"/>
+                        <outlet property="passwordTextField" destination="222-3p-ANZ" id="u0W-CX-XZx"/>
+                        <outlet property="portTableViewCell" destination="n1Z-TI-abn" id="9iA-pg-Jmv"/>
+                        <outlet property="portTextField" destination="QiW-cq-Ryr" id="4oW-xR-wUq"/>
+                        <outlet property="relativePathTextField" destination="IUq-f6-n0a" id="bWC-a0-gxo"/>
                     </connections>
                 </tableViewController>
                 <placeholder placeholderIdentifier="IBFirstResponder" id="TwL-zY-yej" userLabel="First Responder" sceneMemberID="firstResponder"/>

Deluge Remote/Sources/API/DelugeClient.swift

@@ -237,7 +237,7 @@ class DelugeClient {
                             Logger.error(ClientError.other(error))
                             seal.reject(ClientError.other(error))
                         } else if error.isResponseSerializationError {
-                            Logger.error(error.localizedDescription)
+                            Logger.error(error)
                             seal.reject(ClientError.decoding)
                         } else {
                             Logger.error(error)

Deluge Remote/Sources/Models/Deluge/TorrentMetadata.swift

@@ -100,14 +100,14 @@ struct TrackerMetadata: Decodable {
     let last_error: ErrorMetadata?
     let fail_limit: Int?
     let next_announce: Int?
-    let complete_sent: Bool
-    let source: Int
+    let complete_sent: Bool?
+    let source: Int?
     let trackerid: String?
-    let start_sent: Bool
-    let tier: Int
+    let start_sent: Bool?
+    let tier: Int?
     let scrape_complete: Int?
     let message: String?
-    let updating: Bool
+    let updating: Bool?
 }
 
  struct ErrorMetadata: Decodable {

Deluge Remote/Sources/View Controllers/AddClientViewController.swift

@@ -42,9 +42,18 @@ class AddClientViewController: UITableViewController, Storyboarded {
             let relativePath = relativePathTextField.text
         else { return }
 
-        if nickname.isEmpty { showAlert(target: self, title: "Nickname cannot be left empty")}
-        if hostname.isEmpty { showAlert(target: self, title: "Hostname cannot be empty")}
-        if portString.isEmpty { showAlert(target: self, title: "Port cannot be empty")}
+        if nickname.isEmpty {
+            showAlert(target: self, title: "Nickname cannot be left empty")
+            return
+        }
+        if hostname.isEmpty {
+            showAlert(target: self, title: "Hostname cannot be empty")
+            return
+        }
+        if portString.isEmpty {
+            showAlert(target: self, title: "Port cannot be empty")
+            return
+        }
 
         guard let port = Int(portString) else {
             showAlert(target: self, title: "Port must be a numeric type")