doc/bugs/About___37__2F_problem.mdwn

   1 I use inline to create a blog like the below:
   2
   3     \[[!inline pages="./bugs/* and !./bugs/done and !./bugs/discussion and
   4     !link(patch) and !link(./bugs/done) and !./bugs/*/*"
   5     actions=yes rootpage="./bugs" postform="yes" postformtext="请用一句话简述问题，然后点击 Edit 按钮添加具体细节" show=0]]
   6
   7
   8 When I use posform to add a new page, it show:
   9
  10 > Error: bad page name
  11
  12 Its url include a %2F, like below:
  13
  14 > http://172.16.0.109/ikiwiki.cgi?do=blog&from=.%2Fbugs&subpage=1&title=aaa
  15
  16 I use ikiwiki 3.20180311
  17
  18 ----
  19
  20 I have found that it is not "%2F"'s problem, it just that inline directive can
  21 not deal with Chinese char, the below link can work
  22
  23     http://172.16.0.109/ikiwiki.cgi?do=blog&from=aaa%2Fbugs&subpage=1&title=aaa
  24
  25 ---
  26
  27 > I don't think this is actually caused by the Chinese text. The problem is that
  28 > you used `rootpage="./bugs"`, which leads to the `blog` request handler
  29 > generating an invalid page name. If you change it to `rootpage="bugs"` does
  30 > that fix the error?
  31 >
  32 > Ideally either the `inline` directive or the `blog` request handler would
  33 > understand and remove `./`, if it's something that makes sense in this context.
  34 > --[[smcv]]
  35
  36 ---
  37
  38 > I have found the problem, it is inline plugin can not decode_utf8 "from", the below is patch:
  39
  40     From f79dde20b275707f70df2d481919a079abec6c19 Mon Sep 17 00:00:00 2001
  41     From: Feng Shu <tumashu@163.com>
  42     Date: Sun, 2 Dec 2018 08:38:34 +0800
  43     Subject: [PATCH 1/2] Fix inline plugin can no set rootpage to a UTF-8 page
  44
  45     ---
  46      IkiWiki/Plugin/inline.pm | 2 +-
  47      1 file changed, 1 insertion(+), 1 deletion(-)
  48
  49     diff --git a/IkiWiki/Plugin/inline.pm b/IkiWiki/Plugin/inline.pm
  50     index a85cd5d2f..f40956821 100644
  51     --- a/IkiWiki/Plugin/inline.pm
  52     +++ b/IkiWiki/Plugin/inline.pm
  53     @@ -125,7 +125,7 @@ sub sessioncgi ($$) {
  54                         error(gettext("please enter a page title"));
  55                 }
  56                 # if the page already exists, munge it to be unique
  57     -           my $from=$q->param('from');
  58     +           my $from=decode_utf8($q->param('from'));
  59                 my $add="";
  60                 while (exists $IkiWiki::pagecase{lc($from."/".$page.$add)}) {
  61                         $add=1 unless length $add;
  62     --
  63     2.19.0
  64
  65 ---
  66
  67 > [Request for clarification removed]
  68 >
  69 > I've now been able to reproduce this bug, and confirmed that your patch
  70 > fixes it. [[Patch now applied|done]].
  71 >
  72 > (For other maintainers' reference: when testing Unicode bugs that
  73 > relate to page titles, using Unicode that is considered to be
  74 > punctuation, like ¬ or emoji, will probably not work; page titles
  75 > treat `[:alnum:]` and `^[:alnum:]` differently.)
  76 >
  77 > In future bug reports it would be useful if you could provide a minimal
  78 > example or test, perhaps on the [[sandbox]] on this wiki or as a unit
  79 > test in `t/*.t` in the ikiwiki source code, that demonstrates this bug
  80 > and would be fixed by the patch. If you've found multiple bugs, a
  81 > separate example or test for each one would be easiest to deal with.
  82 >
  83 > You can run all the tests with:
  84 >
  85 >     ./Makefile.PL
  86 >     make
  87 >     make test
  88 >
  89 > or a single test with something like:
  90 >
  91 >     ./Makefile.PL
  92 >     make
  93 >     PERL5LIB=. ./t/git-cgi.t
  94 >
  95 > --[[smcv]]