{"issue":{"state":"open","pull_request":{"diff_url":"https://github.com/joomla/joomla-platform/pull/217.diff","html_url":"https://github.com/joomla/joomla-platform/pull/217","patch_url":"https://github.com/joomla/joomla-platform/pull/217.patch"},"user":{"avatar_url":"https://secure.gravatar.com/avatar/cab39d7e547423f2a0c72823eeec95b2?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"cab39d7e547423f2a0c72823eeec95b2","url":"https://api.github.com/users/realityking","login":"realityking","id":628508},"milestone":null,"updated_at":"2011-11-06T03:27:46Z","labels":[],"assignee":null,"comments":9,"closed_at":null,"url":"https://api.github.com/repos/joomla/joomla-platform/issues/217","body":"This is based on the functionality found the installation controller, it was expanded quite a bit for the AJAX installation in 1.7. This is basically the same as found in 1.7 except for some cleanup. (No deprecated APIs, a MIME type bug fixed, some unnecessary code removed)\r\n\r\nThis API makes it somewhat easier to send JSON responses because it takes care of some of the repetitive tasks like sending along any enqueued messages, the token and so on.\r\n\r\nHaving these things always in the same format will also make it possible to add a JS API in the future that takes care of all these things.\r\n\r\nI'm not completely sure about the API. For now I just followed what was present but I see at least three options:\r\n1) Remove JControllerJsonResponse and fold it's functionality into JController::sendJsonResponse().\r\n2) Add JControllerJsonResponse::send() that contains the json encoding and response header parts of JController::sendJsonResponse().\r\n3) The way the pull is done.\r\n\r\nThe advantage of 2) and 3) is that JControllerJsonResponse is also potentially useful for JSON views.","html_url":"https://github.com/joomla/joomla-platform/issues/217","number":217,"title":"Add improved support for JSON responses to JController.","id":1396255,"created_at":"2011-08-12T17:27:15Z"},"action":"created","comment":{"user":{"avatar_url":"https://secure.gravatar.com/avatar/679908c699f06435eb3c407f0259bab1?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"679908c699f06435eb3c407f0259bab1","url":"https://api.github.com/users/joomla-jenkins","login":"joomla-jenkins","id":929228},"updated_at":"2011-11-06T03:27:46Z","url":"https://api.github.com/repos/joomla/joomla-platform/issues/comments/2643769","body":"Build triggered by changes to the base.\n\nTest log missing. Tests failed to execute.\nCheckstyle analysis reported 441 warnings and 19 errors.\n","id":2643769,"created_at":"2011-11-06T03:27:46Z"}}