Skip to content

Added data model validation #503

Closed
wants to merge 1 commit into from

2 participants

@kstenschke

Added data model validation to prevent crashing when customer EAV config is faulty

@kstenschke kstenschke Update README.md - added data model validation
Added data model validation to prevent crashing when customer EAV config is faulty
7f5f4c6
@verklov verklov self-assigned this Feb 14, 2014
@verklov
verklov commented Feb 14, 2014

@kstenschke , thank you for your contribution! The team will look at it and we will update you once the analysis is complete.

@verklov
verklov commented Feb 17, 2014

@kstenschke , the team is reviewing your contribution and asked for additional information. Could you please extend a bit on your understanding of "faulty config"? Thank you!

@verklov
verklov commented Apr 25, 2014

@kstenschke, unfortunately we cannot accept your pull request, because there is no condition to get $dataModel as non-object.

We are closing this pull request.

@verklov verklov closed this Apr 25, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.