From efd64514cb4065a9f81cd9aa6c29532bad082e92 Mon Sep 17 00:00:00 2001
From: Jeff Balogh <me@jeffbalogh.org>
Date: Thu, 6 May 2010 09:56:08 -0700
Subject: [PATCH] put the |f against the string (bug 564140)

---
 apps/addons/templates/addons/review_add_box.html | 2 +-
 apps/addons/tests/test_views.py                  | 8 ++++++++
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/apps/addons/templates/addons/review_add_box.html b/apps/addons/templates/addons/review_add_box.html
index 5cd9f263926..714652d21a3 100644
--- a/apps/addons/templates/addons/review_add_box.html
+++ b/apps/addons/templates/addons/review_add_box.html
@@ -57,7 +57,7 @@ <h3>{{ _('What do you think?') }}</h3>
     <p><a href="{{ remora_url('/pages/review_guide') }}">{{ _('Review Guidelines') }}</a></p>
     <p>
       {# TODO reverse url #}
-      <a href="{{ remora_url('/reviews/add/{0}')|f(addon.id) }}">
+      <a href="{{ remora_url('/reviews/add/{0}'|f(addon.id)) }}">
         {{ _('Detailed Review') }}</a>
     </p>
 
diff --git a/apps/addons/tests/test_views.py b/apps/addons/tests/test_views.py
index 972a549c32b..aaf60267c94 100644
--- a/apps/addons/tests/test_views.py
+++ b/apps/addons/tests/test_views.py
@@ -300,6 +300,14 @@ def test_remove_tag_button(self):
         doc = pq(r.content)
         assert len(doc('#tags li input.removetag'))
 
+    def test_detailed_review_link(self):
+        # TODO(jbalogh): use reverse when we drop remora.
+        self.client.login(username='regular@mozilla.com', password='password')
+        r = self.client.get(reverse('addons.detail', args=[3615]))
+        doc = pq(r.content)
+        href = doc('#review-box a[href*="reviews/add"]').attr('href')
+        assert href.endswith('/reviews/add/3615'), href
+
 
 class TestTagsBox(test_utils.TestCase):
     fixtures = ['base/addontag']