$ phpcs --standard=Drupal --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml post_api/
Xdebug: [Step Debug] Time-out connecting to debugging client, waited: 200 ms. Tried: localhost:9000 (through xdebug.client_host/xdebug.client_port) :-(

FILE: C:\Users\SI-001\Downloads\post_api\post_api.module
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
45 | WARNING | [x] '@todo: add config setting for granular logging.' should
| | match the format '@todo Fix problem X here.'
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

FILE: C:\Users\SI-001\Downloads\post_api\src\Form\PostApiQueueForm.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
19 | WARNING | The class short comment should describe what the class does and
| | not simply repeat the class name
--------------------------------------------------------------------------------

FILE: C:\Users\SI-001\Downloads\post_api\src\Form\PostApiSettingsForm.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
13 | WARNING | The class short comment should describe what the class does and
| | not simply repeat the class name
--------------------------------------------------------------------------------

FILE: ...sers\SI-001\Downloads\post_api\src\Plugin\QueueWorker\PostApiQueueBase.php
--------------------------------------------------------------------------------
FOUND 2 ERRORS AND 1 WARNING AFFECTING 3 LINES
--------------------------------------------------------------------------------
176 | ERROR | [x] Use null coalesce operator instead of ternary operator.
178 | ERROR | [x] Use null coalesce operator instead of ternary operator.
226 | WARNING | [x] '@todo: allow API rate limit customization based on more
| | granular time' should match the format '@todo Fix problem
| | X here.'
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 3 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

FILE: C:\Users\SI-001\Downloads\post_api\src\Service\AddToQueue.php
--------------------------------------------------------------------------------
FOUND 1 ERROR AND 1 WARNING AFFECTING 2 LINES
--------------------------------------------------------------------------------
80 | WARNING | [x] '@todo: display warning if uid is not set for the item.'
| | should match the format '@todo Fix problem X here.'
97 | ERROR | [x] Use null coalesce operator instead of ternary operator.
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

Time: 5.06 secs; Memory: 10MB

Comments

Rakhi Soni created an issue. See original summary.

rakhi soni’s picture

Assigned: rakhi soni » Unassigned
Status: Active » Needs review
StatusFileSize
new3.65 KB

Kindly review patch,,

shamir.vs’s picture

Status: Needs review » Reviewed & tested by the community
StatusFileSize
new44.13 KB

Hi @rakhi-soni, Patch #2 applied successfully and all phpcs issues are fixed

patch

  • swirt committed fc82554a on 2.x
    Issue #3303848 by Rakhi Soni, shamir.vs: Drupal Coding Standard As Per...
swirt’s picture

Issue tags: -

Thank you both for your contributions. This will go out with the 2.0.2 release.

swirt’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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