Bug（バグ） #4188
完了
アクティビティガジェット Web公開していない投稿がWeb公開されてしまう
100%
説明
Overview (現象)¶
プロフィール画面のガジェット設定で、アクティビティガジェットを「web全体に公開」に設定した場合、
タイムラインへの投稿時に「全員に公開」で設定した投稿が、プロフィール画面でWeb全体に公開される。
各バージョンごとの動作確認¶
背景赤色がこのチケットの修正対象。
背景オレンジは通常バグであるので、このチケットの修正公開後に通常バグとしてチケット作成する。
「エラー(※)」は通常バグであるので、このチケットの修正公開後に通常バグとしてチケット作成する。
再現手順¶
- 管理画面→上級者向け設定(3.6.x の場合はSNS設定)→メンバーのプロフィールページ公開範囲設定 を 「メンバーの設定を許可する」もしくは「メンバーの設定を許可しない（Web全体に公開）」に設定
- 管理画面→デザイン設定→ガジェット設定→プロフィール画面ガジェット設定→ガジェット追加 にて アクティビティ を追加し、設定変更ボタンを押下
- アクティビティの設定画面を開き公開範囲を「Web全体に公開」に変更し submit ボタンを押下
- メンバーAのアカウントでSNSメンバー画面にログインし、タイムラインにて「全員に公開」で投稿
- SNSメンバー画面をログアウトするか、別のブラウザにて、「/member/{メンバーAのID}」を確認
Causes (原因)¶
プロフィールページの公開範囲及びガジェットの公開範囲を考慮していないため
Way to fix (修正内容)¶
修正内容を記入
--
補足¶
・OpenPNE-3.6.x では JSON API がないため opTimelinePlugin は動作しない
・タイムライン・アクティビティにて Web公開して投稿するためには、OpenPNE.yml に、「activity_is_open: true」を設定する必要がある
ファイル
kaoru n さんが約7年前に更新
- 関連している Bug（バグ） #4260: opTimelinePlugin 携帯版プロフィール画面のSNSメンバーのタイムラインガジェット Web公開していない投稿がWeb公開されてしまう を追加
kaoru n さんが5年以上前に更新
github にてプルリクエスト作成しました
3.6.x: https://github.com/openpne/OpenPNE3/pull/514
3.8.x: https://github.com/openpne/OpenPNE3/pull/515
master: https://github.com/openpne/OpenPNE3/pull/516
Rimpei Ogawa さんが5年以上前に更新
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
非ログインユーザーに関しては一律で、$flag = self::PUBLIC_FLAG_OPEN;
となるべきで、
間違っているのは、修正部分よりも手前の、
if (is_null($viewerMemberId))
{
$flag = self::PUBLIC_FLAG_OPEN;
}
https://github.com/openpne/OpenPNE3/blob/master/lib/model/doctrine/ActivityDataTable.class.php#L311
の部分だと思います。
おそらく opAnonymousMember::getId() が null ではなく 0 を返すので、動いていない。
member_id は存在する場合は 1 以上の整数なので、null or 0 を弾く意味で、if (!$viewerMemberId) {
でよいと思います。
このファイル内、他にもメンバーIDに対してNULLチェックをかけていてかなり怪しそう・・・
kaoru n さんが5年以上前に更新
- ステータス を Rejected（差し戻し） から Pending Review（レビュー待ち） に変更
プルリクエスト修正しました
3.6.x: https://github.com/openpne/OpenPNE3/pull/514
3.8.x: https://github.com/openpne/OpenPNE3/pull/515
master: https://github.com/openpne/OpenPNE3/pull/516
Rimpei Ogawa さんが5年以上前に更新
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
レビューOKです。
Chiharu Nakajima さんが5年以上前に更新
- ステータス を Pending Testing（テスト待ち） から Pending Merge（マージ待ち） に変更
- 進捗率 を 70 から 80 に変更
動作確認OKです
