Merge branch 'master' into tova

diff --git a/IkiWiki/Plugin/attachment.pm b/IkiWiki/Plugin/attachment.pm
index c6711aa05ac80e2957a982305b3ee187cfd13cce..e08aa36771b8bb927f9a8a5ba50189f545fae8bb 100644 (file)
--- a/IkiWiki/Plugin/attachment.pm
+++ b/IkiWiki/Plugin/attachment.pm
@@ -136,7 +136,7 @@ sub formbuilder (@) { #{{{
                        }
                }
 
-               $filename=IkiWiki::titlepage(
+               $filename=IkiWiki::linkpage(
                        IkiWiki::possibly_foolish_untaint(
                                attachment_location($form->field('page')).
                                IkiWiki::basename($filename)));

diff --git a/debian/changelog b/debian/changelog
index 57a36eeeca5709b3e006e86bc77c65eee607d40e..a0d526f8819adc9cad2d617540e871bb5d05d485 100644 (file)
--- a/debian/changelog
+++ b/debian/changelog
@@ -21,6 +21,7 @@ ikiwiki (2.55) UNRELEASED; urgency=low
     page, to avoid confusion.
   * Split out error messages from editpage.tmpl into several separate
     templates.
+  * attachment: Do not escape _ when determining attachment filenames.
 
  -- Joey Hess <joeyh@debian.org>  Mon, 21 Jul 2008 11:35:46 -0400
 

diff --git a/doc/bugs/attachment:_escaping_underscores_in_filename__63__.mdwn b/doc/bugs/attachment:_escaping_underscores_in_filename__63__.mdwn
new file mode 100644 (file)
index 0000000..d7d1017
--- /dev/null
+++ b/doc/bugs/attachment:_escaping_underscores_in_filename__63__.mdwn
@@ -0,0 +1,19 @@
+I've just noticed that `attachment` plugin escapes the underscore
+characters in attached filenames. For example, when I wanted to add
+`foo_bar_baz.txt` file, then Ikiwiki added file `foo__95__bar__95__baz.txt`
+to my Subversion repo. I hope that the filename is terribly ugly not only
+for me ;)
+
+Is it a bug or security feature? --[[Paweł|ptecza]]
+
+>> Update: It's not only problem with attached filenames. I have
+>> `mysql/myisam_vs_ndb.mdwn` page in my wiki and attached two
+>> images (`myisam_vs_ndb_sql.png` and `myisam_vs_ndb_cpu.png`)
+>> and one OpenDocument file (`myisam_vs_ndb.ods`). Ikiwiki placed
+>> them into `myisam__95__vs__95__ndb` subdirectory as
+>> `myisam__95__vs__95__ndb__95__sql.png`, `myisam__95__vs__95__ndb__95__cpu.png`
+>> and `myisam__95__vs__95__ndb.ods` files. When I click "Attachments" link,
+>> I can't see my uploaded files, because there are in another subdirectory
+>> (`myisam__95__vs__95__ndb` instead of `myisam_vs_ndb`).  --[[Paweł|ptecza]]
+
+> [[done]], uses `linkpage` now.

diff --git a/doc/bugs/map_is_inconsistent_about_bare_directories.mdwn b/doc/bugs/map_is_inconsistent_about_bare_directories.mdwn
new file mode 100644 (file)
index 0000000..479db3e
--- /dev/null
+++ b/doc/bugs/map_is_inconsistent_about_bare_directories.mdwn
@@ -0,0 +1,26 @@
+The [[plugins/map]] plugin has inconsistent behaviour.  In particular, I have in my wiki some directory structures holding files without wikitext pointers (I point directly to the files from elsewhere).  For example, imagine the following file structure in the source dir:
+
+* Assignments.mdwn
+* Assignments
+    * 2004
+        * Assign1.pdf
+        * Assign2.pdf
+    * 2005
+        * Assign1.pdf
+        * Assign2.pdf
+    * 2006
+        * etc., etc.
+
+When I use map to make a map of this, the result looks more like this:
+
+* Assignments        # this is a link to the correct page
+    * 2004                   # this has a create link
+        * Assign1.pdf
+        * Assign2.pdf
+        * Assign1.pdf
+        * Assign2.pdf
+        * etc., etc.
+
+Note that while the 2004 directory exists with a create link, the 2005 and 2006 (etc) directories are missing from the site map.
+
+I could imagine including all 'bare' directories in the map, and I could imagine including no 'bare' directories in the map.  Just including the first bare directory seems a strange intermediate point.

diff --git a/doc/plugins/toggle.mdwn b/doc/plugins/toggle.mdwn
index 5e0b8392a50c20d621dd4292c5c311a5187a78b8..5eac6e4a1ab80a4b2d2a7a84abf57ff3e63603cd 100644 (file)
--- a/doc/plugins/toggle.mdwn
+++ b/doc/plugins/toggle.mdwn
@@ -30,4 +30,4 @@ mutiple toggles that all toggle a single togglable.
 The id has a default value of "default", so can be omitted in simple cases.
 
 If you'd like a toggleable to be displayed by default, and toggle to
-hidden, then pass a parameter "open=true" when setting up the toggleable.
+hidden, then pass a parameter "open=yes" when setting up the toggleable.

diff --git a/doc/tips/wikiannounce.mdwn b/doc/tips/wikiannounce.mdwn
index 0d1973546c09b1a2c4cf7d48c245caa5485cc3a8..6eb142cdf277abcf89bc8a959f38832a26c585ef 100644 (file)
--- a/doc/tips/wikiannounce.mdwn
+++ b/doc/tips/wikiannounce.mdwn
@@ -2,7 +2,7 @@ One thing I use ikiwiki for is the web pages for software projects I
 maintain. Each of my projects has a news page with an announcements feed,
 and to automatically update this when I release a new version, generating
 an announcement from the debian/changelog and debian/NEWS files, I use a
-[wikiannounce](http://git.kitenet.net/?p=joey/home;a=blob_plain;f=bin/wikiannounce)
+[wikiannounce](http://git.kitenet.net/?p=joey/home.git;a=blob_plain;f=bin/wikiannounce)
 program. It's somewhat specific to dealing with Debian packages, and uses a
 special `announcedir` target in debian/rules, but the general idea could be
 useful. --[[Joey]]

diff --git a/doc/todo/mbox.mdwn b/doc/todo/mbox.mdwn
index bae9ea3e846138b2737a1acc02c3337fd41ee1e2..2df7ed87784289559cdc4ab720ba70da21c10986 100644 (file)
--- a/doc/todo/mbox.mdwn
+++ b/doc/todo/mbox.mdwn
@@ -3,4 +3,10 @@ I'd like to be able to drop an unmodified RFC2822 email message into ikiwiki, an
 > We're discussing doing just that (well, whole mailboxes, really) over in
 > [[comment_by_mail]] --[[Joey]]
 
->> I am going to start putting something simple together. [[DavidBremner]]
+>> I am going to start putting something simple together, but 
+>> probably not too quickly.
+>> So far I don't see a way to have ikiwiki process directories 
+>> (i.e. maildirs or mh folders) as a single page.  This not that 
+>> big of a deal, but it means that every mailbox will need 
+>> something like \[[!mailbox type=maildir path=thedir]] in some  
+>> "normal" (e.g. markdown) page -- [[DavidBremner]]