Skip to content

[REF] Post move cleanup - switch from $form to $this #20160

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

Merged
merged 1 commit into from
Apr 27, 2021

Conversation

eileenmcnaughton
Copy link
Contributor

Overview

[REF] Post move cleanup - switch from $form to $this

Before

$form = $this & $form used

After

$this used

Technical Details

Comments

@civibot
Copy link

civibot bot commented Apr 27, 2021

(Standard links)

@seamuslee001
Copy link
Contributor

Test fails relate

CRM_Contribute_Form_Task_PDFLetterCommonTest::testBuildContributionArray
Non-static method CRM_Contribute_Form_Task_PDFLetter::buildContributionArray() should not be called statically

/home/jenkins/bknix-dfl/build/core-20160-18ywr/web/sites/all/modules/civicrm/tests/phpunit/CRM/Contribute/Form/Task/PDFLetterCommonTest.php:155
/home/jenkins/bknix-dfl/build/core-20160-18ywr/web/sites/all/modules/civicrm/tests/phpunit/CiviTest/CiviUnitTestCase.php:226
/home/jenkins/bknix-dfl/extern/phpunit7/phpunit7.phar:615

CRM_Contribute_Form_Task_PDFLetterCommonTest::testPostProcessGroupByContact
Undefined index: contribution

/home/jenkins/bknix-dfl/build/core-20160-18ywr/web/sites/all/modules/civicrm/CRM/Contribute/BAO/Contribution.php:5301
/home/jenkins/bknix-dfl/build/core-20160-18ywr/web/sites/all/modules/civicrm/CRM/Contribute/Form/Task/PDFLetter.php:322
/home/jenkins/bknix-dfl/build/core-20160-18ywr/web/sites/all/modules/civicrm/CRM/Contribute/Form/Task/PDFLetter.php:209
/home/jenkins/bknix-dfl/build/core-20160-18ywr/web/sites/all/modules/civicrm/tests/phpunit/CRM/Contribute/Form/Task/PDFLetterCommonTest.php:288
/home/jenkins/bknix-dfl/build/core-20160-18ywr/web/sites/all/modules/civicrm/tests/phpunit/CiviTest/CiviUnitTestCase.php:226
/home/jenkins/bknix-dfl/extern/phpunit7/phpunit7.phar:615

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants