input method should be named text()

[perl/html-form-simple.git] / lib / HTML / Form / Simple.pm

diff --git a/lib/HTML/Form/Simple.pm b/lib/HTML/Form/Simple.pm
index 66958c3c294d760d7729474a4ba829de9e3e1b10..3a58c799761a27e880891e321a7ec375166b75c4 100644 (file)
--- a/lib/HTML/Form/Simple.pm
+++ b/lib/HTML/Form/Simple.pm
@@ -71,6 +71,16 @@ sub hidden {
        my $self = shift;
        my ($name, $value, $attr) = $self->_attr(2, @_);
 
+       if (ref $name eq 'HASH') {
+               my @return = map { $self->hidden($_, $name->{$_}, $attr) } sort keys %$name;
+               return wantarray ? @return : join(defined $, ? $, : '', @return);
+       }
+
+       if (ref $value eq 'ARRAY') {
+               my @return = map { $self->hidden($name, $_, $attr) } @$value;
+               return wantarray ? @return : join(defined $, ? $, : '', @return);
+       }
+
        $attr->{name } = $name  if defined $name;
        $attr->{value} = $value if defined $value;
        $attr->{type} = 'hidden' unless defined $attr->{type};
@@ -78,7 +88,7 @@ sub hidden {
        return $self->tag(input => $attr);
 }
 
-sub input {
+sub text {
        my $self = shift;
        my ($name, $value, $attr) = $self->_attr(2, @_);
 
@@ -108,6 +118,8 @@ sub select {
 
        my @options = map { ref $_ ? $_ : {value => $_} } @$rows;
 
+       my @return;
+
        if ($attr->{type} eq 'select') {
                delete $attr->{type};
                if (defined $default) {
@@ -115,17 +127,17 @@ sub select {
                                $_->{selected} = 1 if defined $_->{value} and $_->{value} eq $default;
                        }
                }
-               my @return = (
+               @return = (
                        $self->tag(select => $attr),
                        (map { $self->tag(option => $_) } @options),
                        '</select>',
                );
-               return wantarray ? @return : join('', @return);
        }
        else {
                if (defined $attr->{id} and $attr->{id} ne '') {
-                       defined $_->{id} or defined $_->{value} and $_->{id} = $attr->{id}.'_'.$_->{value}
-                               for @options;
+                       defined $_->{id}
+                               or defined $_->{value} and $_->{id} = $attr->{id}.'_'.$_->{value}
+                                       for @options;
                }
                if (defined $attr->{label}) {
                        defined $_->{value} and not defined $_->{label}
@@ -139,14 +151,15 @@ sub select {
                        }
                }
                $_ = {%$attr, %$_} for @options;
-               my @return = map {
+               @return = map {
                        my $label = delete $_->{label};
                        defined $label && $label ne ''
                                ? '<label>'.$self->tag(input => $_)." $label</label>"
                                :           $self->tag(input => $_)
                } @options;
-               return wantarray ? @return : join('', @return);
        }
+
+       return wantarray ? @return : join(defined $, ? $, : '', @return);
 }
 
 sub radio {
@@ -197,7 +210,7 @@ sub check {
 
 =head1 NAME
 
-HTML::Form::Simple
+HTML::Form::Simple - Generate HTML form elements
 
 =head1 SYNOPSIS
 
@@ -222,3 +235,12 @@ HTML::Form::Simple
        );
        say $input->stop; # </form>
 
+=head1 AUTHOR
+
+Mischa POSLAWSKY <perl@shiar.org>
+
+=head1 LICENSE
+
+This module is free software; you can redistribute it and/or modify it
+under the same L<terms|perlartistic> as Perl itself.
+