Fix the issues reported by phpcs

Created on 13 April 2023, almost 2 years ago
Updated 20 November 2023, about 1 year ago

Problem/Motivation

Getting following error/warnings.
FILE: /web/modules/contrib/login_time_restriction/tests/src/Functional/AccessTimeSettingsTest.php
------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------------------------------------------
35 | WARNING | Unused variable $target_entity.
------------------------------------------------------------------------------------------------------------------------------------------------

FILE: /web/modules/contrib/login_time_restriction/login_time_restriction.module
------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
------------------------------------------------------------------------------------------------------------------------------
162 | WARNING | [x] '@TODO Replace FALSE value with config value.' should match the format '@todo Fix problem X here.'
------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------------

Time: 916ms; Memory: 12MB

Steps to reproduce

Run following command
phpcs --standard=Drupal,DrupalPractice --extensions=php,module,inc,install,test,profile,theme,css,info,txt,md,yml modules/contrib/login_time_restriction/

Proposed resolution

Above error/warnings need to be fixed.

📌 Task
Status

Fixed

Version

1.0

Component

Code

Created by

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

Production build 0.71.5 2024