undefined method `delete' for true:TrueClass when executing destroy about her HOT 6 CLOSED

What is the response sent back from the API? The only way you could get this error is if the response body would be true:

json = MultiJson.load('true')
json.delete(:errors) # raises NoMethodError: undefined method `delete' for true:TrueClass

I actually wanted to implement some kind of check for this over the weekend to make sure this error doesn’t happen because I thought that only objects or arrays could be sent back from MultiJson.load. But it looks like simple values like true or "foo" are actually valid JSON data.

Anyway, making the endpoint called by #destroy return something like {} for now should do the trick for now. But I’ll come up with something better soon.

from her.

FWIW, I just pushed a commit in master (30584da) that should provide a better error message when encountering this problem. Will be part of the next release.

from her.

You're right, the response is true

Is there a best practice how such a response message should look like?

from her.

By default, if the server returns an empty hash ({} or { "model_name": {} }, depending on whether you’re using the parse_root_in_json setting), it will keep the model attributes intact and just mark it as destroyed (calling #destroyed? on it will return true).

In my opinion, a good practice would be to return an empty response and send it using the 204 No Content HTTP header. Her will behave the same way as described above.

from her.

I’m going to close this issue now. Please let me know if you have further questions!

from her.

Sorry, but I haven't got time yet to test it. But the case is clear to me and I will change the response messages. Thanks for your work and help!

from her.