From 4d349fbd9d23d6ed58e70abb5461c2a462e1bbe8 Mon Sep 17 00:00:00 2001
From: Bridger Maxwell <bridgeyman@gmail.com>
Date: Fri, 6 Jul 2012 17:42:01 -0400
Subject: [PATCH] Fixed wiki error page again.

---
 lms/djangoapps/simplewiki/views.py            |  4 +--
 .../simplewiki/simplewiki_error.html          | 26 +++++--------------
 2 files changed, 9 insertions(+), 21 deletions(-)

diff --git a/lms/djangoapps/simplewiki/views.py b/lms/djangoapps/simplewiki/views.py
index d7c9558532..8ac7b3c387 100644
--- a/lms/djangoapps/simplewiki/views.py
+++ b/lms/djangoapps/simplewiki/views.py
@@ -461,7 +461,7 @@ def not_found(request, article_path, course):
     d = {'wiki_err_notfound': True,
          'article_path': article_path,
          'namespace' : course.wiki_namespace}
-    update_template_dictionary(request, course)
+    update_template_dictionary(d, request, course)
     return render_to_response('simplewiki/simplewiki_error.html', d)
     
 def get_article(request, article_path, course):
@@ -495,7 +495,7 @@ def check_permissions(request, article, course, check_read=False, check_write=Fa
 	            'wiki_err_nowrite': write_err,
 	            'wiki_err_locked': locked_err,
                 'wiki_err_deleted': deleted_err,}
-        update_template_dictionary(request, course)
+        update_template_dictionary(d, request, course)
         # TODO: Make this a little less jarring by just displaying an error
         #       on the current page? (no such redirect happens for an anon upload yet)
         # benjaoming: I think this is the nicest way of displaying an error, but
diff --git a/lms/templates/simplewiki/simplewiki_error.html b/lms/templates/simplewiki/simplewiki_error.html
index 9610423b2c..0ce0763def 100644
--- a/lms/templates/simplewiki/simplewiki_error.html
+++ b/lms/templates/simplewiki/simplewiki_error.html
@@ -3,8 +3,7 @@
 <%inherit file="simplewiki_base.html"/>
 
 <%!
-  from django.core.urlresolvers import reverse
-  from simplewiki.views import wiki_namespace_reverse
+  from simplewiki.views import wiki_reverse
 %>
 
 <%block name="title"><title>Wiki Error – MITx 6.002x</title></%block>
@@ -22,21 +21,11 @@ ${wiki_error}
 %endif
 
 %if wiki_err_notfound is not UNDEFINED:
-	%if article_path is not UNDEFINED:
-		<p>
-		The page you requested could not be found.
-		Click <a href="${wiki_namespace_reverse("wiki_create", course=course, kwargs={'article_path' : article_path})}">here</a> to create it.
-		</p>
-	%else:
-		<p>
-		Or maybe rather: Congratulations! It seems that there's no root
-		article, which is probably because you just installed simple-wiki
-		and your installation is working. Now you can create the root article.
-		Click <a href="{% url wiki_create "" %}">here</a> to create it.
-		</p>
-	%endif
-%else:
-%if wiki_err_no_namespace is not UNDEFINED and wiki_err_no_namespace:
+	<p>
+	The page you requested could not be found.
+	Click <a href="${wiki_reverse("wiki_create", course=course, kwargs={'article_path' : article_path})}">here</a> to create it.
+	</p>
+%elif wiki_err_no_namespace is not UNDEFINED and wiki_err_no_namespace:
 	<p>
 	You must specify a namespace to create an article in.
 	</p>
@@ -73,7 +62,7 @@ ${wiki_error}
 	</p>
 %elif wiki_err_deleted is not UNDEFINED and wiki_err_deleted:
 	<p>
-	The article you tried to access has been deleted. You may be able to restore it to an earlier version in its <a href="${reverse("wiki_history", args=[wiki_article.get_url(),1])}">history</a>, or <a href="${reverse("wiki_edit", args=[wiki_article.get_url()])}">create a new version</a>.
+	The article you tried to access has been deleted. You may be able to restore it to an earlier version in its <a href="${wiki_reverse("wiki_history", wiki_article, course)}">history</a>, or <a href="${wiki_reverse("wiki_edit", wiki_article, course)}">create a new version</a>.
 	</p>
 %elif wiki_err_norevision is not UNDEFINED:
 	<p>
@@ -84,7 +73,6 @@ ${wiki_error}
 	An error has occured.
 	</p>
 %endif
-%endif
 
 </div>
 </%block>