search

backoffice / BOT-Partial-Payment-Extension

1 stars 1 forks source link

Compatibility with CiviCRM 4.7.x? #2

Open sgladstone opened 8 years ago

sgladstone commented 8 years ago

Has this extension been used successfully with CiviCRM version 4.7.x? Any known issues?

anumgoel commented 8 years ago

No, we have not tested it with v4.7.x Development was done for CiviCRM v4.5 but then we did some testing with v4.6 and it worked.

philmb commented 6 years ago

My experience: Tried installing with 4.7.21 and received the following error when refreshing the list of extensions (before it was even installed). (Joomla 3.7.5)

Parse error in extension: Failed to parse info XML: File=/---------/media/civicrm/custom_ext/BOT-Partial-Payment-Extension-master/bot.partial.payment/info.xml Line=26 Column=13 Code=76: Opening and ending tag mismatch: urls line 9 and extension File=/---------/media/civicrm/custom_ext/BOT-Partial-Payment-Extension-master/bot.partial.payment/info.xml Line=27 Column=1 Code=77: Premature end of data in tag urls line 7 File=/h---------/media/civicrm/custom_ext/BOT-Partial-Payment-Extension-master/bot.partial.payment/info.xml Line=27 Column=1 Code=77: Premature end of data in tag extension line 2

ddoligalski commented 6 years ago

@philmb, that error doesn't appear like it would be related to 4.7, rather it looks like a malformed xml file. We will check it out.

davesbot commented 6 years ago

There was an error in the xml file that previous version just ignored.

Just to clarify, this extension only provides a function that allows some frontend code to make a partial payment on a pending contribution. The code at https://github.com/backoffice/bot.roundlake.paymentui provides and example of a UI that we constructed. It is not meant to be an out of the box solution.

This version should work in 4.7 as long as the core partial payment framework hasn't changed. Based on my review of https://github.com/civicrm/civicrm-core/blob/master/CRM/Contribute/BAO/Contribution.php it looks like that is the case.