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 8 years ago
Updated by 実装案¶
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 over 8 years ago
Updated by - 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 5 years ago
Updated by - 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 5 years ago
- Status changed from New（新規） to Accepted（着手）
#5
Updated by Akihiro KOBAYASHI over 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 almost 5 years ago
Updated by - Assignee deleted (
Akihiro KOBAYASHI)
#7
Updated by isao sano over 4 years ago
Updated by - Related to Backport（バックポート） #3853: プロフィール編集画面にて、郵便番号が日本語文字列で登録できる added
#8
Updated by isao sano over 4 years ago
- Related to Backport（バックポート） #3854: プロフィール編集画面にて、郵便番号が日本語文字列で登録できる added
#9
Updated by Shinichi Urabe about 4 years ago
Updated by - 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 almost 4 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 almost 4 years ago
Updated by - 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 almost 4 years ago
- Status changed from Rejected（差し戻し） to Accepted（着手）
- % Done changed from 50 to 0
#13
Updated by kaoru n almost 4 years ago
https://github.com/openpne/OpenPNE3/pull/364
プルリクエスト更新しました。
レビューをお願いします。
#14
Updated by kaoru n almost 4 years ago
- Status changed from Accepted（着手） to Pending Review（レビュー待ち）
- % Done changed from 0 to 50
#15
Updated by Youichi Kimura almost 4 years ago
- Status changed from Pending Review（レビュー待ち） to Pending Testing（テスト待ち）
- % Done changed from 50 to 70
レビューOKです
#17
Updated by isao sano about 3 years ago
- Status changed from Pending Testing（テスト待ち） to Won't fix（対応せず）
- % Done changed from 70 to 0
OpenPNE 3.8.21 にて対応済みであったため、対応せずとします。