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

[Marta Girdea]

+1

On 9/21/07, Vincent Massol <vincent at massol.net> 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