kriswallsmith/Buzz — pull request #88 by johannac (2012-07-02 11:19:14+00:00)
Title: Don't change error reporting level

Removing the lines which change the error reporting level to zero (off). This completely masks any problems with the URL, request, or response from the server, making debugging difficult. Using error_get_last in the next line to throw an exception does not always provide relevant information as file_get_contents will throw several errors on failure. Most errors seem to be warnings, so will not halt execution of the program and will allow you to see useful debugging information in the logs. Since the FileGetContents stream is the default one, I would expect a lot of people, especially new users like myself, to come up against this problem. This seems to be the only instance where the error reporting level is changed, so perhaps its ok to just get rid of it and let the users control the error reporting levels themselves?