[Yanel-dev] Deprecated methods used in search resource

[Michael Wechner]

Mehmet Birgi wrote:
> On Mon, Jul 12, 2010 at 3:08 PM, Michael Wechner
> <michael.wechner at wyona.com> wrote:
>   
>> Mehmet Birgi wrote:
>>     
>>> On Mon, Jul 12, 2010 at 2:25 PM, Michael Wechner
>>> <michael.wechner at wyona.com> wrote:
>>>
>>>       
>>>> Mehmet Birgi wrote:
>>>>
>>>>         
>>>>> Hi
>>>>>
>>>>> I am currently working on a patch for the search resource, to enable
>>>>> paging through search results. While doing this, I have come accross
>>>>> some uses of deprecated methods, e.g.
>>>>> org.wyona.yanel.core.Resource.getRequest() and
>>>>> org.wyona.yanel.core.Resource.getRequest() in
>>>>>
>>>>>
>>>>>           
>>>> you mean replace by getEnvironment().getRequest() ?
>>>>
>>>> If yes, then please yes :-)
>>>>
>>>>         
>>> That's exactly what I meant :-)
>>>
>>> So I'll quickly do it. Shall I also commit it right away, or might
>>> there be a risk of breaking anything?
>>>
>>>       
>> if you commit it independently of your other planned changes re pagination,
>> then it should be fine,
>> whereas I would be glad to review the pagination changes first.
>>     
>
> I know, and I have already started these, so the commits will have to
> wait. I'll submit a patch when I am done, and the refactorings will be
> included in this.
>   

thanks, looking forward to it :-)

Thanks

Michi
> cheers,
>
> memo
>
>
>   
>> Thanks
>>
>> Michi
>>     
>>>       
>>>> Thanks
>>>>
>>>> MIchi
>>>>
>>>>         
>>>>> /yanel-search-resource-type/src/java/org/wyona/yanel/impl/resources/search/SearchResource.java.
>>>>> Should I try to replace those with the newer methods, while I am at
>>>>> it?
>>>>>
>>>>> Cheers,
>>>>>
>>>>> Memo
>>>>>
>>>>>
>>>>>
>>>>>
>>>>>           
>>>> --
>>>> Yanel-development mailing list Yanel-development at wyona.com
>>>> http://lists.wyona.org/cgi-bin/mailman/listinfo/yanel-development
>>>>
>>>>
>>>>         
>>>
>>>
>>>       
>> --
>> Yanel-development mailing list Yanel-development at wyona.com
>> http://lists.wyona.org/cgi-bin/mailman/listinfo/yanel-development
>>
>>     
>
>
>
>