Role Expire - 6.x-1.12 | 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.

Role Expire - 6.x-1.12

Test status

Status	Result
Last received	Tue, 04/10/2012 - 16:23:37
Last requested	Tue, 04/10/2012 - 16:24:03
Last tested	Tue, 04/10/2012 - 16:24:18
Test count	4

Environment status

MySQL	pass
Code review	fail

Overall Result	pass

Branch information

Identifier	6.x-1.12
Project	Role Expire
Link	Project information

Event logView entire log

ID	Message	Timestamp
3555564	Result retrieved by project client.	3 years 46 weeks ago
3555534	Result retrieved by project client.	3 years 46 weeks ago
3555529	Result received from test client #.	3 years 46 weeks ago
3555524	Result received from test client #659.	3 years 46 weeks ago
3555514	Requested by test client #659.	3 years 46 weeks ago
3555509	Requested by test client #.	3 years 46 weeks ago
3555499	Test request received.	3 years 46 weeks ago
3205358	Result retrieved by project client.	3 years 51 weeks ago
3193198	Result retrieved by project client.	3 years 51 weeks ago
3193178	Result retrieved by project client.	3 years 51 weeks ago

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

25 minor(s), 1 critical(s), and 48 normal(s)

Non-pass

Test name Minor Critical Normal

role_expire.install 0 1 4

Message	Group	Filename	Line	Function
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)	Other	role_expire.install	2	N/A
table names should be enclosed in {curly_brackets}	Other	role_expire.install	177	N/A
Function documentation should be less than 80 characters per line.	Other	role_expire.install	186	N/A
Doxygen uses @todo and @bug to markup things to be done.	Other	role_expire.install	206	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	role_expire.install	210	N/A

role_expire.migrate.inc 0 0 1

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

role_expire.module 23 0 34

Message	Group	Filename	Line	Function
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)	Other	role_expire.module	2	N/A
Doxygen uses @todo and @bug to markup things to be done.	Other	role_expire.module	12	N/A
put a space between the asterisk and the comment text	Other	role_expire.module	16	N/A
Function summaries should be one line only. (Drupal Docs)	Other	role_expire.module	19	N/A
If you define a @param or @return, you should document it as well.	Other	role_expire.module	24	N/A
There should be no trailing spaces	Other	role_expire.module	33	N/A
Function summaries should be one line only. (Drupal Docs)	Other	role_expire.module	52	N/A
If you define a @param or @return, you should document it as well.	Other	role_expire.module	57	N/A
If you define a @param or @return, you should document it as well.	Other	role_expire.module	76	N/A
If you define a @param or @return, you should document it as well.	Other	role_expire.module	89	N/A
Doxygen uses @todo and @bug to markup things to be done.	Other	role_expire.module	107	N/A
Function summaries should be one line only. (Drupal Docs)	Other	role_expire.module	116	N/A
If you define a @param or @return, you should document it as well.	Other	role_expire.module	119	N/A
Function summaries should be one line only. (Drupal Docs)	Other	role_expire.module	144	N/A
put a space between the asterisk and the comment text	Other	role_expire.module	173	N/A
Separate comments from comment syntax by a space.	Other	role_expire.module	175	N/A
Missing parenthesis after function name	Other	role_expire.module	176	N/A
Format should be `* Implementation of hook_foo().`	Other	role_expire.module	176	N/A
Separate comments from comment syntax by a space.	Other	role_expire.module	178	N/A
Separate comments from comment syntax by a space.	Other	role_expire.module	188	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	role_expire.module	244	N/A
Use an indent of 2 spaces, with no tabs	Other	role_expire.module	246	N/A
Use an indent of 2 spaces, with no tabs	Other	role_expire.module	247	N/A
Function summaries should be one line only. (Drupal Docs)	Other	role_expire.module	260	N/A
If you define a @param or @return, you should document it as well.	Other	role_expire.module	262	N/A
@param and @return descriptions begin indented on the next line. (Drupal Docs)	Other	role_expire.module	262	N/A
If you define a @param or @return, you should document it as well.	Other	role_expire.module	263	N/A
@param and @return descriptions begin indented on the next line. (Drupal Docs)	Other	role_expire.module	263	N/A
missing space after comma	Other	role_expire.module	274	N/A
Use an indent of 2 spaces, with no tabs	Other	role_expire.module	277	N/A
Use an indent of 2 spaces, with no tabs	Other	role_expire.module	278	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	role_expire.module	278	N/A
Use an indent of 2 spaces, with no tabs	Other	role_expire.module	298	N/A
There should be no trailing spaces	Other	role_expire.module	315	N/A
Use an indent of 2 spaces, with no tabs	Other	role_expire.module	324	N/A
Use an indent of 2 spaces, with no tabs	Other	role_expire.module	325	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	role_expire.module	325	N/A
Use an indent of 2 spaces, with no tabs	Other	role_expire.module	330	N/A
There should be no trailing spaces	Other	role_expire.module	342	N/A
There should be no trailing spaces	Other	role_expire.module	350	N/A
missing space after comma	Other	role_expire.module	352	N/A
There should be no trailing spaces	Other	role_expire.module	358	N/A
missing space after comma	Other	role_expire.module	371	N/A
There should be no trailing spaces	Other	role_expire.module	389	N/A
in most cases, replace the string function with the drupal_ equivalent string functions	Other	role_expire.module	424	N/A
There should be no trailing spaces	Other	role_expire.module	481	N/A
Doxygen uses @todo and @bug to markup things to be done.	Other	role_expire.module	481	N/A
There should be no trailing spaces	Other	role_expire.module	483	N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote terms	Other	role_expire.module	500	N/A
missing space after comma	Other	role_expire.module	500	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	role_expire.module	500	N/A
There should be no trailing spaces	Other	role_expire.module	502	N/A
Use Drupal's format_date(), not PHP's default date().	Other	role_expire.module	506	N/A
put a space between the asterisk and the comment text	Other	role_expire.module	516	N/A
Function summaries should be one line only. (Drupal Docs)	Other	role_expire.module	519	N/A
Function summaries should be one line only. (Drupal Docs)	Other	role_expire.module	538	N/A
If you define a @param or @return, you should document it as well.	Other	role_expire.module	539	N/A

role_expire.rules.inc 0 0 8

Message	Group	Filename	Line	Function
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)	Other	role_expire.rules.inc	2	N/A
missing space after comma	Other	role_expire.rules.inc	57	N/A
Function summaries should be one line only. (Drupal Docs)	Other	role_expire.rules.inc	65	N/A
Separate comments from comment syntax by a space.	Other	role_expire.rules.inc	79	N/A
missing space after comma	Other	role_expire.rules.inc	92	N/A
missing space after comma	Other	role_expire.rules.inc	96	N/A
Function summaries should be one line only. (Drupal Docs)	Other	role_expire.rules.inc	118	N/A
If you define a @param or @return, you should document it as well.	Other	role_expire.rules.inc	121	N/A

role_expire.views.inc 2 0 1

Message	Group	Filename	Line	Function
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)	Other	role_expire.views.inc	2	N/A
Use an indent of 2 spaces, with no tabs	Other	role_expire.views.inc	131	N/A
Use an indent of 2 spaces, with no tabs	Other	role_expire.views.inc	132	N/A

All