Sandbox: https://drupal.org/sandbox/cameronbprince/1898490
Git clone: git clone http://git.drupal.org/sandbox/cameronbprince/1898490.git commerce_gpg
Drupalcode tree: http://drupalcode.org/sandbox/cameronbprince/1898490.git/tree
Version: 7.x
Commerce GPG provides a credit card payment method for Drupal Commerce that encrypts credit card information using the PHP: GnuPG extension. A common use case would be for shops that would prefer to use an existing terminal for manual card processing or have an accounting system that handles processing. The GPG encrypted block is available in a commerce-order token for embedding in emails generated by the order action. The block is typically decrypted by an email client plugin.
This module was not created to be a method of secure credit card storage for later online use. It purposely provides no method of decryption and is strictly designed for secure transmission of the credit card data only.
Comments
Comment #1
ykyuen CreditAttribution: ykyuen commentedAutomatic Review: (http://ventral.org/pareview/httpgitdrupalorgsandboxcameronbprince1898490git)
1. There is some issue need to be fixed.
2. You should create another branch called 7.x-1.x and set it to default. See http://drupal.org/node/1127732
Manuel Review:
1. Try to add more @param and @return directives in the comments of each function.
Comment #2
alexmoreno CreditAttribution: alexmoreno commentedFirstly thank you very much for your effort.
Just a quickly thing, the url in the description is not right. Please, take note of the correct one:
git clone http://git.drupal.org/sandbox/cameronbprince/1898490.git commerce_gpg
The module itself has been written quite nicely, the code is easily readable, but it could be improved a bit following the Drupal coding standars . Please, read "Doxygen and comment formatting conventions": http://drupal.org/node/1354
The current git branch also must be changed, from master to Drupal 7.
You should also get a review bonus to see your project reviewed fastly: http://drupal.org/node/1410826
And a final issues throwed by the paraview tool (sorry for that):
Thank you.
Comment #3
ain CreditAttribution: ain commentedAutomated review
As stated above, there are a few errors in your module, please see http://ventral.org/pareview/httpgitdrupalorgsandboxcameronbprince1898490git
Manual review
Recommendations
NB! To everyone who has commented above: if you find errors in the applications, please also make sure to set the project status to needs work!
Comment #4
cameron prince CreditAttribution: cameron prince commentedAll issues identified by the automated review have been addressed. The README.txt file and the project page have been expanded to include reference links for the installation of the required PHP GnuPG extension. Additional comments have been added to the module as well. Finally, a 7.x-1.x branch was created and the original master was removed.
Comment #5
bekirdag CreditAttribution: bekirdag commentedI have checked the code, seems fine, didn't see any problems. I think this one is ready to go.
Comment #6
klausiSorry for the delay, but you have not listed any reviews of other project applications in your issue summary as strongly recommended in the application documentation.
manual review:
But that are not application blockers, so ...
Thanks for your contribution, cameronbprince!
I updated your account to let you promote this to a full project and also create new projects as either a sandbox or a "full" project.
Here are some recommended readings to help with excellent maintainership:
You can find lots more contributors chatting on IRC in #drupal-contribute. So, come hang out and get involved!
Thanks, also, for your patience with the review process. Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.
Thanks to the dedicated reviewer(s) as well.
Comment #8
cameron prince CreditAttribution: cameron prince commentedThank you and sorry for the delayed reply. I had an issue with the email account my d.o account was associated with. I will promote the project and make your recommended improvements. Thank you again.
Comment #9
TruthJustice CreditAttribution: TruthJustice commentedHi,I'm using the Commerce GPG with commerce-kickstart drupal 7.
The problem is that the buyer's details are being sent to my mail except for the credit card number. And nothing is encrypted in it.
I've created couple of keys. One is public and the second is private. I've uploaded to the server the public key.
The question is: Why don't I get the credit number to the mail? and nothing is encrypted?
For your rapid answer,
TrustJustice
Comment #10
klausiThis project application is closed. Please report any problems to the module's issue queue, not here.
Comment #10.0
klausiAdded information related to saving card data for reuse.