#17 「いいね解除」された時に投稿が増えるバグ、及び、付随するバグの修正

Merge realizado
senooken mesclou 1 commits de tak4/main em gnusocialjp/main 1 ano atrás
tak4 comentado 1 ano atrás

#10 の残りのバグを修正しました。
mastodonとの会話で増幅するバグについては、再現方法が分らないので未確認ですが、今回の修正で直っている可能性があります。
また、リモートとの「いいね解除」の通信がうまく行かない事があったので、そちらも修正しました。

  • 外部サーバーから「いいね解除」された時の増幅を防止する。
    Activitypub_notice.php
  • 「いいね解除」した事を外部サーバーに通知する。
    Activitypub_like.php
  • 外部サーバーからの「いいね解除」をローカルDBに反映する。
    Activitypub_like.php

( #14 の反映はできていないので、もし #14 を先にマージされるのであれば、その後で修正後、再度プルリクエストします。)

#10 の残りのバグを修正しました。 mastodonとの会話で増幅するバグについては、再現方法が分らないので未確認ですが、今回の修正で直っている可能性があります。 また、リモートとの「いいね解除」の通信がうまく行かない事があったので、そちらも修正しました。 - 外部サーバーから「いいね解除」された時の増幅を防止する。 Activitypub_notice.php - 「いいね解除」した事を外部サーバーに通知する。 Activitypub_like.php - 外部サーバーからの「いいね解除」をローカルDBに反映する。 Activitypub_like.php ( #14 の反映はできていないので、もし #14 を先にマージされるのであれば、その後で修正後、再度プルリクエストします。)
senooken comentado 1 ano atrás
Proprietário

対応ありがとうございます!内容、修正量的に、こちらを先にマージしたいです。PHP 8対応は確認や修正に時間がかかると思います。 他の作業との兼ね合いで、確認に時間がかかるかもしれませんので、お待ちください…

対応ありがとうございます!内容、修正量的に、こちらを先にマージしたいです。PHP 8対応は確認や修正に時間がかかると思います。 他の作業との兼ね合いで、確認に時間がかかるかもしれませんので、お待ちください…
tak4 comentado 1 ano atrás
Autor

承知いたしました。よろしくお願いいたします。

承知いたしました。よろしくお願いいたします。

tak4さんの変更は、PHP8対応をマージ前にも、問題無しでこれをマージ出来るはずです

tak4さんの変更は、PHP8対応をマージ前にも、問題無しでこれをマージ出来るはずです
senooken comentado 1 ano atrás
Proprietário

suwako、コメントありがとうございます。ピンポイントの修正なのでおそらく大丈夫だろうと私も思っています。現状この2個しかマージ予定ないので、修正量の少ないこちらをやはり先にマージします。

suwako、コメントありがとうございます。ピンポイントの修正なのでおそらく大丈夫だろうと私も思っています。現状この2個しかマージ予定ないので、修正量の少ないこちらをやはり先にマージします。
senooken comentado 1 ano atrás
Proprietário

@tak4 修正が入ることで、相手からいいねの解除が来た場合の再送がなくなることを確認できました。

また、再投稿の場合は、二重化はそもそも発生していませんでした。OKです。ありがとうございました。

@tak4 修正が入ることで、相手からいいねの解除が来た場合の再送がなくなることを確認できました。 また、再投稿の場合は、二重化はそもそも発生していませんでした。OKです。ありがとうございました。
O merge deste Pull Request foi aplicado com sucesso!
Faça login para participar desta conversação.
Sem milestone
Não atribuída
3 participantes
Carregando...
Cancelar
Salvar
Ainda não há conteúdo.