--- trunk/lib/Frey/Action.pm	2008/11/17 20:14:12	385
+++ trunk/lib/Frey/Action.pm	2008/11/17 22:40:22	386
@@ -4,6 +4,7 @@
 with 'Frey::Web';
 with 'Frey::Config';
 
+use Clone qw/clone/;
 use Data::Dump qw/dump/;
 
 =head1 DESCRIPTION
@@ -44,7 +45,7 @@
 			$attr->is_required && $attr;
 		} $self->class->meta->get_attribute_list;
 
-	warn "## required = ",dump( map { $_->name } @required ), " for ", $self->class;
+	warn "## required = ",dump( map { $_->name } @required ), " for ", $self->class if @required && $self->debug;
 	return @required if wantarray;
 	return \@required;
 }
@@ -65,14 +66,15 @@
 	my @attrs = @{ $self->attribute_order };
 	@attrs = map {  $a->{$_}++; $_ } @attrs;
 	push @attrs, $_ foreach grep { ! $a->{$_} } map { $_->name } @{ $self->required };
-	warn "# attributes = ",dump( @attrs );
+	warn "# attributes = ",dump( @attrs ) if $self->debug;
 	return @attrs if wantarray;
 	return \@attrs;
 }
 
 =head2 params_form
 
-  my $html = $self->params_form;
+  my  $html                  = $self->params_form;
+  my ($html,$default_params) = $self->params_form;
 
 =cut
 
@@ -81,20 +83,23 @@
 	my @required = $self->required;
 	if ( ! @required ) {
 		warn "all params available ", dump( $self->params ), " not creating form";
+		return (undef,$self->params) if wantarray;
 		return;
 	} else {
-		warn $self->class, " required params ", dump( @required );
+		warn $self->class, " required params ", dump( @required ) if $self->debug;
 	}
 
 	my $class = $self->class;
 
 	$self->load_class( $class );
 
-	my $params = {};
-	$params = $self->config($class);
-	warn "# $class config = ",dump( $params ) if $self->debug;
+	my $default = clone $self->params; # XXX we really don't want to modify params!
 
-	my $html = qq|<h1>$class params</h1><form method="post">|;
+	my $config_params = {};
+	$config_params = $self->config($class);
+	warn "# $class config = ",dump( $config_params ) if $self->debug;
+
+	my $form;
 
 	foreach my $name ( grep { ! $class->meta->get_attribute($_)->is_lazy } $self->attributes ) {
 		my $attr_type;
@@ -103,15 +108,16 @@
 		my $value = '';
 		my $label_title = '';
 		my $value_html = '';
-		if ( ref($params) eq 'HASH' ) {
-			$value = $params->{$name};
-		} elsif ( ref($params) eq 'ARRAY' ) {
+		if ( ref($config_params) eq 'HASH' ) {
+			$value = $config_params->{$name};
+		} elsif ( ref($config_params) eq 'ARRAY' ) {
 			$value_html = qq|<select name="$name">| . join("\n",
 				map {
 					my $v = $_->{$name};
 					qq|<option value="$v">$v</option>|
-				} @$params
+				} @$config_params
 			) . qq|</select>|;
+			$default->{$name} = $config_params->[0]->{$name};
 		} elsif ( my $attr = $class->meta->get_attribute( $name ) ) {
 			$attr_type = $attr->type_constraint->name;
 			if ( $attr->has_type_constraint ) {
@@ -131,11 +137,18 @@
 
 		$value_html = qq|<input type="$type" name="$name" title="$attr_type" value="$value">| unless $value_html;
 
+		$default->{$name} = $value unless defined $default->{$name};
+
 #		warn "# required $name ", $class->meta->get_attribute( $name )->dump( 2 );
-		$html .= qq|<label for="$name"$label_title>$label</label>| . $value_html;
+		$form .= qq|<label for="$name"$label_title>$label</label>| . $value_html;
 	}
-	$html .= qq|<input type="submit" value="Run $class"></form>|;
+	my $html = join("\n",
+		qq|<h1>$class params</h1><form method="post">$form<input type="submit" value="Run $class"></form>|,
+		qq|<span class="frey-popdown">config<span><code>|  . dump( $config_params ) . qq|</span></span>|,
+		qq|<span class="frey-popdown">default<span><code>| . dump( $default )       . qq|</span></span>|,
+	) if $form;
 
+	return ($html,$default) if wantarray;
 	return $html;
 }