Problem/Motivation
After running code sniffer (Drupal and DrupalPractice) I found some issues:
FILE: ...rce-8/modules/contrib/commerce_payeezy/commerce_payeezy.info.yml
----------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 1 LINE
----------------------------------------------------------------------
1 | WARNING | Remove "project" from the info file, it will be added
| | by drupal.org packaging automatically
1 | WARNING | Remove "version" from the info file, it will be added
| | by drupal.org packaging automatically
----------------------------------------------------------------------
FILE: ...rce_payeezy/src/Plugin/Commerce/PaymentGateway/HostedGateway.php
----------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------
143 | WARNING | [x] A comma should follow the last multiline array
| | item. Found: $x_trans_id
----------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------
FILE: ...eezy/src/Plugin/Commerce/PaymentGateway/PayeezyOnsiteGateway.php
----------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
----------------------------------------------------------------------
41 | WARNING | [ ] Possible useless method overriding detected
197 | WARNING | [x] A comma should follow the last multiline array
| | item. Found: ]
436 | WARNING | [x] A comma should follow the last multiline array
| | item. Found: ]
----------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------
Time: 302ms; Memory: 13.25Mb
FILE: ...rce-8/modules/contrib/commerce_payeezy/commerce_payeezy.info.yml
----------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
----------------------------------------------------------------------
6 | WARNING | All dependencies must be prefixed with the project
| | name, for example "drupal:"
7 | WARNING | All dependencies must be prefixed with the project
| | name, for example "drupal:"
8 | WARNING | All dependencies must be prefixed with the project
| | name, for example "drupal:"
----------------------------------------------------------------------
Time: 175ms; Memory: 10Mb
Proposed resolution
Fix the issues.
Remaining tasks
N/A
User interface changes
N/A
API changes
N/A
Data model changes
N/A
Comments
Comment #2
subhojit777Comment #3
subhojit777Comment #5
gauravjeet commentedPatch committed.. Thanks subhojit777