aboutsummaryrefslogtreecommitdiff
path: root/doc/bugs/hardcoded___34__Discussion__34___link.mdwn
blob: fc8406d8704c000cbb569f13bbf5b5fc61656e36 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
I can't translate "Discussion" link in `templates/inlinepage.tmpl`
and `templates/page.tmpl` files, because it's hardcoded in
ikiwiki sources:

    ptecza@horus:~/svn/ikiwiki$ rgrep -i DISCUSSIONLINK . |grep -v '.svn'
    ./templates/inlinepage.tmpl:<TMPL_IF NAME="DISCUSSIONLINK">
    ./templates/inlinepage.tmpl:<li><TMPL_VAR DISCUSSIONLINK></li>
    ./templates/page.tmpl:<TMPL_IF NAME="DISCUSSIONLINK">
    ./templates/page.tmpl:<li><TMPL_VAR DISCUSSIONLINK><br /></li>
    ./IkiWiki/Plugin/inline.pm:                                     $template->param(discussionlink => htmllink($page, $params{page}, "Discussion", 1, 1));
    ./IkiWiki/Render.pm:            $template->param(discussionlink => htmllink($page, $page, "Discussion", 1, 1));

I hope it's a bug, not a feature, because I don't have the same
problem with other links, for example "Edit", "RecentChanges"
or "History".

--Pawel

> There are good reasons for feeding a full html link into the template,
> rather than the urls used for the other links. For one, the Discussion
> link needs to be different if the Discussion page doesn't yet exist.

>> You can always use <tmpl_if> and <tmpl_else> construct in that place ;) --Pawel

>>> Not without duplicating the logic that constructs a link to an
>>> existing/nonexisting page in two places, one in code in ikiwiki and one
>>> in the template. Not good design. --[[Joey]]
 
> As noted in [[patchqueue/l10n]], there are some other places in ikiwiki
> that hard code English strings, and I feel that using standard gettext
> and po files is the best approach for these, although Recai suggested an
> approach of translating the strings using a template file. --[[Joey]]

>> You know that I rather prefer static templates, but it's your choice,
>> of course.
>> 
>> BTW, is there a chance for configurable name of Discussion page?
>> In my wiki I use only Polish name of pages, so I would like to have
>> dyskusja.html page, instead of discussion.html page. --Pawel

>>> I have an incomplete i10n patch for such hardcoded strings in the code, 
>>> which I hope to finish up eventually --[[Joey]]