]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/commitdiff
Merge commit 'origin'
authorintrigeri <intrigeri@boum.org>
Mon, 3 Nov 2008 18:47:55 +0000 (19:47 +0100)
committerintrigeri <intrigeri@boum.org>
Mon, 3 Nov 2008 18:47:55 +0000 (19:47 +0100)
IkiWiki/Plugin/format.pm
doc/bugs/pagetitle_function_does_not_respect_meta_titles.mdwn
doc/plugins/contrib/po.mdwn
doc/todo/New_preprocessor_directive_syntax/discussion.mdwn
doc/todo/syntax_highlighting.mdwn
doc/todo/using_meta_titles_for_parentlinks.html

index a219190e820a4f579a389c223a536a371e357ef7..1e21a0bdc59d01b1768e04a812b6fd59e4afb7b1 100644 (file)
@@ -23,7 +23,8 @@ sub preprocess (@) { #{{{
                error(sprintf(gettext("unsupported page format %s"), $format));
        }
 
-       return IkiWiki::htmlize($params{page}, $params{destpage}, $format, $text);
+       return IkiWiki::htmlize($params{page}, $params{destpage}, $format,
+               IkiWiki::preprocess($params{page}, $params{destpage}, $text));
 } #}}}
 
 1
index a30ab0fa377508c9a0f9f9bac611980732f4672e..77c86eba154e019cead22fef7420b72e1715f4ad 100644 (file)
@@ -1,3 +1,12 @@
 The `IkiWiki::pagetitle` function does not respect title changes via `meta.title`. It really should, so that links rendered with `htmllink` get the proper title in the link text.
 
 --[[madduck]]
+
+> Agreed. [[todo/using_meta_titles_for_parentlinks]] contains a beginning of
+> solution. A few quick notes about it:
+
+> - Using <code>inline</code> would avoid the redefinition + code duplication.
+> - A few plugins would need to be upgraded.
+> - It may be necessary to adapt the testsuite in `t/pagetitle.t`, as well.
+
+> --[[intrigeri]]
index f60b8fbea26bd5b074fc8f4e62d1c232107a0d5e..98cc07178eb6789af0ed0c029a80982a025b2c3d 100644 (file)
@@ -52,10 +52,13 @@ Any thoughts on this?
 >>> [[plugins/write]]. I think you can just inject wrappers about a few ikiwiki
 >>> functions, rather than adding hooks. The `inject` function is pretty
 >>> insane^Wlow level, but seems to work great. --[[Joey]]
->>
+>>>
 >>>> Thanks a lot, it seems to be a nice interface for what I was trying to achieve.
 >>>> I may be forced to wait two long weeks before I have a chance to confirm
 >>>> this. Stay tuned. --[[intrigeri]]
+>>>>
+>>>>> I've updated the plugin to use `inject`. It is now fully self-contained,
+>>>>> and does not modify the core anymore. --[[intrigeri]]
 >>
 >> The Discussion pages issue is something I am not sure about yet. But I will
 >> probably decide that "slave" pages, being only translations, don't deserve
index dda1ff5e9812ef1c9a4c5666fc36fd2257f2f701..f6c0fc0ec6cbe03227985f20e0ac4a6f1bbefc14 100644 (file)
@@ -1,2 +1,19 @@
 Err, is this really fixed in 2.21? I can't find it anywhere in 2.32.3 
 (debian unstable)
+
+-----
+
+I just did a `--dumpsetup` with the current version from the Git repository
+and the default option is
+
+    # use '!'-prefixed preprocessor directives?
+    prefix_directives => 0,
+
+My impression was that this should be enabled by default now.  --[[JasonBlevins]]
+
+> As stated in `debian/NEWS`:
+>> For backward compatibility with existing wikis,
+>> refix_directives currently defaults to false.  In ikiwiki 3.0,
+>> prefix_directives will default to true [...]
+> --[[intrigeri]]
+
index bb1c84f026b76437f9227a4672428a50d2d4dede..5df1857050e0585daf457f0109517af387e59823 100644 (file)
@@ -32,6 +32,10 @@ pages, as well as doing syntax highlighting as a preprocessor directive
   we could use an external plugin..)
 * Currently no single plugin supports both modes of operation (directive
   and whole source file to page).
+
+  > This is now fixed by the [[ikiwiki/directive/format]] directive for all
+  > whole-source-file plugins, right?
+
 * Nothing seems to support 
   [[wiki-formatted_comments|wiki-formatted_comments_with_syntax_plugin]]
   inside source files. Doing this probably means post-processing the 
@@ -45,6 +49,17 @@ pages, as well as doing syntax highlighting as a preprocessor directive
   One approach that's also been requested for eg,
   [[plugins/contrib/mediawiki]] is to allow controlling which linkification
   types a page type can have on it.
+
+  > The previous two points seem to be related.  One thought: instead of
+  > getting the source from the `content` parameter, the plugin could
+  > re-load the page source.  That would stop directives/links from
+  > being processed in the source.  As noted above, comments
+  > could then be parsed for directives/links later.
+  >
+  > Would it be worth adding a `nodirectives` option when registering
+  > an htmlize hook that switches off directive and link processing before
+  > generating the html for a page?
+
 * The whole-file plugins all get confused if there is a `foo.c` and a `foo.h`.
   This is trivially fixable now by passing the keepextension option when
   registering the htmlize hooks, though.
@@ -61,6 +76,28 @@ pages, as well as doing syntax highlighting as a preprocessor directive
   extensions. The workaround is to use a directive on a wiki page, pulling
   in the Makefile.
 
+  > I wonder how hard it would be to make a patch whereby a file with
+  > no `.` in the name, and a name that matches a filetype, and where
+  > that filetype was registered `keepextension`, then the file is just
+  > chosen as the appropriate type.  This would allow `Makefile` to
+  > work.
+
+like this:
+
+    diff --git a/IkiWiki.pm b/IkiWiki.pm
+    index 8d728c9..1bd46a9 100644
+    --- a/IkiWiki.pm
+    +++ b/IkiWiki.pm
+    @@ -618,6 +618,8 @@ sub pagetype ($) { #{{{
+       
+       if ($page =~ /\.([^.]+)$/) {
+               return $1 if exists $hooks{htmlize}{$1};
+    +  } elsif ($hooks{htmlize}{$page}{keepextension}) {
+    +          return $page;
+       }
+       return;
+     } #}}}
+
 ## format directive
 
 Rather than making syntax highlight plugins have to provide a preprocessor
index 651b7fa0fc4e02233c2740455ae55f1d1d494a3b..d04e5a300227bf5db891309372b5e4577e05d325 100644 (file)
@@ -114,3 +114,9 @@ diff -c /usr/share/perl5/IkiWiki/Plugin/meta.pm.distrib /usr/share/perl5/IkiWiki
 
 
 </pre>
+
+<p>
+This is actually a duplicate for
+[[bugs/pagetitle_function_does_not_respect_meta_titles]], where I'm
+following up a bit. --[[intrigeri]]
+</p>
\ No newline at end of file