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

マージされた
senooken 1 年間 前 に 1 件のコミットが tak4/main から gnusocialjp/main にマージされました
tak41 年間 前 にコメントしました

#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 を先にマージされるのであれば、その後で修正後、再度プルリクエストします。)
senooken1 年間 前 にコメントしました
オーナー

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

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

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

承知いたしました。よろしくお願いいたします。
TechnicalSuwako1 年間 前 にコメントしました

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

tak4さんの変更は、PHP8対応をマージ前にも、問題無しでこれをマージ出来るはずです
senooken1 年間 前 にコメントしました
オーナー

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

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

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

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

@tak4 修正が入ることで、相手からいいねの解除が来た場合の再送がなくなることを確認できました。 また、再投稿の場合は、二重化はそもそも発生していませんでした。OKです。ありがとうございました。
このプルプルリクエストは正常にマージされました!
会話に参加するには サインイン してください。
マイルストーンなし
担当者なし
3 参加者
読み込み中…
キャンセル
保存
まだコンテンツがありません