Skip to content

Risk: Fix reserved keyword variable names phpcs reports #8544

New issue
New issue
Open
Open
Risk: Fix reserved keyword variable names phpcs reports#8544
Labels
focus: fraud toolspriority: lowThe issue/PR is low priority—not many people are affected or there’s a workaround, etc.type: technical debtThis issue/PR represents/solves the technical debt of the project.
@reykjalin

Description

@reykjalin
reykjalin
opened on Apr 1, 2024

Part of #8438.

Description

Remove all the Universal.NamingConventions.NoReservedKeywordParameterNames excludes from phpcs.xml.dist and fix the issues reported when running npm run lint:php in the following files:

  • includes/fraud-prevention/models/class-check.php
  • includes/fraud-prevention/models/class-rule.php
  • includes/fraud-prevention/class-fraud-risk-tools.php

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    focus: fraud toolspriority: lowThe issue/PR is low priority—not many people are affected or there’s a workaround, etc.type: technical debtThis issue/PR represents/solves the technical debt of the project.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions