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

Replace deprecated usage of CRM_Core_BAO_Setting #44

Merged
merged 2 commits into from
Sep 19, 2024

Conversation

dontub
Copy link
Contributor

@dontub dontub commented Apr 18, 2024

Additionally add configuration for phpcs, phpstan, and phpunit and a first test.

For now phpcs and phpstan only analyse tests. This should be changed, though.

systopia-reference: 24737

Additionally add configuration for phpcs, phpstan, and phpunit and a
first test.
@dontub dontub requested a review from jensschuppe April 18, 2024 10:25
@jensschuppe jensschuppe added this to the 1.2 milestone Sep 19, 2024
@jensschuppe jensschuppe added status:reviewed and tested Code has received thorough review and test and is ready to be committed/merged enhancement labels Sep 19, 2024
@jensschuppe jensschuppe merged commit ba172d3 into master Sep 19, 2024
9 checks passed
@jensschuppe jensschuppe deleted the replace-deprecated-code-add-test branch September 19, 2024 08:24
@jensschuppe jensschuppe added status:fixed The issue has been resolved (usually by committing/merging code) and removed status:reviewed and tested Code has received thorough review and test and is ready to be committed/merged labels Sep 19, 2024
@jensschuppe jensschuppe linked an issue Sep 19, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement status:fixed The issue has been resolved (usually by committing/merging code)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refactor deprecated: CRM_Core_BAO_Setting::setItem
2 participants