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.

Sign up for GitHub

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

Jump to bottom

Getting non-related error when trying to make new entity because of bad data #6562

Closed
Spriz opened this issue May 14, 2015 · 5 comments
Closed

Getting non-related error when trying to make new entity because of bad data #6562

Spriz opened this issue May 14, 2015 · 5 comments
Assignees
@markstory
Labels
defect
Milestone
3.0.5

Comments

@Spriz
Copy link
Contributor

Spriz commented May 14, 2015

Hi!

When trying to put the following data:

https://gist.github.com/Spriz/3d76313473f180acc424

into a new Entity it dies with the error: https://gist.github.com/Spriz/50270a74888abe7485d4

I guess it should give a more descriptive message - at least IRC ( @lorenzo ) sent me here :-)
@Spriz
Copy link
Contributor Author

Spriz commented May 14, 2015

... weird errors whether I include ['associated' => ['Industires']] in the newEntity() happened...

@lorenzo lorenzo added this to the 3.0.5 milestone May 14, 2015
@lorenzo lorenzo added the defect label May 14, 2015
@markstory
Copy link
Member

Is the problematic key industries or contact_person?

@markstory markstory self-assigned this May 15, 2015
@Spriz
Copy link
Contributor Author

Spriz commented May 15, 2015

industries @markstory

@markstory
Copy link
Member

Thanks, I will take a look into this.

markstory added a commit that referenced this issue May 16, 2015
@markstory
Skip scalar data in belongsToMany relations.
95a0fb6 
Scalar data shouldn't get converted into an entity, as it is likely to
be a mistake/naughty user. Add coverage for other scalar values in both
hasMany and belongsToMany associations.

Refs #6562
@markstory markstory mentioned this issue May 16, 2015
Merged
@markstory
Copy link
Member

Pull request up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@markstory markstory

Labels
defect
Projects
None yet
Milestone
3.0.5
Development

No branches or pull requests
3 participants
@markstory @lorenzo @Spriz