Restricted characters are not displaying properly in validating message.

Added following restricted character "@, test" in configuration form and checked comma to blacklist checkbox, but its showing validation message only for "@, ," while creating node title "test article @, user".

validate

message

Expected result:

The validation message should display all the blacklisted items like "@, ,, test" in validation message.

Comments

Mashir created an issue. See original summary.

mashir’s picture

Assigned: Unassigned » mashir
mashir’s picture

Issue summary: View changes
mashir’s picture

Status: Active » Needs review
StatusFileSize
new1.21 KB

Patch attached.

mashir’s picture

Title: Restricted characters are not displaying properly in validating message. » Restricted characters are not displaying properly in validation message.
neeraprajapati’s picture

Status: Needs review » Needs work

Hello @mashir,

I have applied patch and found following unexpected result with given string:

String: abc @ xyz, mno
Result error: This characters/words are not allowed to enter in the title. - @, ,, @

And restricted comma charactor in error message is showing little bit confusing, so it will be better to use bold tag with these blacklisted values.
so the result should be like @, ,

mashir’s picture

Status: Needs work » Needs review
StatusFileSize
new1.39 KB
new1.07 KB

@neeraprajapati, thank you for reviewing the patch and testing the functionality as well. Attaching a new patch, which will solve the issue and an interdiff.txt as well.

neeraprajapati’s picture

Status: Needs review » Reviewed & tested by the community

Looks good.

  • mahaveer003 committed 8d3d0af on 8.x-1.x authored by Mashir
    Issue #2958723 by Mashir, neeraprajapati: Restricted characters are not...
mahaveer003’s picture

Assigned: mashir » Unassigned
Status: Reviewed & tested by the community » Fixed

Thanks a lot for giving patch and doing RTBC. Patch changes pushed to dev version.

Status: Fixed » Closed (fixed)

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