[Bps-public-commit] r15136 - in Prophet/trunk: .

[spang at bestpractical.com]

Author: spang
Date: Wed Aug 13 14:38:35 2008
New Revision: 15136

Modified:
   Prophet/trunk/   (props changed)
   Prophet/trunk/lib/Prophet/Record.pm

Log:
 r48229 at loki:  spang | 2008-08-13 19:37:59 +0100
 delete_prop is equivalent to setting the prop to '' for now


Modified: Prophet/trunk/lib/Prophet/Record.pm
==============================================================================
--- Prophet/trunk/lib/Prophet/Record.pm	(original)
+++ Prophet/trunk/lib/Prophet/Record.pm	Wed Aug 13 14:38:35 2008
@@ -286,7 +286,7 @@
 =head2 prop $name
 
 Returns the current value of the property C<$name> for this record.
-(This is a convenience method wrapped around L</get_props>.
+(This is a convenience method wrapped around L</get_props>).
 
 =cut
 
@@ -299,8 +299,7 @@
 =head2 delete_prop { name => $name }
 
 Deletes the current value for the property $name.
-
-TODO: how is this different than setting it to an empty value?
+(This is currently equivalent to setting the prop to ''.)
 
 =cut
 
@@ -310,10 +309,12 @@
 
     confess "delete_prop called on a record that hasn't been loaded or created yet." if !$self->uuid;
 
-    $self->handle->delete_record_prop(
-        uuid => $self->uuid,
-        name => $args{'name'}
-    );
+    $self->set_prop(name => $args{'name'}, value => '');
+
+#    $self->handle->delete_record_prop(
+#        uuid => $self->uuid,
+#        name => $args{'name'}
+#    );
 }
 
 =head2 delete