Contrib.social

Fix remaining code style issues

Open on Drupal.org →
Created on 5 April 2024, over 1 year ago

Problem/Motivation

FILE: /var/www/html/web/modules/custom/front/src/EventSubscriber/FrontPageSubscriber.php
-------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 4 WARNINGS AFFECTING 4 LINES
-------------------------------------------------------------------------------------------------
  96 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
 103 | WARNING | User::load calls should be avoided in classes, use dependency injection instead
 121 | WARNING | Line exceeds 80 characters; contains 87 characters
 125 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
-------------------------------------------------------------------------------------------------


FILE: /var/www/html/web/modules/custom/front/src/Form/FrontPageHomeLinksForm.php
------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------
 40 | WARNING | Avoid backslash escaping in translatable strings when possible, use "" quotes instead
------------------------------------------------------------------------------------------------------


FILE: /var/www/html/web/modules/custom/front/src/Form/FrontPageSettingsForm.php
---------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
---------------------------------------------------------------------------------------------------------
  33 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
  57 | WARNING | Role::loadMultiple calls should be avoided in classes, use dependency injection instead
 118 | WARNING | \Drupal calls should be avoided in classes, use dependency injection instead
---------------------------------------------------------------------------------------------------------


FILE: /var/www/html/web/modules/custom/front/src/FrontPagePathProcessor.php
-----------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
-----------------------------------------------------------------------------------------------------------------
 15 | WARNING | The class short comment should describe what the class does and not simply repeat the class name
-----------------------------------------------------------------------------------------------------------------


FILE: /var/www/html/web/modules/custom/front/README.txt
----------------------------------------------------------------------
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
----------------------------------------------------------------------
 15 | WARNING | Line exceeds 80 characters; contains 83 characters
 30 | WARNING | Line exceeds 80 characters; contains 81 characters
---------------------------------------------------------------------

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

📌 Task
Status

Active

Version

10.0

Component

Code

Created by

🇩🇪Germany Anybody Porta Westfalica

Live updates comments and jobs are added and updated live.
  • Novice

    It would make a good project for someone who is new to the Drupal contribution process. It's preferred over Newbie.

Sign in to follow issues

Merge Requests

Comments & Activities

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024