To save time editing manually.
I nearly made this a patch to add the patch to the module directory, but that would've been overkill.
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | _d7.code_coverage.patch | 367 bytes | sun |
| #5 | 293298-core-patch.patch | 4.89 KB | boombatower |
| #4 | 293298-core-patch.patch | 3.2 KB | boombatower |
| #1 | 293298-core-patch.patch | 559 bytes | boombatower |
| coverage.patch | 2.05 KB | catch |
Comments
Comment #1
boombatower commentedMoved the bit of code to the top of bootstrap.inc so it will be called first by xmlrpc.php, index.php, install.php, update.php etc instead of patching those individually.
I'll re-look into what is needed for SimpleTest next.
Comment #2
boombatower commentedLooks like this may not work as well as I thought, but we definitly need all the mentioned locations included in coverage.
Comment #3
boombatower commentedComment #4
boombatower commentedThis includes all files, but does nothing with testing system at the moment.
update.php is not ideal, since the file docblock is not at the top the $Id$ will screw up when the patch file is committed, so for now the first line will never be counted.
in xdebug.inc it does a query in simpletest table that won't work since this code runs before bootstraps, so I'll need to fix that up.
Comment #5
boombatower commentedThat should solve the query issue, but I still need to test with simpletest.
Comment #6
boombatower commenteddrupal_register_shutdown_function() does not exist before bootstraps...so that'll will need to be delt with.
Comment #7
sunI don't understand why #1 didn't work. Placing the require_once into bootstrap.inc seems to be correct, IMO.
Comment #8
sun#983808: Fix Simpletest integration has been committed.
In my testing, the approach of #1 works flawlessly. Here's a new patch.
Also note that I've moved the global variable into the .xdebug.inc file itself. Overall, I'm not sure whether it is useful in any way though. The same could be achieved by testing
function_exists('code_coverage_report')Comment #9
boombatower commentedRe-rolled with format-patch just for kicks and committed. I do not remember for sure, but I believe the reason is the fact that xmlrpc.php used to or still does have code that runs before bootstraps that would always be marked as not run, but we can deal with that later if we really want to.
Comment #10
sunThanks :)
However, coding style issue with if condition control structure in the follow-up commit http://drupalcode.org/project/code_coverage.git/commitdiff/caf6723b57fc3...