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

subhojit777 created an issue. See original summary.

subhojit777’s picture

Version: 8.x-1.0-alpha2 » 8.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new3.41 KB
subhojit777’s picture

gauravjeet’s picture

Status: Needs review » Fixed

Patch committed.. Thanks subhojit777

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.