--- trunk/lib/Frey/Action.pm	2008/11/18 00:54:10	390
+++ trunk/lib/Frey/Action.pm	2008/11/19 01:25:40	435
@@ -82,7 +82,7 @@
 	my ( $self ) = @_;
 	my @required = $self->required;
 	if ( ! @required ) {
-		warn "all params available ", dump( $self->params ), " not creating form";
+		warn "all params available ", dump( $self->params ), " not creating form" if $self->debug;
 		return (undef,$self->params) if wantarray;
 		return;
 	} else {
@@ -102,7 +102,7 @@
 	my $form;
 
 	foreach my $name ( grep { ! $class->meta->get_attribute($_)->is_lazy } $self->attributes ) {
-		my $attr_type;
+		my $attr_type = '';
 		my $type = $name =~ m/^pass/ ? 'password' : 'text';
 		my $label = $name;
 		my $value = '';
@@ -119,8 +119,8 @@
 			) . 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 ) {
+				$attr_type = $attr->type_constraint->name;
 				if ( $attr->type_constraint->can('values') ) {
 					$value_html = qq|<select name="$name">| . join("\n",
 						map { qq|<option value="$_">$_</option>| } @{ $attr->type_constraint->values }