[xwiki-devs] [VOTE] Modify XWiki.search() returned result + other methods returning a collection

[bernaulli]

I am not able to download xwiki. Erro Message is"ObjectWeb Forge Could Not
Connect to Database":

On 9/22/07, Vincent Massol <vincent at massol.net> wrote:
>
> Summary: 4 +1, no 0, no -1.
>
> Done.
>
> Thanks
> -Vincent
>
> On Sep 21, 2007, at 11:43 AM, Vincent Massol wrote:
>
> > Hi,
> >
> > Since this is an API I'm sending a vote.
> >
> > I'd like to modify the following:
> >
> >      public List search(String wheresql) throws XWikiException
> >      {
> >          if (hasProgrammingRights())
> >              return xwiki.search(wheresql, getXWikiContext());
> >          return null;
> >      }
> >
> > To:
> >
> >      public List search(String wheresql) throws XWikiException
> >      {
> >          if (hasProgrammingRights())
> >              return xwiki.search(wheresql, getXWikiContext());
> >          return Collections.EMPTY_LIST;
> >      }
> >
> > Rationale:
> >
> > * A method that returns a List (or a Collection in general) should
> > never return null but always an empty collection instead (or throw an
> > exception).
> >
> > I also propose to modify the other methods to return empty
> > collections instead of null. Those include:
> >
> > *     public List getDeletedDocuments(String fullname, String lang)
> > throws XWikiException
> > *     public List search(String wheresql, int nb, int start) throws
> > XWikiException
> > *     public List searchDocuments(String wheresql, int nb, int start,
> > String selectColumns)
> >
> > Here's my +1
> >
> > Thanks
> > -Vincent
> >
> > _______________________________________________
> > devs mailing list
> > devs at xwiki.org
> > http://lists.xwiki.org/mailman/listinfo/devs
>
> _______________________________________________
> devs mailing list
> devs at xwiki.org
> http://lists.xwiki.org/mailman/listinfo/devs
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.xwiki.org/pipermail/devs/attachments/20070922/60fd1eff/attachment.html