In the 7.x-2.x branch the module '.info' file contains a reference to a non-existent SimpleTest file and could/should also contain a link to an /admin page for the module.

CommentFileSizeAuthor
#6 2397173-diff-4-6.txt651 bytesvijaycs85
#6 2397173-module-info-6.patch1.42 KBvijaycs85
#4 2397173-diff-1-4.txt481 bytesvijaycs85
#4 2397173-module-info-4.patch1020 bytesvijaycs85
#1 tidy_module_info_file-2397173-1.patch539 bytesalex_sansom

Comments

alex_sansom’s picture

alex_sansom commented
StatusFileSize
new tidy_module_info_file-2397173-1.patch539 bytes

Attaching .patch file.

alex_sansom’s picture

alex_sansom commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 1: tidy_module_info_file-2397173-1.patch, failed testing.

vijaycs85’s picture

vijaycs85
Location London, UK
commented
Status: Needs work » Needs review
StatusFileSize
new 2397173-module-info-4.patch1020 bytes
new 2397173-diff-1-4.txt481 bytes

Looks good. I don't think we need CommerceBase anymore...

Status: Needs review » Needs work

The last submitted patch, 4: 2397173-module-info-4.patch, failed testing.

vijaycs85’s picture

vijaycs85
Location London, UK
commented
Status: Needs work » Needs review
StatusFileSize
new 2397173-module-info-6.patch1.42 KB
new 2397173-diff-4-6.txt651 bytes

May be missing dependency modules.

Status: Needs review » Needs work

The last submitted patch, 6: 2397173-module-info-6.patch, failed testing.

vijaycs85’s picture

vijaycs85
Location London, UK
commented

Looks like testbot isn't happy or out (Christmas holidays may be). Tests are fine locally. Committed changes in #1. Thanks @alex_sansom

vijaycs85’s picture

vijaycs85
Location London, UK
commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.