This is for the uc_credit.module. I couldn't find it specifically in the "Project" list to target this post better.

While the fix discussed here ( http://www.ubercart.org/forum/general_discussion/8157/storage_credit_car... ), appears to fix the problem for orders entered through the Checkout / front end, orders entered by Admins in the back end interface fail to save the credit card type.

Attached is a patch which I think fixes this problem for those that need/want to save the Credit Card Type that was used for a particular order.

I've just recently started looking at how to create patches again after having had problems with it a couple years ago. Please let me know if the patch wasn't created correctly for some reason and where I need to change anything to make them easier to work with. Also, should I submit patches here or over at drupal.org?

Thanks,
Stephen

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rszrama’s picture

Assigned: Unassigned » rszrama
Issue tags: +ubercamp sprint
rszrama’s picture

Status: Needs review » Fixed

The module's changed quite a bit since this patch was made, but I was able to apply the changes manually. Thanks!

DigitalFrontiersMedia’s picture

No problem.

Yeah, I noticed you guys had been pretty busy with the Ubercode. Is there a preferred method for handling submitting patches for such a rapidly moving target?

rszrama’s picture

If you have Bzr access, doing it against the latest dev code would be best. Otherwise the latest dev release on Drupal (or latest stable release if there is no dev release) would be preferred.

Status: Fixed » Closed (fixed)
Issue tags: -ubercamp sprint

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