Skip Menu |

This queue is for tickets about the Pod-Markdown CPAN distribution.

Report information
The Basics
Id: 129865
Status: resolved
Priority: 0/
Queue: Pod-Markdown

People
Owner: Nobody in particular
Requestors: ether [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 3.101
Fixed in: 3.200



Subject: local_module_url_prefix might be ''
It should be possible to generate relative URLs to local modules, by setting local_module_url_prefix = ''. However, in the format_perldoc_url sub, local_module_url_prefix is tested for truthiness before it is used, which results in a value of '' not being used and instead perldoc_url_prefix is fallen back to. Instead, it should be checked for defined-ness. A side issue with using no prefix at all is that the target URLs must now be url-escaped -- e.g. <a href="Foo::Bar"> is interpreted by browsers as an ipv6 address, so we must use <a href="Foo%3A%3ABar"> instead.
On Wed Jun 19 12:41:08 2019, ETHER wrote: Show quoted text
> It should be possible to generate relative URLs to local modules, by > setting local_module_url_prefix = ''. > > However, in the format_perldoc_url sub, local_module_url_prefix is > tested for truthiness before it is used, which results in a value of > '' not being used and instead perldoc_url_prefix is fallen back to. > Instead, it should be checked for defined-ness. > > A side issue with using no prefix at all is that the target URLs must > now be url-escaped -- e.g. <a href="Foo::Bar"> is interpreted by > browsers as an ipv6 address, so we must use <a href="Foo%3A%3ABar"> > instead.
How do you feel about https://github.com/rwstauner/Pod-Markdown/pull/18 ?
3.200 released which allows '' and uri-encodes the module names
On 2019-10-09 22:46:39, RWSTAUNER wrote: Show quoted text
> 3.200 released which allows '' and uri-encodes the module names
woohoo, thanks very much!
On Thu Oct 10 08:39:36 2019, ETHER wrote: Show quoted text
> On 2019-10-09 22:46:39, RWSTAUNER wrote:
> > 3.200 released which allows '' and uri-encodes the module names
> > woohoo, thanks very much!
thank you for your help!