[Spacewalk-devel] Patch: add arch to channel.list*Channels

Justin Sherrill jsherril at redhat.com
Wed Apr 7 13:21:50 EDT 2010


On 4/6/10 9:55 AM, Colin Coe wrote:
> Hi all
> 
> As per the API Addition page, I've added 'arch' to the
> channel.list*Channels API calls.
> 
> Comments/criticisms welcome
> 
> CC
> 
> 
> 
> 
> _______________________________________________
> Spacewalk-devel mailing list
> Spacewalk-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/spacewalk-devel
Hey Colin,

Having it do the lookups of package Arch in the Serializer really isn't
the right way to do it as Serializers are just meant to translate the
existing data.

I would either add joins to rhnChannelArch to get the label in each of
the queries or write a elaborator that just gets the channel arch and
make each of those queries use the elaborator.

Thanks,

-Justin


-- 
Justin Sherrill, RHCA          1801 Varsity Drive.
Software Engineer                Raleigh, NC 27603
Red Hat, Inc.




More information about the Spacewalk-devel mailing list