バージョン選択

フォーラム

メニュー

オンライン状況

29 人のユーザが現在オンラインです。 (16 人のユーザが フォーラム を参照しています。)
登録ユーザ: 0
ゲスト: 29
もっと...

サイト内検索

バグ報告 > フロント機能 > モバイルで会員パスワードの文字数制限チェックが行われない

フロント機能

新規スレッドを追加する

スレッド表示 | 新しいものから 前のトピック | 次のトピック | 下へ
投稿者 スレッド
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で確認致しました。
seasoft
投稿日時: 2013/1/24 12:55
対応状況: −−−
登録日: 2008/6/4
居住地:
投稿: 7332
Re: モバイルで会員パスワードの文字数制限チェックが行われない
ソース見てみましたが、スパゲッティ状態で、他にもバグがありそうですね・・・

引用頂いた部分のソースは、本来はそもそも不要っぽいです。他のメソッドの IF 分岐に誤りがありそうです。

もう少し見てみようと思います。


----------------
Seasoft
こちらでの投稿は、アイディア程度に留めさせていただいております。
個別案件の作業は有償で承っております。お気軽にご相談ください。

seasoft
投稿日時: 2013/1/24 15:14
対応状況: −−−
登録日: 2008/6/4
居住地:
投稿: 7332
Re: モバイルで会員パスワードの文字数制限チェックが行われない
改めて確認しましたところ、SC_FormParam#checkError で対応できないチェックを行うため二重に記述するのはやむを得ないようですね。

しかし、SPTAB_CHECK が二重で行なわれていたり、NUM_RANGE_CHECK を行うにもかかわらず MAX_LENGTH_CHECK を行なっていたりと不可解な部分はありますね・・・

とりあえず、バグを含むのは明確なので、チケットを発行しました。
http://svn.ec-cube.net/open_trac/ticket/2053


----------------
Seasoft
こちらでの投稿は、アイディア程度に留めさせていただいております。
個別案件の作業は有償で承っております。お気軽にご相談ください。

seasoft
投稿日時: 2013/1/24 15:52
対応状況: −−−
登録日: 2008/6/4
居住地:
投稿: 7332
Re: モバイルで会員パスワードの文字数制限チェックが行われない
修正をコミットしてみました。
http://svn.ec-cube.net/open_trac/changeset/22340

他の改修と同時コミットとなり、分かりにくくなってしまい申し訳ございません。
本件の部分は、sfCustomerRegisterParam と sfCustomerRegisterErrorCheck 内となっています。


----------------
Seasoft
こちらでの投稿は、アイディア程度に留めさせていただいております。
個別案件の作業は有償で承っております。お気軽にご相談ください。

dayou
投稿日時: 2013/1/24 17:19
対応状況: 解決済
新米
登録日: 2009/8/6
居住地:
投稿: 7
Re: モバイルで会員パスワードの文字数制限チェックが行われない
seasoft様

早々のご対応有難うございます。
文字数制限チェックが行われるのを確認致しました。

またなにかあれば宜しくお願い致します。
スレッド表示 | 新しいものから 前のトピック | 次のトピック | トップ


 



ログイン


EC-CUBEペイメント

公式ストアEC-CUBE4系デザインテンプレート続々リリース中

統計情報

総メンバー数は66,746名です
総投稿数は98,437件です

投稿数ランキング

1
seasoft
7332
2
AMUAMU
2712
3
468
2651
4
nanasess
2107
5
umebius
1758
6
yuh
1612
7
red
1424
8
h_tanaka
1049
9
fukap
907
10
tsuji
863
11
shutta
835
12
tao_s
792
13 ramrun 789
14 karin 657
15 sumida 641
16
homan
633
17 DELIGHT 571
18
patapata
502
19
flealog
485
20 tonton 437


ネットショップの壺

EC-CUBEインテグレートパートナー

Copyright© EC-CUBE CO.,LTD. All Rights Reserved.