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
[php8.2] compliant ajax method for custom data on pledge form #29228
Conversation
🤖 Thank you for contributing to CiviCRM! ❤️ We will need to test and review this PR. 👷 Introduction for new contributors...
Quick links for reviewers...
|
This is in line with civicrm#29228 and civicrm#29241 and already converted forms and 1) handles localised money in custom fields 2) reduces notices 3) calls less legacy code
This is in line with civicrm#29228 and civicrm#29241 and already converted forms and 1) handles localised money in custom fields 2) reduces notices 3) calls less legacy code
This is in line with civicrm#29228 and civicrm#29241 and already converted forms and 1) handles localised money in custom fields 2) reduces notices 3) calls less legacy code
This is in line with civicrm#29228 and civicrm#29241 and already converted forms and 1) handles localised money in custom fields 2) reduces notices 3) calls less legacy code
Deleting a pledge results into a database error after this change. Perhaps, |
thanks @jitendrapurohit I didn't realise delete would be affected - I just pushed up to use |
@jitendrapurohit I also put up #29701 to address that screed of red notices I saw |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have completed r-run and and everything appears to be good during testing 👍
thanks @jitendrapurohit - merging based on your review |
Overview
Use php8.x compliant ajax method for custom data on pledge form
Before
The pledge form is loading custom data in a way which is not php8.2 compliant & calls 3 toxic functions to do so
After
The Custom data is loaded by ajax. I ensured that it works with other currency separators in the process (hence the odd ampersands)
Technical Details
I tested on both new & edit
Comments