Contrib.social

Fix the issues reported by phpcs

Open on Drupal.org →
Created on 7 July 2023, over 1 year ago
Updated 22 January 2024, 12 months ago

Running shows the following errors/warnings which needs to be fixed.

FILE: ./lightning_layout.module
----------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------
 112 | ERROR | [x] Use null coalesce operator instead of ternary
     |       |     operator.
----------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------


FILE: ./modules/lightning_banner_block/tests/src/Functional/DependenciesTest.php
----------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------
 7 | ERROR | Missing short description in doc comment
----------------------------------------------------------------------


FILE: ./modules/lightning_landing_page/tests/src/Functional/DependenciesTest.php
----------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------
 7 | ERROR | Missing short description in doc comment
----------------------------------------------------------------------


FILE: ./modules/lightning_map_block/tests/src/Functional/DependenciesTest.php
----------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
----------------------------------------------------------------------
  7 | ERROR | Missing short description in doc comment
 28 | ERROR | Public method name "DependenciesTest::testSimpleGMap"
    |       | is not in lowerCamel format
----------------------------------------------------------------------


FILE: ./src/Form/SettingsForm.php
----------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------
 40 | ERROR | [x] Do not append variable name "$module_handler" to
    |       |     the type declaration in a member variable comment
----------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------

Time: 156ms; Memory: 6MB
📌 Task
Status

RTBC

Version

3.0

Component

Code

Created by

🇮🇳India roshni27

Live updates comments and jobs are added and updated live.
  • Coding standards

    It involves compliance with, or the content of coding standards. Requires broad community agreement.

Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @roshni27
  • Issue was unassigned.
  • Status changed to Needs review over 1 year ago
  • Open in Jenkins → Open on Drupal.org →
    Core: 9.5.x + Environment: PHP 7.4 & MySQL 5.7
    last update over 1 year ago
    9 pass
  • Comment over 1 year ago →
    🇮🇳India roshni27

    Some comment description remain because which are better know to maintainer and all remaining issues solve by me. Please review the patch.

  • Status changed to Needs work 12 months ago
  • Comment 12 months ago →
    🇮🇳India ashutosh ahirwal India

    Provided patch in #2 get applied cleanly but still there are some remaining errors found.

    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/lightning_layout

    FILE: /Users/ashutoshahirwal/Sites/lando/contribution/web/modules/custom/lightning_layout/modules/lightning_landing_page/tests/src/Functional/DependenciesTest.php
    ------------------------------------------------------------------------------------------------------------------------------------------------------------------
    FOUND 1 ERROR AFFECTING 1 LINE
    ------------------------------------------------------------------------------------------------------------------------------------------------------------------
    7 | ERROR | Missing short description in doc comment
    ------------------------------------------------------------------------------------------------------------------------------------------------------------------

    FILE: /Users/ashutoshahirwal/Sites/lando/contribution/web/modules/custom/lightning_layout/modules/lightning_map_block/tests/src/Functional/DependenciesTest.php
    ---------------------------------------------------------------------------------------------------------------------------------------------------------------
    FOUND 1 ERROR AFFECTING 1 LINE
    ---------------------------------------------------------------------------------------------------------------------------------------------------------------
    7 | ERROR | Missing short description in doc comment
    ---------------------------------------------------------------------------------------------------------------------------------------------------------------

    FILE: /Users/ashutoshahirwal/Sites/lando/contribution/web/modules/custom/lightning_layout/modules/lightning_banner_block/tests/src/Functional/DependenciesTest.php
    ------------------------------------------------------------------------------------------------------------------------------------------------------------------
    FOUND 1 ERROR AFFECTING 1 LINE
    ------------------------------------------------------------------------------------------------------------------------------------------------------------------
    7 | ERROR | Missing short description in doc comment
    ------------------------------------------------------------------------------------------------------------------------------------------------------------------

    Time: 361ms; Memory: 10MB

  • Status changed to Needs review 12 months ago
  • Open in Jenkins → Open on Drupal.org →
    Core: 9.5.x + Environment: PHP 7.4 & MySQL 5.7
    last update 12 months ago
    9 pass
  • Comment 12 months ago →
    🇮🇳India ashutosh ahirwal India

    Providing updated patch with all phpcs issue fixes.
    Please review it

  • Status changed to RTBC 12 months ago
  • Comment 12 months ago →
    🇵🇭Philippines paraderojether

    Hi Ashutosh Ahirwal,

    I reviewed patch #4 and applied it against Lightning Layout 3.0.x-dev with Drupal core version 10.1.7 and confirmed it fixes the issues reported bu phpcs.

    I added screenshots for reference.
    Thank you.

  • Comment 12 months ago →
    🇮🇹Italy apaderno Brescia, 🇮🇹
  • Merge request !24Issue #3373191: Fix the issues reported by phpcs → (Open) created by apaderno
  • Open in Jenkins → Open on Drupal.org →
    Core: 9.5.x + Environment: PHP 7.4 & MySQL 5.7
    last update 12 months ago
    9 pass
  • Comment 12 months ago →
    🇮🇹Italy apaderno Brescia, 🇮🇹
contrib.social Blog FAQ Discussions
Production build 0.71.5 2024