fix default values for config settings
authorJoey Hess <joey@gnu.kitenet.net>
Wed, 17 Dec 2008 22:05:49 +0000 (17:05 -0500)
committerJoey Hess <joey@gnu.kitenet.net>
Wed, 17 Dec 2008 22:05:49 +0000 (17:05 -0500)
defaults cannot be set in getsetup, do it in checkconfig to avoid
uninitialized value warnings.

IkiWiki/Plugin/comments.pm

index 6184c6031281f5c0bd719b2b9dab95ae8337a100..83f67869c06b6d68b58050373ea3e26b9dd9aa00 100644 (file)
@@ -39,7 +39,6 @@ sub getsetup () {
                comments_shown_pagespec => {
                        type => 'pagespec',
                        example => 'blog/*',
-                       default => '',
                        description => 'PageSpec for pages where comments will be shown inline',
                        link => 'ikiwiki/PageSpec',
                        safe => 1,
@@ -48,7 +47,6 @@ sub getsetup () {
                comments_open_pagespec => {
                        type => 'pagespec',
                        example => 'blog/* and created_after(close_old_comments)',
-                       default => '',
                        description => 'PageSpec for pages where new comments can be posted',
                        link => 'ikiwiki/PageSpec',
                        safe => 1,
@@ -56,7 +54,6 @@ sub getsetup () {
                },
                comments_pagename => {
                        type => 'string',
-                       example => 'comment_',
                        default => 'comment_',
                        description => 'Base name for comments, e.g. "comment_" for pages like "sandbox/comment_12"',
                        safe => 0, # manual page moving required
@@ -64,7 +61,6 @@ sub getsetup () {
                },
                comments_allowdirectives => {
                        type => 'boolean',
-                       default => 0,
                        example => 0,
                        description => 'Interpret directives in comments?',
                        safe => 1,
@@ -72,7 +68,6 @@ sub getsetup () {
                },
                comments_allowauthor => {
                        type => 'boolean',
-                       default => 0,
                        example => 0,
                        description => 'Allow anonymous commenters to set an author name?',
                        safe => 1,
@@ -81,7 +76,6 @@ sub getsetup () {
                comments_commit => {
                        type => 'boolean',
                        example => 1,
-                       default => 1,
                        description => 'commit comments to the VCS',
                        # old uncommitted comments are likely to cause
                        # confusion if this is changed
@@ -90,6 +84,17 @@ sub getsetup () {
                },
 }
 
+sub checkconfig () {
+       $config{comments_commit} = 1
+               unless defined $config{comments_commit};
+       $config{comments_shown_pagespec} = ''
+               unless defined $config{comments_shown_pagespec};
+       $config{comments_open_pagespec} = ''
+               unless defined $config{comments_open_pagespec};
+       $config{comments_pagename} = 'comment_'
+               unless defined $config{comments_pagename};
+}
+
 sub htmlize {
        my %params = @_;
        return $params{content};
@@ -208,12 +213,6 @@ sub preprocess {
        return $content;
 }
 
-sub checkconfig () {
-       $config{comments_commit} = 1 unless defined $config{comments_commit};
-       $config{comments_pagename} = 'comment_'
-               unless defined $config{comments_pagename};
-}
-
 # This is exactly the same as recentchanges_link :-(
 sub linkcgi ($) {
        my $cgi=shift;