Skip to content

Fix: remove redundant field variable initialization - #1334

Merged
Rom1-B merged 1 commit into
mainfrom
fix_rector
Nov 6, 2026
Merged

Rom1-B merged 1 commit into
mainfrom
fix_rector

Conversation

@Rom1-B

@Rom1-B Rom1-B commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

Checklist before requesting a review

Please delete options that are not relevant.

  • I have performed a self-review of my code.
  • I have added tests (when available) that prove my fix is effective or that my feature works.
  • I have updated the CHANGELOG with a short functional description of the fix or new feature.
  • This change requires a documentation update.

Description

  • It fixes #N/A
  • Removes a dead initialization reported by Rector in the question type AJAX controller, as the value is always assigned by both branches of the following condition.
    There is no functional change.

Screenshots (if appropriate):

@Rom1-B
Rom1-B requested review from a team, Lainow, RomainLvr and stonebuzz and removed request for a team November 6, 2026 06:24
@Rom1-B
Rom1-B merged commit cdbc9d4 into main Nov 6, 2026
7 checks passed
@Rom1-B
Rom1-B deleted the fix_rector branch November 6, 2026 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants