refs #6463 - hammer was unable to update the org's desc, BZ 1114136

[komidore64]

No description provided.

[daviddavis]

I know you're just fixing a bug but the api doc is very weird for this action. The update action has description and organization[name] as params. I'm not quite sure what the solution is but I think that users will be very confused by this.

[komidore64]

@daviddavis i agree, but this is the api that we shipped with v2, so i'm not sure how much of it we can change (without moving to a v3).

for instance, i would consider moving a param from a nested location to an unnested location to be a breaking change.

[jlsherrill]

This was never working correct?

If not, then there really isn't a worry about 'breaking' updating of description. The doc can be wrong as well as the code, to me in this case both are wrong and can be fixed. If the user previously could update the description with just passing 'description' I'd probably disagree.

there may be other implications I'm not seeing, so feel free to point those out :)

[komidore64]

@jlsherrill good point. okay. i fixed the docs instead of making the code work for incorrect docs.

[komidore64]

@jlsherrill @daviddavis plx2review

[jlsherrill]

ACK from me, @daviddavis ?

[daviddavis]

ACK