Rabbit Hole - 7.x-2.x | Drupal Quality Assurance

Latest changes

So Long and Thanks for all the Tests

1 week 2 days ago

Support

In case of troubles

check the FAQ. Please make sure you've tried the test locally.
search known testbot issues and then create a new issue if yours isn't there already. Please give a link to the test.

Test client logs

No clients to display.

Rabbit Hole - 7.x-2.x

Test status

Status	Result
Last received	Tue, 01/15/2013 - 12:10:34
Last requested	Tue, 01/15/2013 - 12:11:05
Last tested	Tue, 01/15/2013 - 12:11:34
Test count	2

Environment status

MySQL	pass
Code review	fail

Overall Result	pass

Branch information

Identifier	7.x-2.x
Project	Rabbit Hole
Link	Project information

Event logView entire log

ID	Message	Timestamp
5213363	Result retrieved by project client.	3 years 6 weeks ago
5213358	Result retrieved by project client.	3 years 6 weeks ago
5213353	Result received from test client #988.	3 years 6 weeks ago
5213348	Result received from test client #958.	3 years 6 weeks ago
5213343	Requested by test client #958.	3 years 6 weeks ago
5213338	Requested by test client #988.	3 years 6 weeks ago
5213333	Test request received.	3 years 6 weeks ago

SummaryPASSED: [[Coder]]: [Code review] 33 minor(s), 2 critical(s), and 1 normal(s); [[SimpleTest]]: [MySQL] 0 pass(es).

Details

MySQL
Code review

MySQL

Setup environment
Detect an invalid patch URL
Detect invalid repository information
Detect invalid patch format
Detect a non-applicable patch
Detect invalid PHP syntax
Detect a Drupal installation failure
Detect a test run failure
Detect a failing test
Complete test suite with all tests passing

Review log

Code review

Setup environment
Detect an invalid patch URL
Detect invalid repository information
Detect invalid patch format
Detect a non-applicable patch
Detect invalid PHP syntax
Detect a review run failure
Detect coder flags
Ensure that your code follows the Drupal standard and passes a coder review.
Complete coder review with all passed

33 minor(s), 2 critical(s), and 1 normal(s)

Non-pass

Test name

Minor

Critical

Normal

modules/rh_taxonomy/rh_taxonomy.install

Message	Group	Filename	Line	Function	Status
Use update_sql() instead of db_query() in hook_update_N()	Other	rh_taxonomy.install	13	N/A

rabbit_hole.module

Message	Group	Filename	Line	Function
There should be no trailing spaces	Other	rabbit_hole.module	25	N/A
in most cases, replace the string function with the drupal_ equivalent string functions	Other	rabbit_hole.module	31	N/A
There should be no trailing spaces	Other	rabbit_hole.module	32	N/A
There should be no trailing spaces	Other	rabbit_hole.module	37	N/A
There should be no trailing spaces	Other	rabbit_hole.module	43	N/A
There should be no trailing spaces	Other	rabbit_hole.module	97	N/A
There should be no trailing spaces	Other	rabbit_hole.module	103	N/A
There should be no trailing spaces	Other	rabbit_hole.module	136	N/A
There should be no trailing spaces	Other	rabbit_hole.module	139	N/A
There should be no trailing spaces	Other	rabbit_hole.module	148	N/A
Potential problem: FAPI elements '#title' and '#description' only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.	Other	rabbit_hole.module	159	N/A
There should be no trailing spaces	Other	rabbit_hole.module	172	N/A
There should be no trailing spaces	Other	rabbit_hole.module	190	N/A
There should be no trailing spaces	Other	rabbit_hole.module	260	N/A
There should be no trailing spaces	Other	rabbit_hole.module	274	N/A
There should be no trailing spaces	Other	rabbit_hole.module	280	N/A
There should be no trailing spaces	Other	rabbit_hole.module	284	N/A
There should be no trailing spaces	Other	rabbit_hole.module	291	N/A
There should be no trailing spaces	Other	rabbit_hole.module	314	N/A
Using eval() or drupal_eval() in your module's code could have a security risk if the PHP input provided to the function contains malicious code. (Drupal Docs)	Other	rabbit_hole.module	355	N/A
There should be no trailing spaces	Other	rabbit_hole.module	529	N/A
@see references should be separated by "," followed by a single space and with no trailing punctuation	Other	rabbit_hole.module	536	N/A
There should be no trailing spaces	Other	rabbit_hole.module	546	N/A
There should be no trailing spaces	Other	rabbit_hole.module	569	N/A
There should be no trailing spaces	Other	rabbit_hole.module	596	N/A
There should be no trailing spaces	Other	rabbit_hole.module	603	N/A
There should be no trailing spaces	Other	rabbit_hole.module	637	N/A

All

Test name

Minor

Critical

Normal

modules/rh_node/rh_node.module

Message	Group	Filename	Line	Function	Status
There should be no trailing spaces	Other	rh_node.module	34	N/A
@see references should be separated by "," followed by a single space and with no trailing punctuation	Other	rh_node.module	71	N/A

modules/rh_taxonomy/rh_taxonomy.install

Message	Group	Filename	Line	Function	Status
Use update_sql() instead of db_query() in hook_update_N()	Other	rh_taxonomy.install	13	N/A

modules/rh_taxonomy/rh_taxonomy.module

Message	Group	Filename	Line	Function	Status
There should be no trailing spaces	Other	rh_taxonomy.module	56	N/A

modules/rh_user/rh_user.module

Message	Group	Filename	Line	Function	Status
There should be no trailing spaces	Other	rh_user.module	56	N/A

rabbit_hole.install

Message	Group	Filename	Line	Function
There should be no trailing spaces	Other	rabbit_hole.install	44	N/A
There should be no trailing spaces	Other	rabbit_hole.install	65	N/A
There should be no trailing spaces	Other	rabbit_hole.install	74	N/A
There should be no trailing spaces	Other	rabbit_hole.install	95	N/A

rabbit_hole.module

Message	Group	Filename	Line	Function
There should be no trailing spaces	Other	rabbit_hole.module	25	N/A
in most cases, replace the string function with the drupal_ equivalent string functions	Other	rabbit_hole.module	31	N/A
There should be no trailing spaces	Other	rabbit_hole.module	32	N/A
There should be no trailing spaces	Other	rabbit_hole.module	37	N/A
There should be no trailing spaces	Other	rabbit_hole.module	43	N/A
There should be no trailing spaces	Other	rabbit_hole.module	97	N/A
There should be no trailing spaces	Other	rabbit_hole.module	103	N/A
There should be no trailing spaces	Other	rabbit_hole.module	136	N/A
There should be no trailing spaces	Other	rabbit_hole.module	139	N/A
There should be no trailing spaces	Other	rabbit_hole.module	148	N/A
Potential problem: FAPI elements '#title' and '#description' only accept filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.	Other	rabbit_hole.module	159	N/A
There should be no trailing spaces	Other	rabbit_hole.module	172	N/A
There should be no trailing spaces	Other	rabbit_hole.module	190	N/A
There should be no trailing spaces	Other	rabbit_hole.module	260	N/A
There should be no trailing spaces	Other	rabbit_hole.module	274	N/A
There should be no trailing spaces	Other	rabbit_hole.module	280	N/A
There should be no trailing spaces	Other	rabbit_hole.module	284	N/A
There should be no trailing spaces	Other	rabbit_hole.module	291	N/A
There should be no trailing spaces	Other	rabbit_hole.module	314	N/A
Using eval() or drupal_eval() in your module's code could have a security risk if the PHP input provided to the function contains malicious code. (Drupal Docs)	Other	rabbit_hole.module	355	N/A
There should be no trailing spaces	Other	rabbit_hole.module	529	N/A
@see references should be separated by "," followed by a single space and with no trailing punctuation	Other	rabbit_hole.module	536	N/A
There should be no trailing spaces	Other	rabbit_hole.module	546	N/A
There should be no trailing spaces	Other	rabbit_hole.module	569	N/A
There should be no trailing spaces	Other	rabbit_hole.module	596	N/A
There should be no trailing spaces	Other	rabbit_hole.module	603	N/A
There should be no trailing spaces	Other	rabbit_hole.module	637	N/A

Review log