Bug（バグ） #2430
closed
プロフィール編集画面にて、郵便番号が日本語文字列で登録できる
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 (修正内容)¶
郵便番号が日本語文字列で登録できないように修正お願いします。
備考¶
入力した日本語文字列
ああああ
Updated by Yuya Watanabe over 13 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;
}
}
Updated by Yuma Sakata about 13 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 (修正内容)¶
郵便番号が日本語文字列で登録できないように修正お願いします。
Updated by Akihiro KOBAYASHI about 10 years ago
- Assignee set to Akihiro KOBAYASHI
- Target version set to OpenPNE 3.9.0-old
- 3.8 で発生するか set to Yes (はい)
Updated by Akihiro KOBAYASHI about 10 years ago
- Status changed from New（新規） to Accepted（着手）
Updated by Akihiro KOBAYASHI about 10 years ago
- Status changed from Accepted（着手） to Pending Review（レビュー待ち）
- % Done changed from 0 to 50
プルリクエスト： https://github.com/openpne/OpenPNE3/pull/231
原因：入力内容のバリデート時に日本語入力が加味されていないから
対策：入力内容が正しい形式以外で入力された場合(今回では日本語であったり、英字であったり)にエラーを返す
Updated by isao sano about 9 years ago
- Related to Backport（バックポート） #3853: プロフィール編集画面にて、郵便番号が日本語文字列で登録できる added
Updated by isao sano about 9 years ago
- Related to Backport（バックポート） #3854: プロフィール編集画面にて、郵便番号が日本語文字列で登録できる added
Updated by Shinichi Urabe almost 9 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() を使わない意図は何でしょうか
Updated by kaoru n over 8 years ago
- Status changed from Rejected（差し戻し） to Pending Review（レビュー待ち）
- Assignee changed from isao sano to kaoru n
https://github.com/openpne/OpenPNE3/pull/364
にてプルリクエストしました。
レビューをお願いします。
Updated by Youichi Kimura over 8 years ago
- Status changed from Pending Review（レビュー待ち） to Rejected（差し戻し）
opValidatorProfile::doClean() に正規表現を直接書くのではなく、 source:lib/config/config/preset_profile.yml@43190aa1#L161 にある postal_code の定義に正規表現パターンを記述して下さい。
Updated by kaoru n over 8 years ago
- Status changed from Rejected（差し戻し） to Accepted（着手）
- % Done changed from 50 to 0
Updated by kaoru n over 8 years ago
https://github.com/openpne/OpenPNE3/pull/364
プルリクエスト更新しました。
レビューをお願いします。
Updated by kaoru n over 8 years ago
- Status changed from Accepted（着手） to Pending Review（レビュー待ち）
- % Done changed from 0 to 50
Updated by Youichi Kimura over 8 years ago
- Status changed from Pending Review（レビュー待ち） to Pending Testing（テスト待ち）
- % Done changed from 50 to 70
レビューOKです
Updated by isao sano almost 8 years ago
- Status changed from Pending Testing（テスト待ち） to Won't fix（対応せず）
- % Done changed from 70 to 0
OpenPNE 3.8.21 にて対応済みであったため、対応せずとします。