[dokuwiki] New PR: Remove Google Lucky for non-existing interwiki links

  • From: "phy25" <wiki@xxxxxxxxxxxx>
  • To: dokuwiki@xxxxxxxxxxxxx
  • Date: Tue, 18 Jun 2019 02:32:52 +0200 (CEST)

Hi,

phy25 opened a new pull request at 
https://github.com/splitbrain/dokuwiki/pull/2802:

For compatibility renderer will return string '' of $shortcut and $url instead 
of NULL when seeing a non-existing interwiki link. In the meantime, media and 
link output in xhtml renderer is adjusted, to show title text instead when 
src/href is null (doesn't limit to interwiki situation). In interwiki case, the 
title text will be the "reference" part of the interwiki link.

This makes it possible to also support no URL cases in `interwiki.conf`. Before 
it will output a URL as `rawurlencode($reference)`, which doesn't make sense 
since it's encoded. However, I am not sure if it's useful under the current 
behavior (no URL, but text as `$reference`).

Docs needs to be added to warn renderer plugin developers of this situation.

This fixes #2588.

Please help us to review this pull request, so new contributors get feedback in 
a timely manner.

9ac59a00-9160-11e9-98a9-0072fe8cb88a


-- 
DokuWiki mailing list - more info at
http://www.dokuwiki.org/mailinglist

Other related posts:

  • » [dokuwiki] New PR: Remove Google Lucky for non-existing interwiki links - phy25