Fix the warnings/errors reported by PHP_CodeSniffer

Raveen Thakur → made their first commit to this issue’s fork.

added 1 commit

Since this issue is using PHP_CodeSniffer, let us rescope this issue.

The issue summary should always describe what the issue is trying to fix and, in the case of coding standards issues, show which command and arguments have been used and which report that command shown. In this way, project maintainers can verify the patch/MR fixes all the warnings/errors.

Issue summary updated.

Unassigning due to non activity.

Shruu_rao → made their first commit to this issue’s fork.

I would rather first apply the patch given in 📌 Automated Drupal 10 compatibility fixes Needs review , and then fix this.

pray_12 → made their first commit to this issue’s fork.

Hi,
adressed the #13 comment and applied the patch .There are many phpcs errors and warnings that need to be resolved.

@pray_12 please check this if you have time https://git.drupalcode.org/project/paypal_plus/-/merge_requests/2

I left out the warnings/errors about the functions not prefixed by the module machine name, or the ones about using dependency injection, as those require changes that are not backward compatible. I would leave those to separate issues.

The list of warnings/errors has been updated for this merge request.

Let's fix the dependency injection problems in another issue.

I would postpone this issue on 📌 \Drupal calls should be avoided in classes, use dependency injection instead Needs work , which requires less changes.