mapping user email to nickname in MessageResponseDto #80
Conversation
| messageResponseDto.setSentFrom(contactService.findByEmail(email).getNickname()); | ||
| messageResponseDto.setSendTo( | ||
| contactService.findByEmail(messageResponseDto.getSendTo()).getNickname()); | ||
|
|
There was a problem hiding this comment.
немає перевірки на exception. setSendTo в більшості випадків буде викликати ContactNotFoundException бо такого контакта як, наприклад, Topic id=4 немає.
There was a problem hiding this comment.
Перевірка є
Всі поля обєктів DTO валідуються та оброблюються в классі WebsocketExceptionHandler а потім виводится повідомлення про помилку від сервера
There was a problem hiding this comment.
так. не побачив що це приватний у тебе топік.
|
Можливо я щось не так зрозумів але наче нікнейм є в полі sentFrom |
| messageResponseDto.setSentFrom(contactService.findByEmail(email).getNickname()); | ||
| messageResponseDto.setSendTo( | ||
| contactService.findByEmail(messageResponseDto.getSendTo()).getNickname()); | ||
|
|
There was a problem hiding this comment.
так. не побачив що це приватний у тебе топік.
| List<MessageResponseDto> messages = messageService.findAllByTopicId(topicId, pageRequestDto) | ||
| .stream() | ||
| .peek(m -> m.setSentFrom(contactService.findByEmail(m.getSentFrom()).getNickname())) | ||
| .toList(); |
There was a problem hiding this comment.
цей використовується тільки для публічних топіків?
There was a problem hiding this comment.
Це для отримання історії переписки як публічних так і приватних топіків.
There was a problem hiding this comment.
якщо і для приватних то не бачу setSentTo
No description provided.