dailymotion / dailymotion-sdk-js

Dailymotion JavaScript client API
https://developers.dailymotion.com/api/platform-api/
MIT License
34 stars 33 forks source link

[API] fix regression in SDK sending empty fields param when unspecified #14

Closed happypoulp closed 10 years ago

happypoulp commented 10 years ago

Correct behavior: 1) When calling API without a "fields" param, results in a response containing defaults fields. 2) When calling API with an empty "fields" param, results in a response not containing any fields.

Regression:

When not specifying "fields" param to call API through SDK (like case 1) described above), the call issued now contains an empty fields param (instead of the absence of one). This results in the response not containing any fields.

NykO18 commented 10 years ago

Thank you François!