auto.jsの修正お願いします。
211～212行 is("checked") ----> is(':checked')
217～222行 is("checked") ----> is(':checked')
325～330行 is('checked','') ----> attr('checked',false)
331～336行 is('checked',true) ----> attr('checked',true)
338～339行 is('checked',false) ----> attr('checked',false)
341～342行 is('checked',true) ----> attr('checked',true)
2.4.1を見ました。
XSSはどの状況で発生ですか？ソースはどこを変更しました？
また都合のいいときに教えてください。
Re: Re: Re: Re: 機能拡張フォームのconcrete5-5.4.2への対応について
2011年8月28日 at 19:07
218～222行目のif文は戻り値が FALSE となりデータベースへの登録に影響がありましゅ。
325～330行目のattr('checked','')はチェックをはずす処理なのにチェックがはいります。
331～336行目のattr('checked',true)のへの変更は僕の好みです。すみません.
チェックボックス、ラジオボタンはtrue,falseで処理したほうが、コーディング間違いが減ると思います。ほとんどのエディタでtrue、falseは色を変えてくれるので気づきやすい。checkedを使用する場合chekedと打ち間違いをしても、実行エラーすらはいてくれない。''で囲むからよけいにですけど。
jQuery1.6以降はプロパティ参照にはattr()の変わりにprop()を使用することを推奨されているようですが、まだoncrete5-5.4.1.1を使用しているサイトも多いと思いますのでattrを使用したままの稼動が理想と考えます。
以上が説明ですが伝わるように書けていますでしょうか？文章能力の無さがたたっています。すみません。不明点やご指摘あれば言ってください。
tomoac
Re: Re: Re: Re: Re: 機能拡張フォームのconcrete5-5.4.2への対応について
ご指摘の修正をして、リリースしました。
5.4.2.1の日本語化がなされるようですので、それがリリースされたころまた機能アップをしてみます。