Project

General

Profile

Backport(バックポート) #1881

When a value of MemberConfig's "age_public_flag" is not exists, the label of "Settings" is inconsistent with behavior (MemberConfigの "age_public_flag" の値が存在しないとき、「設定変更」の表示と実際の挙動が異なる)

Added by Shogo Kawahara over 8 years ago. Updated about 8 years ago.

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

100%


Description

Overview

When a value of MemberConfig's "age_public_flag" is not exists, the caption of "Settings" is inconsistent with behavior.

MemberConfigの "age_public_flag" の値が空のとき、
「設定変更」での表示は「全員に公開」と表示されているのにも関わらず、
実際は「非公開」として取り扱われる。

非常に紛らわしい状況になっている。

Causes

lib/config/config/member_config.yml の age_public_flag
の Default が "1" になっている。

Way to fix

Default を 3 (private) としておくべき。

(表示の方を現状の挙動に合わせる。
これは、今までリリースしてきた各安定版バージョンが
存在するからである。)

Environment

OpenPNE3.4.x 〜


Related issues

Related to OpenPNE 3 - Bug(バグ) #1872: When a value of MemberConfig's "age_public_flag" is not exists, the label of "Settings" is inconsistent with behavior (MemberConfigの "age_public_flag" の値が存在しないとき、「設定変更」の表示と実際の挙動が異なる) Fixed(完了) 2011-01-17
Related to OpenPNE 3 - Backport(バックポート) #2294: When a value of MemberConfig's "age_public_flag" is not exists, the label of "Settings" is inconsistent with behavior (MemberConfigの "age_public_flag" の値が存在しないとき、「設定変更」の表示と実際の挙動が異なる) Fixed(完了) 2011-01-17

Associated revisions

Revision 84563be9 (diff)
Added by Kousuke Ebihara about 8 years ago

fixed comparing ProfileTable::PUBLIC_FLAG_* to a value to follow conding standard in the Member::getAge() (fixes #1881, BP from #1872)

Revision 10cb44c6 (diff)
Added by Kousuke Ebihara about 8 years ago

fixed comparing ProfileTable::PUBLIC_FLAG_* to a value to follow conding standard in the Member::getAge() (fixes #1881, BP from #1872)

History

#1 Updated by Kousuke Ebihara over 8 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 about 8 years ago

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

#4 Updated by Masato Nagasawa about 8 years ago

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

#5 Updated by Kousuke Ebihara about 8 years ago

  • Due date set to 2011-06-24

#6 Updated by Kousuke Ebihara about 8 years ago

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

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

#7 Updated by Kousuke Ebihara about 8 years ago

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

元チケットを「差し戻し」にしたため、その状況を見てからバックポートします。

#8 Updated by Kousuke Ebihara about 8 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • Assignee set to Kousuke Ebihara

すいません、取り込みの際にコミットメッセージ内のチケット番号を変更し忘れました。

a5bafb22ac71a02ef407bb0356ba14e93c7c2cff9bcca0767963f8b32f43d82d2118404bd8795858 が、本チケットの stable-3.6.x への対応となります。

#9 Updated by Naoya Tozuka about 8 years ago

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

修正点確認しました。問題ないと思われます。

元チケット http://redmine.openpne.jp/issues/1872#note-13 同様、お手数ですが、今回の lib/model/doctrine/Member.class.php の修正箇所の後で、$publicFlag と定数が比較されている部分について、$publicFlag が左辺に来ているのがコーディング規約に反しますので、直してもらっても宜しいでしょうか?

#10 Updated by Kousuke Ebihara about 8 years ago

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

更新履歴 84563be9f2a4e87735e3c5385f9bf775e337651f で適用されました。

#11 Updated by Naoya Tozuka about 8 years ago

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

確認しました。OKです。

#12 Updated by isao sano about 8 years ago

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

テスト完了いたしました。
問題ありませんので、完了いたします。

Also available in: Atom PDF