git.oblomov.eu Git - ikiwiki/blob - doc/bugs/urlto_API_change_breaks_wikis_with_po_plugin.mdwn

   1 The po plugin needs to be updated to match the urlto sub API and
   2 signature changes. Else a wiki with the po plugin enabled cannot be
   3 refreshed / rebuilt because of (correct) Perl errors.
   4
   5 My po branch contains a fix.
   6 --[[intrigeri]]
   7
   8 > The commit looks sane to me, for what it's worth. Joey, please
   9 > consider merging? --[[smcv]]
  10
  11 >> Merged. --[[Joey]]
  12
  13 Also, I fear the lack of any useful `$from` parameter might break some
  14 l10n'd link niceness when using `po_link_to = current` but I have not
  15 investigated this yet.
  16 --[[intrigeri]]
  17
  18 > If `urlto` is called without a second parameter, it means we need
  19 > a URL valid from either the CGI URL or any page in the wiki,
  20 > (so we'd previously have set the third parameter true), but we
  21 > don't *necessarily* need an absolute URL - so return what you'd
  22 > have returned if asked for an absolute URL, but looking like
  23 > `/bugs/` rather than `http://ikiwiki.info/bugs/` if possible.
  24 >
  25 > It looks as though `beautify_urlpath` under `po_link_to = current`,
  26 > and 3-argument `urlto`, aren't tested by `t/po.t` - perhaps you
  27 > could add some test cases there? To test 3-argument `urlto` you'd
  28 > need to add `$config{baseurl} = "http://example.com"` or
  29 > something. --[[smcv]]
  30
  31 >> I'm leaving this bug report open until this can be checked. --[[Joey]]