]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/commitdiff
* Change order of linkify and preprocess; first preprocess and then linkify.
authorjoey <joey@0fa5a96a-9a0e-0410-b3b2-a0fd24251071>
Wed, 23 Aug 2006 20:23:57 +0000 (20:23 +0000)
committerjoey <joey@0fa5a96a-9a0e-0410-b3b2-a0fd24251071>
Wed, 23 Aug 2006 20:23:57 +0000 (20:23 +0000)
  This allows passing a wikilink inside a parameter to a preprocessor
  directive without it being expanded to html, and leaking out of the
  parameter, which had required some non-obvious use of triple-quoting
  to avoid. Note that any preprocessor plugins that output something
  that looks like a wikilink will now have it treated as such; AFAIK
  this doesn't change any behavior though except for the template plugin.
* Enable preprocessor directives when previewing an edit.

IkiWiki/CGI.pm
IkiWiki/Plugin/inline.pm
IkiWiki/Plugin/sidebar.pm
IkiWiki/Render.pm
debian/changelog
doc/plugins/template.mdwn
doc/todo/linkify_and_preprocessor_ordering.mdwn [new file with mode: 0644]

index 7d4ba146fc9de3eb2e8ca08fb7b4d100a29c6bda..120e2fdee7dad588a027f6ef0264d995714e4a5b 100644 (file)
@@ -411,8 +411,12 @@ sub cgi_editpage ($$) { #{{{
                                value => $content, force => 1);
                $form->field(name => "comments",
                                value => $comments, force => 1);
                                value => $content, force => 1);
                $form->field(name => "comments",
                                value => $comments, force => 1);
+               $config{rss}=0; # avoid preview writing an rss feed!
                $form->tmpl_param("page_preview",
                $form->tmpl_param("page_preview",
-                       htmlize($type, linkify($page, "", filter($page, $content))));
+                       htmlize($type,
+                       linkify($page, "",
+                       preprocess($page, $page,
+                       filter($page, $content)))));
        }
        else {
                $form->tmpl_param("page_preview", "");
        }
        else {
                $form->tmpl_param("page_preview", "");
index 935b86b5858d53ab73ce258ac9021c3b3f80aad1..c712f7bb434b80135b1d4742e3d4451de3f6163d 100644 (file)
@@ -116,8 +116,8 @@ sub preprocess_inline (@) { #{{{
                        my $type=pagetype($file);
                        if (defined $type) {
                                $ret.="\n".
                        my $type=pagetype($file);
                        if (defined $type) {
                                $ret.="\n".
-                                     preprocess($page, $params{page},
                                      linkify($page, $params{page},
                                      linkify($page, $params{page},
+                                     preprocess($page, $params{page},
                                      filter($page,
                                      readfile(srcfile($file)))));
                        }
                                      filter($page,
                                      readfile(srcfile($file)))));
                        }
@@ -145,8 +145,8 @@ sub get_inline_content ($$) { #{{{
        my $type=pagetype($file);
        if (defined $type) {
                return htmlize($type,
        my $type=pagetype($file);
        if (defined $type) {
                return htmlize($type,
-                      preprocess($page, $destpage,
                       linkify($page, $destpage,
                       linkify($page, $destpage,
+                      preprocess($page, $destpage,
                       filter($page,
                       readfile(srcfile($file))))));
        }
                       filter($page,
                       readfile(srcfile($file))))));
        }
index a68f5e1da6016ba39a600c571111e8be9e3c3ee9..e0e81526f080cdb502d0f72cfb8e4e4bf145303a 100644 (file)
@@ -29,8 +29,8 @@ sub sidebar_content ($) { #{{{
                my $content=IkiWiki::readfile(IkiWiki::srcfile($sidebar_file));
                return unless length $content;
                return IkiWiki::htmlize($sidebar_type,
                my $content=IkiWiki::readfile(IkiWiki::srcfile($sidebar_file));
                return unless length $content;
                return IkiWiki::htmlize($sidebar_type,
-                      IkiWiki::preprocess($sidebar_page, $page,
                       IkiWiki::linkify($sidebar_page, $page,
                       IkiWiki::linkify($sidebar_page, $page,
+                      IkiWiki::preprocess($sidebar_page, $page,
                       IkiWiki::filter($sidebar_page, $content))));
        }
 
                       IkiWiki::filter($sidebar_page, $content))));
        }
 
index 5351ee7adedc9e09caff4d6df270ee15b9d8b989..cef770c6784025d89c2b1d64ffeca2391d5c3749 100644 (file)
@@ -104,11 +104,24 @@ sub preprocess ($$$;$) { #{{{
                        # Note: preserve order of params, some plugins may
                        # consider it significant.
                        my @params;
                        # Note: preserve order of params, some plugins may
                        # consider it significant.
                        my @params;
-                       while ($params =~ /(?:(\w+)=)?(?:"""\n?(.+)"""|"([^"]+)"|(\S+))(?:\s+|$)/sg) {
-                               my $val=(defined $2 ? $2 : (defined $3 ? $3 : $4));
-                               chomp $val;
-                               if (defined $1) {
-                                       push @params, $1, $val;
+                       while ($params =~ /(?:(\w+)=)?(?:"""(.*?)"""|"([^"]+)"|(\S+))(?:\s+|$)/sg) {
+                               my $key=$1;
+                               my $val;
+                               if (defined $2) {
+                                       $val=$2;
+                                       $val=~s/\r\n/\n/mg;
+                                       $val=~s/^\n+//g;
+                                       $val=~s/\n+$//g;
+                               }
+                               elsif (defined $3) {
+                                       $val=$3;
+                               }
+                               elsif (defined $4) {
+                                       $val=$4;
+                               }
+
+                               if (defined $key) {
+                                       push @params, $key, $val;
                                }
                                else {
                                        push @params, $val, '';
                                }
                                else {
                                        push @params, $val, '';
@@ -125,7 +138,7 @@ sub preprocess ($$$;$) { #{{{
                }
        };
        
                }
        };
        
-       $content =~ s{(\\?)\[\[(\w+)\s+((?:(?:\w+=)?(?:""".+"""|"[^"]+"|[^\s\]]+)\s*)*)\]\]}{$handle->($1, $2, $3)}eg;
+       $content =~ s{(\\?)\[\[(\w+)\s+((?:(?:\w+=)?(?:""".*?"""|"[^"]+"|[^\s\]]+)\s*)*)\]\]}{$handle->($1, $2, $3)}seg;
        return $content;
 } #}}}
 
        return $content;
 } #}}}
 
@@ -268,8 +281,8 @@ sub render ($) { #{{{
                
                $links{$page}=[findlinks($page, $content)];
                
                
                $links{$page}=[findlinks($page, $content)];
                
-               $content=linkify($page, $page, $content);
                $content=preprocess($page, $page, $content);
                $content=preprocess($page, $page, $content);
+               $content=linkify($page, $page, $content);
                $content=htmlize($type, $content);
                
                check_overwrite("$config{destdir}/".htmlpage($page), $page);
                $content=htmlize($type, $content);
                
                check_overwrite("$config{destdir}/".htmlpage($page), $page);
index 0d0b74d054a99b66bb3649b94d97823db0f788be..7931dc98abe0f669496d3d46df10fdd079ecb1ea 100644 (file)
@@ -16,8 +16,16 @@ ikiwiki (1.22) UNRELEASED; urgency=low
   * Make pagespec merge code smarter about merging duplicate pagespecs.
   * Patch from Jordà Polo to make Setup::Standard support hashes in config
     files.
   * Make pagespec merge code smarter about merging duplicate pagespecs.
   * Patch from Jordà Polo to make Setup::Standard support hashes in config
     files.
-
- -- Joey Hess <joeyh@debian.org>  Wed, 23 Aug 2006 14:35:18 -0400
+  * Change order of linkify and preprocess; first preprocess and then linkify.
+    This allows passing a wikilink inside a parameter to a preprocessor
+    directive without it being expanded to html, and leaking out of the
+    parameter, which had required some non-obvious use of triple-quoting
+    to avoid. Note that any preprocessor plugins that output something
+    that looks like a wikilink will now have it treated as such; AFAIK
+    this doesn't change any behavior though except for the template plugin.
+  * Enable preprocessor directives when previewing an edit.
+
+ -- Joey Hess <joeyh@debian.org>  Wed, 23 Aug 2006 15:30:09 -0400
 
 ikiwiki (1.21) unstable; urgency=low
 
 
 ikiwiki (1.21) unstable; urgency=low
 
index 9e2ab089147674285fcf34963664a6a49537295a..6f38e554eca488cfdbc92e5ddb3a6ae132e8b438 100644 (file)
@@ -9,15 +9,18 @@ and inserted into pages in the wiki. Using a template works like this:
 This fills out the template `templates/foo`, filling in the `color` and `age`
 fields on it with the specified values, and inserts the result into the page.
 
 This fills out the template `templates/foo`, filling in the `color` and `age`
 fields on it with the specified values, and inserts the result into the page.
 
-If a value is triple-quoted, it can include any markup that would be
-allowed in the wiki page outside the template. Combined with multi-line
-quoted values, this allows for large chunks of marked up text to be
-embedded into a template:
+(Note that if the template doesn't exist, the page will provide a link that
+can be used to create it.)
+
+A value can include any markup that would be allowed in the wiki page
+outside the template. Triple-quoting the value even allows quotes to be
+included in it. Combined with multi-line quoted values, this allows for
+large chunks of marked up text to be embedded into a template:
 
        \[[template id=foo name="Sally" color="green" age=8 notes="""
        * \[[Charley]]'s sister.
 
        \[[template id=foo name="Sally" color="green" age=8 notes="""
        * \[[Charley]]'s sister.
+       * "I want to be an astronaut when I grow up."
        * Really 8 and a half.
        * Really 8 and a half.
-       * Wants to be an astronaut when she grows up.
        """]]
 
 To create a template, make a page in the wiki named `template/foo`. Note
        """]]
 
 To create a template, make a page in the wiki named `template/foo`. Note
@@ -31,6 +34,8 @@ for the full syntax, but all you really need to know are a few things:
 * To insert the value of a variable, use `<TMPL_VAR variable>`.
 * To make a block of text conditional on a variable being set use
   `<TMPL_IF NAME="variable">text</TMPL_IF>`.
 * To insert the value of a variable, use `<TMPL_VAR variable>`.
 * To make a block of text conditional on a variable being set use
   `<TMPL_IF NAME="variable">text</TMPL_IF>`.
+* To use one block of text if a variable is set and a second if it's not,
+  use `<TMPL_IF NAME="variable">text<TMPL_ELSE>other text</TMPL_IF>`
 
 Here's a sample template:
 
 
 Here's a sample template:
 
diff --git a/doc/todo/linkify_and_preprocessor_ordering.mdwn b/doc/todo/linkify_and_preprocessor_ordering.mdwn
new file mode 100644 (file)
index 0000000..2936d74
--- /dev/null
@@ -0,0 +1,24 @@
+Currently ikiwiki linkifies text, then runs preprocessor directives. This
+allows a directive to contain a wikilink inside a parameter, but since the
+wikilink expands to some arbitrary html, the parameter needs to be
+triple-quoted to avoid quotes in the expanded text from leaking out. This
+is rather non-obvious.
+
+One fix would be to switch the order, since linkification and preprocessing
+are relatively independant. Some directives, like inline, would need to keep
+on linkifiying the inlined pages, to make the links be resolved correctly,
+but that's ok. Any directives that outputed stuff that looked like a
+wikilink, but wasn't, would need to be changed.
+
+> This solution has been implemented and _seems_ ok.
+
+An alternative would be to change the wikilink regexp so it doesn't apply
+to wikilinks that are embedded inside preprocessor directives. I haven't
+found a way to do that yet, since perl doesn't allow variable-width
+negative lookbehind.
+
+Maybe processing wikilinks and preprocessor directives
+as part of the same loop would work, but that probably has its own
+issues.
+
+[[todo/done]]