Weather - 6.x-5.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.

Weather - 6.x-5.x

Test status

Status	Result
Last received	Wed, 05/16/2012 - 20:33:47
Last requested	Wed, 05/16/2012 - 20:39:31
Last tested	Wed, 05/16/2012 - 20:39:40
Test count	36

Environment status

MySQL	pass
Code review	fail

Overall Result	pass

Branch information

Identifier	6.x-5.x
Project	Weather
Link	Project information

Event logView entire log

ID	Message	Timestamp
3713218	Result retrieved by project client.	3 years 40 weeks ago
3713203	Result received from test client #654.	3 years 40 weeks ago
3713198	Requested by test client #654.	3 years 40 weeks ago
3713193	Result received from test client #654.	3 years 40 weeks ago
3713188	Requested by test client #654.	3 years 40 weeks ago
3713173	Test request received.	3 years 40 weeks ago
3534419	Result retrieved by project client.	3 years 46 weeks ago
3534414	Result retrieved by project client.	3 years 46 weeks ago
3534409	Result retrieved by project client.	3 years 46 weeks ago
3534404	Result received from test client #654.	3 years 46 weeks ago

SummaryPASSED: [[SimpleTest]]: [MySQL] 275 pass(es); [[Coder]]: [Code review] 18 minor(s), 0 critical(s), and 54 normal(s).

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

275 pass(es), 0 fail(s), and 0 exception(s)

Test name	Pass	Fail	Exception
Parser (WeatherParserTestCase) [Weather]	111	0	0
Permission (WeatherPermissionTestCase) [Weather]	164	0	0

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

18 minor(s), 0 critical(s), and 54 normal(s)

Non-pass

Test name

Minor

Critical

Normal

tests/parser.test

Message	Group	Filename	Line	Function	Status
@file block missing (Drupal Docs)	Other	parser.test	-1	N/A

tests/permissions.test

Message	Group	Filename	Line	Function
@file block missing (Drupal Docs)	Other	permissions.test	-1	N/A
There should be no trailing spaces	Other	permissions.test	47	N/A
There should be no trailing spaces	Other	permissions.test	130	N/A

weather.module

Message	Group	Filename	Line	Function
put a space between the asterisk and the comment text	Other	weather.module	50	N/A
put a space between the asterisk and the comment text	Other	weather.module	179	N/A
put a space between the asterisk and the comment text	Other	weather.module	212	N/A
Use "elseif" in place of "else if"	Other	weather.module	238	N/A
Use "elseif" in place of "else if"	Other	weather.module	255	N/A
Functions should be called with no spaces between the function name and opening parentheses	Other	weather.module	301	N/A
Use "elseif" in place of "else if"	Other	weather.module	317	N/A
Use "elseif" in place of "else if"	Other	weather.module	408	N/A
Use "elseif" in place of "else if"	Other	weather.module	439	N/A
Use "elseif" in place of "else if"	Other	weather.module	501	N/A
Use "elseif" in place of "else if"	Other	weather.module	505	N/A
Use "elseif" in place of "else if"	Other	weather.module	512	N/A
Use "elseif" in place of "else if"	Other	weather.module	523	N/A
put a space between the asterisk and the comment text	Other	weather.module	551	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	weather.module	805	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	weather.module	806	N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSE	Other	weather.module	809	N/A
Use uppercase for PHP constants, e.g. NULL, TRUE, FALSE	Other	weather.module	817	N/A
in most cases, replace the string function with the drupal_ equivalent string functions	Other	weather.module	974	N/A
in most cases, replace the string function with the drupal_ equivalent string functions	Other	weather.module	1080	N/A
in most cases, replace the string function with the drupal_ equivalent string functions	Other	weather.module	1086	N/A
in most cases, replace the string function with the drupal_ equivalent string functions	Other	weather.module	1088	N/A
in most cases, replace the string function with the drupal_ equivalent string functions	Other	weather.module	1173	N/A
in most cases, replace the string function with the drupal_ equivalent string functions	Other	weather.module	1174	N/A
put a space between the asterisk and the comment text	Other	weather.module	1352	N/A
Use "elseif" in place of "else if"	Other	weather.module	1383	N/A
Use "elseif" in place of "else if"	Other	weather.module	1386	N/A
Use "elseif" in place of "else if"	Other	weather.module	1404	N/A
Use "elseif" in place of "else if"	Other	weather.module	1411	N/A
Use "elseif" in place of "else if"	Other	weather.module	1415	N/A
Use "elseif" in place of "else if"	Other	weather.module	1426	N/A
Use "elseif" in place of "else if"	Other	weather.module	1434	N/A
Use "elseif" in place of "else if"	Other	weather.module	1444	N/A
Use "elseif" in place of "else if"	Other	weather.module	1461	N/A
Use "elseif" in place of "else if"	Other	weather.module	1467	N/A
Use "elseif" in place of "else if"	Other	weather.module	1470	N/A
Use "elseif" in place of "else if"	Other	weather.module	1477	N/A
Use "elseif" in place of "else if"	Other	weather.module	1481	N/A
Use "elseif" in place of "else if"	Other	weather.module	1484	N/A
Use "elseif" in place of "else if"	Other	weather.module	1495	N/A
Use "elseif" in place of "else if"	Other	weather.module	1498	N/A
Functions should be called with no spaces between the function name and opening parentheses	Other	weather.module	1529	N/A
Use "elseif" in place of "else if"	Other	weather.module	1696	N/A
Use "elseif" in place of "else if"	Other	weather.module	1700	N/A
Use "elseif" in place of "else if"	Other	weather.module	1704	N/A
Use "elseif" in place of "else if"	Other	weather.module	1737	N/A
Use "elseif" in place of "else if"	Other	weather.module	1741	N/A
put a space between the asterisk and the comment text	Other	weather.module	1918	N/A
in most cases, replace the string function with the drupal_ equivalent string functions	Other	weather.module	2017	N/A

weather.tpl.php

Message	Group	Filename	Line	Function	Status
@file block missing (Drupal Docs)	Other	weather.tpl.php	-1	N/A

weather_compact.tpl.php

Message	Group	Filename	Line	Function	Status
@file block missing (Drupal Docs)	Other	weather_compact.tpl.php	-1	N/A

weather_parser.inc

Message	Group	Filename	Line	Function
@file block missing (Drupal Docs)	Other	weather_parser.inc	-1	N/A
in most cases, replace the string function with the drupal_ equivalent string functions	Other	weather_parser.inc	39	N/A
Use "elseif" in place of "else if"	Other	weather_parser.inc	211	N/A
Use "elseif" in place of "else if"	Other	weather_parser.inc	234	N/A
Use "elseif" in place of "else if"	Other	weather_parser.inc	248	N/A
Use "elseif" in place of "else if"	Other	weather_parser.inc	285	N/A
Use "elseif" in place of "else if"	Other	weather_parser.inc	305	N/A
Use "elseif" in place of "else if"	Other	weather_parser.inc	327	N/A
Use "elseif" in place of "else if"	Other	weather_parser.inc	332	N/A
Use "elseif" in place of "else if"	Other	weather_parser.inc	346	N/A
Use "elseif" in place of "else if"	Other	weather_parser.inc	382	N/A
Use "elseif" in place of "else if"	Other	weather_parser.inc	392	N/A
Use "elseif" in place of "else if"	Other	weather_parser.inc	400	N/A
Use "elseif" in place of "else if"	Other	weather_parser.inc	541	N/A
Use "elseif" in place of "else if"	Other	weather_parser.inc	600	N/A

All