Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#1546 closed enhancement (fixed)

Pasting links with "http://" results in broken links

Reported by: isaackeyet Owned by:
Priority: trivial Milestone: 3.4
Component: Compose Version:
Keywords: Cc:


Classifying as an Enhancement.

If you copy a link that includes the http protocol into the Link Helper in the Post Editor, the result will be a broken link (e.g. "http://". Solution would be to check for what's been pasted and remove the first http:// if there's two. Note that it also has to work with other types of links (e.g. "https://").

Change History (3)

comment:1 Changed 5 years ago by jkv1

Sent a github pull request with the fix -

The code is already dealing with a URL with the http:// or other protocols already added but it does not deal with spaces before the protocol. I just added a trim to the text field.

comment:2 Changed 5 years ago by koke

  • Resolution set to fixed
  • Status changed from new to closed

(In [4009]) Fixes #1546 - link helper adding duplicate protocol. props jkv1

The URL text field needs to be trimmed of whitespace since the user
could paste text that has spaces in it.

comment:3 Changed 5 years ago by beaucollins

  • Milestone changed from Future Release to 3.4
Note: See TracTickets for help on using tickets.