Validate the username with the list of blacklist which is configured in the config page

CommentFileSizeAuthor
#6 user_name_validation-2711299-6.patch2.75 KBn.kishorekumar
#2 username_validation-2711299-2.patch2.78 KBn.kishorekumar

Comments

N.kishorekumar created an issue. See original summary.

n.kishorekumar’s picture

n.kishorekumar commented
StatusFileSize
new username_validation-2711299-2.patch2.78 KB

Implemented the validation for black characer

n.kishorekumar’s picture

n.kishorekumar commented
Status: Active » Needs review
krknth’s picture

krknth commented
Status: Needs review » Needs work 
  1. +++ b/user_name_validation.module
@@ -1,19 +1,63 @@
+  ¶
...
+  // Finding words in the node title.

    Remove white space 

  2. +++ b/user_name_validation.module
@@ -1,19 +1,63 @@
+    if (Drupal\Component\Utility\Unicode::strlen($char) == 1) {

    You already imported

xaiwant’s picture

xaiwant commented
Issue summary: View changes
n.kishorekumar’s picture

n.kishorekumar commented
Assigned: n.kishorekumar » Unassigned
Status: Needs work » Needs review
StatusFileSize
new user_name_validation-2711299-6.patch2.75 KB

@krknth, dont find any extra whitespace
for the second point, changes are done

Please review the patch

xaiwant’s picture

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

@N.kishorekumar not getting any extra new as per @krknth #4 comment after applying the latest patch(#6).
moving to RTBC

  • N.kishorekumar committed df76d0f on 8.x-1.x 
    Issue #2711299 by N.kishorekumar, xaiwant, krknth: Implement the...
heykarthikwithu’s picture

heykarthikwithu
Primary language English
Location Bengaluru 🌍
commented
Assigned: Unassigned » heykarthikwithu
heykarthikwithu’s picture

heykarthikwithu
Primary language English
Location Bengaluru 🌍
commented
Assigned: heykarthikwithu » Unassigned
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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