they fit in this class i think

CommentFileSizeAuthor
#1 drupal-urlvalidator-2020811-1.patch3.66 KBParisLiakos

Comments

ParisLiakos’s picture

ParisLiakos commented
Status: Active » Needs review
StatusFileSize
new drupal-urlvalidator-2020811-1.patch3.66 KB
dawehner’s picture

dawehner
Primary language German
commented

Just from the naming it seems wrong to have such functionality in the UrlValidator

ParisLiakos’s picture

ParisLiakos commented

url_is_external i think is perfect fit since it uses the strip dangerous protocols method

Status: Needs review » Needs work

The last submitted patch, drupal-urlvalidator-2020811-1.patch, failed testing.

ParisLiakos’s picture

ParisLiakos commented
Status: Needs work » Closed (duplicate)

lets move them over at #2010024: Move url related functions to a new Url component

ParisLiakos’s picture

ParisLiakos commented
Issue summary: View changes

Updated issue summary