Project

General

Profile

Bug(バグ) #2465

性別の英語表記「Man/Female」となっている部分がある

Added by Yuya Watanabe about 8 years ago. Updated over 7 years ago.

Status:
Pending Fixing(修正待ち)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2011-10-04
Due date:
% Done:

0%

3.6 で発生するか:
3.8 で発生するか:
Unknown (未調査)

Description

概要

性別の英語表記「Man/Female」となっている.本来は「Man/Woman」あるいは「Male/Female」という表記になるべきだと思われる.

確認手順

プロフィール項目について表示を確認する条件は下記の通り.
  • 管理画面プロフィール項目(pc_backend.php/profile/list)においてプリセットとして「性別」の項目が追加されている
    • 無い場合は「プロフィール項目登録」(pc_backend.php/profile/edit)のページを開き,「プリセットから選択する」を選び,「プリセット」の項目を「性別」にして「追加」ボタンを選択する.
  • SNSのどこかしらのガジェットに言語切り替えボックスが存在する
    • デフォルトはデザイン設定ガジェットのサイドバナー設定(pc_backend.php/design/gadget/type/sideBanner)によってサイドバナーに表示されている
手順は以下のとおり
  1. プロフィール編集画面(member/edit/profile)を開く
  2. 「性別」の項目が存在していることを確認する
  3. 言語切り替えボックスで「English」を選択する
    • 「Sex」のフォームで「Man」と「Female」が存在することを確認する

原因

lib/config/config/preset_profile.yml 23,24行目など

 19   IsRequired: false
 20   IsEditPublicFlag: true
 21   DefaultPublicFlag: 0
 22   Choices:
 23     "F" : "Female" 
 24     "M" : "Man" 
 25
 26 birthday:
 27   Name: "birthday" 
 28   Caption: "Birthday" 

修正案

デフォルトの状態では「Male/Female」ではなく「Man/Woman」と表記を行う.

確認環境

OpenPNE 3.7.0-dev (master)

Associated revisions

Revision a53ebe7b (diff)
Added by Yuya Watanabe about 8 years ago

(fixes #2465) replace Female to Woman

Revision a5d48423 (diff)
Added by Yuya Watanabe about 8 years ago

(refs #2465) replace 'F' to 'W' for profile shortened form

History

#1 Updated by Yuya Watanabe about 8 years ago

  • Subject changed from プロフィール項目で性別の英語表記「Man/Female」となっている. to 性別の英語表記「Man/Female」となっている部分がある
  • Description updated (diff)

#2 Updated by Yuya Watanabe about 8 years ago

  • Status changed from New(新規) to Accepted(着手)

#3 Updated by wa ta about 8 years ago

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

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

#4 Updated by Minoru Takai about 8 years ago

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

性別の表記に関して、 note-3 でも様々なファイルが修正されていることからも分かりますが、不幸なことに ('M' === $sex ? 'Man' : 'Woman') や ('Man' === $sex ? 'male' : 'female') のように、性別を示す値が、定数(や変数)ではなく文字列リテラルで様々な箇所に直接書かれているようです。尤も、定数であったとしても、定数 MAN と FEMALE のような組が使われていては本末転倒なのですが。

  • man / woman (Man / Woman)
  • male / female (Male / Female)
  • M / W
  • M / F

これらの文字列リテラルがソースコード中に散見されており、特別な理由もなく値が統一されていない状況は好ましくないですね(man/woman の組を使うか male/female の組を使うかという点についてですが、これはこのチケットの内容とは関係ありません)。

このチケットでは man/female という不自然な値の組を使っていることを修正しており、修正するという方針は良いと思います。また、 ('Man' === $sex) や ('M' === $sex) のような比較は見つかるものの、 ('Female' === $sex) や ('F' === $sex) は見当たらないため、この変更による影響は(バンドルプラグインも含めて)なさそうです。(もしかすると見逃しがあるかもしれません。この確認は十分行うべきです。)


'man', 'female', 'm', 'f' などの記述が残っていないかを調べてみましたが、以下で Man/Female の組に相当する記述が残っているようです。

  • $ ack -a "'m'" -i --ignore-dir=vendor での結果の一部
    lib/util/opOpenIDSregProfileExport.class.php
    46:    return ('Man' === $sex ? 'M' : 'F');
    
    lib/util/opOpenIDAxProfileExport.class.php
    79:    return ('Man' === $sex ? 'M' : 'F');
    

これはこのままでよいものなのでしょうか。

コメントのみ行うつもりでしたが、ステータスを差し戻しにしておきます。

#5 Updated by Yuya Watanabe about 8 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)

note-4の指摘部分について a5d48423 で修正.

十分な確認を行うためステータスはAcceptedにしておきます.

#6 Updated by Yuya Watanabe about 8 years ago

  • Status changed from Accepted(着手) to Pending Fixing(修正待ち)

時間が取れないため,一旦ステータスを「修正待ち」にします.

#7 Updated by Shouta Kashiwagi over 7 years ago

  • Target version changed from OpenPNE 3.7.0 to 252

#8 Updated by Shouta Kashiwagi over 7 years ago

  • Target version changed from 252 to OpenPNE 3.8.x

Also available in: Atom PDF