diff --git a/services/web/app/coffee/Features/User/UserInfoController.coffee b/services/web/app/coffee/Features/User/UserInfoController.coffee index a77f575a48..8054f48afe 100644 --- a/services/web/app/coffee/Features/User/UserInfoController.coffee +++ b/services/web/app/coffee/Features/User/UserInfoController.coffee @@ -30,6 +30,8 @@ module.exports = UserController = res.send JSON.stringify(info) formatPersonalInfo: (user, callback = (error, info) ->) -> + if !user? + return {} formatted_user = { id: user._id.toString() } for key in ["first_name", "last_name", "email", "signUpDate", "role", "institution"] if user[key]? diff --git a/services/web/public/coffee/ide/review-panel/controllers/ReviewPanelController.coffee b/services/web/public/coffee/ide/review-panel/controllers/ReviewPanelController.coffee index f68e6518c2..3b9990564e 100644 --- a/services/web/public/coffee/ide/review-panel/controllers/ReviewPanelController.coffee +++ b/services/web/public/coffee/ide/review-panel/controllers/ReviewPanelController.coffee @@ -373,7 +373,9 @@ define [ return comment formatUser = (user) -> - if !user? + id = user?._id or user?.id + + if !id? return { email: null name: "Anonymous" @@ -381,13 +383,13 @@ define [ hue: ColorManager.ANONYMOUS_HUE avatar_text: "A" } - - id = user._id or user.id if id == window.user_id name = "You" isSelf = true else - name = "#{user.first_name} #{user.last_name}" + name = [user.first_name, user.last_name].filter((n) -> n?).join(" ") + if name == "" + name = "Unknown" isSelf = false return { id: id