bit more tweakiness

[git.ikiwiki.info.git] / doc / todo / inline_autotitles.mdwn

diff --git a/doc/todo/inline_autotitles.mdwn b/doc/todo/inline_autotitles.mdwn
index 9846687e953f499d889ae49114c6f7b917910ce6..8bf71deaed5571978e7638d8429ecf5195586d47 100644 (file)
--- a/doc/todo/inline_autotitles.mdwn
+++ b/doc/todo/inline_autotitles.mdwn
@@ -7,26 +7,30 @@ this can either be based on timestamp formatting, or use the already existing
 munging mechanism, which appends numbers to page titles in case that page
 already exists.
 
-two [patches][1] set inline up for that, adding an additional `autotitle`
+two patches ([1], [2]) set inline up for that, adding an additional `autotitle`
 parameter. if that is given, the regular input of the inline postform will be
 replaced with a hidden input of that text. in addition, the empty title is
 permitted (both for autotitle and regular titles, as they go in the same GET
 parameter, `title`). as the empty page title is illegal, munging is used,
 resulting in ascending numeric page titles to be created.
 
-the second patch is actually a one-liner, filtering the title through sprintf.
+the second patch is actually a one-liner, filtering the title through strftime.
 
 ### potential user interaction issues
 
 this has two side effects which have to be considered: first, the empty page
 title is accepted also in normal postforms (previously, this resulted in a "bad
 page name" error); second, entering a percent sign in that field might result
-in unexpexted sprintf substitution (sprintf might not even substitute for
+in unexpexted strftime substitution (strftime might not even substitute for
 common uses of percent as in "reach 10% market share", but might in others as
 in "the 10%-rule").
 
-both can be circumvented by using another GET parameter for autotexts, for
-which i could provide a patch. (as of writing this, i think i'll do that.)
+both can be circumvented by using another GET parameter for autotexts, as
+implemented in [3].
+> this patch still does not work perfectly; especially, it should make a
+> distinction between "autotitle is set but equal ''" (in which case it
+> should create a page named `1.mdwn`, and "autotitle is not set, and title is
+> equal ''" (in which case it should display the old error message) --[[chrysn]]
 
 ### potential security issues
 
@@ -42,4 +46,6 @@ cool (eg for creating an entry with yesterday's date), but would be a bit of a
 problem with static pages. javascript could help with the date part, but name
 munging would be yet another thing.
 
-[1]: http://...
+[1]: http://github.com/github076986099/ikiwiki/commit/b568eb257a3ef5ff49a84ac00a3a7465b643c1e1
+[2]: http://github.com/github076986099/ikiwiki/commit/34bc82f232be141edf036d35e8ef5aa289415072
+[3]: http://github.com/github076986099/ikiwiki/commit/40dc10a4ec7809e401b4497c2abccfba30f7a2af