--- trunk/Estraier.pm	2006/01/04 14:48:11	6
+++ trunk/Estraier.pm	2006/01/04 15:50:08	11
@@ -43,6 +43,8 @@
 
 package Search::Estraier::Document;
 
+use Carp qw/croak confess/;
+
 =head1 Search::Estraier::Document
 
 Document for HyperEstraier
@@ -70,7 +72,7 @@
 
   $doc->add_attr( name => 'value' );
 
-B<FIXME>: delete attribute using
+Delete attribute using
 
   $doc->add_attr( name => undef );
 
@@ -81,8 +83,14 @@
 	my $attrs = {@_};
 
 	while (my ($name, $value) = each %{ $attrs }) {
-		push @{ $self->{attrs}->{_s($name)} }, _s($value);
+		if (! defined($value)) {
+			delete( $self->{attrs}->{_s($name)} );
+		} else {
+			$self->{attrs}->{_s($name)} = _s($value);
+		}
 	}
+
+	return 1;
 }
 
 
@@ -132,6 +140,51 @@
 	return $self->{id};
 }
 
+=head2 attr_names
+
+Returns array with attribute names from document object.
+
+  my @attrs = $doc->attr_names;
+
+=cut
+
+sub attr_names {
+	my $self = shift;
+	croak "attr_names return array, not scalar" if (! wantarray);
+	return sort keys %{ $self->{attrs} };
+}
+
+
+=head2 attr
+
+Returns value of an attribute.
+
+  my $value = $doc->attr( 'attribute' );
+
+=cut
+
+sub attr {
+	my $self = shift;
+	my $name = shift;
+
+	return $self->{'attrs'}->{ $name };
+}
+
+
+=head2 texts
+
+Returns array with text sentences.
+
+  my @texts = $doc->texts;
+
+=cut
+
+sub texts {
+	my $self = shift;
+	confess "attr_names return array, not scalar" if (! wantarray);
+	return @{ $self->{dtexts} };
+}
+
 =head2 dump_draft
 
   print $doc->dump_draft;
@@ -139,6 +192,7 @@
 =cut
 
 sub dump_draft {
+	return 'FIXME';
 }
 
 =head2 delete
@@ -156,6 +210,8 @@
 		delete($self->{$data});
 	}
 
+	$self->{id} = -1;
+
 	return 1;
 }