Fix codings standard

Created on 21 May 2018, over 6 years ago
Updated 22 July 2024, 5 months ago

Please fix the code review error. Please refer the below URL:

https://pareview.sh/pareview/https-git.drupal.org-project-active_facet_p...

🐛 Bug report
Status

Needs work

Component

Code

Created by

🇮🇳India Maheshwaran.j

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • 🇮🇳India riddhi.addweb

    I have applied the "fix-coding-standard-2974189-2" patch but still, there are some errors.
    Please check the screenshot for the same.

    FILE: /var/www/html/web/drupal_8.9/web/modules/contrib/active_facet_pills/src/Plugin/Block/ActiveFacetPillsBlock.php
    --------------------------------------------------------------------------------------------------------------------
    FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
    --------------------------------------------------------------------------------------------------------------------
     38 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
     58 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
     59 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
    --------------------------------------------------------------------------------------------------------------------
    
    
    FILE: /var/www/html/web/drupal_8.9/web/modules/contrib/active_facet_pills/README.txt
    ------------------------------------------------------------------------------------
    FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
    ------------------------------------------------------------------------------------
     5 | WARNING | Line exceeds 80 characters; contains 178 characters
    ------------------------------------------------------------------------------------
    
    
    FILE: /var/www/html/web/drupal_8.9/web/modules/contrib/active_facet_pills/active_facet_pills.info.yml
    -------------------------------------------------------------------------------------------------------------
    FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 1 LINE
    -------------------------------------------------------------------------------------------------------------
     1 | WARNING | Remove "project" from the info file, it will be added by drupal.org packaging automatically
     1 | WARNING | Remove "datestamp" from the info file, it will be added by drupal.org packaging automatically
     1 | WARNING | Remove "version" from the info file, it will be added by drupal.org packaging automatically
    -------------------------------------------------------------------------------------------------------------
    
Production build 0.71.5 2024