GitHub PushEvent at 2014-11-21 01:11:37
Repository: civicrm-core
Actor: davecivicrm
Payload: {"shas":[["bccf4a0035280a132324f7eb4b965f7821968931","dc1e55ee830a357f61796a4d04a0dd2c3bb65d49@civicrm.org","Remove redundant help link","Coleman Watts",true],["9b7e7d4c3abac6f11ab7bd41a6f07cd5fc340720","d277050ff08fa22c2e6749398f40a9cbf59731b8@giantrabbit.com","CRM-15622: Fix for incorrect Fee Amount.\n\nThere's a bunch of fee_amount fields in this section of the code leading\nto a lot of confusion (see CRM-15372). Many of the payment processors set\nthe fee_amount field in the result array they return which sets up the\ncontribution.fee_amount field, but the Authorize.Net AIM API doesn't\nreturn fee information, so Authorize.Net doesn't set this field. Instead\nof setting this field to 0 there, I thought it would be better to not\nrequire that the payment processor set this field. Since some of the\npayment processors return the same array that was passed into them and the\npayment processors don't need the fee_amount field as an argument, I think\nthat setting that to NULL before passing it in gets us the correct output.\n\nNote that we still need the fix from CRM-15372 to get the\nparticipant.fee_amount field set correctly because the code that sets up\nthe pariticpant shares the field array that the contribution creation\nuses even though they don't have the same 'fee_amount' field.","Peter Haight",true],["33a429d4e1ec211e0d02d627ca81e96a04240fba","dc1e55ee830a357f61796a4d04a0dd2c3bb65d49@civicrm.org","CRM-15625 - Add support for contribution api honor_type_id param","Coleman Watts",true],["cbf194f4e1aa0950e9ad7fb62a920b2e19c02b87","dc1e55ee830a357f61796a4d04a0dd2c3bb65d49@civicrm.org","Merge pull request #4591 from colemanw/CRM-15625\n\nCRM-15625 - Add support for contribution api honor_type_id param","colemanw",true],["511f1e3467b3ac06dfaf2e7a75ac65ff12123c74","bfcdf3e6ca6cef45543bfbb57509c92aec9a39fb@civicrm.org","Merge pull request #4586 from giant-rabbit/CRM-15622\n\nCRM-15622: Fix for incorrect Fee Amount.","Dave Greenberg",true]],"size":5,"ref":"refs/heads/4.5","head":"511f1e3467b3ac06dfaf2e7a75ac65ff12123c74"}