Node Ownership - 7.x-1.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.

Node Ownership - 7.x-1.x

Test status

Status	Result
Last received	Mon, 07/23/2012 - 21:37:15
Last requested	Mon, 07/23/2012 - 21:38:02
Last tested	Mon, 07/23/2012 - 21:38:31
Test count	4062

Environment status

MySQL	pass
Code review	fail

Overall Result	pass

Branch information

Identifier	7.x-1.x
Project	Node Ownership
Link	Project information

Event logView entire log

ID	Message	Timestamp
4042613	Result retrieved by project client.	3 years 31 weeks ago
4042608	Result received from test client #654.	3 years 31 weeks ago
4042603	Result received from test client #813.	3 years 31 weeks ago
4042598	Requested by test client #813.	3 years 31 weeks ago
4042593	Requested by test client #654.	3 years 31 weeks ago
4042588	Test request received.	3 years 31 weeks ago
3800403	Result retrieved by project client.	3 years 39 weeks ago
3800398	Result received from test client #.	3 years 39 weeks ago
3800393	Result retrieved by project client.	3 years 39 weeks ago
3800378	Result received from test client #654.	3 years 39 weeks ago

SummaryPASSED: [[Coder]]: [Code review] 57 minor(s), 7 critical(s), and 19 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

57 minor(s), 7 critical(s), and 19 normal(s)

Non-pass

Test name

Minor

Critical

Normal

7.x-1.0/nodeownership.install

Message	Group	Filename	Line	Function	Status
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)	Other	nodeownership.install	2	N/A

7.x-1.0/nodeownership.module

Message	Group	Filename	Line	Function
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)	Other	nodeownership.module	2	N/A
There should be no trailing spaces	Other	nodeownership.module	14	N/A
There should be no trailing spaces	Other	nodeownership.module	17	N/A
put a space between the asterisk and the comment text	Other	nodeownership.module	184	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	nodeownership.module	189	N/A
There should be no trailing spaces	Other	nodeownership.module	192	N/A
SQL keywords should be upper case	Other	nodeownership.module	219	N/A
SQL keywords should be upper case	Other	nodeownership.module	220	N/A
Potential problem: "SELECT FROM {node}" statements should probably be wrapped in db_rewrite_sql() and with the alias for {node} table defined (e.g. {node} n)	Other	nodeownership.module	251	N/A
There should be no trailing spaces	Other	nodeownership.module	293	N/A
There should be no trailing spaces	Other	nodeownership.module	318	N/A
SQL keywords should be upper case	Other	nodeownership.module	342	N/A
SQL keywords should be upper case	Other	nodeownership.module	346	N/A
SQL keywords should be upper case	Other	nodeownership.module	347	N/A
Potential problem: "SELECT FROM {node}" statements should probably be wrapped in db_rewrite_sql() and with the alias for {node} table defined (e.g. {node} n)	Other	nodeownership.module	347	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	nodeownership.module	354	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	nodeownership.module	359	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	nodeownership.module	359	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	nodeownership.module	363	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	nodeownership.module	363	N/A
There should be no trailing spaces	Other	nodeownership.module	366	N/A
SQL keywords should be upper case	Other	nodeownership.module	367	N/A
There should be no trailing spaces	Other	nodeownership.module	389	N/A
SQL keywords should be upper case	Other	nodeownership.module	419	N/A
Potential problem: "SELECT FROM {node}" statements should probably be wrapped in db_rewrite_sql() and with the alias for {node} table defined (e.g. {node} n)	Other	nodeownership.module	419	N/A
SQL keywords should be upper case	Other	nodeownership.module	420	N/A
Missing parenthesis after function name	Other	nodeownership.module	510	N/A

nodeownership.module

Message	Group	Filename	Line	Function
There should be no trailing spaces	Other	nodeownership.module	23	N/A
Menu item titles and descriptions should NOT be enclosed within t(). (Drupal Docs)	Other	nodeownership.module	53	N/A
There should be no trailing spaces	Other	nodeownership.module	54	N/A
Menu item titles and descriptions should NOT be enclosed within t(). (Drupal Docs)	Other	nodeownership.module	69	N/A
There should be no trailing spaces	Other	nodeownership.module	71	N/A
Menu item titles and descriptions should NOT be enclosed within t(). (Drupal Docs)	Other	nodeownership.module	77	N/A
There should be no trailing spaces	Other	nodeownership.module	78	N/A
There should be no trailing spaces	Other	nodeownership.module	79	N/A
There should be no trailing spaces	Other	nodeownership.module	106	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	nodeownership.module	204	N/A
There should be no trailing spaces	Other	nodeownership.module	213	N/A
SQL keywords should be upper case	Other	nodeownership.module	220	N/A
SQL keywords should be upper case	Other	nodeownership.module	221	N/A
There should be no trailing spaces	Other	nodeownership.module	223	N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.	Other	nodeownership.module	225	N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.	Other	nodeownership.module	229	N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.	Other	nodeownership.module	233	N/A
There should be no trailing spaces	Other	nodeownership.module	248	N/A
There should be no trailing spaces	Other	nodeownership.module	249	N/A
The $string argument to t() should not begin or end with a space. (Drupal Docs)	Other	nodeownership.module	256	N/A
There should be no trailing spaces	Other	nodeownership.module	260	N/A
There should be no trailing spaces	Other	nodeownership.module	265	N/A
There should be no trailing spaces	Other	nodeownership.module	272	N/A
There should be no trailing spaces	Other	nodeownership.module	278	N/A
There should be no trailing spaces	Other	nodeownership.module	288	N/A
There should be no trailing spaces	Other	nodeownership.module	299	N/A
There should be no trailing spaces	Other	nodeownership.module	301	N/A
There should be no trailing spaces	Other	nodeownership.module	302	N/A
There should be no trailing spaces	Other	nodeownership.module	303	N/A
There should be no trailing spaces	Other	nodeownership.module	304	N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.	Other	nodeownership.module	310	N/A
There should be no trailing spaces	Other	nodeownership.module	316	N/A
There should be no trailing spaces	Other	nodeownership.module	323	N/A
SQL keywords should be upper case	Other	nodeownership.module	327	N/A
SQL keywords should be upper case	Other	nodeownership.module	331	N/A
SQL keywords should be upper case	Other	nodeownership.module	332	N/A
There should be no trailing spaces	Other	nodeownership.module	333	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	nodeownership.module	339	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	nodeownership.module	344	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	nodeownership.module	348	N/A
SQL keywords should be upper case	Other	nodeownership.module	351	N/A
SQL keywords should be upper case	Other	nodeownership.module	375	N/A
SQL keywords should be upper case	Other	nodeownership.module	377	N/A
SQL keywords should be upper case	Other	nodeownership.module	379	N/A
SQL keywords should be upper case	Other	nodeownership.module	380	N/A
SQL keywords should be upper case	Other	nodeownership.module	382	N/A
SQL keywords should be upper case	Other	nodeownership.module	385	N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.	Other	nodeownership.module	395	N/A
There should be no trailing spaces	Other	nodeownership.module	397	N/A
SQL keywords should be upper case	Other	nodeownership.module	399	N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.	Other	nodeownership.module	410	N/A
The $message argument to drupal_set_message() should be enclosed within t() so that it is translatable.	Other	nodeownership.module	414	N/A
There should be no trailing spaces	Other	nodeownership.module	427	N/A
Missing parenthesis after function name	Other	nodeownership.module	459	N/A

All