diff --git a/sites/scrapy.org/scrapyorg/article/models.py b/sites/scrapy.org/scrapyorg/article/models.py
index 9eb34a372..fd73b660f 100644
--- a/sites/scrapy.org/scrapyorg/article/models.py
+++ b/sites/scrapy.org/scrapyorg/article/models.py
@@ -49,8 +49,8 @@ class Article(models.Model):
# ugly, but django-admin isn't very versatile right now
def position_link(self):
- return _("%(position)s (Up" \
- " | Down)") % \
+ return _("%(position)s (Up" \
+ " | Down)") % \
{ "position": self.position, "id": self.id }
position_link.short_description = u"position"
position_link.allow_tags = True
diff --git a/sites/scrapy.org/scrapyorg/article/views.py b/sites/scrapy.org/scrapyorg/article/views.py
index 736650404..e461e2345 100644
--- a/sites/scrapy.org/scrapyorg/article/views.py
+++ b/sites/scrapy.org/scrapyorg/article/views.py
@@ -1,15 +1,17 @@
from django.shortcuts import get_object_or_404
from django.http import HttpResponseRedirect
+from django.contrib.admin.views.decorators import staff_member_required
from scrapyorg.article.models import Article
+@staff_member_required
def position_up(request, article_id):
article = get_object_or_404(Article, pk=article_id)
article.position_up()
return HttpResponseRedirect("/admin/article/article/")
-
+@staff_member_required
def position_down(request, article_id):
article = get_object_or_404(Article, pk=article_id)
article.position_down()
diff --git a/sites/scrapy.org/scrapyorg/download/models.py b/sites/scrapy.org/scrapyorg/download/models.py
index 97102eb9b..6ca684699 100644
--- a/sites/scrapy.org/scrapyorg/download/models.py
+++ b/sites/scrapy.org/scrapyorg/download/models.py
@@ -32,9 +32,8 @@ class DownloadLink(models.Model):
# ugly, but django-admin isn't very versatile right now
def public_link(self):
- return _("%(status)s (toggle)") % \
- { 'status': self.public and _("Yes") or _("No"),
- 'id': self.id }
+ return _("%s (toggle)") % \
+ (self.public and _("Yes") or _("No"), self.id )
public_link.short_description = u"public"
public_link.allow_tags = True
diff --git a/sites/scrapy.org/scrapyorg/download/urls.py b/sites/scrapy.org/scrapyorg/download/urls.py
index e8d1e4700..59deb47f4 100644
--- a/sites/scrapy.org/scrapyorg/download/urls.py
+++ b/sites/scrapy.org/scrapyorg/download/urls.py
@@ -4,5 +4,5 @@ from scrapyorg.download.views import *
urlpatterns = patterns('',
- (r"^(?P\d+)/public/toggle/$", toggle_public),
+ (r"^(?P\d+)/toggle/$", toggle_public),
)
diff --git a/sites/scrapy.org/scrapyorg/download/views.py b/sites/scrapy.org/scrapyorg/download/views.py
index f0b6ae097..bc4b56215 100644
--- a/sites/scrapy.org/scrapyorg/download/views.py
+++ b/sites/scrapy.org/scrapyorg/download/views.py
@@ -1,9 +1,11 @@
from django.shortcuts import get_object_or_404
from django.http import HttpResponseRedirect
+from django.contrib.admin.views.decorators import staff_member_required
from scrapyorg.download.models import DownloadLink
+@staff_member_required
def toggle_public(request, link_id):
link = get_object_or_404(DownloadLink, pk=link_id)
link.toggle_public()
diff --git a/sites/scrapy.org/scrapyorg/urls.py b/sites/scrapy.org/scrapyorg/urls.py
index 3cfe2d689..97316b1a0 100644
--- a/sites/scrapy.org/scrapyorg/urls.py
+++ b/sites/scrapy.org/scrapyorg/urls.py
@@ -6,9 +6,11 @@ from django.conf import settings
urlpatterns = patterns('',
(r"^$", direct_to_template, { "template": "home.html" }),
(r"^article/", include("scrapyorg.article.urls")),
- (r"^download/", include("scrapyorg.download.urls")),
(r"^weblog/", include("scrapyorg.blog.urls")),
+ # admin
+ (r"^admin/article/article/", include("scrapyorg.article.urls")),
+ (r"^admin/download/downloadlink/", include("scrapyorg.download.urls")),
(r"^admin/", include("django.contrib.admin.urls")),
)