Problem/Motivation

FormSubscriber calls itself FormValidator in describing its constructor statement. (And when you lie to grep, you confuse developers who are desperately looking for a way to override a tiny piece of FormValidator!)

CommentFileSizeAuthor
#2 drupal-3229012-fix-form-subscriber-comment.patch534 bytesmlncn

Comments

mlncn created an issue. See original summary.

mlncn’s picture

mlncn commented
Status: Active » Needs review
StatusFileSize
new drupal-3229012-fix-form-subscriber-comment.patch534 bytes

One line comment only patch!

wolcen’s picture

wolcen commented
Status: Needs review » Reviewed & tested by the community

It's those little things that'll confuse someone eventually. +1

  • catch committed 8fd69fc on 9.3.x 
    Issue #3229012 by mlncn, wolcen: Fix copy-paste mistake in code comment

  • catch committed 7c4ed97 on 9.2.x 
    Issue #3229012 by mlncn, wolcen: Fix copy-paste mistake in code comment...
catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 9.3.x and cherry-picked to 9.2.x, thanks!

Status: Fixed » Closed (fixed)

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