Problem/Motivation

FILE: /Users/ashutoshahirwal/Sites/lando/contribution/web/modules/custom/webform_product/webform_product.routing.yml
--------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
--------------------------------------------------------------------------------------------------------------------
6 | WARNING | Open page callback found, please add a comment before the line why there is no access restriction
13 | WARNING | Open page callback found, please add a comment before the line why there is no access restriction
20 | WARNING | Open page callback found, please add a comment before the line why there is no access restriction
--------------------------------------------------------------------------------------------------------------------

FILE: /Users/ashutoshahirwal/Sites/lando/contribution/web/modules/custom/webform_product/src/WebFormProductFormHelper.php
-------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------------------
27 | WARNING | Unused variable $element_info.
-------------------------------------------------------------------------------------------------------------------------

Time: 532ms; Memory: 16MB

User interface changes

API changes

Data model changes

Comments

roshni27 created an issue. See original summary.

roshni27’s picture

Issue summary: View changes
roshni27’s picture

Status: Active » Needs review
StatusFileSize
new9.26 KB

I have fixed all errors ,Please review patch.

ashutosh ahirwal’s picture

Status: Needs review » Needs work
StatusFileSize
new165.93 KB

Hi Provided patch get applied cleanly but still found some error.
Used command with arguments:-
./vendor/bin/phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,info,txt,md,css,js,yml web/modules/custom/webform_product

FILE: /Users/ashutoshahirwal/Sites/lando/contribution/web/modules/custom/webform_product/webform_product.routing.yml
--------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
--------------------------------------------------------------------------------------------------------------------
6 | WARNING | Open page callback found, please add a comment before the line why there is no access restriction
13 | WARNING | Open page callback found, please add a comment before the line why there is no access restriction
20 | WARNING | Open page callback found, please add a comment before the line why there is no access restriction
--------------------------------------------------------------------------------------------------------------------

FILE: /Users/ashutoshahirwal/Sites/lando/contribution/web/modules/custom/webform_product/src/WebFormProductFormHelper.php
-------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------------------
27 | WARNING | Unused variable $element_info.
-------------------------------------------------------------------------------------------------------------------------

Time: 532ms; Memory: 16MB

zkhan.aamir’s picture

Issue summary: View changes

Issue summary updated.

nagy.balint’s picture

StatusFileSize
new8.33 KB

Updated patch

nagy.balint’s picture

  • nagy.balint committed 756eb1c1 on 3.1.x
    Issue #3372977 by roshni27, nagy.balint, Ashutosh Ahirwal: Fix the...
nagy.balint’s picture

Status: Needs work » Fixed

  • nagy.balint committed 8fa9a9d5 on 3.1.x
    Issue #3372977 by nagy.balint: Fix the issues reported by phpcs.
    
nagy.balint’s picture

Now it only does not like the api.php, but it should not analyse that anyways.

Status: Fixed » Closed (fixed)

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