CiviCRM Community Forums (archive)

Many moons ago I asked about adding a surcharge to credit card payments, and worked with a friend to make it happen.  In reality, he worked, I told him what I wanted, tested, complained, tested again.  It turns out to be MUCH more complicated than originally thought.  The result is both hook and hack.  I leave it to the real developers to decide what should be added to the core. 

The patch, drupal module, and fully modified civi module (3.1.3) is available at: http://github.com/mariagwyn/civiCC.  Please feel free to branch and modify.  This is only a beginning.  Please don't modify my version.  I will update it if others offer improvements (hint hint).

GOAL: The patch and accompanying drupal module allows a 5% surcharge to be added to all CC payments.  The goal was to automatically add a preset percentage, and create a line item both in the civi record and the emailed receipt.  These goals were met with more and less success.

For EVENTS the patch meets all goals.
For MEMBERSHIP, the patch is really a hack.  IT does not allow line-item description, and depending on the method of adding/renewing the membership, it does not always show the total in the emailed receipt.  It does show the total in the civi payment record.

Here are the files modified:
 civicrm/CRM/Contribute/Form/Contribution/Main.php |    2
 civicrm/CRM/Member/Form/Membership.php            |    2
 civicrm/CRM/Member/Form/MembershipRenewal.php     |    2
 civicrm/CRM/Price/BAO/Set.php                     |    2
 civicrm/CRM/Utils/Hook.php                        |   20

Surely the most offending modifications are to all MEMBERSHIP files.  There seemed to be no other way than add 5% into the core files.  Bad form, I know, and really not worthy of a commit.  This is the area that needs work by experts.

The precessAmount HOOK seems worthy of a commit (main.php; set.php; hook.php) as that is what is called by the drupal module, where the amount can be modified, as well as text, labels, etc.

Attached are the patch for Civi, and the drupal module files (called civihooks, though on my site it lives in the 'dirtyfixes' folder with other hacks).

- maria

According to Brad (developer) the hook gets called when the build amount doesn't get called, which only gets called if there is no price set.  Since events use price sets (on my setup), the processAmount gets called instead of build amount.  I am not sure what happens when there is no price set on events, we didn't test for that.  Also, the membership payment setup is way different, so we couldn't get a hook to work at all there, which is why it is coded in the core (blech).

Issue started: http://issues.civicrm.org/jira/browse/CRM-5935

why not just add a price set with a manditory field as a surcharge?

The attached (see above) surcharge works on 3.1.4.  I am in the process of upgrading to 3.3, and will let you know if it works on 3.3. 

A very late update: The patch works up through CiviCRM 3.3.3.  The line numbers of the modified files are different but the code/files are the same.

Actually, after reading this: http://forum.civicrm.org/index.php?topic=17758.0  It seems that a membership page is a contribution page at heart.  Either way, this doesn't work for our needs but very informative and instructional.  Thanks!

Just wanted to post a thank you for this....saved me loads of time implementing a similar requirement. I had to tweak mine for a Joomla environment but having this as an example saved my bacon! Many Thanks!

Any chance you could post your Joomla solution?