Revert "Revert "Fix a bug with links to pages whose names contained colons.""

This reverts commit 810a355308d276ef922ff4312bff0c7935676633.

I double-checked, and the change seems ok after all, actually.

still exclude discussion

oops

no deep pages

link

enlarge

announce

Revert "Fix a bug with links to pages whose names contained colons."

This reverts commit e25c3a0a7caa9783c657efe86611929ecb7bd4a3.

I didn't think that through far enough; beautify_url may be called on full
urls, which don't start with "/"!

Conflicts:

debian/changelog

beautify url recentchanges_link redirs to

This ensures that the same link is reached as is used on pages,
so browsers will know that the link on pages has been visited, and color it
appropriately.

bug

whitespace

fix use ordering

The recent setup revamp exposed some latent bugs in use/package ordering
that caused some symbols to not the exported into the correct scope.

whitespace

web commit by intrigeri: thanks + note about git collaborative work

web commit by intrigeri: thanks

web commit by intrigeri: answer + fix escaping in example wiki page

question

escape

Fixes creation of pages when clicking on WikiLinks starting with "/".

template: Add support for a BASENAME variable.

web commit by intrigeri: initial bug submission

web commit by intrigeri: added opinion + patch

work around CGI::Session constructor issues

The constructor can fail with a useless error message if module fail to
load. Work around this by evaling it, and checking for failures, and
printing CGI::Session->errstr to get a more useful message.

web commit by intrigeri: removed misunderstood references to pagetemplate, added patch against plugins/template

debian/changelog: fix typo in old entry

ikiwiki-transition: Fix command-line processing so the prefix_directives transition works again.

reviewed this again

document setup plugins

clarifications, comments

fix changelog

Make it possible to load setup files w/o running them. Code needing to do so can call IkiWiki::Setup::load, and the values will be loaded into %IkiWiki::Setup::setup.

add news item for ikiwiki 2.53

releasing version 2.53

improve error message if virus checker fails w/o output

web commit by intrigeri: initial wishlist

web commit by http://jcflack.myopenid.com/

web commit by http://jcflack.myopenid.com/

web commit by http://ptecza.myopenid.com/: * Confirmation and thanks

remove patch tag

close

close

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

testing

web commit by bremner: fix my name

mention github mirror

testing

testing

testing

testing

testing

otl: Support utf-8 files. (Recai Oktaş)

response

web commit by https://id.mayfirst.org/jamie/

web commit by https://id.mayfirst.org/jamie/

response

web commit by AndrewGreenberg: bad formatting on bug report, sorry.

web commit by AndrewGreenberg: Added nasty URL edit bug in 2.52.

not yet released

response

web commit by mrled

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

Conflicts:

doc/bugs/attachment:_failed_to_get_filehandle.mdwn

attachment: Support perl 5.8's buggy version of CGI.pm.

This is truely horribly disgusting. CGI::tmpFileName, in current perls, is
an undocumented function (which should be a clue..) that takes the original
filename of an uploaded attachment, and returns the name of the tempfile
that CGI has stored it in.

In old perls, though, CGI::tmpFileName does not take a filename. It takes
a key from the object's {'.tmpfiles'} hash. This key is something
crazy like '*Fh::fh00001group' -- apparently the stringification of a
filehandle object.

Just to add to the fun, tmpFileName doesn't take the key, it expects a
refernce to the key. Argh?!

But the fun doesn't stop there, because in perl 5.8, CGI.pm is also broken
in two other ways. The upload() method is supposed to return a filehandle
to the temp file. It doesn't. The param() method is supposed to return
a filehandle to the temp file, that stringifies to the original filename.
It returns just the original filename, no filehandle.

Combine all these bugs, and you end up with this disgusting commit. Since
I have no way to get the filehandle, I *need* to get the tempfile name.
If I had the filehandle, I could probably pass it into tmpFileName, and
it might strigify to the right key name. But I don't, so the only way to
determine the key is to grub through the .tmpfiles hash ourselves.

And finally, one the temp file name is discovered, a filehandle can finally
be obtained by (re)opening it.

I recommend that this commit be reverted when perl 5.8 is a mercifully
faded memory.

I'm really, really, really glad I'm actually being paid for working on
this right now!

explicitly set form type to multipart

This is the default in unstable's formbuilder, but not in stable's,
and CGI multipart is needed for fule uploads.

Include ikiwiki.setup in examples in the debian package.

web commit by http://ptecza.myopenid.com/: * Thanks!

web commit by HenrikBrixAndersen: This happens on FreeBSD as well

reproduced

response

check for tempfile being empty

rm

web commit by http://ptecza.myopenid.com/: * The bug still occurs

add dist-bugs

merge

response

web commit by mrled

web commit by mrled

web commit by mrled

web commit by mrled

web commit by mrled

undef $fh before opening

web commit by http://ptecza.myopenid.com/: * Still no success

response

another try at dealing with CGI.pm problem

open the temp file if all attempts to get a fd fail

web commit by http://ptecza.myopenid.com/: * Debugging again

possible fix

attachment: Support old versions of CGI.pm that lack an upload method.

web commit by http://ptecza.myopenid.com/: * Next debbuging

new thing to try

fix fix

close

Fix a bug with links to pages whose names contained colons.

So the problem is that ikiwiki would generate a relative link like
href="colon:problem", which web browsers treat as being in the "colon:"
uri scheme.

The best fix seems to be to make url beautification fix this, by slapping
a "./" in front.

web commit by http://ptecza.myopenid.com/: * Missing Apache2 version

web commit by http://ptecza.myopenid.com/: * Response

split out separate bug

response

attachment: Fix an uninitialised value warning when editing a page that currently has no attachments.

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

web commit by http://jcflack.myopenid.com/

web commit by http://ptecza.myopenid.com/: * Problem with colon in report page name

web commit by http://ptecza.myopenid.com/: * Problem with uploading attachments