Code deduplication fin find_src_files()

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

diff --git a/IkiWiki/Plugin/edittemplate.pm b/IkiWiki/Plugin/edittemplate.pm
index c381940e5edd4056b5622c0daeb10cab1d6423a8..a163b0d84c3b28cee03ab3eecd24744ee5a2fc65 100644 (file)
--- a/IkiWiki/Plugin/edittemplate.pm
+++ b/IkiWiki/Plugin/edittemplate.pm
@@ -3,11 +3,11 @@ package IkiWiki::Plugin::edittemplate;
 
 use warnings;
 use strict;
 
 use warnings;
 use strict;

-use IkiWiki 2.00;
+use IkiWiki 3.00;

 use HTML::Template;
 use Encode;
 
 use HTML::Template;
 use Encode;
 

-sub import { #{{{
+sub import {

        hook(type => "getsetup", id => "edittemplate",
                call => \&getsetup);
        hook(type => "needsbuild", id => "edittemplate",
        hook(type => "getsetup", id => "edittemplate",
                call => \&getsetup);
        hook(type => "needsbuild", id => "edittemplate",


@@ -16,17 +16,17 @@ sub import { #{{{
                call => \&preprocess);
        hook(type => "formbuilder", id => "edittemplate",
                call => \&formbuilder);
                call => \&preprocess);
        hook(type => "formbuilder", id => "edittemplate",
                call => \&formbuilder);

-} #}}}
+}

 
 

-sub getsetup () { #{{{
+sub getsetup () {

        return
                plugin => {
                        safe => 1,
                        rebuild => undef,
                },
        return
                plugin => {
                        safe => 1,
                        rebuild => undef,
                },

-} #}}}
+}

 
 

-sub needsbuild (@) { #{{{
+sub needsbuild (@) {

        my $needsbuild=shift;
 
        foreach my $page (keys %pagestate) {
        my $needsbuild=shift;
 
        foreach my $page (keys %pagestate) {


@@ -40,9 +40,9 @@ sub needsbuild (@) { #{{{
                        }
                }
        }
                        }
                }
        }

-} #}}}
+}

 
 

-sub preprocess (@) { #{{{
+sub preprocess (@) {

         my %params=@_;
 
        return "" if $params{page} ne $params{destpage};
         my %params=@_;
 
        return "" if $params{page} ne $params{destpage};


@@ -54,24 +54,23 @@ sub preprocess (@) { #{{{
                error gettext("match not specified")
        }
 
                error gettext("match not specified")
        }
 

-       $pagestate{$params{page}}{edittemplate}{$params{match}}=$params{template};
+       my $link=linkpage($params{template});
+       my $bestlink=bestlink($params{page}, $link);
+       $pagestate{$params{page}}{edittemplate}{$params{match}}=$bestlink;

 
        return "" if ($params{silent} && IkiWiki::yesno($params{silent}));
 
        return "" if ($params{silent} && IkiWiki::yesno($params{silent}));

+       add_depends($params{page}, $link, deptype("presence"));
+       return sprintf(gettext("edittemplate %s registered for %s"),
+               htmllink($params{page}, $params{destpage}, $link),
+               $params{match});
+}

 
 

-       my $link=IkiWiki::linkpage($params{template});
-       add_depends($params{page}, $link);
-       my $linkHTML = htmllink($params{page}, $params{destpage}, $link);
-
-       return sprintf(gettext("edittemplate: %s registered for %s"),
-               $linkHTML, $params{match});
-} # }}}
-
-sub formbuilder (@) { #{{{
+sub formbuilder (@) {

        my %params=@_;
        my $form=$params{form};
 
        return if $form->field("do") ne "create" ||
        my %params=@_;
        my $form=$params{form};
 
        return if $form->field("do") ne "create" ||

-               length $form->field("editcontent");
+               (defined $form->field("editcontent") && length $form->field("editcontent"));

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


@@ -84,29 +83,33 @@ sub formbuilder (@) { #{{{
        foreach my $field ($form->field) {
                if ($field eq 'page') {
                        @page_locs=$field->def_value;
        foreach my $field ($form->field) {
                if ($field eq 'page') {
                        @page_locs=$field->def_value;

-                       push @page_locs, $field->options;
+
+                       # FormBuilder is on the bad crack. See #551499
+                       my @options=map { ref $_ ? @$_ : $_ } $field->options;
+
+                       push @page_locs, @options;

                }
        }
                }
        }

-

        foreach my $p (@page_locs) {
                foreach my $registering_page (keys %pagestate) {
                        if (exists $pagestate{$registering_page}{edittemplate}) {
                                foreach my $pagespec (sort keys %{$pagestate{$registering_page}{edittemplate}}) {
                                        if (pagespec_match($p, $pagespec, location => $registering_page)) {
        foreach my $p (@page_locs) {
                foreach my $registering_page (keys %pagestate) {
                        if (exists $pagestate{$registering_page}{edittemplate}) {
                                foreach my $pagespec (sort keys %{$pagestate{$registering_page}{edittemplate}}) {
                                        if (pagespec_match($p, $pagespec, location => $registering_page)) {

+                                               my $template=$pagestate{$registering_page}{edittemplate}{$pagespec};

                                                $form->field(name => "editcontent",
                                                $form->field(name => "editcontent",

-                                                        value => filltemplate($pagestate{$registering_page}{edittemplate}{$pagespec}, $page));
+                                                        value =>  filltemplate($template, $page));

                                                $form->field(name => "type",
                                                $form->field(name => "type",

-                                                        value => pagetype($pagesources{$pagestate{$registering_page}{edittemplate}{$pagespec}}))
-                                                               if $pagesources{$pagestate{$registering_page}{edittemplate}{$pagespec}};
+                                                        value => pagetype($pagesources{$template}))
+                                                               if $pagesources{$template};

                                                return;
                                        }
                                }
                        }
                }
        }
                                                return;
                                        }
                                }
                        }
                }
        }

-} #}}}
+}

 
 

-sub filltemplate ($$) { #{{{
+sub filltemplate ($$) {

        my $template_page=shift;
        my $page=shift;
 
        my $template_page=shift;
        my $page=shift;
 


@@ -137,6 +140,6 @@ sub filltemplate ($$) { #{{{
        $template->param(name => $page);
 
        return $template->output;
        $template->param(name => $page);
 
        return $template->output;

-} #}}}
+}

 
 1
 
 1