1.2.0 Request to change return value names (2 methods)
rkalla
Registered Users Posts: 108 Major grins
smugmug.images.changePosition, changeSettings and delete all return an argument named "Image" with a boolean value to indicate if the action completed correctly.
I was hoping we could change this to "result" or something else, because many other image operations return a struct "Image" with a single field "id", so it can be confusing that both of these have the same name but are infact different objects (JSON for example).
Also it seems strange to query a return result for the field "Image" to see if an operation completed or not.
Just my 2-cents and my O.C.D. nature :uhoh
I was hoping we could change this to "result" or something else, because many other image operations return a struct "Image" with a single field "id", so it can be confusing that both of these have the same name but are infact different objects (JSON for example).
Also it seems strange to query a return result for the field "Image" to see if an operation completed or not.
Just my 2-cents and my O.C.D. nature :uhoh
0
Comments
kallasoft | The "Break It Down" Blog