Drupal Sync - 7.x-1.x | Drupal Quality Assurance

Latest changes

So Long and Thanks for all the Tests

1 week 1 day 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.

Drupal Sync - 7.x-1.x

Test status

Status	Result
Last received	Tue, 01/15/2013 - 15:27:10
Last requested	Tue, 01/15/2013 - 15:28:05
Last tested	Tue, 01/15/2013 - 15:28:36
Test count	2

Environment status

MySQL	pass
Code review	fail

Overall Result	pass

Branch information

Identifier	7.x-1.x
Project	Drupal Sync
Link	Project information

Event logView entire log

ID	Message	Timestamp
5214293	Result retrieved by project client.	3 years 6 weeks ago
5214288	Result retrieved by project client.	3 years 6 weeks ago
5214278	Result received from test client #988.	3 years 6 weeks ago
5214273	Result received from test client #978.	3 years 6 weeks ago
5214268	Requested by test client #978.	3 years 6 weeks ago
5214263	Requested by test client #988.	3 years 6 weeks ago
5214258	Test request received.	3 years 6 weeks ago

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

22 minor(s), 1 critical(s), and 10 normal(s)

Non-pass

Test name Minor Critical Normal

drupal_sync.module 6 1 2

Message	Group	Filename	Line	Function
put a space between the asterisk and the comment text	Other	drupal_sync.module	88	N/A
put a space between the asterisk and the comment text	Other	drupal_sync.module	123	N/A
Format should be `* Implementation of hook_foo().`	Other	drupal_sync.module	126	N/A
Potential problem: drupal_set_message() only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.	Other	drupal_sync.module	143	N/A
put a space between the asterisk and the comment text	Other	drupal_sync.module	165	N/A
use <br /> instead of <br>	Other	drupal_sync.module	257	N/A
use <br /> instead of <br>	Other	drupal_sync.module	277	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	drupal_sync.module	277	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	drupal_sync.module	278	N/A

modules/drupal_sync_manager/drupal_sync_manager.module 5 0 8

Message	Group	Filename	Line	Function
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	drupal_sync_manager.module	109	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	drupal_sync_manager.module	110	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	drupal_sync_manager.module	111	N/A
use <br /> instead of <br>	Other	drupal_sync_manager.module	128	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	drupal_sync_manager.module	128	N/A
use <br /> instead of <br>	Other	drupal_sync_manager.module	130	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	drupal_sync_manager.module	130	N/A
Use an indent of 2 spaces, with no tabs	Other	drupal_sync_manager.module	190	N/A
use <br /> instead of <br>	Other	drupal_sync_manager.module	223	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	drupal_sync_manager.module	223	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	drupal_sync_manager.module	269	N/A
use <br /> instead of <br>	Other	drupal_sync_manager.module	286	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	drupal_sync_manager.module	286	N/A

All

Test name Minor Critical Normal

drupal_sync.admin.inc 3 0 0

Message	Group	Filename	Line	Function
put a space between the asterisk and the comment text	Other	drupal_sync.admin.inc	168	N/A
put a space between the asterisk and the comment text	Other	drupal_sync.admin.inc	254	N/A
put a space between the asterisk and the comment text	Other	drupal_sync.admin.inc	260	N/A

drupal_sync.api.php 0 0 0

drupal_sync.install 0 0 0

drupal_sync.module 6 1 2

Message	Group	Filename	Line	Function
put a space between the asterisk and the comment text	Other	drupal_sync.module	88	N/A
put a space between the asterisk and the comment text	Other	drupal_sync.module	123	N/A
Format should be `* Implementation of hook_foo().`	Other	drupal_sync.module	126	N/A
Potential problem: drupal_set_message() only accepts filtered text, be sure to use check_plain(), filter_xss() or similar to ensure your $variable is fully sanitized.	Other	drupal_sync.module	143	N/A
put a space between the asterisk and the comment text	Other	drupal_sync.module	165	N/A
use <br /> instead of <br>	Other	drupal_sync.module	257	N/A
use <br /> instead of <br>	Other	drupal_sync.module	277	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	drupal_sync.module	277	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	drupal_sync.module	278	N/A

includes/drupal_sync.api.inc 0 0 0

includes/drupal_sync.entity.inc 3 0 0

Message	Group	Filename	Line	Function
Use an indent of 2 spaces, with no tabs	Other	drupal_sync.entity.inc	35	N/A
Use an indent of 2 spaces, with no tabs	Other	drupal_sync.entity.inc	37	N/A
Use an indent of 2 spaces, with no tabs	Other	drupal_sync.entity.inc	121	N/A

includes/drupal_sync.forms.inc 0 0 0

includes/drupal_sync.queue.inc 0 0 0

includes/drupal_sync.xmlrpc.inc 1 0 0

Message	Group	Filename	Line	Function	Status
Format should be `* Implementation of hook_foo().`	Other	drupal_sync.xmlrpc.inc	383	N/A

modules/drupal_sync_additional_taxonomy_rules/drupal_sync_additional_taxonomy_rules.admin.inc 0 0 0

modules/drupal_sync_additional_taxonomy_rules/drupal_sync_additional_taxonomy_rules.install 0 0 0

modules/drupal_sync_additional_taxonomy_rules/drupal_sync_additional_taxonomy_rules.module 0 0 0

modules/drupal_sync_initial_setup/drupal_sync_initial_setup.admin.inc 0 0 0

modules/drupal_sync_initial_setup/drupal_sync_initial_setup.install 0 0 0

modules/drupal_sync_initial_setup/drupal_sync_initial_setup.module 1 0 0

Message	Group	Filename	Line	Function	Status
use <br /> instead of <br>	Other	drupal_sync_initial_setup.module	116	N/A

modules/drupal_sync_manager/drupal_sync_manager.module 5 0 8

Message	Group	Filename	Line	Function
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	drupal_sync_manager.module	109	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	drupal_sync_manager.module	110	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	drupal_sync_manager.module	111	N/A
use <br /> instead of <br>	Other	drupal_sync_manager.module	128	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	drupal_sync_manager.module	128	N/A
use <br /> instead of <br>	Other	drupal_sync_manager.module	130	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	drupal_sync_manager.module	130	N/A
Use an indent of 2 spaces, with no tabs	Other	drupal_sync_manager.module	190	N/A
use <br /> instead of <br>	Other	drupal_sync_manager.module	223	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	drupal_sync_manager.module	223	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	drupal_sync_manager.module	269	N/A
use <br /> instead of <br>	Other	drupal_sync_manager.module	286	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	drupal_sync_manager.module	286	N/A

modules/drupal_sync_menu/drupal_sync_menu.module 3 0 0

Message	Group	Filename	Line	Function
put a space between the asterisk and the comment text	Other	drupal_sync_menu.module	32	N/A
Format should be `* Implementation of hook_foo().`	Other	drupal_sync_menu.module	210	N/A
put a space between the asterisk and the comment text	Other	drupal_sync_menu.module	332	N/A

Review log