/*
 * Copyright 2020 Google LLC
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *     https://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

package com.google.android.ground.persistence.remote.firestore.schema

import com.google.android.ground.model.User

/** Converts between Firestore objects and [User] instances.  */
internal object UserConverter {
    fun toNestedObject(user: User): UserNestedObject {
        return UserNestedObject(user.id, user.email, user.displayName)
    }

    fun toUser(ud: UserNestedObject?): User {
        // Degrade gracefully when user missing in remote db.
        val ud2 = if (ud?.id == null) UserNestedObject.UNKNOWN_USER else ud
        return User.builder()
            .setId(ud2.id)
            .setEmail(ud2.email)
            .setDisplayName(ud2.displayName)
            .build()
    }
}

/*
 * Copyright 2020 Google LLC
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *     https://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

package com.google.android.ground.persistence.remote.firestore.schema

import com.google.android.ground.model.User

/** Converts between Firestore objects and [User] instances.  */
internal object UserConverter {
    fun toNestedObject(user: User): UserNestedObject {
        return UserNestedObject(user.id, user.email, user.displayName)
    }

    fun toUser(ud: UserNestedObject?): User {
        return User.builder()
            .setId(ud?.id.orEmpty())
            .setEmail(ud?.email.orEmpty())
            .setDisplayName(ud?.displayName.orEmpty())
            .build()
    }
}

Use orEmpty() instead of setting as null
Use orEmpty() instead of setting as null

google/ground-android,google/ground-android,google/ground-android