Bug(バグ) #2430
プロフィール編集画面にて、郵便番号が日本語文字列で登録できる
0%
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
History
#1
Updated by Yuya Watanabe over 11 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 about 11 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 over 8 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 over 8 years ago
- Status changed from New(新規) to Accepted(着手)
#5
Updated by Akihiro KOBAYASHI over 8 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 7 years ago
- Assignee deleted (
Akihiro KOBAYASHI)
#7
Updated by isao sano about 7 years ago
- Related to Backport(バックポート) #3853: プロフィール編集画面にて、郵便番号が日本語文字列で登録できる added
#8
Updated by isao sano about 7 years ago
- Related to Backport(バックポート) #3854: プロフィール編集画面にて、郵便番号が日本語文字列で登録できる added
#9
Updated by Shinichi Urabe almost 7 years ago
- Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
- Assignee set to isao sano
- 文字リテラルは シングルクオートで囲んでください コーディング規約違反 https://github.com/openpne/OpenPNE3-doc/blob/master/coding-standards/coding-standards.ja.rst#文字列リテラル
- sfValidatorRegex() を使わない意図は何でしょうか
#10
Updated by kaoru n over 6 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 6 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 6 years ago
- Status changed from Rejected(差し戻し) to Accepted(着手)
- % Done changed from 50 to 0
#13
Updated by kaoru n over 6 years ago
https://github.com/openpne/OpenPNE3/pull/364
プルリクエスト更新しました。
レビューをお願いします。
#14
Updated by kaoru n over 6 years ago
- Status changed from Accepted(着手) to Pending Review(レビュー待ち)
- % Done changed from 0 to 50
#15
Updated by Youichi Kimura over 6 years ago
- Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
- % Done changed from 50 to 70
レビューOKです
#17
Updated by isao sano almost 6 years ago
- Status changed from Pending Testing(テスト待ち) to Won't fix(対応せず)
- % Done changed from 70 to 0
OpenPNE 3.8.21 にて対応済みであったため、対応せずとします。