Changeset 66 in bookmarks for trunk/BookmarkController.pm

Timestamp:

02/04/14 15:21:17 (10 years ago)

Author:

peter

Message:

• the bookmark.tt template takes a hash or object named bookmark as its only variable
• moved the calculation of a human-readable timestamp and the ISO timestamps for the WayBack machine from the BookmarkController class into the Bookmark class as methods (created, updated, created_iso, and updated_iso
• moved the exists flag into the Bookmark class as an attribute

File:

• trunk/BookmarkController.pm (modified) (2 diffs)

trunk/BookmarkController.pm

@@ -119 +119 @@
         $template->process(
             'bookmark.tt',
-            {
-                uri   => $self->request->param('uri'),
-                title => $self->request->param('title') || '',
+            { 
+                bookmark => {
+                    uri   => $self->request->param('uri'),
+                    title => $self->request->param('title') || '',
+                },
             },
             \my $output,
@@ -337 +339 @@
         } else {
             # display the bookmark form for this bookmark
-            # TODO: these should probably be methods on the Bookmark class
-            $bookmark->{exists} = 1;
-            $bookmark->{created} = localtime($bookmark->ctime);
-            $bookmark->{updated} = localtime($bookmark->mtime);
-            ($bookmark->{created_iso} = time2isoz($bookmark->ctime)) =~ s/\D//g;
-            ($bookmark->{updated_iso} = time2isoz($bookmark->mtime)) =~ s/\D//g;
             my $template = Template->new;
             $template->process(
                 'bookmark.tt',
-                $bookmark,
+                { bookmark => $bookmark },
                 \my $output,
             );