git.oblomov.eu Git - ikiwiki/blob - doc/todo/inline_postform_autotitles.mdwn

   1 [[!tag wishlist patch plugins/inline]]
   2 [[!template id=gitbranch branch=chrysn/patches author="[[chrysn]]"]]
   3
   4 for postforms in inlines of pages which follow a certain scheme, it might not
   5 be required to set the title for each individual post, but to automatically set
   6 the title and show no input box prompting for it.
   7 this can either be based on timestamp formatting, or use the already existing
   8 munging mechanism, which appends numbers to page titles in case that page
   9 already exists.
  10
  11 two patches ([1], [2]) set inline up for that, adding an additional `autotitle`
  12 parameter. if that is given, the regular input of the inline postform will be
  13 replaced with a hidden input of that text. in addition, the empty title is
  14 permitted (both for autotitle and regular titles, as they go in the same GET
  15 parameter, `title`). as the empty page title is illegal, munging is used,
  16 resulting in ascending numeric page titles to be created.
  17
  18 the second patch is actually a one-liner, filtering the title through strftime.
  19
  20 ### potential user interaction issues
  21
  22 this has two side effects which have to be considered: first, the empty page
  23 title is accepted also in normal postforms (previously, this resulted in a "bad
  24 page name" error); second, entering a percent sign in that field might result
  25 in unexpexted strftime substitution (strftime might not even substitute for
  26 common uses of percent as in "reach 10% market share", but might in others as
  27 in "the 10%-rule").
  28
  29 both can be circumvented by using another GET parameter for autotexts, as
  30 implemented in [3].
  31 > this patch still does not work perfectly; especially, it should make a
  32 > distinction between "autotitle is set but equal ''" (in which case it
  33 > should create a page named `1.mdwn`, and "autotitle is not set, and title is
  34 > equal ''" (in which case it should display the old error message) --[[chrysn]]
  35
  36 ### potential security issues
  37
  38 * the autotitle's value is directly output through the template (but that's
  39   done in other places as well, so i assume it's safe)
  40 * i don't know if anything bad can happen if unfiltered content is passed to
  41   POSIX::strftime.
  42
  43 ### further extension
  44
  45 having a pre-filled input field instead of an unchangable hidden input might be
  46 cool (eg for creating an entry with yesterday's date), but would be a bit of a
  47 problem with static pages. javascript could help with the date part, but name
  48 munging would be yet another thing.
  49
  50 [1]: http://github.com/github076986099/ikiwiki/commit/b568eb257a3ef5ff49a84ac00a3a7465b643c1e1
  51 [2]: http://github.com/github076986099/ikiwiki/commit/34bc82f232be141edf036d35e8ef5aa289415072
  52 [3]: http://github.com/github076986099/ikiwiki/commit/40dc10a4ec7809e401b4497c2abccfba30f7a2af