{"action":"created","issue":{"url":"https://api.github.com/repos/RESTful-Drupal/restful/issues/739","labels_url":"https://api.github.com/repos/RESTful-Drupal/restful/issues/739/labels{/name}","comments_url":"https://api.github.com/repos/RESTful-Drupal/restful/issues/739/comments","events_url":"https://api.github.com/repos/RESTful-Drupal/restful/issues/739/events","html_url":"https://github.com/RESTful-Drupal/restful/pull/739","id":116548461,"number":739,"title":"[BUGFIX] Make sure the description property is always available in the base RestfulException object.","user":{"login":"e0ipso","id":1140906,"avatar_url":"https://avatars.githubusercontent.com/u/1140906?v=3","gravatar_id":"","url":"https://api.github.com/users/e0ipso","html_url":"https://github.com/e0ipso","followers_url":"https://api.github.com/users/e0ipso/followers","following_url":"https://api.github.com/users/e0ipso/following{/other_user}","gists_url":"https://api.github.com/users/e0ipso/gists{/gist_id}","starred_url":"https://api.github.com/users/e0ipso/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/e0ipso/subscriptions","organizations_url":"https://api.github.com/users/e0ipso/orgs","repos_url":"https://api.github.com/users/e0ipso/repos","events_url":"https://api.github.com/users/e0ipso/events{/privacy}","received_events_url":"https://api.github.com/users/e0ipso/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/RESTful-Drupal/restful/labels/needs%202.x%20port","name":"needs 2.x port","color":"d4c5f9"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-11-12T13:27:35Z","updated_at":"2015-11-12T14:02:21Z","closed_at":"2015-11-12T13:35:41Z","pull_request":{"url":"https://api.github.com/repos/RESTful-Drupal/restful/pulls/739","html_url":"https://github.com/RESTful-Drupal/restful/pull/739","diff_url":"https://github.com/RESTful-Drupal/restful/pull/739.diff","patch_url":"https://github.com/RESTful-Drupal/restful/pull/739.patch"},"body":"From the original PR:\r\nThe `RestfulException` class contains a method `getDescription` that accesses a property called `description`. This property is not always available and in certain cases, logs errors like these:\r\n\r\n    Notice: Undefined property: RestfulException::$description in RestfulException->getDescription() (line 38 of /.../restful/exceptions/RestfulException.php)\r\n\r\nOn investigating, I found this method was introduced in 484a1744 with the description property added in child classes of `RestfulException`. If an application is extending that class and not defining the property, it results in the error above. While you may want to make it explicit that the description should be set in the child classes, it is not enforced which results in these notices. I think it is a good idea to at least keep it in the base class so that if a child class chooses not to use this, it doesn't have to.\r\n\r\nOn that note, why not just use the `message` property? I am just curious."},"comment":{"url":"https://api.github.com/repos/RESTful-Drupal/restful/issues/comments/156109472","html_url":"https://github.com/RESTful-Drupal/restful/pull/739#issuecomment-156109472","issue_url":"https://api.github.com/repos/RESTful-Drupal/restful/issues/739","id":156109472,"user":{"login":"hussainweb","id":1040271,"avatar_url":"https://avatars.githubusercontent.com/u/1040271?v=3","gravatar_id":"","url":"https://api.github.com/users/hussainweb","html_url":"https://github.com/hussainweb","followers_url":"https://api.github.com/users/hussainweb/followers","following_url":"https://api.github.com/users/hussainweb/following{/other_user}","gists_url":"https://api.github.com/users/hussainweb/gists{/gist_id}","starred_url":"https://api.github.com/users/hussainweb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hussainweb/subscriptions","organizations_url":"https://api.github.com/users/hussainweb/orgs","repos_url":"https://api.github.com/users/hussainweb/repos","events_url":"https://api.github.com/users/hussainweb/events{/privacy}","received_events_url":"https://api.github.com/users/hussainweb/received_events","type":"User","site_admin":false},"created_at":"2015-11-12T14:02:21Z","updated_at":"2015-11-12T14:02:21Z","body":"@e0ipso, I think this is not needed in 2.x. The RestfulException class there already has the description property."}}