Project

General

Profile

Actions

Bug(バグ) #4188

closed

アクティビティガジェット Web公開していない投稿がWeb公開されてしまう

Added by kaoru n over 7 years ago. Updated over 5 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Assignee:
Target version:
Start date:
2017-04-21
Due date:
% Done:

100%

Estimated time:
3.6 で発生するか:
Yes (はい)
3.8 で発生するか:
Yes (はい)

Description

Overview (現象)

プロフィール画面のガジェット設定で、アクティビティガジェットを「web全体に公開」に設定した場合、
タイムラインへの投稿時に「全員に公開」で設定した投稿が、プロフィール画面でWeb全体に公開される。

各バージョンごとの動作確認

背景赤色がこのチケットの修正対象。
背景オレンジは通常バグであるので、このチケットの修正公開後に通常バグとしてチケット作成する。
「エラー(※)」は通常バグであるので、このチケットの修正公開後に通常バグとしてチケット作成する。

再現手順

  1. 管理画面→上級者向け設定(3.6.x の場合はSNS設定)→メンバーのプロフィールページ公開範囲設定 を 「メンバーの設定を許可する」もしくは「メンバーの設定を許可しない(Web全体に公開)」に設定
  2. 管理画面→デザイン設定→ガジェット設定→プロフィール画面ガジェット設定→ガジェット追加 にて アクティビティ を追加し、設定変更ボタンを押下
  3. アクティビティの設定画面を開き公開範囲を「Web全体に公開」に変更し submit ボタンを押下
  4. メンバーAのアカウントでSNSメンバー画面にログインし、タイムラインにて「全員に公開」で投稿
  5. SNSメンバー画面をログアウトするか、別のブラウザにて、「/member/{メンバーAのID}」を確認

Causes (原因)

プロフィールページの公開範囲及びガジェットの公開範囲を考慮していないため

Way to fix (修正内容)

修正内容を記入

--

補足

・OpenPNE-3.6.x では JSON API がないため opTimelinePlugin は動作しない
・タイムライン・アクティビティにて Web公開して投稿するためには、OpenPNE.yml に、「activity_is_open: true」を設定する必要がある


Files

t4188.png (68.5 KB) t4188.png kaoru n, 2018-01-15 14:24

Related issues 1 (0 open1 closed)

Related to OpenPNE 3 - Bug(バグ) #4260: opTimelinePlugin 携帯版プロフィール画面のSNSメンバーのタイムラインガジェット Web公開していない投稿がWeb公開されてしまうFixed(完了)kaoru n2017-11-15

Actions
Actions #1

Updated by kaoru n about 7 years ago

  • Subject changed from opTimlinePlugin Web公開していない投稿がWeb公開されてしまう to アクティビティガジェット Web公開していない投稿がWeb公開されてしまう
Actions #2

Updated by kaoru n about 7 years ago

  • Description updated (diff)
Actions #3

Updated by kaoru n about 7 years ago

  • Description updated (diff)
Actions #4

Updated by kaoru n about 7 years ago

  • Description updated (diff)
  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to kaoru n
Actions #5

Updated by kaoru n about 7 years ago

  • Related to Bug(バグ) #4260: opTimelinePlugin 携帯版プロフィール画面のSNSメンバーのタイムラインガジェット Web公開していない投稿がWeb公開されてしまう added
Actions #6

Updated by kaoru n about 7 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50
Actions #7

Updated by kaoru n about 7 years ago

Actions #8

Updated by kaoru n about 7 years ago

  • 3.6 で発生するか changed from Unknown (未調査) to Yes (はい)
Actions #10

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チェックをかけていてかなり怪しそう・・・

Actions #11

Updated by kaoru n almost 6 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)
Actions #12

Updated by Rimpei Ogawa almost 6 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

レビューOKです。

Actions #13

Updated by Chiharu Nakajima almost 6 years ago

  • Status changed from Pending Testing(テスト待ち) to Pending Merge(マージ待ち)
  • % Done changed from 70 to 80

動作確認OKです

Actions #14

Updated by kaoru n over 5 years ago

  • Status changed from Pending Merge(マージ待ち) to Fixed(完了)
  • % Done changed from 80 to 100

マージしました

Actions #15

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
Actions #16

Updated by isao sano over 5 years ago

  • Target version changed from OpenPNE 3.8.32 to OpenPNE-3.9.5
Actions #17

Updated by isao sano over 5 years ago

  • Target version changed from OpenPNE-3.9.5 to OpenPNE 3.8.32
Actions

Also available in: Atom PDF