Currently, the maximum length is set to 128 characters, the default for a textfield. The attached patch increases it to 1024 characters as we were easily running into the 128 character limit with longer domains and subdomains.

CommentFileSizeAuthor
#1 extlink_remove_regex_maxlength.patch1.04 KBquicksketch
increase_regex_length.patch899 bytesdeviantintegral

Comments

quicksketch’s picture

quicksketch
he/him
commented
StatusFileSize
new extlink_remove_regex_maxlength.patch1.04 KB

A very reasonable request. Would you be opposed to removing the "maxlength" property entirely? Using NULL ensures that no value is added at all.

quicksketch’s picture

quicksketch
he/him
commented
Status: Needs review » Fixed

I've committed the patch from #1.

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

Anonymous (not verified) commented
Status: Closed (fixed) » Active

I also ran into this problem, glad to find a committed patch!
Do we get a new release to go with it?

quicksketch’s picture

quicksketch
he/him
commented
Status: Active » Fixed

Sure, doesn't look like there have been any other issues filed in a few months, I'll make a new release tonight.

Anonymous’s picture

Anonymous (not verified) commented

Great, thanks!

Status: Fixed » Closed (fixed)

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