バグ報告 > フロント機能 > モバイルで会員パスワードの文字数制限チェックが行われない |
フロント機能
スレッド表示 | 古いものから | 前のトピック | 次のトピック | 下へ |
投稿者 | スレッド |
---|---|
dayou |
投稿日時: 2013/1/24 17:19
対応状況: 解決済
|
新米 登録日: 2009/8/6 居住地: 投稿: 7 |
Re: モバイルで会員パスワードの文字数制限チェックが行われない seasoft様
早々のご対応有難うございます。 文字数制限チェックが行われるのを確認致しました。 またなにかあれば宜しくお願い致します。 |
seasoft |
投稿日時: 2013/1/24 15:52
対応状況: −−−
|
神 登録日: 2008/6/4 居住地: 投稿: 7367 |
Re: モバイルで会員パスワードの文字数制限チェックが行われない 修正をコミットしてみました。
http://svn.ec-cube.net/open_trac/changeset/22340 他の改修と同時コミットとなり、分かりにくくなってしまい申し訳ございません。 本件の部分は、sfCustomerRegisterParam と sfCustomerRegisterErrorCheck 内となっています。
|
seasoft |
投稿日時: 2013/1/24 15:14
対応状況: −−−
|
神 登録日: 2008/6/4 居住地: 投稿: 7367 |
Re: モバイルで会員パスワードの文字数制限チェックが行われない 改めて確認しましたところ、SC_FormParam#checkError で対応できないチェックを行うため二重に記述するのはやむを得ないようですね。
しかし、SPTAB_CHECK が二重で行なわれていたり、NUM_RANGE_CHECK を行うにもかかわらず MAX_LENGTH_CHECK を行なっていたりと不可解な部分はありますね・・・ とりあえず、バグを含むのは明確なので、チケットを発行しました。 http://svn.ec-cube.net/open_trac/ticket/2053
|
seasoft |
投稿日時: 2013/1/24 12:55
対応状況: −−−
|
神 登録日: 2008/6/4 居住地: 投稿: 7367 |
Re: モバイルで会員パスワードの文字数制限チェックが行われない ソース見てみましたが、スパゲッティ状態で、他にもバグがありそうですね・・・
引用頂いた部分のソースは、本来はそもそも不要っぽいです。他のメソッドの IF 分岐に誤りがありそうです。 もう少し見てみようと思います。
|
dayou |
投稿日時: 2013/1/24 12:27
対応状況: −−−
|
新米 登録日: 2009/8/6 居住地: 投稿: 7 |
モバイルで会員パスワードの文字数制限チェックが行われない パスワードの文字数制限を変更した時に気付いてのですが、
モバイルサイトでは会員登録・変更時にパスワードの文字数制限のチェックが行われていませんでした。 下記のチケットで修正された内容です。 http://svn.ec-cube.net/open_trac/ticket/1598 また、会員登録・変更時のパスワードの最大文字数チェックで、 最大文字数が STEXT_LEN となっているので PASSWORD_MAX_LEN の変更に対応できないと思います。 SC_Helper_Customerクラス --------- /** * 会員登録共通 * * @param SC_FormParam $objFormParam SC_FormParam インスタンス * @param boolean $isAdmin true:管理者画面 false:会員向け * @param boolean $is_mypage マイページの場合 true * @return void */ function sfCustomerRegisterParam(&$objFormParam, $isAdmin = false, $is_mypage = false) { $objFormParam->addParam('パスワード', 'password', STEXT_LEN, 'a', array('EXIST_CHECK', 'SPTAB_CHECK', 'ALNUM_CHECK', 'MAX_LENGTH_CHECK')); --- ver2.12.2で確認致しました。 |
スレッド表示 | 古いものから | 前のトピック | 次のトピック | トップ |