Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#17347 closed defect (bug) (fixed)

WP autolink problem on wpdevel

Reported by: denis-de-bernardy's profile Denis-de-Bernardy Owned by: sivel's profile sivel
Milestone: WordPress.org Priority: normal
Severity: normal Version:
Component: WordPress.org Site Keywords:
Focuses: Cc:

Description (last modified by sivel)

As seen here:

http://wpdevel.wordpress.com/2011/05/05/dev-chat-summary-may-4-2011/#comment-18781

The ticket number provided within the parentheses was not auto linked to trac.

Attachments (2)

trac-links.php (1.2 KB) - added by sivel 14 years ago.
trac-links.1.0.2.php (1.2 KB) - added by sivel 14 years ago.

Download all attachments as: .zip

Change History (15)

#1 @Denis-de-Bernardy
14 years ago

  • Component changed from General to WordPress.org site

#2 @scribu
14 years ago

  • Summary changed from WP autolink problem on wordpress.com to WP autolink problem on wpdevel

#3 @sivel
14 years ago

  • Owner set to sivel
  • Status changed from new to accepted

I'll take this. I have the plugin that is in use on wpdevel for doing this. If I can't get to it, or need help, I'll post the plugin here to get it fixed.

#4 follow-up: @Denis-de-Bernardy
14 years ago

Is the plugin downloadable anywhere?

#5 in reply to: ↑ 4 @sivel
14 years ago

Replying to Denis-de-Bernardy:

Is the plugin downloadable anywhere?

As of this moment it is not.

#6 @sivel
14 years ago

  • Description modified (diff)

@sivel
14 years ago

#7 @sivel
14 years ago

Updated regex, passes all of my tests. Needs deployed to wpdevel.

#8 @Viper007Bond
14 years ago

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

Committed and deployed.

#9 @Viper007Bond
14 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

And reverted. It's linking HTML entities and needs more testing it seems. :)

#10 @Viper007Bond
14 years ago

The raw text:

I opened a ticket for the out of date browser nag (#17323).  There's a patch there, but you can't test it since the API it's supposed to work with doesn't exist yet.

That becomes this due to wptexturize and such:

I opened a ticket for the out of date browser nag (#17323).  There’s a patch there, but you can’t test it since the API it’s supposed to work with doesn’t exist yet.

The plugin is then turning those HTML entities into Trac links:

I opened a ticket for the out of date browser nag (<a href="http://core.trac.wordpress.org/ticket/17323">#17323</a>).  There&<a href="http://core.trac.wordpress.org/ticket/8217">#8217</a>;s a patch there, but you can&<a href="http://core.trac.wordpress.org/ticket/8217">#8217</a>;t test it since the API it&<a href="http://core.trac.wordpress.org/ticket/8217">#8217</a>;s supposed to work with doesn&<a href="http://core.trac.wordpress.org/ticket/8217">#8217</a>;t exist yet.

This is why it required spaces, heh.

Last edited 14 years ago by Viper007Bond (previous) (diff)

#11 @sivel
14 years ago

I'll update the regexes to include a negative look behind later.

#12 @sivel
14 years ago

Sorry, I don't have this in version control so I cannot submit updates as patches as easily. Anyway, probably easier to post the whole thing anyway for the time being.

New version just does a negative look behind for the '&' symbol as well as 'amp;' and '38;'

I'm sure there are hundreds of other variations that could just cause more issues, but this should resolve the issues that you were seeing.

Last edited 14 years ago by nacin (previous) (diff)

#13 @Viper007Bond
14 years ago

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

Tested (I forgot this step last time :P), committed, and deployed. :)

Note: See TracTickets for help on using tickets.