Closed (won't fix)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
27 Mar 2012 at 02:03 UTC
Updated:
20 Dec 2012 at 23:33 UTC
This project creates installments for Commerce orders. One can add, edit, delete and pay installments. Installments are written with using entity api. Installments are supported by views and one can get detailed reports about installments.
http://drupalcode.org/sandbox/zulfierken/1502028.git
This is a drupal 7 project.
Prject link : http://drupal.org/sandbox/zulfierken/1502028
I added the zipped module if there was a problem in git
| Comment | File | Size | Author |
|---|---|---|---|
| commerce_order_installment.zip | 72.34 KB | zulfierken |
Comments
Comment #1
adammaloneYou can find the result of the autoreview at this URL: http://ventral.org/pareview/httpgitdrupalorgsandboxzulfierken1502028git
The main issues reported are:
When you've corrected all the coding standards issues and pushed your changes simply set the tag to 'needs review' and the application can proceed.
Comment #2
zulfierken commentedI committed with changing codes
Comment #3
jpontani commentedYou still have a substantial amount of coding style issues to fix.
Your branch should be named "7.x-1.x", not 7.x-1.x-dev. See Moving from a master to a major version branch.
Also, your repo base directory should be inside your module, it should not include the "commerce_order_installment" directory.
Comment #4
sokrplare commentedLove the idea for this module, but looks like a lot of client-specific code still included (ie school fees related code). That should ideally be cleared out too so this can be generic and not specifically for a school tuition system.
Would love to see this become a full project!
Comment #5
joshmillerJust spent 30 minutes looking for this sandbox... @jpontani pointed me in the right direction. Multiple Payments in Drupal Commerce are something that was considered very difficult by @rszrama: http://drupal.org/node/1096504
I can assure you the idea of this module is very needed in the community. I'll see if we can rally a bit more coding support.
Comment #6
klausiClosing due to lack of activity. Feel free to reopen if you are still working on this application.