remote function get_distros vs. find_distro

[Vreman, Peter]

What shall I do with the remote XMLRPC functions for retrieving items with optional filter
criteria and paging?

I prefer to only have a
get_distros(criteria={},expand=False,page=None,results_per_page=None) that handles
everything. The find_distro() can then be removed or reimplemented to find a single
distribution based on a name.

Any comments?

Regards,
Peter




This e-mail and any attachment is for authorised use by the intended recipient(s) only. It
may contain proprietary material, confidential information and/or be subject to legal
privilege. It should not be copied, disclosed to, retained or used by, any other party. If
you are not an intended recipient then please promptly delete this e-mail and any
attachment and all copies and inform the sender. Thank you.

[Michael DeHaan]

Vreman, Peter - Acision wrote:
>  What shall I do with the remote XMLRPC functions for retrieving items 
>  with optional filter criteria and paging?
>
>   
>
>  I prefer to only have a 
>  get_distros(criteria={},expand=False,page=None,results_per_page=None) 
>  that handles everything. The find_distro() can then be removed or 
>  reimplemented to find a single distribution based on a name.
>
>   
>
>  Any comments?
>
>   
>
>  Regards,
>
>  Peter
>
>   
>  
Whatever we do, we can't break the API contract.   Adding additional 
parameters with default values is acceptable, removing methods is 
something we can't do.

Implementing one function with another is perfectly ok.

I'm ok with cleaning things up like you say above so that all functions 
have paging capability.  That would be nice.

However what each function returns and so forth ... we need to be very 
careful that it stays the same.   Internal implementations don't matter 
so much.

>  This e-mail and any attachment is for authorised use by the intended 
>  recipient(s) only. It may contain proprietary material, confidential 
>  information and/or be subject to legal privilege. It should not be 
>  copied, disclosed to, retained or used by, any other party. If you are 
>  not an intended recipient then please promptly delete this e-mail and 
>  any attachment and all copies and inform the sender. Thank you.
>  ------------------------------------------------------------------------
>
>  _______________________________________________
>  cobbler-devel mailing list
>  cobbler-devel(a)lists.fedorahosted.org
>  https://fedorahosted.org/mailman/listinfo/cobbler-devel
>