[Seasar-user:16268] Re: [S2BeanUtils]CharSequence型指定時にexcludesメソッドが利かない

[Junichi Kato]

田山さん、
加藤です。

ご指摘の不具合を修正してみました。
SNAPSHOTを発行しましたので、お手数ですがご確認をいただいてもよろしいでしょうか？
よろしくお願いいたします。

http://maven.seasar.org/maven2-snapshot/org/seasar/container/s2-tiger/2.4.33-SNAPSHOT/s2-tiger-2.4.33-20081110.090525-2.jar

# s2-tigerだけで大丈夫だと思いますが、念のため以下も用意しました。
http://maven.seasar.org/maven2-snapshot/org/seasar/container/s2-framework/2.4.33-SNAPSHOT/s2-framework-2.4.33-20081110.090149-3.jar
http://maven.seasar.org/maven2-snapshot/org/seasar/container/s2-extension/2.4.33-SNAPSHOT/s2-extension-2.4.33-20081110.090149-2.jar


2008/11/10 15:38 Junichi Kato <[E-MAIL ADDRESS DELETED]>:
> 田山さん、
> 加藤です。
>
> ご報告ありがとうございます。
> 修正を試みますので、しばらくお時間をいただきたいと思います。
> よろしくお願いします。
>
> 2008/11/10 14:12 hirokazu tayama <[E-MAIL ADDRESS DELETED]>:
>> いつもお世話になっております。田山と申します。
>>
>> Seasar2 2.4.31よりBeansのexcludesの引数に
>> CharSequence型を指定できるようになったと思います。
>>
>> https://www.seasar.org/issues/browse/CONTAINER-294
>>
>> こちらですが、String以外のCharSequence型指定時、
>> excludesが正常に機能していないように思います。
>>
>> public class Hoge {
>>        public String prop;
>> }
>>
>> public class Fuga {
>>        public String prop;
>> }
>>
>> public class Prop implements CharSequence{
>>        :
>>        @Override
>>        public String toString(){
>>                return "prop";
>>        }
>>        :
>> }
>>
>> Hoge hoge = new Hoge();
>> hoge.prop = "hogeProp";
>> Fuga fuga = new Fuga();
>> fuga.prop = "fugaProp";
>>
>> Beans.copy(hoge, fuga).excludes(new Prop()).execute();
>> System.out.println(fuga.prop); //hogeProp
>>
>> コードを追ってみましたが、AbstractCopy#isTargetPropertyにて
>> 引数のname（String型）とexcludePropertyNames（CharSequence型）
>> をequalsで比較しているのが原因かと思いました。
>> （恐らく、includesも同様？）
>>
>> 以上、ご確認よろしくお願いいたします。
>> _______________________________________________
>> Seasar-user mailing list
>> [E-MAIL ADDRESS DELETED]
>> https://ml.seasar.org/mailman/listinfo/seasar-user
>>
>
>
>
> --
> /**
>  * name  : Junichi Kato
>  * email : [E-MAIL ADDRESS DELETED]
>  *
>  * blog  : http://d.hatena.ne.jp/j5ik2o/
>  *
>  * commiting projects :
>  * http://jiemamy.org/
>  * http://s2chronos.sandbox.seasar.org/
>  * http://s2config.sandbox.seasar.org/
>  */
>



-- 
/**
 * name  : Junichi Kato
 * email : [E-MAIL ADDRESS DELETED]
 *
 * blog  : http://d.hatena.ne.jp/j5ik2o/
 *
 * commiting projects :
 * http://jiemamy.org/
 * http://s2chronos.sandbox.seasar.org/
 * http://s2config.sandbox.seasar.org/
 */