{"issue":{"created_at":"2011-12-15T16:30:15Z","state":"open","updated_at":"2011-12-15T16:43:17Z","user":{"gravatar_id":"539ce7165882106a7016e789dbfd3560","id":608408,"url":"https://api.github.com/users/PHPGangsta","avatar_url":"https://secure.gravatar.com/avatar/539ce7165882106a7016e789dbfd3560?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","login":"PHPGangsta"},"comments":1,"body":"Hi Evert,\r\n\r\nI used \"Code inspection\" of PHPStorm to find problems in the SabreDAV code. I fixed the following:\r\n\r\n- fixed most of PHPDoc parameters\r\n- Spelling mistakes\r\n- Removed ?> from the bottom of php files\r\n- Removed trailing spaces at the end of each line (my IDE did this automatically)\r\n\r\nI don't know the your Code Style Guide and I don't know your convention regarding \"space after comma\" in parameter lists and so on, otherwise I would have changed that, too. The same for \"if(\" or \"if (\" etc.\r\nSo if you tell me the style that has to be used I would try to fix them.","assignee":null,"id":2569063,"number":3,"pull_request":{"patch_url":"https://github.com/evert/SabreDAV/pull/3.patch","html_url":"https://github.com/evert/SabreDAV/pull/3","diff_url":"https://github.com/evert/SabreDAV/pull/3.diff"},"labels":[],"title":"Spelling and php doc","html_url":"https://github.com/evert/SabreDAV/issues/3","closed_at":null,"url":"https://api.github.com/repos/evert/SabreDAV/issues/3","milestone":null},"action":"created","comment":{"created_at":"2011-12-15T16:43:17Z","updated_at":"2011-12-15T16:43:17Z","user":{"gravatar_id":"7d76e2bb6f8c962a5628093c9f5bc6fb","id":178960,"url":"https://api.github.com/users/evert","avatar_url":"https://secure.gravatar.com/avatar/7d76e2bb6f8c962a5628093c9f5bc6fb?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","login":"evert"},"body":"Thanks!\r\n\r\nManually ran through.. had to make sure nothing else was changed :)\r\n\r\nAs for your other points.. if( should really be if (. I have no strong feelings about spaces after comma's though. Tend to add them when I feel it's beneficial for legibility, but it's not always needed.\r\n\r\n","id":3165039,"url":"https://api.github.com/repos/evert/SabreDAV/issues/comments/3165039"}}