Bug #63860 for Pod-Advent:

Sun Dec 12 21:35:06 2010 jpierce [...] cpan.org - Ticket created

RT#63525 wasn't implemented quite how I thought, the current version uses the possibly abbreviated text as the search cpan query. This patch turns M<App::perlzonji|perlzonji> into <tt><a href="http://search.cpan.org/perldoc?App::perlzonji"><tt title="App::perlzonji">perlzonji</tt></a></tt> The title itself isn't necessarily required (they can view the URL when hovering) but not bad. The double tt is odd... if keeping the title, apply it to the anchor instead?

Subject:

patch

Download patch
application/octet-stream 1k

Message body not shown because it is not plain text.

Sun Dec 12 22:05:07 2010 jpierce [...] cpan.org - Correspondence added

The patch was buggy, here's an unbuggy implementation of that M block }elsif( $mode eq 'M' ){ my($real, $alt); unless( ($real, $alt) = split /\|/, $text, 2 ){ $real = $text; } if( $M_values_seen{$real}++ ){ $parser->add( sprintf('<span title="%s">%s</span>', $real, $alt) ); }else{ $parser->add( sprintf('<a href="http://search.cpan.org/perldoc?%s" title="%s">%s</a>',$real,$real, (defined $alt ? $alt : $real) ) ); }

Sun Dec 12 22:05:08 2010 jpierce [...] cpan.org - Status changed from 'new' to 'open'

Sun Dec 12 22:23:31 2010 jpierce [...] cpan.org - Correspondence added

Show quoted text

> The title itself isn't necessarily required (they can view the URL > when hovering) but not bad. The double tt is odd... if keeping the > title, apply it to the anchor instead?

Oh, I guess it is necessary for subsequent M's, but the new supplied code handles it.

Mon Dec 13 09:28:18 2010 davidrw [...] cpan.org - Taken

Tue Dec 14 08:35:14 2010 davidrw [...] cpan.org - Fixed in 0.23 added

Tue Dec 14 08:35:36 2010 davidrw [...] cpan.org - Correspondence added

Fixed in v0.23

Tue Dec 14 08:35:38 2010 davidrw [...] cpan.org - Status changed from 'open' to 'resolved'