erunion / showtimes

[deprecated] a movie showtimes API
MIT License
52 stars 14 forks source link

Changed all false to null, and empty arrays to [] #38

Closed wes-nz closed 8 years ago

wes-nz commented 8 years ago

Hi,

I believe instead of returning false the JSON should return null for empty properties. I have also changed arrays from false to empty arrays [] Returning false changes the value type, this is an issue when parsing JSON with a strictly typed language.

http://jsonapi.org/format/

Happy to discuss. Cheers! 🎉

wes-nz commented 8 years ago

Also genre is a String when there is only one value, and an Array when multiple one.

gvf6xbmhovqra

It's now always an array

erunion commented 8 years ago

Awesome, thanks for this. Sorry for the delay! I've been uh... busy.