Fixed getting posts for a thread. Implemented thread getting test.

boards/models.py

@@ -41,10 +41,13 b' class PostManager(models.Manager):'
         return threads
 
     def get_thread(self, opening_post_id):
-        opening_post = self.get(opening_post_id)
-        replies =  self.filter(parent = opening_post_id)
+        opening_post = self.get(id = opening_post_id)
+        replies = self.filter(parent = opening_post_id)
 
-        return [opening_post, replies]
+        thread = [opening_post]
+        thread.extend(replies)
+
+        return thread
 
     def exists(self, post_id):
         posts = self.filter(id = post_id)

boards/tests.py

@@ -86,3 +86,14 b' class BoardTests(TestCase):'
 
         admin.delete()
 
+    def test_get_thread(self):
+        opening_post = self._create_post()
+        op_id = opening_post.id
+
+        for i in range(0, 2):
+            Post.objects.create_post('title', 'text',
+                parent_id = op_id)
+
+        thread = Post.objects.get_thread(op_id)
+
+        self.assertEqual(3, thread.__len__())

boards/urls.py

@@ -12,9 +12,9 b" urlpatterns = patterns('',"
     url(r'^logout$', views.logout, name='logout'),
 
     # /boards/tag/
-    url(r'^(?P<tag>\w+)/$', views.tag, name = 'tag'),
+    url(r'^tag/(?P<tag>\w+)/$', views.tag, name = 'tag'),
     # /boards/post_id/
-    url(r'^(?P<post>\w+)/$', views.thread, name = 'thread'),
+    url(r'^post/(?P<id>\w+)/$', views.thread, name = 'thread'),
     # /boards/tag/post/
     url(r'^post.html$', views.new_post,
 	    name='post'),

boards/views.py

@@ -47,18 +47,17 b' def tag(request):'
     return render(request, 'posting_general.html',
         context)
 
-def thread(request):
+def thread(request, id):
     """Get all thread posts"""
 
     # TODO Show 404 if there is no such thread
 
-    opening_post_id = request.GET['id']
-    posts = Post.objects.get_thread(opening_post_id)
+    posts = Post.objects.get_thread(id)
 
     context = RequestContext(request)
     context['posts'] = posts
 
-    return render(request), 'thread.html'
+    return render(request, 'thread.html', context)
 
 def login(request):
     """Log in as admin"""

templates/posting_general.html

@@ -6,7 +6,8 b''
 		{% for thread in threads %}
             <span class="title">{{ thread.title }}</span><br />
             <span class="text">{{ thread.text }}</span><br />
-            <a class="link" href="/boards/id/{{ thread.id }}/">View thread</a>
+            <a class="link" href="/boards/post/{{ thread.id }}/">View
+                thread</a>
             <hr />
 		{% endfor %}
 	{% else %}