project_issue_file_review 6.x-2.6

A whole pile of superb new features and bugfixes by @jthorson.

Issue #1235096 by jthorson: Fixed Manual test form wrongly assumes Pifr_coder() is active.
Issue #1234946 by jthorson | rfay: Fixed Use 'branch/tag' terminology (or identifier?) where appropriate.
Issue #1126112 by jthorson: Fix core test detection
Issue #1126112 by jthorson: detect core tests before allowing --all argument on run-tests.sh
Issue # 1126112 by jthorson: Use scan instead of syntax_files() to determine simpletest patch tests to run
Issue #1234226 by jthorson: Fixed syntax_files() operation to support coder branch tests
Issue #1234918 by rfay: Enforce pifr_coder dependency on coder & coder_tough_love
Issue # 1234936 by rfay: Added @debug counter to results array
Issue #1236394 by jthorson: Removed extraneous D6 dependencies from manual test form
Issue #1231918 by jthorson: Added directory argument to manual testbot branch tests
Issue # 1234928 by rfay: Fixed bad 'DEFINE' references in manual coder processing
Issue #1231936 by jthorson: Log run-tests.sh command before execution
Issue # 1120228 by jthorson: Fixed typo in client certification 'fail' test
Issue #1230614 by jthorson: Added missing db_set_active() call in client/review/db/mysql.inc
Issue #1229516 by jthorson: Updated admin/pifr client tests core repository reference
Issue #1229062-07 by rfay: Make PIFR side be more resilient in allowing trailing comma
by jthorson: Added missing variable_del() to manual test simpletest runs
by rfay: Two minor typos discovered
by jthorson: Fixed issue with constants introduced in manual tests commit
Issue #1227386 by jthorson: Fixed filepath issue causing files to be skipped during syntax operation on contrib patch tests
Issue #1176982 by jthorson: Provide manual test interface on testbots
Issue #1207076 by jthorson: Return TRUE instead of array() for non-select queries in database.mysql.inc
Issue #1108896 by jthorson: Fixed Test Status table 'status' display issues
- #1221620 by jthorson: Fixed pifr_client().review.inc references non-existant chart 'master' branch.
#1214774 by jthorson: Fixed simpletest test_info_parse() broken for local tests.
- #1217066 by jthorson: Fixed cron run kills run-tests.sh when running local testbot tests.
Issue #1217800-01 by jthorson: Fix warning about pass-by-reference
Issue #1214818 by jthorson: Fix quick local tests by using simpletest.tests as argument
Issue #1192680 by jthorson: Fix /tmpfs filling up by cleaning up test dbs
Issue #1206758 by jthorson: Update coder confirmation patches
Issue #1205236 by jthorson: Strict checking of coder.reviews
Issue #1205878 by jthorson: Allow override of coder severity level
Issue #1018032 by neilnz, fiaso: Update Postgres support
by rfay: Writeup on PIFR testing system. This probably won't be here for long but needs to be under source control
Issue #1206984 by jthorson, rfay: Fix
Issue #1207010 by jhorson: Fix bad variable
Issue #694876 by jthorson: Fix up hook_views_data()
Issue #1206758 by jthorson: Fix coder confirmation patches to git style
Issue #1172750 by jthorson | boombatower: Fixed review/client.inc should not assume there is a patch (file) to be applied.