Changeset 88 in bookmarks for trunk/lib/Bookmarks.pm

Timestamp:

06/04/15 22:00:25 (10 years ago)

Author:

peter

Message:

Better separation of the model for searching and the web app controller/representation layer

• renamed Bookmarks::get_bookmarks() to search()
• Bookmarks::search() now returns a Bookmarks::Search object instead of a Bookmarks::List
• search results now reside in the Bookmarks::Search class

File:

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

trunk/lib/Bookmarks.pm

@@ -7 +7 @@
 
 use Bookmark;
-use Bookmarks::List;
 use Bookmarks::Search;
 
@@ -100 +99 @@
 }
 
-sub get_bookmarks {
+sub search {
     my $self = shift;
     my $params = shift || {};
@@ -130 +129 @@
         });
     }
-    return Bookmarks::List->new({
-        bookmarks => $self,
-        search    => $search,
-        results   => \@resources,
-    });
+    $search->results(\@resources);
+
+    return $search;
 }