the rename hooks names waltz seems over

Another set of hooks would not endure such heavy renaming, but the rename* hooks
actually do ;)

Signed-off-by: intrigeri <intrigeri@boum.org>

revert renamepage's hook original name

Signed-off-by: intrigeri <intrigeri@boum.org>

need_global_renamepage_hook: s/suggest/will

Signed-off-by: intrigeri <intrigeri@boum.org>

contrib/po: update after implementing suggested changes

Signed-off-by: intrigeri <intrigeri@boum.org>

po: bring back a useful use of scalar

... that was removed in 68869d664b978b063c9181d024edb34a63306c33

Without this scalar, a two-cells array is passed to $template->param, which
builds a hash with an odd number of elements.

Signed-off-by: intrigeri <intrigeri@boum.org>

plugins/po: talk about the po4a version in the first lines

... as it seems it can be missed if I keep it only in the security section.

Signed-off-by: intrigeri <intrigeri@boum.org>

suggest a po4a version not subject to DoS

Signed-off-by: intrigeri <intrigeri@boum.org>

only pass named parameters to the canremove hook

Signed-off-by: intrigeri <intrigeri@boum.org>

only pass named parameters to the canrename hook

Signed-off-by: intrigeri <intrigeri@boum.org>

po(formbuilder): stop hunting down non-existent cases

After some thinking about it, I can't find why the type of a page being created
in the CGI could be restricted to po. So the previous case seems enough.

Signed-off-by: intrigeri <intrigeri@boum.org>

Merge commit 'upstream/master' into prv/po

po and doc/todo/need_global_renamepage_hook: response

Signed-off-by: intrigeri <intrigeri@boum.org>

new bug report

review

add mising hooks and fix syntax error

remove unnecessary returns

check_canremove/canrename is called only for its side effect (of failing if
removal is not allowed), its return value is never used and returning
something makes that unclear

don't add new exported warning function

Nothing in the po plugin actually uses this function.

The benefit of adding the function in general is debatable, but I'd
prefer to keep the changes involved in merging po at a minimum.

remove redundant use

use is file-scoped so warnings and strict are already enabled
inside the second package, and IkiWiki is already loaded
(though not imported into this context)

non-code changes

more unnecessary parens

avoid keeping running if a DOS attack is possible

remove unnecessary parens

another useless use of scalar

scalar(undef) is undef, so using scalar here had no effect.
Instead make the function return "", probably avoiding an uninitialized
value warning.

remove useless uses of scalar

if (scalar @array) is written idiomatically in perl as
if (@array).

improve some english

I'm still not happy with the clarity of this warning message.
I don't understand when it could happen or why a warning is needed.

avoid standalone -e or foo syntax

I prefer to use either of the other two syntaxes perl offers, and not this
one.

avoid mixing english with translation

"google search" is english which would get mixed in with translated text.
Instead, usr just the plugin name.

Merge branch 'master' into po

git: Fix malformed utf8 recieved from git.

If git log outputs malformed utf8 in, eg, usernames, detect it and fix it
up. This avoids commits such as f71abc92aa279fbe0b7578b8c4752d775dd4a259
breaking things.

update

update

typo

format moderation queue only at end, avoid O(N^3) bug

It was calling format hooks for each comment on the page.
When relativedate is enabled, that made it insert <script> tags
for each comment. And the browser loaded the same script over and over,
which was slow on its own. But that was nothing compared to running
the onload even over and over.. especially since the hook system
added a new call to the hook each time it loaded.

For a page with 10 comments, that caused the relativedate DOM parsing
code to run 1000 times, I think. Anyway, it was sloow. Now it runs once.

add reject all marked defer checkbox

sort comment queue by time, newest first

add a button to prefs page for comment moderation

clean up comment preview

Remove actions from it, and avoid a broken title link.

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

comments: Add a moderation web interface.

remove hr under comment actions

The horizontal line was only meant to be displayed under
the actions at the top of a page.

add alt tag

comments: If comment content checks fail, store the comment (in .ikiwiki/comments_pending) for moderator review.

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

blogspam: Fix use of blogspam_options and blogspam_server config settings.

vote for Template::Toolkit

clarify description of verbose

It enables debug prints, so not just during building, but at runtime.

typo

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

mention identica group

followup on pagespec negation

typo

fix uninitialized value warnings

I suspect these are only triggered by spammers.

fix typo

response

question on locked_pages and users() in pagespec

one fleeting second thought

response

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

Question about user tests

add

img: only provide alt text if it was specified

if suitable alternate text is unknown, then it should not be given.
empty alt text is suitable mainly for purely decorative images.
(cherry picked from commit 3cd7f67f0cf894f4fd5ba16f68e82e4f7bdbfdc5)

added version

add human-readable error when unknown pagespec function is called

It used to replace unknown functions with "0" when translating a pagespec.
Instead, replace it with a FailReason object. This way, the pagespec will
still evaluate as before (possibly successfully if other terminals exist),
but a human-readable error will be shown if the result is displayed.

Also, an empty pagespec used to be replaced with "0", to avoid a eval
error. Also use a FailReason here.

cleanup

response

Remove nonstandard css. Closes: #512378

fix removal form display

The form was misdisplayed when displayed via comment removal.

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

po: let's walk the last steps needed before merge!

Signed-off-by: intrigeri <intrigeri@boum.org>

blogspam: Log spam info on failure.

Updated spanish translation

remove cansave plugin, now replaced by Joey's checkcontent one

Signed-off-by: intrigeri <intrigeri@boum.org>

make checkcontent compatible with hooks that need the full content

Always pass the full (modified) content in `content` named parameter. When the
user edits an existing wiki page, also pass a `diff` named parameter, which
includes only the lines that they added to the page, or modified.

Signed-off-by: intrigeri <intrigeri@boum.org>

Add meta field "updated", which can alter the <updated> Atom element

Some aggregators, like Planet, sort by mtime rather than ctime. This
means that posts with modified content come to the top (which seems odd
to me, but is presumably what the aggregator's author or operator
wants), but it also means that posts with insignificant edits (like
adding tags) come to the top too. Atom defines <updated> to be the date
of the last *significant* change, so it's fine that ikiwiki defaults to
using the mtime, but it would be good to have a way for the author to
say "that edit was insignificant, don't use that mtime".

Avoid feeding decoded unicode to Term::ReadLine

That resulted in double encoded display when using perl's stub
readline module. Apparently that module unconditionally upgrades
text to utf8, in a quite braindead way.

(Term::ReadLine::Gnu::Perl worked ok.)

overriding visible modification time (to avoid insignificant edits flooding planets)

Some more thoughts about a gallery plugin

Add myself to ikiwikiusers

Document underlay plugin as included

Add underlay plugin

fix typo, mention git branch

underlay plugin, a command-line interface for add_underlay()

Merge commit 'upstream/master' into prv/po

I really ought to RTFM before I make todo requests, eh?

Created user page.