[xwiki-dev] [Lucene] Fixing the rebuildIndex API?

[Ludovic Dubost]

+1

I started rewriting the whole API leaving the older ones in 
LucenePluginAPI as deprecated but I have to redo it all because of your 
styling commit :(
I'm in the process of adding the sort parameter for lucene which is 
needed on curriki

http://jira.xwiki.org/jira/browse/XWIKI-795

Ludovic

Vincent Massol a écrit :
> Hi everyone,
>
> I can see the following API in LucenePluginAPI:
>
>     public int rebuildIndex(XWiki wiki, Context context)
>
> I can see 2 problems with that:
>
> 1) I don't see the need for exposing the context, especially as it's 
> known internally by the plugin.
> 2) Passing both the context and the wiki object seems redundant as 
> it's possible to call context.getXWiki()
>
> I'm thus proposing to deprecate this method and instead replace it with:
>
>     public int rebuildIndex()
>
> Ok for everyone?
>
> Thanks
> -Vincent
>
>
>     
>
>     
>        
> ___________________________________________________________________________Yahoo! 
> Mail réinvente le mail ! Découvrez le nouveau Yahoo! Mail et son 
> interface révolutionnaire.
> http://fr.mail.yahoo.com
>
> ------------------------------------------------------------------------
>
>
> --
> You receive this message as a subscriber of the xwiki-dev at objectweb.org mailing list.
> To unsubscribe: mailto:xwiki-dev-unsubscribe at objectweb.org
> For general help: mailto:sympa at objectweb.org?subject=help
> ObjectWeb mailing lists service home page: http://www.objectweb.org/wws
>   


-- 
Ludovic Dubost
Blog: http://www.ludovic.org/blog/
XWiki: http://www.xwiki.com
Skype: ldubost GTalk: ldubost 
AIM: nvludo Yahoo: ludovic