Fix several places that did not properly handle capitalization of the discussionpage setting.

Specifically, fixes discussion actions on discussion pages, and unbreaks the opendiscussion plugin.
master
Joey Hess 2010-01-02 15:42:20 -05:00
parent ed67ff3b84
commit 7dd074e2e7
5 changed files with 13 additions and 5 deletions

View File

@ -357,9 +357,9 @@ sub preprocess_inline (@) {
my $file = $pagesources{$page}; my $file = $pagesources{$page};
my $type = pagetype($file); my $type = pagetype($file);
if ($config{discussion}) { if ($config{discussion}) {
if ($page !~ /.*\/\Q$config{discussionpage}\E$/ && if ($page !~ /.*\/\Q$config{discussionpage}\E$/i &&
(length $config{cgiurl} || (length $config{cgiurl} ||
exists $pagesources{$page."/".$config{discussionpage}})) { exists $pagesources{$page."/".lc($config{discussionpage})})) {
$template->param(have_actions => 1); $template->param(have_actions => 1);
$template->param(discussionlink => $template->param(discussionlink =>
htmllink($page, htmllink($page,

View File

@ -23,7 +23,7 @@ sub canedit ($$) {
my $cgi=shift; my $cgi=shift;
my $session=shift; my $session=shift;
return "" if $page=~/(\/|^)\Q$config{discussionpage}\E$/; return "" if $page=~/(\/|^)\Q$config{discussionpage}\E$/i;
return undef; return undef;
} }

View File

@ -92,7 +92,7 @@ sub genpage ($$) {
$actions++; $actions++;
} }
if ($config{discussion}) { if ($config{discussion}) {
if ($page !~ /.*\/\Q$config{discussionpage}\E$/ && if ($page !~ /.*\/\Q$config{discussionpage}\E$/i &&
(length $config{cgiurl} || (length $config{cgiurl} ||
exists $links{$page."/".$config{discussionpage}})) { exists $links{$page."/".$config{discussionpage}})) {
$template->param(discussionlink => htmllink($page, $page, $config{discussionpage}, noimageinline => 1, forcesubpage => 1)); $template->param(discussionlink => htmllink($page, $page, $config{discussionpage}, noimageinline => 1, forcesubpage => 1));

7
debian/changelog vendored
View File

@ -1,4 +1,6 @@
ikiwiki (3.20091219) UNRELEASED; urgency=low ikiwiki (3.20100102.3) UNRELEASED; urgency=low
Happy palindrome day!
* pagestats: Add show parameter. Closes: #562129 (David Paleino) * pagestats: Add show parameter. Closes: #562129 (David Paleino)
* toc: Add startlevel parameter. (kerravonsen) * toc: Add startlevel parameter. (kerravonsen)
@ -14,6 +16,9 @@ ikiwiki (3.20091219) UNRELEASED; urgency=low
* linkmap: Add option to omit disconnected pages from the map. * linkmap: Add option to omit disconnected pages from the map.
* inline: Fix bug that limited displayed pages when feedshow was * inline: Fix bug that limited displayed pages when feedshow was
specified w/o show. specified w/o show.
* Fix several places that did not properly handle capitalization of
the discussionpage setting. Specifically, fixes discussion actions
on discussion pages, and unbreaks the opendiscussion plugin.
-- Joey Hess <joeyh@debian.org> Fri, 25 Dec 2009 14:31:22 -0500 -- Joey Hess <joeyh@debian.org> Fri, 25 Dec 2009 14:31:22 -0500

View File

@ -9,6 +9,9 @@ I do want discussion subpage, but I don't want to have, for example: discussion/
>> This bug appears to have returned. For example, >> This bug appears to have returned. For example,
>> [[plugins/contrib/unixauth/discussion]] has a Discussion link. -- [[schmonz]] >> [[plugins/contrib/unixauth/discussion]] has a Discussion link. -- [[schmonz]]
>>> Lots of case issues this time. Audited for and fixed them all. [[done]]
>>> --[[Joey]]
>>> Joey, I've just seen that you closed that bug in ikiwiki 1.37, but it seems >>> Joey, I've just seen that you closed that bug in ikiwiki 1.37, but it seems
>>> you fixed it only for English "discussion" page. The bug still occurs >>> you fixed it only for English "discussion" page. The bug still occurs
>>> for the international "discussion" pages. I have backported ikiwiki 1.40 >>> for the international "discussion" pages. I have backported ikiwiki 1.40