Bug 465554 - software.channel.listLatestPackages returns id as Intetger not String
Summary: software.channel.listLatestPackages returns id as Intetger not String
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Satellite 5
Classification: Red Hat
Component: API
Version: 530
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Brad Buckingham
QA Contact: Jay Dobies
URL:
Whiteboard:
Depends On:
Blocks: 456996
TreeView+ depends on / blocked
 
Reported: 2008-10-03 20:43 UTC by Justin Sherrill
Modified: 2009-09-10 19:53 UTC (History)
2 users (show)

Fixed In Version: sat530
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-09-10 19:53:29 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Justin Sherrill 2008-10-03 20:43:28 UTC
the docs for software.channel.listLatestPackages says it returns 'id' as a string, but this is actually an int.

Comment 1 Brad Buckingham 2009-02-11 13:57:36 UTC
Updated with git commit: 7259087459fcc0663ede9e74182af929dfb94202

Comment 2 John Matthews 2009-02-17 14:55:44 UTC
Should the below be "latest" instead of "largest"?

"Lists the packages with the largest version (including release and epoch) for the given channel"





ISO: Satellite-5.3.0-RHEL5-re20090213.1-i386-embedded-oracle.iso

Method: listLatestPackages
Description:
Lists the packages with the largest version (including release and epoch) for the given channel

Parameters:

    * string sessionKey
    * string channelLabel - channel to query

Returns:

    * array:
          o struct - package
                + string "name"
                + string "version"
                + string "release"
                + string "epoch"
                + int "id"
                + string "arch_label"

Comment 3 Brad Buckingham 2009-02-17 16:27:52 UTC
Either 'largest' or 'latest' is correct; however, 'latest' would be the better word for the apidoc.  I can update that.  Unfortunately, I didnt' catch that in the original doc when fixing the bug reported. :(

Comment 4 John Matthews 2009-02-17 22:12:08 UTC
Moving this back to ASSIGNED to address the "/s/largest/latest/" suggestion.  

Note:  Other changes looked good on ISO: Satellite-5.3.0-RHEL5-re20090213.1-i386-embedded-oracle.iso, once "largest" is changed to "latest" it can be VERIFIED

Comment 5 Brad Buckingham 2009-02-18 15:34:14 UTC
git commit: 0bdda3aaa696dbe22884e5b8e6e939e1f9f51d7f

New doc:

Method: listLatestPackages
Description:
Lists the packages with the latest version (including release and epoch) for the given channel

Parameters:

    * string sessionKey
    * string channelLabel - channel to query

Returns:

    * array:
          o struct - package
                + string "name"
                + string "version"
                + string "release"
                + string "epoch"
                + int "id"
                + string "arch_label"

Comment 6 John Matthews 2009-02-25 01:09:09 UTC
Verified 
ISO: Satellite-5.3.0-RHEL5-re20090220.1-i386-embedded-oracle.iso

Method: listLatestPackages
Description:
Lists the packages with the latest version (including release and epoch) for the given channel

Parameters:

    * string sessionKey
    * string channelLabel - channel to query

Returns:

    * array:
          o struct - package
                + string "name"
                + string "version"
                + string "release"
                + string "epoch"
                + int "id"
                + string "arch_label"

Comment 7 Jay Dobies 2009-08-13 18:01:34 UTC
Verified on test1182.

Comment 8 Brandon Perkins 2009-09-10 19:53:29 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHEA-2009-1434.html


Note You need to log in before you can comment on or make changes to this bug.