Changeset 200

Timestamp:

02/11/07 10:12:59 (3 years ago)

Author:

gboggs

Message:

pluginbuilder.pl:
- Simplified syntax. Removed cargo-cult artifact. Added standard plugin registration metadata.
PluginBuilder/App.pm & pluginbuilder.tmpl:
- Pass validated plugin metadata back and forth between the templates.

Location:

trunk/PluginBuilder/plugins/PluginBuilder

Files:

• lib/PluginBuilder/App.pm (modified) (3 diffs)
• pluginbuilder.pl (modified) (1 diff)
• tmpl/collect-metadata.tmpl (added)
• tmpl/plugin.tmpl (added)
• tmpl/pluginbuilder.tmpl (modified) (1 diff)
• tmpl/readme.tmpl (added)

trunk/PluginBuilder/plugins/PluginBuilder/lib/PluginBuilder/App.pm

@@ -1 +1 @@
 # $Id$
-
 package PluginBuilder::App;
-
 use strict;
 use warnings;
-
 our $VERSION = '0.00_4';
-
 use MT::App;
 @PluginBuilder::App::ISA = 'MT::App';
@@ -15 +11 @@
     $app->SUPER::init(@_) or return;
     $app->add_methods( show_config => \&show_config );
+    $app->add_methods( set_metadata => \&set_metadata );
     $app->{default_mode} = 'show_config';
     $app->{requires_login} = 1;
-
     return $app;
 }
@@ -24 +20 @@
     my $app = shift;
     my $param = {
-        version => $VERSION,
-        user => $app->user,
-        blog => $app->blog,
+        pb_author_link => 'http://',
+        pb_author_name => $app->user->nickname || $app->user->name || $app->user->email,
+        pb_description => 'Frobnicate Whirlygigs',
+        pb_doc_link    => 'http://',
+        pb_plugin_link => 'http://',
+        pb_plugin_name => 'Foo',
+        pb_plugin_version => '0.00_0',
+        plugin_version => $VERSION,
+    };
+    $app->build_page( 'pluginbuilder.tmpl', $param );
+}
+
+sub set_metadata {
+    my $app = shift;
+    my $q = $app->param;
+    my $param = {
+        have_metadata => 1,
+        pb_author_link => $q->param('pb_author_link'),
+        pb_author_name => $q->param('pb_author_name'),
+        pb_description => $q->param('pb_description'),
+        pb_doc_link => $q->param('pb_doc_link'),
+        pb_plugin_link => $q->param('pb_plugin_link'),
+        pb_plugin_name => $q->param('pb_plugin_name'),
+        pb_plugin_version => $q->param('pb_plugin_version'),
+        plugin_version => $VERSION,
     };
     $app->build_page( 'pluginbuilder.tmpl', $param );

trunk/PluginBuilder/plugins/PluginBuilder/pluginbuilder.pl

@@ -2 +2 @@
 # $Id$
 use strict;
-my $PLUGIN_NAME = "PluginBuilder";
 require MT::Plugin;
 my $plugin = MT::Plugin->new(
-    name => $PLUGIN_NAME, 
-    dir_name => lc($PLUGIN_NAME),
+    author_link => 'http://www.sixapart.com/',
+    author_name => 'Gene Boggs',
+    config_link => 'mt-pluginbuilder.cgi',
     description => 'Plugin Stubber',
-    config_link => 'mt-'. lc($PLUGIN_NAME). '.cgi',
+    doc_link    => 'http://www.lifewiki.net/sixapart/PluginBuilder documentation',
+    name        => 'PluginBuilder', 
+    plugin_link => 'http://code.sixapart.com/svn/mtplugins/trunk/PluginBuilder/',
 );
 return MT->add_plugin($plugin);

trunk/PluginBuilder/plugins/PluginBuilder/tmpl/pluginbuilder.tmpl

@@ -1 +1 @@
 <TMPL_INCLUDE NAME="header.tmpl">
 
-<h2>Plugin metadata:</h2>
-<ul>
-<li>Plugin name: <TMPL_VAR NAME=PLUGIN_NAME></li>
-<li>Plugin version: <TMPL_VAR NAME=PLUGIN_VERSION></li>
-<!-- <li>Plugin link: <TMPL_VAR NAME=PLUGIN_LINK></li> -->
-<li>Author name: <TMPL_VAR NAME=AUTHOR_NAME></li>
-<li>Author link: <TMPL_VAR NAME=AUTHOR_LINK></li>
-<li>Description: <TMPL_VAR NAME=DESCRIPTION></li>
-<!-- <li>Blog config template: <TMPL_VAR NAME=BLOG_CONFIG_TEMPLATE></li> -->
-<li>Current blog: <TMPL_VAR NAME=BLOG></li>
-<li>Current user: <TMPL_VAR NAME=USER></li>
-</ul>
+<script type="text/javascript">
+<!--
+function validate (f) {
+/*
+    if (!is_valid_url(f.pb_author_link.value)){
+        alert('<MT_TRANS phrase="You must set a valid Author link." escape="singlequotes">');
+        return false;
+    } else if (!is_valid_url(f.pb_doc_link.value)){
+        alert('<MT_TRANS phrase="You must set a valid Documentation link." escape="singlequotes">');
+        return false;
+    } else if (!is_valid_url(f.pb_plugin_link.value)){
+        alert('<MT_TRANS phrase="You must set a valid Plugin link." escape="singlequotes">');
+        return false;
+    } else
+*/
+    if (!f.pb_author_name.value){
+        alert('<MT_TRANS phrase="You must set a valid Author name." escape="singlequotes">');
+        return false;
+    } else if (!f.pb_description.value){
+        alert('<MT_TRANS phrase="You must set a valid Description." escape="singlequotes">');
+        return false;
+    } else if (!f.pb_plugin_name.value) {
+        alert('<MT_TRANS phrase="You must set your Plugin name." escape="singlequotes">');
+        return false;
+    } else if (!f.pb_plugin_version.value) {
+        alert('<MT_TRANS phrase="You must set your Plugin version." escape="singlequotes">');
+        return false;
+    }
+    return true;
+}
+function is_valid_url(url_){
+    return url_.match( /^https?:\/\/[A-Za-z0-9!$%()=_.:,;@~-]+/ );
+}
+function is_valid_path(path_){
+    for(i = 0; i < path_.length; i++){
+        var buf = escape(path_.substr(i, 1));
+        if (buf.match(/^%u.*/)){
+            return false;
+        }
+        if (buf.match(/(%.{2}){2}?/)){
+            return false;
+        }
+    }
+    return true;
+}
+//-->
+</script>
+
+<TMPL_IF NAME=have_metadata>
+
+<TMPL_INCLUDE NAME="readme.tmpl">
+<hr/>
+<TMPL_INCLUDE NAME="plugin.tmpl">
+
+<TMPL_ELSE>
+
+<TMPL_INCLUDE NAME="collect-metadata.tmpl">
+
+</TMPL_IF>
 
 <hr/>
 <p>
-<MT_TRANS phrase="PluginBuilder"> version <TMPL_VAR NAME=VERSION>
-<br/>
-Please see the
-<a href="http://www.lifewiki.net/sixapart/PluginBuilder">PluginBuilder documentation</a>.
+<a href="http://www.lifewiki.net/sixapart/PluginBuilder"><MT_TRANS phrase="PluginBuilder"></a>
+version <TMPL_VAR NAME=PLUGIN_VERSION>
 </p>