responses

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

Update todo/comments with today's fixes, and thoughts about the remaining items

clarify

pagespec thoughts

note about comments feed

aggregate: If a feed fails to be downloaded, try again immediatly next time aggregation is run, even if the usual time has not passed. Closes: #508622 (Michael Gold)

close

deletion

jump to comments anchor after post

Not ideal, it would be nicer to jump to the actual comment posted, but no
anchor is available.

typo

fix

simplify pagespec

two comments ui suggestions

add Comments link when displaying a page inline

This link will supplant the usual Discussion link for pages
that have comments enabled.

fix test when comments_closed_pagespec is empty

don't rely on $@

A malformed pagespec will cause $@ to be set when translated, but if
it is used a second time, the memoization will defeat that check. Better to
check for the result not being defined.

change around comments pagespecs

I think it is clearer to have one pagespec that controls all pages with
comments, and a separate pagespec that can be used to close new comments on
a subset of those pages.

fix default values for config settings

defaults cannot be set in getsetup, do it in checkconfig to avoid
uninitialized value warnings.

clarification

Coding style change: Remove explcit vim folding markers.

checksessionexpiry: rework

This function as factored out was a bit confusing, I think this makes more
sense.

more comments doc updates

Moved todo items to a todo page, mark the old todo item about comments as
done, etc.

Merge branch 'master' into comments

monotone: When getting the log, tell monotone how many entries we want, rather than closing the pipe, which it dislikes. (thm)

rename: Fix double-escaping of page name in edit box.

titlepage normally escapes, but so does formbuilder.

rename bugs/bugfix_for:___38____35__34__59__mtn:_operation_canceled:_Broken_pipe__38____35__34__59_____40__patch__41__.mdwn to bugs/bugfix_for:___34__mtn:_operation_canceled:_Broken_pipe__34_____40__patch__41__.mdwn

rename bugs/bugfix_for:___34__mtn:_operation_canceled:_Broken_pipe__34__.mdwn to bugs/bugfix_for:___38____35__34__59__mtn:_operation_canceled:_Broken_pipe__38____35__34__59_____40__patch__41__.mdwn

add a meta title

not putting cgi in destdir

elide unnecessary variables

remove fixme

sessioncgi hooks are always called with the wiki locked

fix redefinition of $author

move related code together

whitespace

don't explicitly use inline

loadplugin("inline") should take care of that

avoid unnecessary variable

get rid of the [postcomment] hack

I think that using a flag variable is sorta cleaner.

(This is untested.)

improve documentation of postcomment

move getsetup to usual place

first pass through comments documentation

Moved documentation out of contrib.

Mostly tweaked some wording. Moved documentation of various bits to other
pages (pagespec, etc), and linked to those.

Documented the new templates in wikitemplates.

Small quantities of documentation were removed. Particularly the list of
template variables, which I think is fairly obvious when editing the
template.

update changelog

update copyright

Merge branch 'master' into comments

add log entry

htmlbalance: don't compact whitespace, and set misc other options

Not compacting whitespace is the most important one: now that we run
sanitize hooks on individual posted comments in the comments plugin,
whitespace that is significant to Markdown (but not HTML) is lost.
(cherry picked from commit cb5aaa3cee8b35d6fc6e88a7449a9477a6587c7a)

Merge branch 'master' into comments

meta: Pass info to htmlscrubber so htmlscrubber_skip can take effect.

Another TODO item

update with some more code improvements

comments: Store dates in GMT and in ISO-8601

The [[!_comment]] directive is a serialization format, not something for
presentation to users, so we should use the least ambiguous possible
representation.

adding setup note

Fix too much quoting.

Fix cut'n'past-o.

simplifiy logic

comments: Change internal-use-only directive from [[!comment]] to [[!_comment]]

comments: Use new feedfile, emptyfeeds options to inline

comments: don't interpolate IP into anonymous user's "name"

If an admin wants the IP in the comment display template they can still
get it (the default template shows it).

comments_display.tmpl: use less nice formatting to avoid whitespace in output

comments_display.tmpl: preferentially use confirmed username, but fall back to claimed author if unauthenticated

This still isn't quite right (it displays the IP address twice if the user
doesn't specify a name).

comments: fix invocation of possibly_foolish_untaint

comments_form.tmpl: show labels for name, website

comments: avoid warning if there's no subject

comments: Optionally allow anonymous commenters to set their name/URL.

Also provide a way for the comment template to pick up the verified
username/IP.

comments: Remove some dead code

comments: rename main field to "editcontent" consistent with editpage

This has the side-effect that Ikiwiki's default style.css gives the text
box 100% width.

comments: instead of hard-coding mdwn, allow any supported page format

comments: Save comments as a file with one big [[!comment]] directive.

This delays all comment formatting until the last possible time, allows
us to set metadata without worrying that commenters may be able to evade
it, and means that changes to how a comment is saved can be handled
gracefully. It also gives us somewhere to put the commenter's username
or IP address for later reference.

Remove dead code for preprocessing [[!comments]]

comments: Duplicate logic and CGI hook from recentchanges to link user pages correctly

Qualify name of formattime() correctly

Delay checking for session expiry til we actually post a comment

comments: record the time at which each comment was posted

comments: Use a checkconfig hook to get the default value of comments_pagename

Remove comments_embed.tmpl (no longer needed)

comments: render comments/commenturl in page.tmpl

comments: use global config to decide whether commenting is allowed, and for name of page

Also:
* decide comment page name sooner
* set permalink on it

comments: use global configuration for allow_directives, commit, and pagename

comments: Add some global configuration

comments: make preprocess a no-op

page.tmpl: optionally include "add comment" link

comments_comment.tmpl: allow permalink, anchor to be passed in

comments: document what linkuser does

comments: add a stub pagetemplate hook to show the comments

comments: Use HTML entities to escape directives

Embed comments into comments_embed.tmpl rather than concatenating in perl

comments_embed.tmpl: include the inlined comments if present

comments: use CGI module's checksessionexpiry

editpage: factor out checksessionexpiry into IkiWiki::CGI

Add comments to page.tmpl

htmlbalance: don't compact whitespace, and set misc other options

Not compacting whitespace is the most important one: now that we run
sanitize hooks on individual posted comments in the comments plugin,
whitespace that is significant to Markdown (but not HTML) is lost.

comments: remove allowhtml option, just switch it on all the time

Now that posts are individually sanitized, that should be safe.

comments: load inline and mdwn lazily

comments: don't rely on mdwn getting loaded first

comments: sanitize the body of each comment before posting it

This should ensure that users can't "break out" from the enclosing
<div>, making it impossible to forge comments (assuming htmlscrubber
is enabled, and so is either htmlbalance or htmltidy).

Fix typo that led to comments being blanked