Re: [PATCH 2/7] flickr: Implement search all elements



On Thu, 2011-02-03 at 07:22 +0000, Iago Toral wrote:
> On Wed,  2 Feb 2011 21:26:36 +0100, "Juan A. Suarez Romero" 
>  <jasuarez igalia com> wrote:
> > In case of public Flickr source, as search all is not supported, 
> > search
> > all will return the latest photos uploaded to Flickr.
> 
>  Is this consistent? If NULL text search is supposed to return *all 
>  available media*, then shouldn't it do just that?
>  If it is possible for a user to browse/search media items from the 
>  flickr plugin that are not returned with this implementation for 
>  search(NULL) then we are basically changing the semantics of the 
>  search(NULL) for this plugin and I am not sure that is a good idea...


Yes, it is consistent.

"return latest photos uploaded to Flickr" means that the operation will
return the photos sorted by upload time.

If you would bee able to do a search with count==infinitum, it would
return all content from Flickr, sorted by upload time.

Flickr service decided to get rid of search(NULL) and move to
getRecent(), because at the end both operations will return all
elements, but in different order. So keeping one of them was enough for
them.

	J.A.




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]