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

[hirokazu tayama]

加藤様

お世話になっております。田山です。

SNAPSHOTにて、正常にexcludesされることを確認しました。
迅速なご対応ありがとうございました。


2008/11/10 18:10 Junichi Kato <[E-MAIL ADDRESS DELETED]>:
> 田山さん、
> 加藤です。
>
> ご指摘の不具合を修正してみました。
> 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/
>  */
> _______________________________________________
> Seasar-user mailing list
> [E-MAIL ADDRESS DELETED]
> https://ml.seasar.org/mailman/listinfo/seasar-user
>