inesh@dinesh:/var/www/html/drupalvb$ vendor/bin/phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig web/modules/contrib/civicrm_entity_leaflet/

FILE: /var/www/html/drupalvb/web/modules/contrib/civicrm_entity_leaflet/src/Plugin/views/style/CivicrmEntityAddressLeafletMap.php
------------------------------------------------------------------------------------------------------------------------------------------
FOUND 4 ERRORS AFFECTING 4 LINES
------------------------------------------------------------------------------------------------------------------------------------------
414 | ERROR | [x] Use null coalesce operator instead of ternary operator.
493 | ERROR | [x] Use null coalesce operator instead of ternary operator.
647 | ERROR | [ ] The array declaration extends to column 93 (the limit is 80). The array content should be split up over multiple lines
677 | ERROR | [ ] The array declaration extends to column 91 (the limit is 80). The array content should be split up over multiple lines
------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------------------------

Time: 1.05 secs; Memory: 16MB

CommentFileSizeAuthor
#2 phpcsissues-3353192-2.patch2.28 KBdineshkumarbollu

Comments

dineshkumarbollu created an issue. See original summary.

dineshkumarbollu’s picture

Assigned: dineshkumarbollu » Unassigned
Status: Active » Needs review
StatusFileSize
new2.28 KB

Hi

I created patch for this issue please review.

Thanks

avpaderno’s picture

Priority: Normal » Minor
Issue tags: -Fix the issues reported by phpcs +Coding standards
markusa’s picture

Oh wow, that's nice. Thank you!

skt-001’s picture

Status: Needs review » Reviewed & tested by the community

patch #2 fixes all the PHPCS error so moving status to RTBC.

dsdeiz’s picture

Status: Reviewed & tested by the community » Fixed

Committed. Thanks!

Status: Fixed » Closed (fixed)

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