Project

General

Profile

Bug(バグ) #2430

プロフィール編集画面にて、郵便番号が日本語文字列で登録できる

Added by Yuma Sakata about 8 years ago. Updated over 2 years ago.

Status:
Won't fix(対応せず)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
Due date:
% Done:

0%

3.6 で発生するか:
Yes (はい)
3.8 で発生するか:
Yes (はい)

Description

Overview (現象)

プロフィール編集画面にて、郵便番号が日本語文字列で登録できる

Environment (再現環境)

OpenPNE 3.6RC1

Way to repro (再現手順)

1. 管理画面プロフィール項目登録ページ(/pc_backend.php/profile/edit)にアクセスする
2. 「プリセットから選ぶ」を選択して「郵便番号」を選択する
3. SNSログイン後、プロフィール編集ページ(/member/edit/profile)にアクセスする
4. 郵便番号を日本語文字列で登録する
5. 郵便番号が日本語文字列で登録できる

Way to fix (修正内容)

郵便番号が日本語文字列で登録できないように修正お願いします。

備考

入力した日本語文字列

ああああ

Related issues

Related to OpenPNE 3 - Backport(バックポート) #3853: プロフィール編集画面にて、郵便番号が日本語文字列で登録できる Fixed(完了) 2015-11-06
Related to OpenPNE 3 - Backport(バックポート) #3854: プロフィール編集画面にて、郵便番号が日本語文字列で登録できる Fixed(完了) 2015-11-06

History

#1 Updated by Yuya Watanabe about 8 years ago

実装案

diff --git a/lib/validator/opValidatorProfile.class.php b/lib/validator/opValidatorProfile.class.php
index 5609f03..abc76e2 100644
--- a/lib/validator/opValidatorProfile.class.php
+++ b/lib/validator/opValidatorProfile.class.php
@@ -59,6 +59,11 @@ class opValidatorProfile extends sfValidatorBase
       }
     }

+    if ('op_preset_postal_code' === $this->profile->getName() && !preg_match("/^\d{3}-\d{4}$/", $value['value']))
+    {
+        throw new sfValidatorError($this, 'not postal code');
+    }
+
     return $clean;
   }
 }

#2 Updated by Yuma Sakata almost 8 years ago

  • Description updated (diff)
  • 3.6 で発生するか set to Yes (はい)
  • 3.4 で発生するか set to Yes (はい)

再現確認できました。

Environment (再現バージョン)

OpenPNE3.4.18
OpenPNE3.6.1

Way to repro (再現手順)

1. 管理画面プロフィール項目登録ページ(/pc_backend.php/profile/edit)にアクセスする
2. 「プリセットから選ぶ」を選択して「郵便番号」を選択する
3. SNSログイン後、プロフィール編集ページ(/member/edit/profile)にアクセスする
4. 郵便番号を日本語文字列で登録する
5. 郵便番号が日本語文字列で登録できる

Way to fix (修正内容)

郵便番号が日本語文字列で登録できないように修正お願いします。

#3 Updated by Akihiro KOBAYASHI about 5 years ago

  • Assignee set to Akihiro KOBAYASHI
  • Target version set to OpenPNE 3.9.0-old
  • 3.8 で発生するか set to Yes (はい)

#4 Updated by Akihiro KOBAYASHI about 5 years ago

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

#5 Updated by Akihiro KOBAYASHI about 5 years ago

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

プルリクエスト: https://github.com/openpne/OpenPNE3/pull/231
原因:入力内容のバリデート時に日本語入力が加味されていないから
対策:入力内容が正しい形式以外で入力された場合(今回では日本語であったり、英字であったり)にエラーを返す

#6 Updated by kaoru n over 4 years ago

  • Assignee deleted (Akihiro KOBAYASHI)

#7 Updated by isao sano about 4 years ago

#8 Updated by isao sano about 4 years ago

#9 Updated by Shinichi Urabe almost 4 years ago

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

#10 Updated by kaoru n over 3 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)
  • Assignee changed from isao sano to kaoru n

https://github.com/openpne/OpenPNE3/pull/364
にてプルリクエストしました。
レビューをお願いします。

#11 Updated by Youichi Kimura over 3 years ago

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

opValidatorProfile::doClean() に正規表現を直接書くのではなく、 source:lib/config/config/preset_profile.yml@43190aa1#L161 にある postal_code の定義に正規表現パターンを記述して下さい。

#12 Updated by kaoru n over 3 years ago

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

#13 Updated by kaoru n over 3 years ago

https://github.com/openpne/OpenPNE3/pull/364
プルリクエスト更新しました。
レビューをお願いします。

#14 Updated by kaoru n over 3 years ago

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

#15 Updated by Youichi Kimura over 3 years ago

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

レビューOKです

#17 Updated by isao sano over 2 years ago

  • Status changed from Pending Testing(テスト待ち) to Won't fix(対応せず)
  • % Done changed from 70 to 0

OpenPNE 3.8.21 にて対応済みであったため、対応せずとします。

Also available in: Atom PDF