cherry-pick uri security fix

[git.ikiwiki.info.git] / IkiWiki / Plugin / edittemplate.pm

diff --git a/IkiWiki/Plugin/edittemplate.pm b/IkiWiki/Plugin/edittemplate.pm
index 20c948eb1af75d30b6913efafce70198b3b604b5..b7651ce02b3c71ac96b4a561f3ef5e22738a6a5b 100644 (file)
--- a/IkiWiki/Plugin/edittemplate.pm
+++ b/IkiWiki/Plugin/edittemplate.pm
@@ -12,8 +12,8 @@ sub import { #{{{
                call => \&needsbuild);
        hook(type => "preprocess", id => "edittemplate",
                call => \&preprocess);
                call => \&needsbuild);
        hook(type => "preprocess", id => "edittemplate",
                call => \&preprocess);

-       hook(type => "formbuilder_setup", id => "edittemplate",
-               call => \&formbuilder_setup);
+       hook(type => "formbuilder", id => "edittemplate",
+               call => \&formbuilder);

 } #}}}
 
 sub needsbuild (@) { #{{{
 } #}}}
 
 sub needsbuild (@) { #{{{


@@ -21,7 +21,8 @@ sub needsbuild (@) { #{{{
 
        foreach my $page (keys %pagestate) {
                if (exists $pagestate{$page}{edittemplate}) {
 
        foreach my $page (keys %pagestate) {
                if (exists $pagestate{$page}{edittemplate}) {

-                       if (grep { $_ eq $pagesources{$page} } @$needsbuild) {
+                       if (exists $pagesources{$page} && 
+                           grep { $_ eq $pagesources{$page} } @$needsbuild) {

                                # remove state, it will be re-added
                                # if the preprocessor directive is still
                                # there during the rebuild
                                # remove state, it will be re-added
                                # if the preprocessor directive is still
                                # there during the rebuild


@@ -49,32 +50,23 @@ sub preprocess (@) { #{{{
                $params{template}, $params{match});
 } # }}}
 
                $params{template}, $params{match});
 } # }}}
 

-sub formbuilder_setup (@) { #{{{
+sub formbuilder (@) { #{{{

        my %params=@_;
        my $form=$params{form};
        my %params=@_;
        my $form=$params{form};

-       
-       return if $form->title ne "editpage"
-                 || $form->field("do") ne "create";
-       
+
+       return if $form->field("do") ne "create";

        my $page=$form->field("page");
        my $page=$form->field("page");

-       my $from=$form->field("from");

        
        # The tricky bit here is that $page is probably just the base
        # page name, without any subdir, but the pagespec for a template
        # probably does include the subdir (ie, "bugs/*"). We don't know
        
        # The tricky bit here is that $page is probably just the base
        # page name, without any subdir, but the pagespec for a template
        # probably does include the subdir (ie, "bugs/*"). We don't know

-       # what subdir the user will pick to put the page in. So, generate
-       # an ordered list and the first template to match will be used.
-       #
-       # This code corresponds to the code in editpage() that generates
-       # the list of possible page names, unfortunatly, that code runs
-       # later, so that list can't be simply reused.
+       # what subdir the user will pick to put the page in. So, try them
+       # all, starting with the one that was made default.

        my @page_locs=$page;
        my @page_locs=$page;

-       if (defined $from) {
-               push @page_locs, "$from/$page";
-               my $dir=$from.="/";
-               while (length $dir) {
-                       $dir=~s![^/]+/+$!!;
-                       push @page_locs, $dir.$page;
+       foreach my $field ($form->field) {
+               if ($field eq 'page') {
+                       @page_locs=$field->def_value;
+                       push @page_locs, $field->options;

                }
        }
 
                }
        }