We added a username format validator in #3077523: Add username validator. That would work but we could refactor it a bit to have it match the Drupal 8 version while still meeting our needs.

CommentFileSizeAuthor
#4 3142792-4.patch4.84 KBhussainweb
#2 3142792-2.patch4.71 KBhussainweb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hussainweb created an issue. See original summary.

hussainweb’s picture

hussainweb
Primary language English
CreditAttribution: hussainweb at Axelerant commented
Status: Active » Needs review
FileSize
3142792-2.patch4.71 KB

Status: Needs review » Needs work

The last submitted patch, 2: 3142792-2.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

hussainweb’s picture

hussainweb
Primary language English
CreditAttribution: hussainweb at Axelerant commented
Status: Needs work » Needs review
FileSize
3142792-4.patch4.84 KB

Status: Needs review » Needs work

The last submitted patch, 4: 3142792-4.patch, failed testing. View results

  • hussainweb committed bccbd8b on 8.x-1.x 
    Issue #3142792 by hussainweb: Refactor UsernameFormatValidator to match...
hussainweb’s picture

hussainweb
Primary language English
CreditAttribution: hussainweb at Axelerant commented
Status: Needs work » Fixed

Committing as this is required for tests to work in #3142809: Add a unit test for constraint.

Status: Fixed » Closed (fixed)

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