{"action":"created","issue":{"url":"https://api.github.com/repos/ramsey/uuid/issues/99","repository_url":"https://api.github.com/repos/ramsey/uuid","labels_url":"https://api.github.com/repos/ramsey/uuid/issues/99/labels{/name}","comments_url":"https://api.github.com/repos/ramsey/uuid/issues/99/comments","events_url":"https://api.github.com/repos/ramsey/uuid/issues/99/events","html_url":"https://github.com/ramsey/uuid/pull/99","id":131834079,"number":99,"title":"Migrated isValid to the UuidFactoryInterface","user":{"login":"bradynpoulsen","id":5748275,"avatar_url":"https://avatars.githubusercontent.com/u/5748275?v=3","gravatar_id":"","url":"https://api.github.com/users/bradynpoulsen","html_url":"https://github.com/bradynpoulsen","followers_url":"https://api.github.com/users/bradynpoulsen/followers","following_url":"https://api.github.com/users/bradynpoulsen/following{/other_user}","gists_url":"https://api.github.com/users/bradynpoulsen/gists{/gist_id}","starred_url":"https://api.github.com/users/bradynpoulsen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bradynpoulsen/subscriptions","organizations_url":"https://api.github.com/users/bradynpoulsen/orgs","repos_url":"https://api.github.com/users/bradynpoulsen/repos","events_url":"https://api.github.com/users/bradynpoulsen/events{/privacy}","received_events_url":"https://api.github.com/users/bradynpoulsen/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2016-02-06T08:05:08Z","updated_at":"2016-03-27T18:13:41Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ramsey/uuid/pulls/99","html_url":"https://github.com/ramsey/uuid/pull/99","diff_url":"https://github.com/ramsey/uuid/pull/99.diff","patch_url":"https://github.com/ramsey/uuid/pull/99.patch"},"body":"Almost all of the static methods on the `Uuid` class references the `UuidFactory`. I love this approach and believe it should continue with `isValid(string $uuid) : boolean` as well.\r\n\r\nIn my projects, I pass around an instance of UuidFactory for dependency injection and type-hint against UuidFactoryInterface in case there were ever a time I wanted to use another implementation. Everything I needed was available in the factory EXCEPT the `Uuid::isValid` method. This won't break any users that utilize the Uuid method or `VALID_PATTERN` constant as they are still there, but instead reference UuidFactoryInterface class intead."},"comment":{"url":"https://api.github.com/repos/ramsey/uuid/issues/comments/202116548","html_url":"https://github.com/ramsey/uuid/pull/99#issuecomment-202116548","issue_url":"https://api.github.com/repos/ramsey/uuid/issues/99","id":202116548,"user":{"login":"ramsey","id":42941,"avatar_url":"https://avatars.githubusercontent.com/u/42941?v=3","gravatar_id":"","url":"https://api.github.com/users/ramsey","html_url":"https://github.com/ramsey","followers_url":"https://api.github.com/users/ramsey/followers","following_url":"https://api.github.com/users/ramsey/following{/other_user}","gists_url":"https://api.github.com/users/ramsey/gists{/gist_id}","starred_url":"https://api.github.com/users/ramsey/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ramsey/subscriptions","organizations_url":"https://api.github.com/users/ramsey/orgs","repos_url":"https://api.github.com/users/ramsey/repos","events_url":"https://api.github.com/users/ramsey/events{/privacy}","received_events_url":"https://api.github.com/users/ramsey/received_events","type":"User","site_admin":false},"created_at":"2016-03-27T18:13:41Z","updated_at":"2016-03-27T18:13:41Z","body":"I like @aztech-dev's suggestion of the UuidValidator class with a `getValidator()` method on UuidFactoryInterface. @bradynpoulsen, thoughts?\r\n\r\nFor now, I'm adding this to the 4.0.0 milestone."}}