Project

General

Profile

Backport(バックポート) #1882

管理画面の公開設定変更の可否を「メンバー選択」から「固定」に変更すると、公開設定のデフォルト値でメンバー側の設定が更新されてしまう

Added by isao sano almost 9 years ago. Updated over 8 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Target version:
Start date:
2011-01-18
Due date:
2011-06-24
% Done:

100%


Description

現象

管理画面の公開設定変更の可否を「メンバー選択」から「固定」に変更すると、公開設定デフォルト値でメンバー側の設定を上書きしてしまう。
メンバー側が非公開にしている項目が、管理側の公開設定デフォルト値次第では公開されてしまう恐れがある。

バージョン

  • 3.7.x
  • 3.6.x
  • 3.4.x
  • 3.2.x¶

再現手順

1.管理画面の設定で誕生日のプロフィール項目の公開設定変更の可否を「メンバー選択」にしておく
2.メンバー側で誕生日の公開範囲を「公開しない」にする
3.管理画面の誕生日の公開設定変更の可否を「固定」に戻す
4.管理画面の誕生日の公開設定デフォルト値は「全員に公開」のままにしておく
5.手順2で設定した非公開設定が手順3の設定で上書きされ「全員の公開」になる

補足

  • メンバー側の誕生日の公開範囲を「フレンドまで公開」にして、非フレンドユーザーから見た場合でも同じことが起きる(確認済み)
  • 他のプロフィールでも同様の現象がおきる事が予想できます。(未確認)

環境

OS Windows XP
ブラウザ Firefox 3.6.13

Causes

固定の公開範囲を保存するときに、ProfileForm::save() が
MemberProfileの "public_flag" の値を デフォルトものに上書きしています。

Way to fix

プロフィール表示・検索機能にて、メンバーの設定値を上書きしないで管理画面の公開設定デフォルト値が適用される様に修正する

1. ProfileForm::save() で MemberProfile の "public_flag" の値を上書きしないようにする。
2. MemberProfile の表示判定を 公開範囲の固定に備えて変更する。
3. 検索時の抽出判定を 公開範囲の固定に備えて変更する。


Related issues

Related to OpenPNE 3 - Bug(バグ) #1874: 管理画面の公開設定変更の可否を「メンバー選択」から「固定」に変更すると、公開設定のデフォルト値でメンバー側の設定が更新されてしまう Fixed(完了) 2011-01-18
Related to OpenPNE 3 - Backport(バックポート) #2295: 管理画面の公開設定変更の可否を「メンバー選択」から「固定」に変更すると、公開設定のデフォルト値でメンバー側の設定が更新されてしまう Fixed(完了) 2011-01-18

Associated revisions

Revision 712bb89c (diff)
Added by Shogo Kawahara over 8 years ago

fixed the ProfileForm, because it overwrite value of MemberProfile's "public_flag" (fixes #1882, BP from #1874)

Revision ada4b37f (diff)
Added by Maki Takahashi over 8 years ago

fixed the reflecting the IsEditPublicFlag of Profile in the Member Search (refs #1882, BP from #1874)

Revision 0975456d (diff)
Added by Shogo Kawahara over 8 years ago

fixed the ProfileForm, because it overwrite value of MemberProfile's "public_flag" (fixes #1882, BP from #1874)
(cherry picked from commit 712bb89cda93a348e675218e3841f1d29c16a940)

Revision 14ab4205 (diff)
Added by Maki Takahashi over 8 years ago

fixed the reflecting the IsEditPublicFlag of Profile in the Member Search (refs #1882, BP from #1874)
(cherry picked from commit ada4b37f312ff4b61d52666953b9bca0fc3745be)

History

#1 Updated by Kousuke Ebihara almost 9 years ago

  • Target version changed from OpenPNE 3.7.0 to OpenPNE3.6beta9

#2 Updated by Kousuke Ebihara over 8 years ago

  • Target version changed from OpenPNE3.6beta9 to OpenPNE3.6beta11

#3 Updated by Masato Nagasawa over 8 years ago

  • Target version changed from OpenPNE3.6beta11 to OpenPNE 3.6.0

#4 Updated by Masato Nagasawa over 8 years ago

  • Target version changed from OpenPNE 3.6.0 to OpenPNE3.6beta11

#5 Updated by Kousuke Ebihara over 8 years ago

  • Due date set to 2011-06-24

#6 Updated by Kousuke Ebihara over 8 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Kousuke Ebihara

バックポート引き受けます

#7 Updated by Shogo Kawahara over 8 years ago

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

更新履歴 712bb89cda93a348e675218e3841f1d29c16a940 で適用されました。

#8 Updated by Shingo Yamada over 8 years ago

  • Subject changed from メンバーの設定した公開範囲よりも管理画面の公開設定デフォルト値が優先される to 管理画面の公開設定変更の可否を「メンバー選択」から「固定」に変更すると、公開設定のデフォルト値でメンバー側の設定が上書かれてしまう

#9 Updated by Shingo Yamada over 8 years ago

  • Subject changed from 管理画面の公開設定変更の可否を「メンバー選択」から「固定」に変更すると、公開設定のデフォルト値でメンバー側の設定が上書かれてしまう to 管理画面の公開設定変更の可否を「メンバー選択」から「固定」に変更すると、公開設定のデフォルト値でメンバー側の設定が更新されてしまう

#10 Updated by Naoya Tozuka over 8 years ago

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

プロフィール検索時に管理画面の公開設定デフォルト値が適用されていないようです。
http://redmine.openpne.jp/issues/1874#note-14 同様の修正が必要と思われます。

#11 Updated by Kousuke Ebihara over 8 years ago

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

#12 Updated by Naoya Tozuka over 8 years ago

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

対応どうもありがとうございます。レビューOKです。

#13 Updated by isao sano over 8 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

テスト完了いたしました。
問題ないので完了にいたします。

Also available in: Atom PDF