CVS edit link for covenantd

The module we have prepared provides an important additional feature to Ubercart shopping cart. We have developed this module several months and it is already in use. We are certain that no other Ubercart module can provide the same functionality.

It allows store administrators to add partial payment possibility as a product feature. They can assign down payment amount and the due date to any product, and manage down payments and partially paid orders in the admin section.

For these products buyers can select if they wish to pay the full price or only the down payment. If they choose down payment, a certain number of days before the due date, as determined by the store administrators, they will receive an email remainder about the remaining payment balance. They can see the information about their partial payments under a new tab in My Account where they can add the remaining balance to the cart and finalize the payment.

At the Ubercart discussion boards we have realized that there is a popular demand for this functionality, so we would like to share this module by releasing it on Drupal.org, under the same license as Drupal.

Thank you.

Comments

sokrplare’s picture

StatusFileSize
new13.02 KB
sokrplare’s picture

Status: Postponed (maintainer needs more info) » Needs review
avpaderno’s picture

Issue tags: +Ubercart, +Module review
sokrplare’s picture

Hi there. Any news on the application review? Is there anything we can do from our side? We would really appreciate your support as we have promised to publish the module soon. Thank you.

awolfey’s picture

On a quick review, you may want to validate that the payment amount is numeric in admin_payment_form.
Messages in drupal_set_message() should use the t() function.
I'm not convinced of the necessity to use views_customfield only to create the downpayments link. I suggest you just make a field handler for that.

Otherwise, this looks good and clean. This looks like a great addition to UC.

I did not test it, and won't have time to.

awolfey’s picture

Forgot one thing. You say "we" in the motivation statement, but just to be clear cvs access is granted to individuals, not companies. From your site it looks like you are an individual, so I'm just checking on this.

avpaderno’s picture

Status: Needs review » Needs work

I am changing status as per previous comments.

I can confirm that CVS accounts are provided to individuals, not companies/organizations; the same is true for drupal.org user accounts.

sokrplare’s picture

Thanks for the responses - I appreciate you taking the time to look at it awolfey!

Yes, I'm an individual :) Multiple people worked on the module so for the statement "we" was used to acknowledge that.

Will take a look at getting those changes made and put up a new version shortly so we can hopefully get this added ASAP. Thanks!

sokrplare’s picture

Status: Needs work » Needs review

Thanks for your comments and advice.

I have modified the module so now there is form validation making sure that the payment amount has numeric value, in both admin_payment_form and feature_form. In both forms amount and due date are now required fields.

Also, t() function has been added to the instance of drupal_set_message() where it was missing.

Regarding views_customfield, it is not used only to create the down payments link, but more importantly to load the remaining balance from the database.

Finally, payment options selection next to "Add to Cart" has been modified so now it shows the actual amount next to "Full Payment" option.

Please find attached the updated module for your review.

Thank you.

sokrplare’s picture

It is not possible to upload the file. I have tried with tar.gz and as a zip archive. The error message is:

"Validation error, please try again. The file you attempted to upload may be too large. If this error persists, please contact the site administrator."

The file is only 14K.

Any ideas what may be wrong?

sokrplare’s picture

StatusFileSize
new13.14 KB
sokrplare’s picture

OK, it worked when I selected the file and clicked on "Save" without clicking on "Attach".

sokrplare’s picture

*Bump*

We've got people who chipped in to make this happen waiting for it to get up (plus submitting feature enhancements that we'd love to get included, but it is getting messy just doing things by email). Any ETA on when this can get approved? Do you need anything else from me?

sokrplare’s picture

Need anything from me? How can I get this approved?

sokrplare’s picture

Hey folks! Just bumping this up - people are asking when this will be released so hoping this can get approved soon! Thanks!

sokrplare’s picture

*Bump*

Just had another request for this module. Would love to be able to direct them to drupal.org to grab it. Thanks all!

mo6’s picture

If you'd like, I could make this project live for you and make you maintainer as soon as your CVS application is approved?

avpaderno’s picture

@george@dynapres.nl: You would create the module before it gets approved.

mo6’s picture

I can create the project on drupal.org for you. Apart from that you need to apply for a CVS account, which has to be approved, which will take some time, after that you can maintain your project via CVS. You can also wait until after the d.o. git migration which will happen in the next couple of months, after which anyone with an d.o. account will have access to the Drupal repository.

sokrplare’s picture

That would be great, George! Obviously, it would be good to get a CVS, but the biggest deal is just getting this module up!

mo6’s picture

avpaderno’s picture

I thought it was clear a project used for a CVS application cannot be created before it gets approved; it's not up the applicant to decide if the project can be created or not.

mo6’s picture

@kiamlaluno I'm sorry if I'm acting prematurely here, but there's definitely a demand for this module. There's an enormous backlog in processing for CVS applications (so the CVS application process is completely broken imho) and within a couple of months the git migration will have taken place, with it all d.o. users will have the rights to create projects.

Meanwhile, thumbs up for the uc_downpayment module: clearly a nice piece of work. If a maintainer with enough d.o. access comes along in the next couple of months: approve this application and change the ownership of the module to @coventantd. If not: we'll all head into git territory.

avpaderno’s picture

The GIT migration will allow new users to have GIT access when they ask for it, but they will not be able to create new projects.
To create new projects, the project will need to be reviewed in some way.

Since then, a module that is being used for a CVS application cannot be created before it gets approved.
Contrary to what it could seem, it's not just the users that are under review, but also the proposed module/theme; in fact, a CVS maintainer can decline a CVS application because the module duplicate the work done in an already existing project.

mo6’s picture

@kiamlaluno Thanks for the update.

Here's some background on the module that is currently subject of this application: http://www.ubercart.org/forum/bounties/15764/1000_partial_payment_module

The module works very nice.

sokrplare’s picture

@george@dynapres.nl - thanks for getting that up, I know there are a lot of people on the thread who have been waiting a long time and appreciate it!

@kiamlaluno - I'm definitely not trying to circumvent any process here. What needs to happen to get the CVS application approved? Folks in the community are using and have paid to support this module - it's hurting the community to not have it up there and let us maintain it and get some improvements put in place. Thanks for keeping an eye on things here Alberto!

webmestre’s picture

Where and when should the "downpayment" information appear ?

at "Home → Administer → Store administration", I get an empty page when just a title : "Partially paid orders"
When Checking out, nothing appends.

sokrplare’s picture

The downpayment can be setup on a product as a product "feature". Just edit the product and then go to the features tab and add the downpayment info. After that the rest will fall into place (ie a drop-down will appear on the product, etc.)

fossle’s picture

Component: Miscellaneous » miscellaneous

Does the user have to be paying by Paypal or can they use credit card/ check too? This sounds like a great module. If we wanted to test should I use the 12/13 attachment?

avpaderno’s picture

Component: miscellaneous » new project application
sokrplare’s picture

The user can use any payment method I believe. Have mainly tested with PayPal, but it isn't written to be payment gateway specific. Yes, the 12/13 attachement (comment #11) is the one to go for! Would love to see any review / tweaks you might have so we can hopefully get this approved ASAP!

mrconnerton’s picture

Just to clarify the new GIT features when the migration goes live: ANY Drupal user will have the ability to create "sandbox" projects, which are stripped down Drupal projects without releases or a short name. This will let them commit code and have an issue queue dedicated to the sandbox for development. When the module is ready to become an official project, it can then be "promoted" into one.

In a related note, I was asked to look at this module as I have developed something similar. I will make a plan to install this module this/next week to go over it.

sokrplare’s picture

Thanks for the GIT clarification, mrconnerton.

Will be looking forward to hearing your feedback on this module when you get a minute to look through it next week!

sokrplare’s picture

Any progress on this? I'm starting to give up hope modules are ever approved anymore!

sokrplare’s picture

With Git in place, do we need to do a new process to get this module added/approved. Have the username covenantd setup for Git, per comment #32 should we make a sandbox project so it is ready for promotion?

avpaderno’s picture

Every Drupal.org user can create sandboxes; http://drupal.org/documentation/git reports more information.

arianek’s picture

Status: Needs review » Postponed

Hi. Please read all the following and the links provided as this is very important information about your CVS Application:

Drupal.org has moved from CVS to Git! This is a very significant change for the Drupal community and for your application. Please read the following documentation on how this affects and benefits you and the application process:
Migrating from CVS Applications to (Git) Full Project Applications

  • The status of this application will be put to "postponed" and by following the instructions in the above link, you will be able to reopen it.
  • Or if your application has been "needs work" for more than 5 weeks, your application will be marked as "closed (won't fix)". You can still reopen it, by reading the instructions above.
le_petit_basil’s picture

@covenantd,
+1
This is Arthur here (one of the module $ contributors) ... I sincerely do hope you go through the Git Project Application process for this. It's a great start, that could grow into something, especially if it gets the added momentum of additional contributors.
Best of Luck.

-A

sokrplare’s picture

Hi Arthur!

We're hoping to kick that off in the next week or so - would love to see this actually live!

Joshua

sokrplare’s picture

Status: Postponed » Closed (fixed)

For anyone that stumbles across this in the future, we've setup the project in a Git sandbox at: http://drupal.org/sandbox/covenantd/1103142

Hooray for the Promised Land of Git!

Closing this accordingly and will open a new ticket under the Project applications queue soon!