{"action":"created","issue":{"url":"https://api.github.com/repos/Art4/json-api-client/issues/15","labels_url":"https://api.github.com/repos/Art4/json-api-client/issues/15/labels{/name}","comments_url":"https://api.github.com/repos/Art4/json-api-client/issues/15/comments","events_url":"https://api.github.com/repos/Art4/json-api-client/issues/15/events","html_url":"https://github.com/Art4/json-api-client/issues/15","id":104642815,"number":15,"title":"Add Utils\\Helper::isValid() for easier validation","user":{"login":"Art4","id":2162994,"avatar_url":"https://avatars.githubusercontent.com/u/2162994?v=3","gravatar_id":"","url":"https://api.github.com/users/Art4","html_url":"https://github.com/Art4","followers_url":"https://api.github.com/users/Art4/followers","following_url":"https://api.github.com/users/Art4/following{/other_user}","gists_url":"https://api.github.com/users/Art4/gists{/gist_id}","starred_url":"https://api.github.com/users/Art4/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Art4/subscriptions","organizations_url":"https://api.github.com/users/Art4/orgs","repos_url":"https://api.github.com/users/Art4/repos","events_url":"https://api.github.com/users/Art4/events{/privacy}","received_events_url":"https://api.github.com/users/Art4/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/Art4/json-api-client/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-09-03T07:59:13Z","updated_at":"2015-11-03T16:11:12Z","closed_at":null,"body":"```php\r\n$wrong_jsonapi = '{\"data\":{},\"meta\":{\"info\":\"This is wrong JSON API. `data` has to be `null` or containing at least `type` and `id`.\"}}';\r\n\r\nif ( ! \\Art4\\JsonApiClient\\Utils\\Helper::isValid($wrong_jsonapi) )\r\n{\r\n    // string is not in valid jsonapi format.\r\n}\r\n```"},"comment":{"url":"https://api.github.com/repos/Art4/json-api-client/issues/comments/153400738","html_url":"https://github.com/Art4/json-api-client/issues/15#issuecomment-153400738","issue_url":"https://api.github.com/repos/Art4/json-api-client/issues/15","id":153400738,"user":{"login":"fcastillo-es","id":2611437,"avatar_url":"https://avatars.githubusercontent.com/u/2611437?v=3","gravatar_id":"","url":"https://api.github.com/users/fcastillo-es","html_url":"https://github.com/fcastillo-es","followers_url":"https://api.github.com/users/fcastillo-es/followers","following_url":"https://api.github.com/users/fcastillo-es/following{/other_user}","gists_url":"https://api.github.com/users/fcastillo-es/gists{/gist_id}","starred_url":"https://api.github.com/users/fcastillo-es/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fcastillo-es/subscriptions","organizations_url":"https://api.github.com/users/fcastillo-es/orgs","repos_url":"https://api.github.com/users/fcastillo-es/repos","events_url":"https://api.github.com/users/fcastillo-es/events{/privacy}","received_events_url":"https://api.github.com/users/fcastillo-es/received_events","type":"User","site_admin":false},"created_at":"2015-11-03T16:11:12Z","updated_at":"2015-11-03T16:11:12Z","body":"Hi, I've finally decided to go with a modified version of your client, mostly because adding fractal to get the output would add a second validation, which I would like to avoid.\r\n\r\nI'm doing some work extracting validators outside of the main classes. First just to create concrete Validator classes, then maybe add a Validator class similar to the Factory, with the list of classes for each resource. That way the manager could also manage the validator, and that way the code could support switching validations. For example, a client could use the request headers to load validators for a specific version of json-api.\r\n\r\nThis is the first commit in that line: https://github.com/fcastillo-es/json-api-client/commit/279fb0fb92e249c371a536584ede06432090d01b. As I think it's related with this issue please tell me if you would accept this kind of changes if I complete them and add unit testing. Else I would just carry on with my fork.\r\n\r\nThanks."}}