プロジェクト

全般

プロフィール

Bug(バグ) #3736

PC版のPCメールアドレス設定エラーが翻訳されていない

kaoru n9年以上前に追加. 7年以上前に更新.

ステータス:
Invalid(無効)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2014-11-12
期日:
進捗率:

0%

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

説明

Overview (現象)

PC版のPCメールアドレス設定にて、既に別のメンバーにて登録されているメールアドレスを入力し送信した際のエラーメッセージが「invaild pc_address.」と表示される。

Causes (原因)

翻訳の設定がされていない

Way to fix (修正内容)

/path/to/OpenPNE3/i18n/messages.ja.xml
85行目辺りのmobileの場合の設定と同様に下記を追加する。

      <trans-unit id="">
        <source>Invalid pc_address.</source>
        <target>メールアドレスが無効です。</target>
      </trans-unit>


関連するチケット

関連している OpenPNE 3 - Backport(バックポート) #3869: PC版のPCメールアドレス設定エラーが翻訳されていない Invalid(無効) 2015-11-09
関連している OpenPNE 3 - Backport(バックポート) #3870: PC版のPCメールアドレス設定エラーが翻訳されていない Invalid(無効) 2015-11-09
関連している OpenPNE 3 - Bug(バグ) #3900: 携帯版の携帯メールアドレス設定エラーが全角カタカナで表示される Invalid(無効) 2016-03-09
関連している OpenPNE 3 - Bug(バグ) #3899: 携帯版のPCメールアドレス設定エラーが翻訳されていない Invalid(無効) 2016-03-09
関連している OpenPNE 3 - Bug(バグ) #4000: MemberConfigFormのIsUnique制約に対するエラーメッセージが適切に出力されていない Fixed(完了) 2016-11-15

履歴

#1 kaoru n9年以上前に更新

  • ステータスNew(新規) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

https://github.com/openpne/OpenPNE3/pull/236
にてプルリクエストを行いました。

#2 isao sano8年以上前に更新

#3 isao sano8年以上前に更新

#4 Shinichi Urabe約8年前に更新

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

携帯にもPCアドレス設定が有り、同様のエラーメッセージが表示されます
携帯の場合は半角カタカナでエラーを出す必要があります

#5 kaoru n約8年前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更
  • 進捗率50 から 0 に変更

#6 kaoru n約8年前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

https://redmine.openpne.jp/issues/3736#note-4
のご指摘については、ご指摘内容も含めて下記の2パターンの翻訳に問題があるため、携帯版向けのチケットをそれぞれ作成しそちらで対応します。

(1) 携帯版の携帯メールアドレス設定にて、既に別のメンバーにて登録されているメールアドレスを入力し送信した際のエラーメッセージが全角カタカナで表示される。
(2) 携帯版のPCメールアドレス設定にて、既に別のメンバーにて登録されているメールアドレスを入力し送信した際のエラーメッセージが「invaild pc_address.」と表示される。

#7 kaoru n約8年前に更新

  • 関連している Bug(バグ) #3900: 携帯版の携帯メールアドレス設定エラーが全角カタカナで表示される を追加

#8 kaoru n約8年前に更新

  • 関連している Bug(バグ) #3899: 携帯版のPCメールアドレス設定エラーが翻訳されていない を追加

#9 Youichi Kimura7年以上前に更新

この不具合について英日の翻訳の問題として修正されていますが、そもそも「Invalid pc_address.」のように OpenPNE 内部で使用する設定名がエラーメッセージにそのまま使われていることが問題ではないかと思います。

このエラーメッセージは下記のコードによって出力されています。

source:lib/form/doctrine/MemberConfigForm.class.php@43190aa1#L232:

    throw new sfValidatorError($validator, 'Invalid %name%.', array('name' => $name));

つまり本来は「Invalid pc_address.」や「Invalid mobile_address.」に対してではなく「Invalid %name%.」に対して翻訳を行わなければなりませんが、例えばこれを「不正な %name% です」と翻訳すると「不正な pc_address です」「不正な mobile_address です」と表示することになり、ユーザーに対して表示するエラーメッセージとしてはやはり適切ではないと思います。

一方で、類似の機能を持つ MemberProfileForm では下記のように「The inputted value is already exist.」(ja_JP では「すでに登録されています。」と表示される)が使用されており、識別名がそのままエラーメッセージとして使われることはありません。

source:lib/validator/opValidatorProfile.class.php@43190aa1#L55:

      $isDuplicate = Doctrine::getTable('MemberProfile')->isDuplicatedProfileValue($profileId, $value['value'], $memberId);
      if ($isDuplicate)
      {
        throw new sfValidatorError($this, 'The inputted value is already exist.');
      }

よって、当チケットの修正としては Invalid pc_address.Invalid mobile_address. に対する翻訳を追加するのではなく、MemberConfigForm が出力するエラーメッセージを MemberProfileForm と同一のものに合わせるのが適切ではないかと考えます。

#10 kaoru n7年以上前に更新

この不具合について英日の翻訳の問題として修正されていますが、

と指摘されていますが、このチケットは英日の翻訳の問題を修正するチケットであるので、エラーメッセージ自体を修正する方針とするのであれば、別チケットで扱ったほうが良いと思います。

#11 Youichi Kimura7年以上前に更新

  • ステータスPending Review(レビュー待ち) から Invalid(無効) に変更
  • 進捗率50 から 0 に変更

#3736-9 で指摘した問題点について、エラーメッセージの実装の問題を扱う #4000 と翻訳文の改善を扱う #3999 にチケットを分割しました。以後の議論は対応するチケットにて行って下さい。

#12 Shinichi Urabeほぼ7年前に更新

  • 関連している Bug(バグ) #4000: MemberConfigFormのIsUnique制約に対するエラーメッセージが適切に出力されていない を追加

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