バージョン選択

フォーラム

メニュー

オンライン状況

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

サイト内検索

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

フロント機能

新規スレッドを追加する

スレッド表示 | 新しいものから 前のトピック | 次のトピック | 下へ
投稿者 スレッド
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
居住地:
投稿: 7365
Re: モバイルで会員パスワードの文字数制限チェックが行われない
ソース見てみましたが、スパゲッティ状態で、他にもバグがありそうですね・・・

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

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


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

seasoft
投稿日時: 2013/1/24 15:14
対応状況: −−−
登録日: 2008/6/4
居住地:
投稿: 7365
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
居住地:
投稿: 7365
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公式 Amazon Payプラグイン

統計情報

総メンバー数は88,308名です
総投稿数は109,692件です

投稿数ランキング

1
seasoft
7365
2
468
3217
3
AMUAMU
2712
4
nanasess
2303
5
umebius
2085
6
yuh
1818
7
h_tanaka
1610
8
red
1568
9
mcontact
1240
10
tsuji
958
11
fukap
907
12
shutta
835
13
tao_s
796
14 ramrun 789
15 karin 689
16 sumida 641
17
homan
633
18 DELIGHT 572
19
patapata
502
20
flealog
485


ネットショップの壺

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

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