Ogone | Ubercart payment - 6.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.

Ogone | Ubercart payment - 6.x-2.x

Test status

Status	Result
Last received	Tue, 04/03/2012 - 09:29:38
Last requested	Tue, 04/03/2012 - 09:30:11
Last tested	Tue, 04/03/2012 - 09:30:19
Test count	2

Environment status

MySQL	pass
Code review	fail

Overall Result	pass

Branch information

Identifier	6.x-2.x
Project	Ogone \| Ubercart payment
Link	Project information

Event logView entire log

ID	Message	Timestamp
3528514	Result retrieved by project client.	3 years 47 weeks ago
3528504	Result retrieved by project client.	3 years 47 weeks ago
3528494	Result retrieved by project client.	3 years 47 weeks ago
3528484	Result received from test client #659.	3 years 47 weeks ago
3528479	Result received from test client #654.	3 years 47 weeks ago
3528469	Requested by test client #659.	3 years 47 weeks ago
3528459	Requested by test client #654.	3 years 47 weeks ago
3528444	Test request received.	3 years 47 weeks ago

SummaryPASSED: [[SimpleTest]]: [MySQL] 0 pass(es); [[Coder]]: [Code review] 34 minor(s), 0 critical(s), and 123 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

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

34 minor(s), 0 critical(s), and 123 normal(s)

Non-pass

Test name

Minor

Critical

Normal

theme/uc-ogone-secure-payment-form.tpl.php

Message	Group	Filename	Line	Function
Commits to the Git repository do not require the CVS $Id$ keyword in each file. (Drupal Docs)	Other	uc-ogone-secure-payment-form.tpl.php	2	N/A
@file description should be on the following line (Drupal Docs)	Other	uc-ogone-secure-payment-form.tpl.php	5	N/A
Function documentation should be less than 80 characters per line.	Other	uc-ogone-secure-payment-form.tpl.php	7	N/A
Function documentation should be less than 80 characters per line.	Other	uc-ogone-secure-payment-form.tpl.php	9	N/A

uc_ogone_payment.install

Message	Group	Filename	Line	Function
@file block missing (Drupal Docs)	Other	uc_ogone_payment.install	-1	N/A
There should be no trailing spaces	Other	uc_ogone_payment.install	78	N/A
There should be no trailing spaces	Other	uc_ogone_payment.install	170	N/A

uc_ogone_payment.module

119

Message	Group	Filename	Line	Function
Remove the empty commented line in your function documentation.	Other	uc_ogone_payment.module	11	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	21	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	53	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	77	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	78	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	80	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	86	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	87	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	91	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	92	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	94	N/A
put a space between the asterisk and the comment text	Other	uc_ogone_payment.module	106	N/A
Control statements should have one space between the control keyword and opening parenthesis	Other	uc_ogone_payment.module	112	N/A
Potential problem: use the Form API to prevent against CSRF attacks. If you need to use $_POST variables, ensure they are fully sanitized if displayed by using check_plain(), filter_xss() or similar. (Drupal Docs)	Other	uc_ogone_payment.module	121	N/A
else statements should begin on a new line	Other	uc_ogone_payment.module	160	N/A
There should be no trailing spaces	Other	uc_ogone_payment.module	171	N/A
Control statements should have one space between the control keyword and opening parenthesis	Other	uc_ogone_payment.module	173	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	213	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	215	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	219	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	232	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	233	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	234	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	235	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	236	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	237	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	238	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	239	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	240	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	241	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	242	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	243	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	244	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	245	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	246	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	247	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	248	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	249	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	250	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	253	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	256	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	257	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	258	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	259	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	260	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	261	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	262	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	264	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	265	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	268	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	269	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	270	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	271	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	272	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	274	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	276	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	279	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	280	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	281	N/A
else statements should begin on a new line	Other	uc_ogone_payment.module	286	N/A
Use the matching Drupal theme functions, not raw HTML.	Other	uc_ogone_payment.module	294	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	360	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	373	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	381	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	387	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	392	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	404	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	410	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	416	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	422	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	428	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	458	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	464	N/A
The $text argument to l() should be enclosed within t() so that it is translatable.	Other	uc_ogone_payment.module	471	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	489	N/A
There should be no trailing spaces	Other	uc_ogone_payment.module	491	N/A
Missing period	Other	uc_ogone_payment.module	496	N/A
If you define a @param or @return, you should document it as well.	Other	uc_ogone_payment.module	541	N/A
Function documentation should be less than 80 characters per line.	Other	uc_ogone_payment.module	541	N/A
There should be no trailing spaces	Other	uc_ogone_payment.module	563	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	563	N/A
There should be no trailing spaces	Other	uc_ogone_payment.module	564	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	571	N/A
There should be no trailing spaces	Other	uc_ogone_payment.module	572	N/A
There should be no trailing spaces	Other	uc_ogone_payment.module	576	N/A
There should be no trailing spaces	Other	uc_ogone_payment.module	595	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	598	N/A
There should be no trailing spaces	Other	uc_ogone_payment.module	614	N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote terms	Other	uc_ogone_payment.module	615	N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote terms	Other	uc_ogone_payment.module	616	N/A
in most cases, replace the string function with the drupal_ equivalent string functions	Other	uc_ogone_payment.module	617	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	617	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	624	N/A
else statements should begin on a new line	Other	uc_ogone_payment.module	628	N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote terms	Other	uc_ogone_payment.module	632	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	636	N/A
Use an indent of 2 spaces, with no tabs	Other	uc_ogone_payment.module	661	N/A
Use an indent of 2 spaces, with no tabs	Other	uc_ogone_payment.module	662	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	677	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	680	N/A
in most cases, replace the string function with the drupal_ equivalent string functions	Other	uc_ogone_payment.module	681	N/A
in most cases, replace the string function with the drupal_ equivalent string functions	Other	uc_ogone_payment.module	682	N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote terms	Other	uc_ogone_payment.module	683	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	694	N/A
There should be no trailing spaces	Other	uc_ogone_payment.module	696	N/A
Use the matching Drupal theme functions, not raw HTML.	Other	uc_ogone_payment.module	709	N/A
use a space between the closing parenthesis and the open bracket	Other	uc_ogone_payment.module	723	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	724	N/A
Arrays should be formatted with a space separating each element and assignment operator	Other	uc_ogone_payment.module	739	N/A
in most cases, replace the string function with the drupal_ equivalent string functions	Other	uc_ogone_payment.module	756	N/A
in most cases, replace the string function with the drupal_ equivalent string functions	Other	uc_ogone_payment.module	760	N/A
in most cases, replace the string function with the drupal_ equivalent string functions	Other	uc_ogone_payment.module	761	N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote terms	Other	uc_ogone_payment.module	762	N/A
in most cases, replace the string function with the drupal_ equivalent string functions	Other	uc_ogone_payment.module	766	N/A
There should be no trailing spaces	Other	uc_ogone_payment.module	767	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	768	N/A
use a space between the closing parenthesis and the open bracket	Other	uc_ogone_payment.module	769	N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote terms	Other	uc_ogone_payment.module	770	N/A
missing space after comma	Other	uc_ogone_payment.module	770	N/A
There should be no trailing spaces	Other	uc_ogone_payment.module	770	N/A
There should be no trailing spaces	Other	uc_ogone_payment.module	781	N/A
There should be no trailing spaces	Other	uc_ogone_payment.module	782	N/A
Use an indent of 2 spaces, with no tabs	Other	uc_ogone_payment.module	904	N/A
string concatenation should be formatted with a space separating the operators (dot .) and non-quote terms	Other	uc_ogone_payment.module	942	N/A
missing space after comma	Other	uc_ogone_payment.module	942	N/A
There should be no trailing spaces	Other	uc_ogone_payment.module	942	N/A
Use sentence case, not title case, for end-user strings. (Wikipedia)	Other	uc_ogone_payment.module	957	N/A
There should be no trailing spaces	Other	uc_ogone_payment.module	968	N/A
Use Drupal's format_date(), not PHP's default date().	Other	uc_ogone_payment.module	977	N/A
Use Drupal's format_date(), not PHP's default date().	Other	uc_ogone_payment.module	979	N/A
use a space between the closing parenthesis and the open bracket	Other	uc_ogone_payment.module	1003	N/A
else statements should begin on a new line	Other	uc_ogone_payment.module	1006	N/A
There should be no trailing spaces	Other	uc_ogone_payment.module	1007	N/A
use a space between the closing parenthesis and the open bracket	Other	uc_ogone_payment.module	1009	N/A
There should be no trailing spaces	Other	uc_ogone_payment.module	1012	N/A
Function summaries should be one line only. (Drupal Docs)	Other	uc_ogone_payment.module	1017	N/A
If you define a @param or @return, you should document it as well.	Other	uc_ogone_payment.module	1018	N/A
@param and @return descriptions begin indented on the next line. (Drupal Docs)	Other	uc_ogone_payment.module	1018	N/A
There should be no trailing spaces	Other	uc_ogone_payment.module	1019	N/A
If you define a @param or @return, you should document it as well.	Other	uc_ogone_payment.module	1019	N/A
@param and @return descriptions begin indented on the next line. (Drupal Docs)	Other	uc_ogone_payment.module	1019	N/A
use a space between the closing parenthesis and the open bracket	Other	uc_ogone_payment.module	1021	N/A
missing space after comma	Other	uc_ogone_payment.module	1021	N/A
use a space between the closing parenthesis and the open bracket	Other	uc_ogone_payment.module	1022	N/A
There should be no trailing spaces	Other	uc_ogone_payment.module	1024	N/A
use a space between the closing parenthesis and the open bracket	Other	uc_ogone_payment.module	1026	N/A
There should be no trailing spaces	Other	uc_ogone_payment.module	1028	N/A
Arrays should be formatted with a space separating each element and assignment operator	Other	uc_ogone_payment.module	1034	N/A
Separate comments from comment syntax by a space.	Other	uc_ogone_payment.module	1039	N/A
missing space after comma	Other	uc_ogone_payment.module	1042	N/A

All