Skip to content
This repository has been archived by the owner on May 9, 2021. It is now read-only.

Auto link creation bug #30

Closed
ddreier opened this issue Apr 25, 2012 · 6 comments
Closed

Auto link creation bug #30

ddreier opened this issue Apr 25, 2012 · 6 comments

Comments

@ddreier
Copy link
Contributor

ddreier commented Apr 25, 2012

Tiny Issue auto-detects URLs in Issues and comments and that is awesome! But it also includes single periods in URLs if they are at the end of a sentence. Ex: http://d.pr/i/qSKy

I just noticed that for "url.com." the period isn't included. But for "url.com/test." the period is.

mikelbring added a commit that referenced this issue May 1, 2012
@ddreier
Copy link
Contributor Author

ddreier commented May 2, 2012

Just noticed something else. It looks like pretty much [anything].[anything] is being turned into a URL as well. It's not a big deal, but I thought I'd bring it up. Ex: T-SQL [Table Name].[Field Name] http://d.pr/i/iSkK

@deplorableword
Copy link
Contributor

ah interesting, perhaps it's worth finding a long list of testcase URLs and strings we can use to ensure the parser is working correctly?

@ddreier
Copy link
Contributor Author

ddreier commented May 3, 2012

Figure out a pattern that matches URLs based on the list of TLDs? Since a URL won't always start with 'www.' but always has to end in '.TLD.'

@ddreier
Copy link
Contributor Author

ddreier commented May 24, 2012

Another example of incorrect link creation: http://d.pr/i/SwxL
Which is turned into http://path/to/tinyissue/index.php/project/#/issue/clear...it

Oops, just realized that I haven't updated that server to the latest version.

@bertiful
Copy link
Contributor

bertiful commented Jun 9, 2012

Is it me or this bug got fixed? The T-SQL example listed above doesn't cause the link bug for me. @ddreier @deplorableword

@ddreier
Copy link
Contributor Author

ddreier commented Jun 9, 2012

I think it is fixed. My last post is from an older version that I thought I had updated but hadn't.

@ddreier ddreier closed this as completed Jun 9, 2012
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants