[Yanel-dev] Re: [PATCH] Pagination of search results

[Michael Wechner]

Mehmet Birgi wrote:
> OK, I'm not sure if it's a bug or a feature. This was the problem:
>
> In the sample application that I am using for testing the patched
> search resource, the normal search field was not appending the request
> parameter "page" which is required if pagination parameters are
> defined in the resource configuration, thus causing an exception.

I have removed this exception and set it to currentPageNumber = 1 instead

Cheers

Michi
>  I
> have now added this parameter (with value=1) as a hidden input field
> to the search box, and it is working. I believe that this behaviour is
> OK, but I am not sure.
>
> For now, I will not change the patch, but if you feel that this should
> be changed, please comment, and I will implement it differently.
>
> Thanks,
>
> Memo
>
>
>
> On Wed, Jul 14, 2010 at 4:31 PM, Mehmet Birgi <mehmet.birgi at wyona.com> wrote:
>   
>> Sorry, there is still a bug in this patch that I have just found, I
>> will resubmit the patch after fixing this bug.
>>
>>
>>
>> On Wed, Jul 14, 2010 at 4:13 PM, Mehmet Birgi <mehmet.birgi at wyona.com> wrote:
>>     
>>> Hi All
>>>
>>> I am attaching to this posting a patch for
>>> /yanel-search-resource-type/src/java/org/wyona/yanel/impl/resources/search/SearchResource.java.
>>> The XML delivered by the patched resource will have additional content
>>> that can be used by XSLT for paginating search results. The changes
>>> should be completely backward compatible as the already existing
>>> content has not changed.
>>>
>>> Cheers,
>>>
>>> Memo
>>>
>>>
>>>
>>> --
>>> Mehmet Birgi
>>> www.wyona.com
>>>
>>>       
>>
>> --
>> Mehmet Birgi
>> www.wyona.com
>>
>>     
>
>
>
>