{"push_id":738925652,"size":17,"distinct_size":17,"ref":"refs/heads/master","head":"1e2ec44922d9cb325ba7bf2507edbdf5ca90246e","before":"11767b0062992f3fc0c171378e5326df273f100c","commits":[{"sha":"0de7de91bd0ca35d35e6d8c7540fb3411bab8ee0","author":{"email":"5c2dd944dde9e08881bef0894fe7b22a5c9c4b06@joshbetz.com","name":"Josh Betz"},"message":"s/Adbusters/adbusters\n\n`wpcom_vip_load_plugin()` is case sensitive.\r\n\r\ncc @mdrovdahl","distinct":true,"url":"https://api.github.com/repos/danielwrobert/vip-scanner/commits/0de7de91bd0ca35d35e6d8c7540fb3411bab8ee0"},{"sha":"5d44ef300786633f86b100bc3a5217836f347b37","author":{"email":"9041512d0a619ef22b0ff61342bee6c689ff8e3a@gmail.com","name":"Devon Liu"},"message":"Fix inconsistent caching of scan results\n\nChange cache structure from storing theme path in post name to storing\nin post meta instead. This is because WP_Query does not behave as\nintended when dealing with duplicate post names (it assumes all post\nname to be unique) and in this case would ignore taxonomy criteria if\nthere's a post name match, resulting in wrong cache returned.\n\nThis also addresses issue #144 and allows toolbar to display correct\ncount from cache as intended.","distinct":true,"url":"https://api.github.com/repos/danielwrobert/vip-scanner/commits/5d44ef300786633f86b100bc3a5217836f347b37"},{"sha":"4afdacb20cc59f6e63e511a6e0dfeb3a57b61882","author":{"email":"9041512d0a619ef22b0ff61342bee6c689ff8e3a@gmail.com","name":"Devon Liu"},"message":"Change scan result cache structure\n\nChange cache structure to store \"theme path\" and \"review type\" as one\nstring in post_name, elimination using meta or taxonomy. This allows for\nsimplicity and more efficient WP_Query.","distinct":true,"url":"https://api.github.com/repos/danielwrobert/vip-scanner/commits/4afdacb20cc59f6e63e511a6e0dfeb3a57b61882"},{"sha":"64a95ca22904bb0745adac0fde3e0f66814ba00b","author":{"email":"9041512d0a619ef22b0ff61342bee6c689ff8e3a@gmail.com","name":"Devon Liu"},"message":"Add checks for trigger_error and set_error_handler\n\nAdd checks per issue #272:\n> trigger_error() doesn't make much sense, as there is no access to the\nlog file this will produce...and if you intentionally throw a fatal\nerror, that's doubly bad.\n> set_error_handler() is not supported.\n\nTest cases also added","distinct":true,"url":"https://api.github.com/repos/danielwrobert/vip-scanner/commits/64a95ca22904bb0745adac0fde3e0f66814ba00b"},{"sha":"d1e7a98476c03d5b6cbcadf6770a27f5166d5b98","author":{"email":"9041512d0a619ef22b0ff61342bee6c689ff8e3a@gmail.com","name":"Devon Liu"},"message":"Code formatting fix-up for spaces vs tabs\n\nUse spaces instead of tabs to be consistent with the rest of the code","distinct":true,"url":"https://api.github.com/repos/danielwrobert/vip-scanner/commits/d1e7a98476c03d5b6cbcadf6770a27f5166d5b98"},{"sha":"3c0b5feb37ef5a7e2b025156ead7a262aff38d92","author":{"email":"9041512d0a619ef22b0ff61342bee6c689ff8e3a@gmail.com","name":"Devon Liu"},"message":"Add trigger_error set_error_handler to test file\n\nAdd trigger_error and set_error_handler to test file for use in\ntest-VIPRestrictedPatternsCheck","distinct":true,"url":"https://api.github.com/repos/danielwrobert/vip-scanner/commits/3c0b5feb37ef5a7e2b025156ead7a262aff38d92"},{"sha":"ece3ee330fdda8a8607c58bb6641e084dd4e8203","author":{"email":"49b036f87406d9cc297b0eec1431eaa2772be9b5@gmail.com","name":"Nick Daugherty"},"message":"Merge pull request #281 from devonliu/issue272_error_handler_check\n\nAdd checks for trigger_error and set_error_handler","distinct":true,"url":"https://api.github.com/repos/danielwrobert/vip-scanner/commits/ece3ee330fdda8a8607c58bb6641e084dd4e8203"},{"sha":"bedf0873d18b82ebdb2fe582aeb0529fec5995ff","author":{"email":"9041512d0a619ef22b0ff61342bee6c689ff8e3a@gmail.com","name":"Devon Liu"},"message":"Refactor scan result post name generation\n\nAbstract the formation of post name for scan result into a reusable\nfunction, with a spacer in between review type and theme path in the\npost name.","distinct":true,"url":"https://api.github.com/repos/danielwrobert/vip-scanner/commits/bedf0873d18b82ebdb2fe582aeb0529fec5995ff"},{"sha":"ac8d044ffbacb41cba28a1c57e5cb636e1559be3","author":{"email":"49b036f87406d9cc297b0eec1431eaa2772be9b5@gmail.com","name":"Nick Daugherty"},"message":"Merge pull request #280 from devonliu/master\n\nFix inconsistent caching of scan results","distinct":true,"url":"https://api.github.com/repos/danielwrobert/vip-scanner/commits/ac8d044ffbacb41cba28a1c57e5cb636e1559be3"},{"sha":"452066e338253fd2bd7cd766bceef3fe7a6d0bea","author":{"email":"2c83b170016aedfbeae2b0517e2d44e0d2ff6a3f@gmail.com","name":"Daniel W. Robert"},"message":"Merge remote-tracking branch 'upstream/master'\n\n* upstream/master:\n  Refactor scan result post name generation\n  Add trigger_error set_error_handler to test file\n  Code formatting fix-up for spaces vs tabs\n  Add checks for trigger_error and set_error_handler\n  Change scan result cache structure\n  Fix inconsistent caching of scan results\n  s/Adbusters/adbusters","distinct":true,"url":"https://api.github.com/repos/danielwrobert/vip-scanner/commits/452066e338253fd2bd7cd766bceef3fe7a6d0bea"},{"sha":"a8cfd37cfb46164569c095bae5759bfccf8f1b0d","author":{"email":"bebba253c233e35f88be9159978e8a28ab32f1a5@booya.lan","name":"Filipe Varela"},"message":"Adds icon and banner assets","distinct":true,"url":"https://api.github.com/repos/danielwrobert/vip-scanner/commits/a8cfd37cfb46164569c095bae5759bfccf8f1b0d"},{"sha":"41f66b009ae8882c28c250fa997c53dccad2d5dc","author":{"email":"5c2dd944dde9e08881bef0894fe7b22a5c9c4b06@joshbetz.com","name":"Josh Betz"},"message":"Merge pull request #284 from Automattic/add/icon-and-banner\n\nAdds icon and banner assets","distinct":true,"url":"https://api.github.com/repos/danielwrobert/vip-scanner/commits/41f66b009ae8882c28c250fa997c53dccad2d5dc"},{"sha":"4fb1278f469e3f57431bfd9815dfc78342ad2e84","author":{"email":"1a73af9e7ae00182733b2292511b814be66f065f@fklein.info","name":"Fränk Klein"},"message":"Add additional test for the CustomizerCheck\n\nThe test ensures that semicolons in descriptions don’t throw the\ndetection off.\n\nThis is no longer the case since moving away from regular expression,\nbut we should just make sure there’s no regression here.\n\nFixes #250.","distinct":true,"url":"https://api.github.com/repos/danielwrobert/vip-scanner/commits/4fb1278f469e3f57431bfd9815dfc78342ad2e84"},{"sha":"9fb7ea7f75725588e93ea7afc90749e3620df7e1","author":{"email":"1a73af9e7ae00182733b2292511b814be66f065f@fklein.info","name":"Fränk Klein"},"message":"Correct text domains.\n\nThese were still left overs from the Theme Check plugin.\n\nSee #179.","distinct":true,"url":"https://api.github.com/repos/danielwrobert/vip-scanner/commits/9fb7ea7f75725588e93ea7afc90749e3620df7e1"},{"sha":"1e32df8f02f750163b2145910897558e2e959f4d","author":{"email":"1a73af9e7ae00182733b2292511b814be66f065f@fklein.info","name":"Fränk Klein"},"message":"Remove deprecated `screen_icon()` function.","distinct":true,"url":"https://api.github.com/repos/danielwrobert/vip-scanner/commits/1e32df8f02f750163b2145910897558e2e959f4d"},{"sha":"798b815048bf6f1d71ae54bf679099d89141f8c3","author":{"email":"6f5ccf8c3f31fed36c8213408711400d90769bde@raz.or.at","name":"Bernhard Reiter"},"message":"Check node type in function call checks\n\nBefore invoking `$node->name->toString()` on a node of type\n`PhpParser\\Node\\Expr\\FuncCall`, make sure that `$node->name`\nreally has a `toString()` member, by checking that it is of type\n`PhpParser\\Node\\Name`.\n\nIn cases where the function invoked is specified by a variable\n(`PhpParser\\Node\\Expr\\Variable`, as in `$function();`)\ninstead of a name, the check would otherwise throw, as a variable\ndoesn't have a `toString()` member (and we can't compare that variable\nto a blacklist of forbidden function names anyway).\n\nProps to @fklein-lu for reporting this issue.","distinct":true,"url":"https://api.github.com/repos/danielwrobert/vip-scanner/commits/798b815048bf6f1d71ae54bf679099d89141f8c3"},{"sha":"1e2ec44922d9cb325ba7bf2507edbdf5ca90246e","author":{"email":"2c83b170016aedfbeae2b0517e2d44e0d2ff6a3f@gmail.com","name":"Daniel W. Robert"},"message":"Merge remote-tracking branch 'upstream/master'\n\n* upstream/master:\n  Check node type in function call checks\n  Remove deprecated `screen_icon()` function.\n  Correct text domains.\n  Add additional test for the CustomizerCheck\n  Adds icon and banner assets","distinct":true,"url":"https://api.github.com/repos/danielwrobert/vip-scanner/commits/1e2ec44922d9cb325ba7bf2507edbdf5ca90246e"}]}