review

comment

tag as reviewed

respond

Exclude reviewed patches from this list

close

append javascript after CSS

Javascript resources should be presented to browsers after CSS, and
"after the fold" (ATF) according to the best practices:

https://developers.google.com/speed/docs/insights/mobile#PutStylesBeforeScripts

This change allows the browser to download Javascript files in
parallel, by including Javascript on the *closing* </body> tag instead
of the opening tag.

We also improve the regex to tolerate spaces before the body tag, as
some templates have (proper) indentation for the tag.

Remove unreachable git repositories

Signed-off-by: Simon McVittie <smcv@debian.org>

Mark as applied

Signed-off-by: Simon McVittie <smcv@debian.org>

remove the "add comment" button from printed media

Add a missing changelog entry.

Update changelog

Signed-off-by: Simon McVittie <smcv@debian.org>

Allow Breezy as alternative to Bazaar.

(cherry picked from commit a07f048d9fc99928ebbb74b34f5d1932ff3d7884)

comments.t: Assert that comments get permalink metadata

Signed-off-by: Simon McVittie <smcv@debian.org>

comments.t: Exercise post-2009 comment naming

Since commit 6af6c89d, comments are in files whose names contain a hash.

Signed-off-by: Simon McVittie <smcv@debian.org>

old regexp would have failed for old comment page names

git-cgi.t: Exercise an alphanumeric, but non-ASCII, root page

My previous attempt to reproduce this bug used a non-alphanumeric
ASCII character. This is not currently considered to be a valid
value for rootpage, although for a "do what I mean" approach, perhaps
we should accept it and pass it through titlepage() or linkpage().

Using Chinese characters (which are considered to match [[:alnum:]]
even though the Chinese script is not, strictly speaking, an alphabet),
as in the original bug report, reproduces the bug.

Signed-off-by: Simon McVittie <smcv@debian.org>

Fix inline plugin for non-ASCII rootpage

t: Exercise Chinese and Cyrillic page titles

Signed-off-by: Simon McVittie <smcv@debian.org>

trail: Allow unescaped punctuation in pagenames

By processing the pagenames through linkpage, we let users specify
page names that contain non-alphanumerics in a more natural way.

Signed-off-by: Simon McVittie <smcv@debian.org>

trail.t: Exercise numeric escapes in pagenames parameter

Signed-off-by: Simon McVittie <smcv@debian.org>

linkpage.t: Assert we can link to pages with literal underscore

Signed-off-by: Simon McVittie <smcv@debian.org>

t: Consistently remove temp directory before testing, not after

When a test fails, it's useful to be able to inspect the output.

Signed-off-by: Simon McVittie <smcv@debian.org>

build: Add `make reset-generated`

Signed-off-by: Simon McVittie <smcv@debian.org>

pagetitle.t, linkpage.t, titlepage.t: Exercise Unicode more

Signed-off-by: Simon McVittie <smcv@debian.org>

link to recently-added tests

I'm confused about what the bug is, and what's being fixed. Can you give a complete test or example?

close bug

Update changelog

Add a simple test for non-ASCII in tables

Signed-off-by: Simon McVittie <smcv@debian.org>

Fix table plugin can handle UTF-8 csv format

respond

Fix syntax and escaping

git-cgi.t: Add a failing test for a blog with a non-ASCII rootpage

This is one of several possible bug reports on
"doc/bugs/About %2F problem" (I'm not sure what the actual bug being
reported is).

Signed-off-by: Simon McVittie <smcv@debian.org>

git-cgi.t: Add a simple test for blog posts from a root page

This is the working base case for "doc/bugs/About %2F problem".

Signed-off-by: Simon McVittie <smcv@debian.org>

git-cgi.t: Print query string as a TAP diagnostic

Signed-off-by: Simon McVittie <smcv@debian.org>

Reset example

This reverts commit 0b53772b99248d2868fac688cdaa227ad61ac08e

inline is another option of course

show tagging example

link to the map plugin, which i find more useful to show tagged pages

followup

fix permalink to comments

This commit was sponsored by Thomas May.

Merge branch 'master' of ssh://git.ikiwiki.info

bug report

User link and description changed

markdown backquotes

followup

de-indenting

example for inlining only one level of subpages

fail to inject IkiWiki::showform

patch works for me

ack

rename and remove plug can not work well with gettext

Suggest let table plugin support a kind of longtable

Add a patch deal with UTF-8 csv table's problem

Add a patch decode_utf8 inline's root page

add redirection

update for rename of bugs/Re-use_translated_content_instead_of_skipping_if_previously_translated/20180628-patch.txt to bugs/po:_second_or_subsequent_inline_of_translated_page_inlines_.po_file__44___not_translated_content/20180628-patch.txt

rename bugs/Re-use_translated_content_instead_of_skipping_if_previously_translated.mdwn to bugs/po:_second_or_subsequent_inline_of_translated_page_inlines_.po_file__44___not_translated_content.mdwn

update for rename of todo/Re-use_translated_content_instead_of_skipping_if_previously_translated/20180628-patch.txt to bugs/Re-use_translated_content_instead_of_skipping_if_previously_translated/20180628-patch.txt

rename todo/Re-use_translated_content_instead_of_skipping_if_previously_translated.mdwn to bugs/Re-use_translated_content_instead_of_skipping_if_previously_translated.mdwn

refer to a branch

ikiwiki.info doesn't have fenced code blocks yet

Escape directives in code

Review

po: Improve test coverage

Signed-off-by: Simon McVittie <smcv@debian.org>

po: Add failing test for Debian bug #911356

Signed-off-by: Simon McVittie <smcv@debian.org>

Fix broken link in basewiki

Links from pages in the basewiki (which is also a subset of ikiwiki's
own wiki) to pages on ikiwiki's own wiki have to use [[!iki]] so that
they will not become broken links when the page in question is copied
into other installations.

Detected by ./t/basewiki_brokenlinks.t.

Signed-off-by: Simon McVittie <smcv@debian.org>

I don't think the Chinese text causes this

comments (no review yet)

Revert spam

This reverts commit c600ed0e710d82601acf5ae4005b940f5cba8c08

related issue

error output when trying

rename postvote to postlink

This better explains what it contains, which is a wikilink to the page
to go to after posting the vote. And postlink is more consistent a name
with posttrail.

poll: Added postvote and posttrail options for better multi-page polls.

Modern web users probably expect the poll to move on automatically to the
next question, and this allows for that behavior.

Note that bestlink() runs at vote time, which avoids needing to make the
page containing the poll depend on the page that sets up a trail, as the
current trail at vote time will be used.

This commit was sponsored by Eric Drechsel on Patreon.

avoiding ban?

update my entry

update description & remove tag patch

more spam issues. this is definitely not done.

aggregate: Fix aggregation of posts without a title

When an aggregated post lacked a title, the code first prepended the
$feed->{dir} to it, and only then checked if it had zero length. So,
that check could never succeed and it was possible to end up with
$page="dir/", and writing to that would of course fail.

(Same problem could also occur when the whole title got sanitized away by the
wiki_file_regexp.)

Fixed by simply checking earlier if $page is empty.

Based on a patch by Alexandre Oliva which got lost in a maze of email
folders all alike for over two years despite him mentioning it to me at
least once in person.

Give graph an optional "file" param, like table's.

removed

Added a comment: Could someone document the steps to achieve this?

another thing to remove from prints