{"issue":{"state":"open","pull_request":{"diff_url":"https://github.com/Solutions-Nitriques/oembed_field/pull/14.diff","html_url":"https://github.com/Solutions-Nitriques/oembed_field/pull/14","patch_url":"https://github.com/Solutions-Nitriques/oembed_field/pull/14.patch"},"milestone":null,"user":{"avatar_url":"https://secure.gravatar.com/avatar/6be8e584a0f454ac2dc7600bdeaca7b7?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"6be8e584a0f454ac2dc7600bdeaca7b7","url":"https://api.github.com/users/brendo","login":"brendo","id":69268},"assignee":null,"updated_at":"2011-11-14T23:23:41Z","labels":[],"closed_at":null,"comments":2,"url":"https://api.github.com/repos/Solutions-Nitriques/oembed_field/issues/14","body":"Sorry, I introduced a bug in my last commit by omitting a comma, this pull request fixes that.\r\n\r\nI might recommend that some of your update logic in future releases be pulled back into `extension.driver.php` as it's a little out of place in the Field class.","html_url":"https://github.com/Solutions-Nitriques/oembed_field/issues/14","title":"Fixes my bug","number":14,"id":2207911,"created_at":"2011-11-11T09:58:07Z"},"action":"created","comment":{"user":{"avatar_url":"https://secure.gravatar.com/avatar/6be8e584a0f454ac2dc7600bdeaca7b7?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"6be8e584a0f454ac2dc7600bdeaca7b7","url":"https://api.github.com/users/brendo","login":"brendo","id":69268},"updated_at":"2011-11-14T23:23:41Z","url":"https://api.github.com/repos/Solutions-Nitriques/oembed_field/issues/comments/2738445","body":"I took them out because the [spec](http://oembed.com/) defines the keys should be of those names.  In all of the drivers implemented, none deviate from the spec so it's useless abstraction.\r\n\r\nI left in the method that defines the root node (so the `hash` driver still works).","id":2738445,"created_at":"2011-11-14T23:23:41Z"}}