doc/patchqueue/fallback_for_templates.mdwn

   1 It would be nice if you didn't have to copy all templates to your custom templatedir so you only have your customizations there. And then have it fall back to the location as configured at build time (like /usr/share/ikiwiki/templates/). This would be similar to underlaydir. I don't think any extra configuration directive is needed. As it is now, RecentChanges will have Perl CGI error and Preferences will be an empty page, if you don't have corresponding templates.
   2
   3 I have been testing the following. Sorry if the don't match up as I may have other changes (unrelated) also.
   4
   5 (no markdown for patch, so open in editor to real)
   6
   7 --- IkiWiki.pm.save     2007-01-12 09:05:48.000000000 +0900
   8 +++ IkiWiki.pm  2007-01-12 09:08:24.000000000 +0900
   9 @@ -638,7 +638,9 @@
  10                         my $text_ref = shift;
  11                         $$text_ref=&Encode::decode_utf8($$text_ref);
  12                 },
  13 -               filename => "$config{templatedir}/$filename",
  14 +               filename => (-e "$config{templatedir}/$filename" ?
  15 +                       "$config{templatedir}/$filename" :
  16 +                       "$installdir/share/ikiwiki/templates/$filename"),
  17                 loop_context_vars => 1,
  18                 die_on_bad_params => 0,
  19                 @_;
  20
  21 --- IkiWiki/CGI.pm.orig 2007-01-01 07:18:26.000000000 +0900
  22 +++ IkiWiki/CGI.pm      2007-01-12 10:29:14.000000000 +0900
  23 @@ -291,8 +294,7 @@
  24                 params => $q,
  25                 action => $config{cgiurl},
  26                 table => 0,
  27 -               template => (-e "$config{templatedir}/editpage.tmpl" ?
  28 -                               {template_params("editpage.tmpl")} : ""),
  29 +               template => {template_params("editpage.tmpl")},
  30         );
  31
  32         run_hooks(formbuilder_setup => sub {
  33
  34
  35 Note I don't do prefs nor signin because they are not real users of templates.
  36 Also, the above is a little redundant as by default it will use same path for both.
  37
  38 Once something like this is in source then usage can be updated