プロジェクト

全般

プロフィール

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" の値が存在しないとき、「設定変更」の表示と実際の挙動が異なる)

Shogo Kawahara13年以上前に追加. ほぼ13年前に更新.

ステータス:
Fixed(完了)
優先度:
High(高め)
担当者:
対象バージョン:
開始日:
2011-01-17
期日:
2011-06-24
進捗率:

100%


説明

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 〜


関連するチケット

関連している 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
関連している 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

関係しているリビジョン

リビジョン 84563be9 (差分)
Kousuke Ebiharaほぼ13年前に追加

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

リビジョン 10cb44c6 (差分)
Kousuke Ebiharaほぼ13年前に追加

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

履歴

#1 Kousuke Ebihara13年以上前に更新

  • 対象バージョンOpenPNE 3.7.0 から OpenPNE3.6beta9 に変更

#2 Kousuke Ebihara約13年前に更新

  • 対象バージョンOpenPNE3.6beta9 から OpenPNE3.6beta11 に変更

#3 Masato Nagasawaほぼ13年前に更新

  • 対象バージョンOpenPNE3.6beta11 から OpenPNE 3.6.0 に変更

#4 Masato Nagasawaほぼ13年前に更新

  • 対象バージョンOpenPNE 3.6.0 から OpenPNE3.6beta11 に変更

#5 Kousuke Ebiharaほぼ13年前に更新

  • 期日2011-06-24 にセット

#6 Kousuke Ebiharaほぼ13年前に更新

  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者Kousuke Ebihara にセット

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

#7 Kousuke Ebiharaほぼ13年前に更新

  • ステータスAccepted(着手) から New(新規) に変更
  • 担当者 を削除 (Kousuke Ebihara)

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

#8 Kousuke Ebiharaほぼ13年前に更新

  • ステータスNew(新規) から Pending Review(レビュー待ち) に変更
  • 担当者Kousuke Ebihara にセット

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

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

#9 Naoya Tozukaほぼ13年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更

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

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

#10 Kousuke Ebiharaほぼ13年前に更新

  • ステータスRejected(差し戻し) から Pending Review(レビュー待ち) に変更

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

#11 Naoya Tozukaほぼ13年前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

確認しました。OKです。

#12 isao sanoほぼ13年前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

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

他の形式にエクスポート: Atom PDF