Problem/Motivation

FILE: ...l/vb/d_cont/simple_sitemap_page_manager/simple_sitemap_page_manager.module
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
--------------------------------------------------------------------------------
9 | WARNING | [x] Unused use statement
12 | WARNING | [ ] Doc comment indicates hook_form_alter() but function
| | signature is
| | "simple_sitemap_page_manager_form_page_manage_variant_configure_form_alter"
| | instead of "simple_sitemap_page_manager_form_alter". Did
| | you mean hook_form_FORM_ID_alter()?
53 | WARNING | [ ] #description values usually have to run through t() for
| | translation
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

FILE: ...imple_sitemap_page_manager/src/Controller/SimpleSitemapPagesController.php
--------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
--------------------------------------------------------------------------------
30 | ERROR | Parameter $settings is not described in comment
33 | ERROR | Doc comment for parameter $form_helper does not match actual
| | variable name $settings
--------------------------------------------------------------------------------

FILE: ...ml/vb/d_cont/simple_sitemap_page_manager/src/SimpleSitemapPagesManager.php
--------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
--------------------------------------------------------------------------------
11 | WARNING | The class short comment should describe what the class does and
| | not simply repeat the class name
--------------------------------------------------------------------------------

FILE: ..._page_manager/src/Plugin/simple_sitemap/UrlGenerator/PagesUrlGenerator.php
--------------------------------------------------------------------------------
FOUND 4 ERRORS AFFECTING 4 LINES
--------------------------------------------------------------------------------
49 | ERROR | [ ] Parameter $settings is not described in comment
58 | ERROR | [ ] Doc comment for parameter $generator does not match actual
| | variable name $pages_manager
103 | ERROR | [x] Expected 1 space before opening brace; found 2
156 | ERROR | [x] Expected 1 space before opening brace; found 2
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 2 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

Time: 113ms; Memory: 10MB

Steps to reproduce

Execute the command: phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml,twig simple_sitemap_page_manager/

Proposed resolution

Fix the issues reported for Drupal and DrupalPractice coding standards

Remaining tasks

Patch Review

Comments

urvashi_vora created an issue. See original summary.

arpitk’s picture

Assigned: Unassigned » arpitk

Reviewing this.

jay jangid’s picture

Assigned: arpitk » jay jangid
jay jangid’s picture

Assigned: jay jangid » Unassigned

Hey @arpitk Thanks for your patch.
After applied patch , all phpcs issues are resolved.
Thank you.

arpitk’s picture

Hi @urvashi_vora I reviewed the patch. After applying the patch few errors in simple_sitemap_page_manager.module are reported. Attaching before and after patch results. I have fixed those errors and updated the patch. Please review.

Thanks!

murilohp’s picture

Status: Needs review » Reviewed & tested by the community

Thanks for your contribution @Jay Jangid, @arpitk and @urvashi_vora, I've reviewed the patch and looks good to me

murilohp’s picture

  • arpitk authored 00b0a627 on 2.0.x
    Issue #3366174 by arpitk, urvashi_vora, Jay Jangid, murilohp: Fix the...
murilohp’s picture

Status: Reviewed & tested by the community » Fixed

Commited to 2.0.x, thanks!

Status: Fixed » Closed (fixed)

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