Closed (won't fix)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
7 Jul 2015 at 06:45 UTC
Updated:
19 Sep 2015 at 10:26 UTC
Jump to comment: Most recent
Comments
Comment #1
impol commentedHello, thanks for the module.
Please change the title of this task to proper D7 -> [D7]
Please wrap 'git' code into 'code' tag.
Please set the correct default branch https://www.drupal.org/node/1659588
README.txt is missing
Module has a lot of code style defects during review from http://pareview.sh
I think you needs to exclude 'nbproject' folder from project root directory and don't need to store all modules in 'DrupalCommerce' folder.
Code review:
You don't need have an empty .install files.
(*) At now I think you needs to follow drupal coding standards before manual code review.
Comment #2
impol commentedComment #3
multisafepay commented@Impol, Thanks for your fast answer. I will check your feedback and update it as soon as possible.
Comment #4
multisafepay commentedComment #5
PA robot commentedClosing due to lack of activity. If you are still working on this application, you should fix all known problems and then set the status to "Needs review". (See also the project application workflow).
I'm a robot and this is an automated message from Project Applications Scraper.