Problem/Motivation

This is for variable names in non-tests that are misspelled.

Steps to reproduce

Proposed resolution

  1. annot
  2. autofix
  3. colonpos
  4. frombracket
  5. pluralforms
  6. topop

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3392439

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

quietone created an issue. See original summary.

quietone’s picture

Issue summary: View changes
quietone’s picture

Assigned: Unassigned » quietone
quietone’s picture

Issue summary: View changes
Status: Active » Needs review

There are more variable names to fix but they are they will require more discussion. In other words, these should be easy one so let's get this completed.

smustgrave’s picture

Status: Needs review » Reviewed & tested by the community

Replacement for the 6 words seem fine.

longwave’s picture

Version: 11.x-dev » 10.2.x-dev
Status: Reviewed & tested by the community » Fixed

These all look self-contained and uncontroversial. Renaming function arguments is OK as we don't support named arguments except in attribute constructors.

Committed ca9af3d and pushed to 11.x. Thanks!

Also backported to 10.2.x to keep things in sync, there are no BC concerns here.

  • longwave committed 84181889 on 10.2.x
    Issue #3392439 by quietone: Fix variable name spelling in non-tests,...

  • longwave committed ca9af3d9 on 11.x
    Issue #3392439 by quietone: Fix variable name spelling in non-tests,...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

quietone’s picture

Assigned: quietone » Unassigned