Problem/Motivation

Getting following errors.

FILE: /var/www/html/modules/contrib/taxonomy_depth_widget/src/Plugin/Field/FieldWidget/TermOptionsButtonsWidget.php
--------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------------------------------
118 | ERROR | The array declaration extends to column 87 (the limit is 80). The array content should be split up over multiple lines
--------------------------------------------------------------------------------------------------------------------------------------

FILE: /var/www/html/modules/contrib/taxonomy_depth_widget/src/Plugin/Field/FieldWidget/TermOptionsSelectWidget.php
--------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------------------------------
121 | ERROR | The array declaration extends to column 87 (the limit is 80). The array content should be split up over multiple lines
--------------------------------------------------------------------------------------------------------------------------------------

Time: 942ms; Memory: 6MB

Steps to reproduce

Run following command

phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml modules/contrib/taxonomy_depth_widget/

Proposed resolution

Above errors need to be fixed.

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

  • 3355066-phpcs Comparecompare

Comments

samit.310@gmail.com created an issue. See original summary.

samitk’s picture

Assigned: samitk » Unassigned
Status: Needs work » Needs review
StatusFileSize
new1.25 KB

Issue fixed.

clarkssquared’s picture

Status: Needs review » Needs work
StatusFileSize
new88.34 KB
new46.69 KB

Hi samit.310@gmail.com

I applied your patch #2 to the "Taxonomy Depth Widget" module against version 2.1.0. and confirmed that the mentioned phpcs errors were fixed, however, another new phpcs error was being flagged relating to taxonomy_depth_widget.info.yml.

Please look at the screenshot for your reference,

Thank you.

sakthi_dev made their first commit to this issue’s fork.

ashutosh ahirwal’s picture

Status: Needs work » Needs review
StatusFileSize
new102.54 KB
new29.18 KB

Hi @samit.310@gmail.com i have retested your patch with cloning the module from git clone command available on version control tab and it looks fine to me.
@clarkssquared I guess you have tested the patch after using the composer command to clone/download the module by using the composer command the "version, project, datestamp" get added by drupal in module info file.

Moving to RTBC.

fabiansierra5191’s picture

Status: Needs review » Fixed

Fixed on version 2.1.1, thanks for the patch and the tests!

fabiansierra5191’s picture

Status: Fixed » Closed (fixed)