Project

General

Profile

Bug(バグ) #3736

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

Added by kaoru n over 4 years ago. Updated almost 3 years ago.

Status:
Invalid(無効)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2014-11-12
Due date:
% Done:

0%

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

Description

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>


Related issues

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

History

#1 Updated by kaoru n over 4 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#2 Updated by isao sano over 3 years ago

#3 Updated by isao sano over 3 years ago

#4 Updated by Shinichi Urabe over 3 years ago

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

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

#5 Updated by kaoru n over 3 years ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)
  • % Done changed from 50 to 0

#6 Updated by kaoru n over 3 years ago

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

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

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

#7 Updated by kaoru n over 3 years ago

  • Related to Bug(バグ) #3900: 携帯版の携帯メールアドレス設定エラーが全角カタカナで表示される added

#8 Updated by kaoru n over 3 years ago

  • Related to Bug(バグ) #3899: 携帯版のPCメールアドレス設定エラーが翻訳されていない added

#9 Updated by Youichi Kimura almost 3 years ago

この不具合について英日の翻訳の問題として修正されていますが、そもそも「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 Updated by kaoru n almost 3 years ago

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

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

#11 Updated by Youichi Kimura almost 3 years ago

  • Status changed from Pending Review(レビュー待ち) to Invalid(無効)
  • % Done changed from 50 to 0

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

#12 Updated by Shinichi Urabe about 2 years ago

  • Related to Bug(バグ) #4000: MemberConfigFormのIsUnique制約に対するエラーメッセージが適切に出力されていない added

Also available in: Atom PDF