{"action":"closed","issue":{"url":"https://api.github.com/repos/Geeklog-Core/geeklog/issues/499","repository_url":"https://api.github.com/repos/Geeklog-Core/geeklog","labels_url":"https://api.github.com/repos/Geeklog-Core/geeklog/issues/499/labels{/name}","comments_url":"https://api.github.com/repos/Geeklog-Core/geeklog/issues/499/comments","events_url":"https://api.github.com/repos/Geeklog-Core/geeklog/issues/499/events","html_url":"https://github.com/Geeklog-Core/geeklog/issues/499","id":125173534,"number":499,"title":"[Feature Requests] Replacement for COM_refresh","user":{"login":"mystralkk","id":2015377,"avatar_url":"https://avatars.githubusercontent.com/u/2015377?v=3","gravatar_id":"","url":"https://api.github.com/users/mystralkk","html_url":"https://github.com/mystralkk","followers_url":"https://api.github.com/users/mystralkk/followers","following_url":"https://api.github.com/users/mystralkk/following{/other_user}","gists_url":"https://api.github.com/users/mystralkk/gists{/gist_id}","starred_url":"https://api.github.com/users/mystralkk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mystralkk/subscriptions","organizations_url":"https://api.github.com/users/mystralkk/orgs","repos_url":"https://api.github.com/users/mystralkk/repos","events_url":"https://api.github.com/users/mystralkk/events{/privacy}","received_events_url":"https://api.github.com/users/mystralkk/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/Geeklog-Core/geeklog/labels/low%20priority","name":"low priority","color":"ededed"},{"url":"https://api.github.com/repos/Geeklog-Core/geeklog/labels/minor","name":"minor","color":"ededed"}],"state":"closed","locked":false,"assignee":{"login":"mystralkk","id":2015377,"avatar_url":"https://avatars.githubusercontent.com/u/2015377?v=3","gravatar_id":"","url":"https://api.github.com/users/mystralkk","html_url":"https://github.com/mystralkk","followers_url":"https://api.github.com/users/mystralkk/followers","following_url":"https://api.github.com/users/mystralkk/following{/other_user}","gists_url":"https://api.github.com/users/mystralkk/gists{/gist_id}","starred_url":"https://api.github.com/users/mystralkk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mystralkk/subscriptions","organizations_url":"https://api.github.com/users/mystralkk/orgs","repos_url":"https://api.github.com/users/mystralkk/repos","events_url":"https://api.github.com/users/mystralkk/events{/privacy}","received_events_url":"https://api.github.com/users/mystralkk/received_events","type":"User","site_admin":false},"assignees":[{"login":"mystralkk","id":2015377,"avatar_url":"https://avatars.githubusercontent.com/u/2015377?v=3","gravatar_id":"","url":"https://api.github.com/users/mystralkk","html_url":"https://github.com/mystralkk","followers_url":"https://api.github.com/users/mystralkk/followers","following_url":"https://api.github.com/users/mystralkk/following{/other_user}","gists_url":"https://api.github.com/users/mystralkk/gists{/gist_id}","starred_url":"https://api.github.com/users/mystralkk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mystralkk/subscriptions","organizations_url":"https://api.github.com/users/mystralkk/orgs","repos_url":"https://api.github.com/users/mystralkk/repos","events_url":"https://api.github.com/users/mystralkk/events{/privacy}","received_events_url":"https://api.github.com/users/mystralkk/received_events","type":"User","site_admin":false}],"milestone":null,"comments":1,"created_at":"2016-01-06T13:08:31Z","updated_at":"2016-08-29T08:29:18Z","closed_at":"2016-08-29T08:29:18Z","body":"**Reported by @dhaun on 22 May 2011 08:36**\r\n\r\n**Version:** 1.8.0\r\n\r\n**Description:**\r\nCOM_refresh returns an HTML snippet for a meta refresh, used to redirect the user's browser to a new URL. This was the standard way of doing this back in Geeklog's early days, but these days all browsers understand an HTTP Location header.\r\n\r\nHowever, we can't simply start sending additional HTTP headers in COM_refresh, as we learned in Geeklog 1.7.0, when a Content-Type header was added to COM_refresh and caused problems with plugins such as the Forum plugin (the problem is that there's code out there that calls COM_refresh at one point but then throws away the result and decides to send something else to the browser instead).\r\n\r\nSuggestion:\r\n- introduce a new function, COM_redirect\r\n- have it send both a Location header and the HTML meta refresh (and a proper Content-Type header)\r\n- the function should send those immediately and should not return\r\n- keep COM_refresh but mark it as deprecated\r\n- replace all occurences of COM_refresh in Geeklog with COM_redirect\r\n\r\n[Mantis Bugtracker #1350](http://project.geeklog.net/tracking/view.php?id=1350)"}}
{"actor":{"display_login":"mystralkk"}}