Code review changes: public declaration not needed for cell variables, guard instead of if let

Author: mindgraffiti

WooCommerce/Classes/ViewRelated/Orders/OrderDetails/CustomerInfoTableViewCell.swift

@@ -18,7 +18,7 @@ class CustomerInfoTableViewCell: UITableViewCell {
     }
 
 
-    public var title: String? {
+    var title: String? {
         get {
             return titleLabel.text
         }
@@ -27,7 +27,7 @@ class CustomerInfoTableViewCell: UITableViewCell {
         }
     }
 
-    public var name: String? {
+    var name: String? {
         get {
             return nameLabel.text
         }
@@ -36,7 +36,7 @@ class CustomerInfoTableViewCell: UITableViewCell {
         }
     }
 
-    public var address: String? {
+    var address: String? {
         get {
             return addressLabel.text
         }

Yosemite/Yosemite/Model/Storage/Order+ReadOnlyConvertible.swift

@@ -93,7 +93,7 @@ extension Storage.Order: ReadOnlyConvertible {
     // MARK: - Private Helpers
 
     private func createReadOnlyBillingAddress() -> Yosemite.Address? {
-        if billingCountry == nil {
+        guard let billingCountry = billingCountry else {
             return nil
         }
 
@@ -105,15 +105,16 @@ extension Storage.Order: ReadOnlyConvertible {
                        city: billingCity ?? "",
                        state: billingState ?? "",
                        postcode: billingPostcode ?? "",
-                       country: billingCountry ?? "",
+                       country: billingCountry,
                        phone: billingPhone ?? "",
                        email: billingEmail ?? "")
     }
 
     private func createReadOnlyShippingAddress() -> Yosemite.Address? {
-        if shippingCountry == nil {
+        guard let shippingCountry = shippingCountry else {
             return nil
         }
+
         return Address(firstName: shippingFirstName ?? "",
                        lastName: shippingLastName ?? "",
                        company: shippingCompany ?? "",
@@ -122,7 +123,7 @@ extension Storage.Order: ReadOnlyConvertible {
                        city: shippingCity ?? "",
                        state: shippingState ?? "",
                        postcode: shippingPostcode ?? "",
-                       country: shippingCountry ?? "",
+                       country: shippingCountry,
                        phone: shippingPhone ?? "",
                        email: shippingEmail ?? "")
     }