Project

General

Profile

Bug(バグ) #4188

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

Added by kaoru n over 2 years ago. Updated 5 months ago.

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

100%

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」を設定する必要がある

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


Related issues

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

History

#1 Updated by kaoru n almost 2 years ago

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

#2 Updated by kaoru n almost 2 years ago

  • Description updated (diff)

#3 Updated by kaoru n almost 2 years ago

  • Description updated (diff)

#4 Updated by kaoru n almost 2 years ago

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

#5 Updated by kaoru n almost 2 years ago

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

#6 Updated by kaoru n almost 2 years ago

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

#7 Updated by kaoru n almost 2 years ago

#8 Updated by kaoru n almost 2 years ago

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

#10 Updated by Rimpei Ogawa 7 months 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チェックをかけていてかなり怪しそう・・・

#11 Updated by kaoru n 7 months ago

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

#12 Updated by Rimpei Ogawa 7 months ago

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

レビューOKです。

#13 Updated by Chiharu Nakajima 7 months ago

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

動作確認OKです

#14 Updated by kaoru n 5 months ago

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

マージしました

#15 Updated by kaoru n 5 months ago

  • Project changed from OpenPNE Security Issue to OpenPNE 3
  • Category deleted (権限不備)
  • Target version set to OpenPNE 3.8.32

#16 Updated by isao sano 5 months ago

  • Target version changed from OpenPNE 3.8.32 to OpenPNE-3.9.5

#17 Updated by isao sano 5 months ago

  • Target version changed from OpenPNE-3.9.5 to OpenPNE 3.8.32

Also available in: Atom PDF