[Yanel-dev] [PATCH] Pagination of search results
Michael Wechner
michael.wechner at wyona.com
Thu Jul 15 10:29:59 CEST 2010
Mehmet Birgi wrote:
> Yes, you are right in all points. The fact is that I used a customer
> project for testing, so I couldn't include these files in this list,
> but the way you suggest is far better, I just didn't think of it. I
> can still do it, if you like, or haven't done it already.
>
it's fine for the moment, next time ;-)
Thanks
Michi
>
>
> On Thu, Jul 15, 2010 at 9:29 AM, Michael Wechner
> <michael.wechner at wyona.com> wrote:
>
>> Mehmet Birgi 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.
>>>
>>>
>> thanks very much for this patch. I am currently reviewing it. Just a few
>> notes:
>>
>> - it would be nice if you could create a patch with the full path in the
>> future, because it's easier to apply, it makes
>> sure to be unique and allows to contains several changed files
>>
>> - in order to simplify review it would be great to also have the files to
>> actually test it and containing documentation, e.g.
>> - src/contributions/resources/search/resource.xml
>> -
>> src/realms/from-scratch-realm-template/res-configs/en/search.html.yanel-rc
>> - src/contributions/resources/search/htdocs/results2xhtml.xsl
>>
>> - and a refactored test:
>> - http://127.0.0.1:8080/yanel/test/use-cases/#search-with-yarep
>> -
>> http://127.0.0.1:8080/yanel/test/use-cases/yanel/search.html?q=Hello&provider=yanel
>>
>> Having said this any patch/improvement is of course very welcome :-)
>>
>> Thanks
>>
>> Michi
>>
>>> Cheers,
>>>
>>> Memo
>>>
>>>
>>>
>>>
>>>
>> --
>> Yanel-development mailing list Yanel-development at wyona.com
>> http://lists.wyona.org/cgi-bin/mailman/listinfo/yanel-development
>>
>>
>
>
>
>
More information about the Yanel-development
mailing list