blake
14.Август.2020 23:45:48
3
Я создал PR для решения этой проблемы:
master ← remove-new-post-owner-self-likes
merged 03:21PM - 19 Aug 20 UTC
If a user has liked a post that has passed the
`post_undo_action_window_mins` s… ystem setting window and you transfer ownership
of that post to that user you will be the owner of a post that you have
liked, but cannot unlike resulting in a weird UI behavior. This commit
fixes this issue.
The existing tests didn't check for the timeout window for unliking
posts so I added that in.
I couldn't find a good way to do this logic inside of the guardian class
so rather than duplicating behavior of the `PostActionDestroyer` class
inside of the `PostOwnerChanger` I decided to pass in a "bypass"
variable that could be used to check if the calling class is the
'post_owner_changer' and bypass the guardian instead. I went this route
because the guardian `can_delete_post_action` method has no way of
distinguishing how to allow a user to be able to unlike their own posts
after the timeout window but only on a post owner change.
Я выяснил, что передача прав собственности отлично работает при удалении лайков, если ещё не истёк тайм-аут на снятие лайка. Однако, если этот срок уже прошёл, как в случае со старым постом, лайки не удаляются, поскольку не проходит проверка Guardian на возможность снять лайк со своего собственного поста.
2 лайка