Changeset 40 in bookmarks

Timestamp:

05/29/13 01:04:45 (11 years ago)

Author:

peter

Message:

• include the "format=text" query parameter for the text/uri-list list link
• HTML-escape URIs in the templates

Location:

trunk

Files:

• BookmarkApp.pm (modified) (1 diff)
• bookmark.tt (modified) (1 diff)
• list.tt (modified) (2 diffs)

trunk/BookmarkApp.pm

@@ -83 +83 @@
             query => {
                 tag => $tags,
+                format => 'text',
             },
         }

trunk/bookmark.tt

@@ -20 +20 @@
         <strong>New bookmark:</strong>
       [% END %]
-      <a href="[% uri %]" target="_blank">[% title or uri %]</a>
+      <a href="[% uri | html %]" target="_blank">[% title or uri %]</a>
       [% IF exists %]
         <p class="timestamps">[% created %]</p>

trunk/list.tt

@@ -102 +102 @@
             (<a href="[% resource.id %]" onclick="window.open(this.href, 'edit_bookmark', 'width=800,height=250').focus(); return false;">Edit</a>)
           </span>
-          <a href="[% resource.uri %]" title="[% resource.title | html %] ([% resource.tags.join(', ') %])">[% resource.title or resource.uri %]</a>
+          <a href="[% resource.uri | html %]" title="[% resource.title | html %] ([% resource.tags.join(', ') %])">[% resource.title or resource.uri %]</a>
           <div class="tags">
             [% FOREACH tag IN resource.tags %]
@@ -114 +114 @@
       [% FOREACH link IN links %]
         <li>
-          <a href="[% link.href.canonical %]" type="[% link.type %]" rel="[% link.rel %]">[% link.text %]</a>
+          <a href="[% link.href.canonical | html %]" type="[% link.type %]" rel="[% link.rel %]">[% link.text %]</a>
         </li>
       [% END %]