Changeset 127 in bookmarks for trunk

Timestamp:

10/20/16 21:35:13 (8 years ago)

Author:

peter

Message:

Make it clear that add takes a hash not an object.

File:

• trunk/lib/Bookmarks.pm (modified) (2 diffs)

trunk/lib/Bookmarks.pm

@@ -227 +227 @@
 sub add {
     my $self = shift;
-    my $bookmark = shift;
+    my $params = shift;
 
     #TODO: accept a pre-made Bookmark object in addition to a hash
-    my $uri = $bookmark->{uri};
-    my $title = $bookmark->{title};
-    my $ctime = $bookmark->{ctime} || time;
-    my $mtime = $bookmark->{mtime} || $ctime;
-    my $id = $bookmark->{id};
+    my $uri = $params->{uri};
+    my $title = $params->{title};
+    my $ctime = $params->{ctime} || time;
+    my $mtime = $params->{mtime} || $ctime;
+    my $id = $params->{id};
 
     # create an entry for the resource
@@ -271 +271 @@
     }
 
-    my $changed_tags = $self->_update_tags($uri, $bookmark->{tags});
+    my $changed_tags = $self->_update_tags($uri, $params->{tags});
 
     if ($bookmark_exists && $changed_tags) {