{"action":"created","issue":{"url":"https://api.github.com/repos/concrete5/concrete5/issues/2838","labels_url":"https://api.github.com/repos/concrete5/concrete5/issues/2838/labels{/name}","comments_url":"https://api.github.com/repos/concrete5/concrete5/issues/2838/comments","events_url":"https://api.github.com/repos/concrete5/concrete5/issues/2838/events","html_url":"https://github.com/concrete5/concrete5/issues/2838","id":98069413,"number":2838,"title":"Implement deprecated log type","user":{"login":"Mnkras","id":212649,"avatar_url":"https://avatars.githubusercontent.com/u/212649?v=3","gravatar_id":"","url":"https://api.github.com/users/Mnkras","html_url":"https://github.com/Mnkras","followers_url":"https://api.github.com/users/Mnkras/followers","following_url":"https://api.github.com/users/Mnkras/following{/other_user}","gists_url":"https://api.github.com/users/Mnkras/gists{/gist_id}","starred_url":"https://api.github.com/users/Mnkras/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Mnkras/subscriptions","organizations_url":"https://api.github.com/users/Mnkras/orgs","repos_url":"https://api.github.com/users/Mnkras/repos","events_url":"https://api.github.com/users/Mnkras/events{/privacy}","received_events_url":"https://api.github.com/users/Mnkras/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-07-30T02:04:41Z","updated_at":"2015-07-30T14:17:50Z","closed_at":null,"body":"We should implement a new log type for deprecated functions/methods\r\n\r\nBasically anywhere we use deprecated functionality that can get called (like where we are going to use func_get_args() for backwards compatibility) we should log that this happens so that it can be fixed.\r\n\r\nIn the PRB we can enforce addons to not call deprecated methods and having logs would make that much easier."},"comment":{"url":"https://api.github.com/repos/concrete5/concrete5/issues/comments/126344334","html_url":"https://github.com/concrete5/concrete5/issues/2838#issuecomment-126344334","issue_url":"https://api.github.com/repos/concrete5/concrete5/issues/2838","id":126344334,"user":{"login":"mlocati","id":928116,"avatar_url":"https://avatars.githubusercontent.com/u/928116?v=3","gravatar_id":"","url":"https://api.github.com/users/mlocati","html_url":"https://github.com/mlocati","followers_url":"https://api.github.com/users/mlocati/followers","following_url":"https://api.github.com/users/mlocati/following{/other_user}","gists_url":"https://api.github.com/users/mlocati/gists{/gist_id}","starred_url":"https://api.github.com/users/mlocati/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mlocati/subscriptions","organizations_url":"https://api.github.com/users/mlocati/orgs","repos_url":"https://api.github.com/users/mlocati/repos","events_url":"https://api.github.com/users/mlocati/events{/privacy}","received_events_url":"https://api.github.com/users/mlocati/received_events","type":"User","site_admin":false},"created_at":"2015-07-30T14:17:50Z","updated_at":"2015-07-30T14:17:50Z","body":"1. What about log size? If it's enabled for production sites, the log would become huge. So, I'd enable it only during development/for core and package developers.\r\n2. Wouldn't this slow down execution time?\r\n"}}