[Horizon] Add profile disable name fields

Core/Core/Features/People/APIUser.swift

@@ -146,6 +146,17 @@ public struct APIProfile: Codable, Equatable {
     public let uuid: String?
     public let account_uuid: String?
     public let time_zone: String?
+    public let permissions: APIProfile.Permissions?
+
+    public struct Permissions: Codable, Equatable {
+        public let canUpdateName: Bool?
+        public let canUpdateAvatar: Bool?
+
+        enum CodingKeys: String, CodingKey {
+            case canUpdateName = "can_update_name"
+            case canUpdateAvatar = "can_update_avatar"
+        }
+    }
 }
 
 #if DEBUG
@@ -244,7 +255,8 @@ extension APIProfile {
         k5_user: Bool? = nil,
         uuid: String? = nil,
         account_uuid: String? = nil,
-        time_zone: String? = nil
+        time_zone: String? = nil,
+        permissions: APIProfile.Permissions? = nil
     ) -> APIProfile {
         return APIProfile(
             id: id,
@@ -259,7 +271,8 @@ extension APIProfile {
             k5_user: k5_user,
             uuid: uuid,
             account_uuid: account_uuid,
-            time_zone: time_zone
+            time_zone: time_zone,
+            permissions: permissions
         )
     }
 }

Core/Core/Features/Profile/UserProfile.swift

@@ -33,6 +33,8 @@ public final class UserProfile: NSManagedObject {
     @NSManaged public var uuid: String?
     @NSManaged public var accountUUID: String?
     @NSManaged public var defaultTimeZone: String?
+    @NSManaged public var canUpdateName: Bool
+    @NSManaged public var canUpdateAvatar: Bool
 }
 
 extension UserProfile: WriteableModel {
@@ -42,14 +44,15 @@ extension UserProfile: WriteableModel {
         model.id = item.id.value
         model.name = item.name
         model.shortName = item.short_name
-        model.email = item.primary_email
+        model.email = item.primary_email ?? model.email
         model.locale = item.locale
         model.loginID = item.login_id
         model.avatarURL = item.avatar_url?.rawValue
         model.calendarURL = item.calendar?.ics
         model.pronouns = item.pronouns
         model.isK5User = (item.k5_user == true)
-
+        model.canUpdateName = item.permissions?.canUpdateName ?? model.canUpdateName
+        model.canUpdateAvatar = item.permissions?.canUpdateAvatar ?? model.canUpdateAvatar
         // The "/users/self/profile" api does not return accountUUID and since they share
         // the same Core Data entity, it would get overriden with a null value after fetching "/users/self"
         if model.uuid == nil {

Core/Core/Resources/Database.xcdatamodeld/Database.xcdatamodel/contents

@@ -1282,6 +1282,8 @@
         <attribute name="accountUUID" optional="YES" attributeType="String"/>
         <attribute name="avatarURL" optional="YES" attributeType="URI"/>
         <attribute name="calendarURL" optional="YES" attributeType="URI"/>
+        <attribute name="canUpdateAvatar" attributeType="Boolean" usesScalarValueType="YES"/>
+        <attribute name="canUpdateName" attributeType="Boolean" usesScalarValueType="YES"/>
         <attribute name="defaultTimeZone" optional="YES" attributeType="String"/>
         <attribute name="email" optional="YES" attributeType="String"/>
         <attribute name="id" attributeType="String"/>

Horizon/Horizon/Resources/Localizable.xcstrings

@@ -282,6 +282,9 @@
     },
     "Display Name" : {
 
+    },
+    "Display Name can only be changed by your institution." : {
+
     },
     "Done" : {
 
@@ -369,6 +372,9 @@
     },
     "Full Name" : {
 
+    },
+    "Full Name can only be changed by your institution." : {
+
     },
     "Georgetown (-04:00/-04:00)" : {
 
@@ -737,9 +743,6 @@
     },
     "Regenerate Flashcards" : {
 
-    },
-    "Required" : {
-
     },
     "Result: %@" : {
 

Horizon/Horizon/Sources/Common/Domain/GetUserInteractor.swift

@@ -21,6 +21,7 @@ import Core
 
 protocol GetUserInteractor {
     func getUser() -> AnyPublisher<UserProfile, Error>
+    func canUpdateName() -> AnyPublisher<Bool, Error>
 }
 
 final class GetUserInteractorLive: GetUserInteractor {
@@ -32,4 +33,11 @@ final class GetUserInteractorLive: GetUserInteractor {
         .compactMap { $0.first }
         .eraseToAnyPublisher()
     }
+
+    func canUpdateName() -> AnyPublisher<Bool, Error> {
+        ReactiveStore(useCase: GetSelfUserIncludingUUID())
+            .getEntities(ignoreCache: true)
+            .map { $0.first?.canUpdateName ?? false }
+            .eraseToAnyPublisher()
+    }
 }

Horizon/Horizon/Sources/Common/Domain/GetUserInteractorPreview.swift

@@ -35,5 +35,11 @@ final class GetUserInteractorPreview: GetUserInteractor {
             .setFailureType(to: Error.self)
             .eraseToAnyPublisher()
     }
+
+    func canUpdateName() -> AnyPublisher<Bool, any Error> {
+        Just(true)
+            .setFailureType(to: Error.self)
+            .eraseToAnyPublisher()
+    }
 }
 #endif

Horizon/Horizon/Sources/Features/Account/Profile/View/ProfileView.swift

@@ -42,14 +42,31 @@ struct ProfileView: View {
                 .padding(.horizontal, .huiSpaces.space24)
                 .padding(.vertical, .huiSpaces.space48)
             }
+            .alert(isPresented: $viewModel.isAlertErrorPresented) {
+                Alert(title: Text(viewModel.errorMessage), message: Text(viewModel.errorMessage))
+            }
         }
+        .overlay { loaderView }
     }
 
+    @ViewBuilder
+    private var loaderView: some View {
+        if viewModel.isLoaderVisible {
+            ZStack {
+                Color.huiColors.surface.pageSecondary
+                    .ignoresSafeArea()
+                HorizonUI.Spinner(size: .small, showBackground: true)
+            }
+        }
+    }
     private var nameView: some View {
         HorizonUI.TextInput(
             $viewModel.name,
             label: String(localized: "Full Name", bundle: .horizon),
             error: viewModel.nameError,
+            helperText: !viewModel.canUpdateName
+            ? String(localized: "Full Name can only be changed by your institution.", bundle: .horizon)
+            : nil,
             disabled: viewModel.nameDisabled
         )
     }
@@ -59,7 +76,9 @@ struct ProfileView: View {
             $viewModel.displayName,
             label: String(localized: "Display Name", bundle: .horizon),
             error: viewModel.displayNameError,
-            helperText: String(localized: "Required", bundle: .horizon),
+            helperText: !viewModel.canUpdateName
+            ? String(localized: "Display Name can only be changed by your institution.", bundle: .horizon)
+            : nil,
             disabled: viewModel.displayNameDisabled
         )
     }
@@ -76,7 +95,7 @@ struct ProfileView: View {
     private var saveButton: some View {
         SavingButton(
             title: String(localized: "Save Changes", bundle: .horizon),
-            isLoading: $viewModel.isLoading,
+            isLoading: $viewModel.saveLoaderIsVisiable,
             isDisabled: $viewModel.isSaveDisabled,
             onSave: viewModel.save
         )

Horizon/Horizon/Sources/Features/Account/Profile/View/ProfileViewModel.swift

@@ -31,22 +31,29 @@ final class ProfileViewModel {
         }
     }
     var nameDisabled: Bool {
-        isLoading
+        saveLoaderIsVisiable || !canUpdateName
     }
     var nameError: String = " "
+    private(set) var errorMessage: String = ""
+    private(set) var canUpdateName = true
+    private(set) var isLoaderVisible = true
     var displayName: String = "" {
         didSet {
             validateDisplayName()
             updateSaveDisabled()
         }
     }
     var displayNameDisabled: Bool {
-        isLoading
+        saveLoaderIsVisiable || !canUpdateName
     }
     var displayNameError: String = " "
     var email: String = ""
     var isSaveDisabled: Bool = true
-    var isLoading: Bool = true
+    var saveLoaderIsVisiable: Bool = false
+
+    // MARK: - Inputs / Output
+
+    var isAlertErrorPresented: Bool = false
 
     // MARK: - Private
 
@@ -72,29 +79,34 @@ final class ProfileViewModel {
     ) {
         self.updateUserProfileInteractor = updateUserProfileInteractor
 
-        getUserInteractor
-            .getUser()
-            .sink(
-                receiveCompletion: { _ in },
-                receiveValue: { [weak self] user in
-                    self?.nameOriginal = user.name
-                    self?.displayNameOriginal = user.shortName ?? ""
-                    self?.email = user.email ?? ""
-                    self?.validate()
-                    self?.isLoading = false
+        let userProfile = getUserInteractor.getUser()
+        let userPermission = getUserInteractor.canUpdateName()
+
+        Publishers.Zip(userProfile, userPermission)
+            .sink { [weak self] completion in
+                if case .failure(let error) = completion {
+                    self?.errorMessage = error.localizedDescription
+                    self?.isAlertErrorPresented = true
                 }
-            )
+                self?.isLoaderVisible = false
+            } receiveValue: { [weak self] (user, canUpdateName) in
+                self?.canUpdateName = canUpdateName
+                self?.nameOriginal = user.name
+                self?.displayNameOriginal = user.shortName ?? ""
+                self?.email = user.email ?? ""
+                self?.validate()
+            }
             .store(in: &subscriptions)
     }
 
     // MARK: - Input Actions
 
     func save() {
-        isLoading = true
+        saveLoaderIsVisiable = true
         updateUserProfileInteractor.set(name: name, shortName: displayName)
             .sink(
                 receiveCompletion: { [weak self] _ in
-                    self?.isLoading = false
+                    self?.saveLoaderIsVisiable = false
                 },
                 receiveValue: { [weak self] userProfile in
                     self?.nameOriginal = userProfile.name