improved error message

switch to using template_depends

It now handles all the details of finding a page template that this used to
need to implement.

use same error string as template.pm does

allow a bare page name to be specified as a template

fix logic

add

refactor to remove template_params

template_file will be kept separate, since it needs to be memoized

look for templates in srcdir and underlays, first

This entailed changing template_params; it no longer takes the template
filename as its first parameter.

Add template_depends to api and replace calls to template() with
template_depends() in appropriate places, where a dependency should be
added on the template.

Other plugins don't use template(), so will need further work.

Also, includes are disabled for security. Enabling includes only when using
templates from the templatedir would be nice, but would add a lot of
complexity to the implementation.

remove add_templates option

Templates are moving into the srcdir, and will also be searched for in
configured underlays, so this is redundant.

clarify

add

only remove page from influences when influences are not static

This matches what add_depends done

update

update

format

Merge branch 'autotag'

force rebuild from pre 20100422

to ensure those wrong massive depends_simple don't linger on systems that
rebuilt for the other reasons already

note that the new version will have some optimisation fixes

improved fix for depends_simple_mixup

Avoid adding the page matched against as an influence for
currently failing pagespec matches, while still adding
any other influences.

This avoids bloating depends_simple with lots of bogus influences when
matching eg, "!link(done)". It's only necessary for the page being tested
to be an influence of that if the page matches.

add influence info for failed metadata matches

This is needed so that when a negated pagespec like "!author(foo)"
stops matching, due to the page being changed, ikiwiki knows that
the match was influenced by the page content.

fix test cases for dynamic influence calculation

added a test case for this bug

Further analysis suggests fixing this might not be as dreadful as I first
thought!

backlink influence data seems ok

update

more wrongness

link fix

argh. head exploding.

indent

my autotag branch seems ready

turn on tag_autocreate by default if tagbase is set

Merge branch 'master' into autotag

reword news

Merge branch 'master' into autotag

Conflicts:
IkiWiki/Plugin/tag.pm

remove broken ./tag support

The commit that added the (working) support for using /tag to override
tagbase also tried to make ./tag work. Problem is, tags are links,
and ./foo is not a valid link (though I think there's a wishlist about it).

So, using ./tag really resulted in tag creation links that led to a
"bad page name" error. And even if the tag were created in the right place,
the link didn't go to it.

fixes

clarify

Merge branch 'master' into autotag

Conflicts:
IkiWiki/Plugin/tag.pm

pass pagespec parameters along from match_tagged

add missing undef guard in derel

fix false positive in link_types_changed

loadindex does not bother populating oldtypedlinks if there is no link
type. However, the code in link_types_changed assumed that if oldtypedlinks
is not defined, and typedlinks is, they must differ.

add separate template variable for tag page and sanitize displayed tag name

remember autofiles that were tried and failed to be added

This way, if an autofile is registered for a file that already exists,
it is remembered that it was tried, and it doesn't get recreated when
removed.

response

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

note re includes

suppressing auto-creation can be quite counter-intuitive

elide an older patch that's no longer under discussion

update

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

commit autocreated tag pages

update

remember autofiles always

This fixes the problem that it did not remember if an autofile is deleted,
unless a plugin happened to register the autofile at the same time.

With the new code, we just never recreate an autofile more than once.

Only downside is that the list of autofiles is never pruned either.
And I don't really see a way to prune it.

bugfixes

Merge branch 'master' into autotag

update, tag deletion bug

document tag_autocreate

spelling

update; my branch is (partially) debugged now

debugging

remove verify_src_file

Splitting out this function bothered me. It is conceptially similar to
file_pruned, and yet also very specific to exactly the security needs of
find_src_files.

I liked that it got rid of duplicate code in the latter function. So
instead, put a helper sub in that, which I think allows refactoring
things more cleanly, and with less boilerplate.

As to the needs of gen_autofile, I'm not convinced this needs to handle
the same set of problems that verify_src_file did. So I sat down and
wrote a custom validator for autofiles, which turned out to seem to just
need three things: Make sure the candidate filename is not something
that would be pruned; untaint the candidate filename; and make sure that
srcdir doesn't already have something with its name. (Plus, of course,
all the other checks that were already in gen_autofile.)

(In passing, also fixed a bunch of bugs I had introduced in this branch.)

add_autofile filename should be relative to srcdir

Merge branch 'master' into autotag

applied

layout

Always give createlink class to links to nonexistent pages

With this change, the <span> with class createlink is always created
around the link text, even when no CGI URL is defined. This allows
styling of these 'links' in this case too. The same class is used as when
CGI URL is defined so that e.g. clones of the same ikiwiki, one with CGI
and one without, display in the same way (modulo the missing question mark
link).
(cherry picked from commit 290d1b498f00f63e6d41218ddb76d87e68ed5081)

Missing links with no CGI

Merge branch 'master' into autotag

Merge branch 'file_pruned_revamp'

bugfix

So this is the one place that does need to canonpath and compare,
as two argument file_pruned used to.

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

darcs: ensure whole darcs query manifest output is consumed

By a stroke of luck, after a long & full day, I happened to
remember that in the morning, I had seen someone on irc mention
that darcs query manifest doesn't like it if its full output
is not consumed.

So contrary to the usual case where bug reports sent via irc are like
messages written in sand before the new tide, this one was seen and
fixed.

(But use http://ikiwiki.info/bugs/ next time!)

bugfix

Avoid file_pruned triggering on absolute paths causing the whole
comments_pending directory being pruned.

Simplify code.

Merge branch 'master' into file_pruned_revamp

fix uninitialized value warning

$cgi->params('do') may not be defined. The CSRF code may delete all
cgi params. This uninitalized value was introduced when do=register
support was added recently.

Clarifications

typo

fix stat to use unmunged filename

oops, fix for no_chdir mode

$_ will be absolute then

remove 2 argument form of file_pruned

switch to 1 parameter version of file_pruned

Another bit of code that didn't realize that File::Find sets $_ to the
relative filename.

remove explicit absolute test

file_pruned now tests for that

use one parameter form of file_pruned here too

In File::Find, $_ is relative to the current directory, so that is ok.
Also, the directory name doesn't need to be stripped from $_.

clarify why absolute is tested & stripped here

file_prune also fails on absolute filenames now

Merge branch 'master' into file_pruned_revamp

internal pages: the revenge?

clarify whether the raw files would be put in destdir

fix minor bug if a page's name is "0"

similarity to internal pages

no need to return content from postscan hook

response

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

change wording

This is more accurate when a file that is not a page is what is removed.

non comprend

formatting

further clarification

response

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

response