OpenPNE 3

この不具合について英日の翻訳の問題として修正されていますが、そもそも「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 と同一のものに合わせるのが適切ではないかと考えます。

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

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