Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Only call prepareRow if $row is an array. #456

Conversation

chasingmaxwell
Copy link

I'm not sure if this is the right way to handle this, but I thought I'd put something out there to get the ball rolling on a solution. This fixes #331.

@amitaibu
Copy link
Member

amitaibu commented Apr 3, 2015

RestfulFormatterHalJson isn't properly tested. @chasingmaxwell would you be up for adding some test coverage so we stop breaking it over and over again? :)

@e0ipso
Copy link
Member

e0ipso commented Apr 14, 2015

@amitaibu, @chasingmaxwell merging this and creating a follow up to add testing.

@e0ipso e0ipso closed this Apr 14, 2015
e0ipso pushed a commit to e0ipso/restful that referenced this pull request Apr 14, 2015
commit d569551
Author: Peter Sieg <[email protected]>
Date:   Wed Apr 1 16:44:04 2015 -0400

    Only call prepareRow if $row is an array.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants