689862 - Only return the Classification name from Product.get, instead of all the classification info

Status: RESOLVED FIXED

(Bugzilla :: WebService, defect)

Description

[Max Kanat-Alexander]

In Product.get, milestones, versions, and components are returned as hashes along with the get response. This is because they are unique items that can only be accessed in this way. On the other hand, returning classification as hash seems like a waste of response size to me, because when you get it for multiple products the information will be duplicated over and over.

Instead, it would probably be best to simply return the classification name, and then later we can add a get_classification or something like that to Product.

Comment 1

[Frédéric Buclin]

The size doesn't seem a valid argument to me. One classification is negligible compared to tens of versions, milestones and components. IMHO, it doesn't hurt to keep the data in Product.get.

Comment 2

[Frédéric Buclin]

That being said, I'm fine with this change.

Comment 3

[Tiago Mello [:timello]]

Attachment: v1

Comment 4

[Frédéric Buclin]

Comment on attachment 565649 [details] [diff] [review]
v1

Looks good to me. r=LpSolit

Comment 5

[Tiago Mello [:timello]]

Committing to: bzr+ssh://[e-mail]/bugzilla/4.2/                                                                                                             
modified Bugzilla/WebService/Product.pm
Committed revision 7947.

Committing to: bzr+ssh://[e-mail]/bugzilla/trunk/                                                                                                           
modified Bugzilla/WebService/Product.pm
Committed revision 7985.