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 almost 13 years ago. Updated over 12 years ago.

Status:
Fixed（完了）
Priority:
High（高め）
Assignee:
Kousuke Ebihara
Target version:
OpenPNE3.6beta11
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 over 12 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 over 12 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 almost 13 years ago

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

#2 Updated by Kousuke Ebihara over 12 years ago

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

#3 Updated by Masato Nagasawa over 12 years ago

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

#4 Updated by Masato Nagasawa over 12 years ago

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

#5 Updated by Kousuke Ebihara over 12 years ago

  • Due date set to 2011-06-24

#6 Updated by Kousuke Ebihara over 12 years ago

  • Status changed from New（新規） to Accepted（着手）
  • Assignee set to Kousuke Ebihara

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

#7 Updated by Kousuke Ebihara over 12 years ago

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

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

#8 Updated by Kousuke Ebihara over 12 years ago

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

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

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

#9 Updated by Naoya Tozuka over 12 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 over 12 years ago

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

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

#11 Updated by Naoya Tozuka over 12 years ago

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

確認しました。OKです。

#12 Updated by isao sano over 12 years ago

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

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

Also available in: Atom PDF