А каст у всех работает нормально или через раз?

  • deep-purple
    0·
    3 years ago

    Что мешает проверять наличие каста пользака к сообщению? Что мешает удалять имеющийся каст при редактировании?

    • deep-purple
      0·
      3 years ago

      Я не жавист. Я там ноги переломаю себе. Но вообще реализация не ожидается сильно сложной.

    • Bóbr kurwa
      0·
      3 years ago

      Я бы завёл отдельную таблицу с кастами, со связью один ко многим, где на каждый пост был бы список кастов. Когда юзер создаёт или редактирует пост, проверяются все касты, связанные с этим постом - и если какого-то каста ещё небыло, то он добавляется в таблицу и пользователю отсылается уведомление, а если уже такой был - то ничего не происходит.

      • deep-purple
        0·
        3 years ago

        Таким образом, при создании/редактировании поста проще делать запрос на удаление вида:

        DELETE FROM tbl WHERE post_id=N;

        Так, если это новый пост, то запрос ничего не удалит, а если пост редактируется, то все касты поста будут удалены.

        Вторым шагом делаем запрос вида:

        INSERT INTO tbl (post_id, user_id) VALUES (N, A), (N, B), (N, C);
        • Bóbr kurwa
          0·
          3 years ago

          Нихуя не понял. Нам нужно не посылать касты повторно.

          • deep-purple
            0·
            3 years ago

            Даже если каст отправлен, но позднее было редактирование – это не страшно, пост ведь существует. О повторном касте речи не идет.

            • Bóbr kurwa
              0·
              3 years ago

              Как не идёт, если идёт? Пользователь при редактировании может добавить новый каст или уже существующий каст, или удалить каст, или добавить вновь каст, который был удалён ранее. Всё это легко разруливается при помощи связанной таблицы кастов.

              • Bóbr kurwa
                0·
                3 years ago

                @cocucka а где исходники вашего говна лежат? Что-то я в глаза долблюсь.

                • cocucka
                  0·
                  3 years ago

                  Оно спрятано от макскома, а то он фичи ворует

                  • Bóbr kurwa
                    0·
                    3 years ago

                    Опенсорс, который мы заслужили.

                    • cocucka
                      0·
                      3 years ago

                      Ага, сначала они банят/замораживают за любое упоминание о ЛТ, а потом у нас фичи берут.

                      • crypt
                        0·
                        3 years ago

                        @metalbeaver ты можешь оригинальные исходники глянуть и сделать патч, либо, я думаю, соиска тебя к исходникам пустит, если ты захочешь в этом копаться.

                        • Bóbr kurwa
                          0·
                          3 years ago

                          Я сейчас глянул исходники. А подскажите, разве это не буквально то самое, о чём я говорил?

                          /* кастовать только тех, кто добавился. Существующие ранее не кастуются */

                          Включите, блядь, лоркод.

                          /**

                          • Редактирование комментария.
                          • @param oldComment данные старого комментария
                          • @param newComment данные нового комментария
                          • @param commentBody текст нового комментария
                          • @param remoteAddress IP-адрес, с которого был добавлен комментарий
                          • @param xForwardedFor IP-адрес через шлюз, с которого был добавлен комментарий */ @Transactional(rollbackFor = Exception.class, propagation = Propagation.REQUIRED) public void edit(Comment oldComment, Comment newComment, String commentBody, String remoteAddress, String xForwardedFor, User editor, MessageText originalMessageText) { commentDao.changeTitle(oldComment, newComment.getTitle()); msgbaseDao.updateMessage(oldComment.getId(), commentBody);
                          /* кастование пользователей */
Set<User> newUserRefs = textService.mentions(MessageText.apply(commentBody, originalMessageText.markup()));

MessageText messageText = msgbaseDao.getMessageText(oldComment.getId());
Set<User> oldUserRefs = textService.mentions(messageText);
Set<User> userRefs = new HashSet<>();
/* кастовать только тех, кто добавился. Существующие ранее не кастуются */
for (User user :newUserRefs) {
  if (!oldUserRefs.contains(user)) {
    userRefs.add(user);
  }
}

if (permissionService.isUserCastAllowed(editor)) {
  userEventService.addUserRefEvent(userRefs, oldComment.getTopicId(), oldComment.getId());
}

/* Обновление времени последнего изменения топика для того, чтобы данные в кеше автоматически обновились  */
topicDao.updateLastmod(oldComment.getTopicId(), false);

addEditHistoryItem(editor, oldComment, originalMessageText.text(), newComment, commentBody);

updateLatestEditorInfo(editor, oldComment, newComment);

String logMessage = makeLogString("Изменён комментарий " + oldComment.getId(), remoteAddress, xForwardedFor);
logger.info(logMessage);

                          }