Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Major refactor and cleanup for improved stability #80

Open
wants to merge 84 commits into
base: master
Choose a base branch
from

Conversation

mattwire
Copy link

We've done some major refactoring to make this extension much more stable and update it to the latest coding standards / requirements. And added a feature to automatically calculate the gift-aid amount when creating/editing contributions.

Changes:

  • Drop support for older versions of CiviCRM and PHP.
  • Regenerate DAO/civix files.
  • Convert to short array syntax.
  • Cleanup code.

Features:

  • Calculate gift aid when creating/editing a contribution.

@mattwire mattwire force-pushed the mjwconsulting branch 2 times, most recently from 58865b0 to 2a622d2 Compare June 30, 2019 10:49
@tapashdatta
Copy link

This branch throws an error when "Adding to the batch"

"Error: Call to undefined method CRM_Utils_Type::T_DECIMAL() in CRM_Civigiftaid_DAO_BatchSettings::fields() (line 143 of /home/webadmin/public_html/cmm-uat/sites/default/files/civicrm/ext/uk.co.compucorp.civicrm.giftaid/CRM/Civigiftaid/DAO/BatchSettings.php).'

Warning: Invalid argument supplied for foreach() in CRM_Civigiftaid_Utils_GiftAid::isContributionEligible() (line 508 of /home/webadmin/public_html/cmm-uat/sites/default/files/civicrm/ext/uk.co.compucorp.civicrm.giftaid/CRM/Civigiftaid/Utils/GiftAid.php).
--

@mattwire
Copy link
Author

@tapashdatta Thanks - that should be resolved - there was an issue with the schema definition.

@mattwire mattwire force-pushed the mjwconsulting branch 4 times, most recently from ee738bc to 2047f6e Compare July 28, 2019 15:58
@@ -0,0 +1,21 @@
## Release 3.3
**In this release we update profiles to use the declaration eligibility field instead of the contribution.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mattwire I'm not sure I understand this change? Does this mean the field "eligible for gift aid" on the contribution is no longer used? I think users need this to find the contributions that should be eligible for gift aid to add to batch...

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The user needs to create a gift aid declaration - and there are situations where you want to create a gift aid declaration but not apply it to the contribution. By displaying the declaration Yes/Yes past 4 years/No to the user they are explicitly updating the declaration and contributions will by default use that. The contribution now has "Yes/No" because the 3rd option makes no sense at a contribution level.

…r in PHP meaning that we always ensure they are up to date with the latest extension
mattwire and others added 30 commits May 4, 2020 17:50
…bly doesn't work anymore and adds complexity to the code.
Fix install setting default batch when it shouldn't; add first two tests
Add more thorough tests for set eligibility and fix an issue found
…ed-at-runtime

Handle multiple contribs created at runtime
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants