Hi all, great module! As per topic title, we have to separate words in the black list using "," (comma), but what if is the "," (comma) char itself the char that I want to blacklist? How to add the "," (comma) char to the blacklist?

Thank you for attention
Francesco

CommentFileSizeAuthor
#4 2688317-4.patch2.79 KBheykarthikwithu
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

francescosciamanna created an issue. See original summary.

krknth’s picture

krknth CreditAttribution: krknth as a volunteer and at Valuebound commented

@francescosciamanna : Hi, Thanks for reporting this.

As considering about existing users, I'm going to give an configurable option for comma as checkbox. I think this will be the best solution.

We will update this in next release :)

heykarthikwithu’s picture

heykarthikwithu
Primary language English
Location Bengaluru 🌍
CreditAttribution: heykarthikwithu as a volunteer and at Valuebound commented
Assigned: Unassigned » heykarthikwithu
Status: Active » Needs work
heykarthikwithu’s picture

heykarthikwithu
Primary language English
Location Bengaluru 🌍
CreditAttribution: heykarthikwithu as a volunteer and at Valuebound commented
Assigned: heykarthikwithu » Unassigned
Status: Needs work » Needs review
FileSize
2688317-4.patch2.79 KB

Added a checkbox "Add comma to blacklist. " to the configuration page. So that this configuration will allow/disallow the comma character in the title.

francescosciamanna’s picture

francescosciamanna CreditAttribution: francescosciamanna commented

I've tested in a couple of my sites, and seems like working great!
Thank you both again!
Francesco

krknth’s picture

krknth CreditAttribution: krknth as a volunteer and at Valuebound commented
Status: Needs review » Reviewed & tested by the community

krknth’s picture

krknth CreditAttribution: krknth as a volunteer and at Valuebound commented
Status: Reviewed & tested by the community » Fixed

Thanks @francescosciamanna and @karthik :)

Status: Fixed » Closed (fixed)

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