Bug(バグ) #4188
closedアクティビティガジェット Web公開していない投稿がWeb公開されてしまう
100%
Description
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」を設定する必要がある
Files
Updated by kaoru n about 7 years ago
- Subject changed from opTimlinePlugin Web公開していない投稿がWeb公開されてしまう to アクティビティガジェット Web公開していない投稿がWeb公開されてしまう
Updated by kaoru n about 7 years ago
- Description updated (diff)
- Status changed from New(新規) to Accepted(着手)
- Assignee set to kaoru n
Updated by kaoru n about 7 years ago
- Related to Bug(バグ) #4260: opTimelinePlugin 携帯版プロフィール画面のSNSメンバーのタイムラインガジェット Web公開していない投稿がWeb公開されてしまう added
Updated by kaoru n about 7 years ago
- Status changed from Accepted(着手) to Pending Review(レビュー待ち)
- % Done changed from 0 to 50
各ブランチについて Merge Request を作成しました。レビューをお願いします。
Updated by kaoru n about 7 years ago
- 3.6 で発生するか changed from Unknown (未調査) to Yes (はい)
Updated by kaoru n almost 6 years ago
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
Updated by Rimpei Ogawa almost 6 years ago
- Status changed from Pending Review(レビュー待ち) to 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チェックをかけていてかなり怪しそう・・・
Updated by kaoru n almost 6 years ago
- Status changed from Rejected(差し戻し) to 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
Updated by Rimpei Ogawa almost 6 years ago
- Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
- % Done changed from 50 to 70
レビューOKです。
Updated by Chiharu Nakajima almost 6 years ago
- Status changed from Pending Testing(テスト待ち) to Pending Merge(マージ待ち)
- % Done changed from 70 to 80
動作確認OKです
Updated by kaoru n over 5 years ago
- Status changed from Pending Merge(マージ待ち) to Fixed(完了)
- % Done changed from 80 to 100
マージしました
Updated by kaoru n over 5 years ago
- Project changed from 22 to OpenPNE 3
- Category deleted (
権限不備) - Target version set to OpenPNE 3.8.32
Updated by isao sano over 5 years ago
- Target version changed from OpenPNE 3.8.32 to OpenPNE-3.9.5
Updated by isao sano over 5 years ago
- Target version changed from OpenPNE-3.9.5 to OpenPNE 3.8.32