[CUBBY-62] FileRegexpValidatorとRegexpValidatorでマッチフラグ対応 Created: 11/Jan/08  Updated: 12/Jan/08  Resolved: 12/Jan/08

Status: Closed
Project: Cubby
Component/s: Core, Examples
Affects Version/s: 0.9.2
Fix Version/s: 1.0.0-RC1

Type: Improvement Priority: Major
Reporter: jfut Assignee: agata
Resolution: Fixed Votes: 0
Labels: None


 Description   

FileUploadActionの例にある

add("file", new FileRegexpValidator(".+\\.(png|jpg)"));

だと hoge.PNG や hoge.JPG など小文字以外が通らないので、Examplesとしては、

add("file", new FileRegexpValidator(".+\\.(?i)(png|jpg)"));
add("file", new FileRegexpValidator(".+\\.(png|PNG|jpg|JPG)"));

などとしておいた方がサンプルコードを見た方が何が出来て、何が出来ないか判り易くて良いかなと思います。

関連して、FileRegexpValidatorとRegexpValidatorでマッチフラグを引数に取れるようにするのも良いと思いますがどうでしょう?

public FileRegexpValidator(final String regex);
public FileRegexpValidator(final String regex, final int flags);
public FileRegexpValidator(final String regex, final String messageKey);
public FileRegexpValidator(final String regex, final int flags, final String messageKey);
add("file", new FileRegexpValidator(".+\\.(png|jpg)"), Pattern.CASE_INSENSITIVE);


 Comments   
Comment by agata [ 11/Jan/08 ]

いいですね。コメントやサンプルコードへの追加賛成です。
Cubbyではぜひこういう細かいところでサポートしていきたいなと思っています。

マッチフラグはちょっとリッチすぎるかなーという気もしました。
コンストラクタでPatternオブジェクトを渡せるようにするとかでもいいかも。

Comment by jfut [ 12/Jan/08 ]

コンストラクタでPatternオブジェクトを渡せるようにするとかでもいいかも。

こちらの方が良さそうですね。こちらでお願いします。

Comment by agata [ 12/Jan/08 ]

r633で対応しました。

Comment by jfut [ 12/Jan/08 ]

ありがとうございまーす。

Generated at Sun May 26 00:14:21 JST 2019 using JIRA 7.9.2#79002-sha1:3bb15b68ecd99a30eb364c4c1a393359bcad6278.