]> git.openstreetmap.org Git - osqa.git/commitdiff
Adds better permalinks to answers, computing the position of an answer in the list.
authorhernani <hernani@0cfe37f9-358a-4d5e-be75-b63607b5c754>
Wed, 7 Jul 2010 01:33:19 +0000 (01:33 +0000)
committerhernani <hernani@0cfe37f9-358a-4d5e-be75-b63607b5c754>
Wed, 7 Jul 2010 01:33:19 +0000 (01:33 +0000)
git-svn-id: http://svn.osqa.net/svnroot/osqa/trunk@494 0cfe37f9-358a-4d5e-be75-b63607b5c754

forum/models/answer.py
forum/skins/default/templates/node/post_controls.html
forum/templatetags/node_tags.py
forum/urls.py
forum/utils/pagination.py
forum/views/readers.py

index e5f166bd50657a994505adc5d5b9cf687a507010..04735a4cc537c3bf81aacac7318749fb4d71e2c7 100644 (file)
@@ -16,7 +16,7 @@ class Answer(Node):
         return self.question.headline
 
     def get_absolute_url(self):
-        return '%s#%s' % (self.question.get_absolute_url(), self.id)
+        return '%s/%s' % (self.question.get_absolute_url(), self.id)
 
 
 class AnswerRevision(NodeRevision):
index e756fff3be992e219c70909ddca5f5e507064fd3..bbe1e22effa35c85664a322183ebbc9bc706b776 100644 (file)
@@ -2,7 +2,7 @@
 {% spaceless %}\r
 {% for control in controls %}\r
     <span class="action-link">\r
-        <a title="{{ control.title }}" class="{{ control.classes }}" href="{{ control.url }}">{{ control.text }}</a>\r
+        <a rel="nofollow" title="{{ control.title }}" class="{{ control.classes }}" href="{{ control.url }}">{{ control.text }}</a>\r
     </span>\r
     {% ifnotequal controls|last control %}\r
         <span class="action-link-separator">|</span>\r
@@ -14,7 +14,7 @@
         <span id="node-{{ post.id }}-menu-trigger" class="action-link context-menu-trigger">{% trans "more" %} &#9660;</span>\r
         <ul id="node-{{ post.id }}-menu-dropdown" class="context-menu-dropdown">\r
             {% for item in menu %}\r
-            <li class="item"><a class="{{ item.classes }}" href="{{ item.url }}" title="{{ item.title }}" >{{ item.text }}</a></li>\r
+            <li class="item"><a rel="nofollow" class="{{ item.classes }}" href="{{ item.url }}" title="{{ item.title }}" >{{ item.text }}</a></li>\r
             {% endfor %}\r
         </ul>\r
     </span>\r
index a840a2c27bafcfc06970c0254edd15014dce3b15..3124f3aa7ccc4887cb0d4f0ab2064abf8bbf0b49 100644 (file)
@@ -65,7 +65,7 @@ def post_controls(post, user):
         post_type = post.node_type\r
 \r
         if post_type == "answer":\r
-            controls.append(post_control(_('permanent link'), '#%d' % post.id, title=_("answer permanent link")))\r
+            controls.append(post_control(_('permanent link'), post.get_absolute_url(), title=_("answer permanent link")))\r
 \r
         edit_url = reverse('edit_' + post_type, kwargs={'id': post.id})\r
         if user.can_edit_post(post):\r
index 79f7c1159e8b90339ca9d50828cadef758ef8f23..92c0d6bf6a18f1347d9af9c656fb49b4d1aeae48 100644 (file)
@@ -93,8 +93,10 @@ urlpatterns += patterns('',
 
                         url(r'^%s(?P<id>\d+)/(?P<slug>[\w-]*)$' % _('question/'),
                             'django.views.generic.simple.redirect_to', {'url': '/questions/%(id)s/%(slug)s'}),
-                        url(r'^%s(?P<id>\d+)/(?P<slug>[\w-]*)$' % _('questions/'), app.readers.question, name='question'
-                            ),
+                        url(r'^%s(?P<id>\d+)/(?P<slug>[\w-]*)$' % _('questions/'), app.readers.question, name='question'),
+                        url(r'^%s(?P<id>\d+)/(?P<slug>[\w-]*)/(?P<answer>\d+)$' % _('questions/'), app.readers.question),
+
+
                         url(r'^%s$' % _('tags/'), app.readers.tags, name='tags'),
                         url(r'^%s(?P<tag>.*)/$' % _('tags/'), app.readers.tag, name='tag_questions'),
 
index 393fac0d0a9584aeb700a6bd5142b008fdead540..afadb285ab465e81a3782d2a3bea0ef784aa40bf 100644 (file)
@@ -17,6 +17,14 @@ class SimpleSort(object):
     def apply(self, objects):
         return objects.order_by(self.order_by)
 
+class DummySort(object):
+    def __init__(self, label, description=''):
+        self.label = label
+        self.description = description
+
+    def apply(self, objects):
+        return objects
+
 
 class PaginatorContext(object):
     visible_page_range = 5
@@ -24,7 +32,8 @@ class PaginatorContext(object):
 
     base_path = None
 
-    def __init__(self, id, sort_methods=None, default_sort=None, pagesizes=None, default_pagesize=None):
+    def __init__(self, id, sort_methods=None, default_sort=None, force_sort = None, sticky_sort=False,
+                 pagesizes=None, default_pagesize=None):
         self.id = id
         if sort_methods:
             self.has_sort = True
@@ -49,9 +58,72 @@ class PaginatorContext(object):
         else:
             self.has_pagesize = False
 
-        self.force_sort = None
-        self.sticky_sort = False
+        self.force_sort = force_sort
+        self.sticky_sort = sticky_sort
+
+    def session_preferences(self, request):
+        return request.session.get('paginator_%s' % self.id, {})
+
+    def pagesize(self, request, session_prefs=None):
+        if not session_prefs:
+            session_prefs = self.session_preferences(request)
+
+
+        if self.has_pagesize:
+            if request.GET.get(labels.PAGESIZE, None):
+                try:
+                    pagesize = int(request.GET[labels.PAGESIZE])
+                except ValueError:
+                    logging.error('Found invalid page size "%s", loading %s, refered by %s' % (
+                        request.GET.get(labels.PAGESIZE, ''), request.path, request.META.get('HTTP_REFERER', 'UNKNOWN')
+                    ))
+                    raise Http404()
+
+                session_prefs[labels.PAGESIZE] = pagesize
+            else:
+                pagesize = session_prefs.get(labels.PAGESIZE, self.default_pagesize)
+
+            if not pagesize in self.pagesizes:
+                pagesize = self.default_pagesize
+        else:
+            pagesize = 30
+
+        return pagesize
+
+    def page(self, request):
+        try:
+            return int(request.GET.get(labels.PAGE, 1))
+        except ValueError:
+            logging.error('Found invalid page number "%s", loading %s, refered by %s' % (
+                request.GET.get(labels.PAGE, ''), request.path, request.META.get('HTTP_REFERER', 'UNKNOWN')
+            ))
+            raise Http404()
+
+    def sort(self, request, session_prefs=None):
+        if not session_prefs:
+            session_prefs = self.session_preferences(request)
+
+        sort = None
+        if self.has_sort:
+            if request.GET.get(labels.SORT, None):
+                sort = request.GET[labels.SORT]
+                if self.sticky_sort or session_prefs.get('sticky_sort', False):
+                    session_prefs[labels.SORT] = sort
+            else:
+                sort = self.force_sort or session_prefs.get(labels.SORT, self.default_sort)
+
+            if not sort in self.sort_methods:
+                sort = self.default_sort
+
+        return sort
 
+    def sorted(self, objects, request, session_prefs=None):
+        sort = self.sort(request, session_prefs)
+
+        if sort:
+            objects = self.sort_methods[sort].apply(objects)
+
+        return sort, objects
 
 
 class labels(object):
@@ -64,49 +136,13 @@ page_sizes_template = template.loader.get_template('paginator/page_sizes.html')
 sort_tabs_template = template.loader.get_template('paginator/sort_tabs.html')
 
 def paginated(request, list_name, context, tpl_context):
-    session_prefs = request.session.get('paginator_%s' % context.id, {})
-    objects = tpl_context[list_name]
-
-    if context.has_pagesize:
-        if request.GET.get(labels.PAGESIZE, None):
-            try:
-                pagesize = int(request.GET[labels.PAGESIZE])
-            except ValueError:
-                logging.error('Found invalid page size "%s", loading %s, refered by %s' % (
-                    request.GET.get(labels.PAGESIZE, ''), request.path, request.META.get('HTTP_REFERER', 'UNKNOWN')
-                ))
-                raise Http404()
-
-            session_prefs[labels.PAGESIZE] = pagesize
-        else:
-            pagesize = session_prefs.get(labels.PAGESIZE, context.default_pagesize)
-
-        if not pagesize in context.pagesizes:
-            pagesize = context.default_pagesize
-    else:
-        pagesize = 30
-
-    try:
-        page = int(request.GET.get(labels.PAGE, 1))
-    except ValueError:
-        logging.error('Found invalid page number "%s", loading %s, refered by %s' % (
-            request.GET.get(labels.PAGE, ''), request.path, request.META.get('HTTP_REFERER', 'UNKNOWN')
-        ))
-        raise Http404()
-
-    sort = None
-    if context.has_sort:
-        if request.GET.get(labels.SORT, None):
-            sort = request.GET[labels.SORT]
-            if context.sticky_sort or session_prefs.get('sticky_sort', False):
-                session_prefs[labels.SORT] = sort
-        else:
-            sort = context.force_sort or session_prefs.get(labels.SORT, context.default_sort)
+    session_prefs = context.session_preferences(request)
 
-        if not sort in context.sort_methods:
-            sort = context.default_sort
+    objects = tpl_context[list_name]
 
-        objects = context.sort_methods[sort].apply(objects)
+    pagesize = context.pagesize(request, session_prefs)
+    page = context.page(request)
+    sort, objects = context.sorted(objects, request, session_prefs)
 
     paginator = Paginator(objects, pagesize)
 
index eed8fd597877ee9a4cedc23b736325f8a077e4e9..7bfd75af2a34f054e7a9810c81128d4fd15bc332 100644 (file)
@@ -52,15 +52,20 @@ class QuestionListPaginatorContext(pagination.PaginatorContext):
             (_('mostvoted'), pagination.SimpleSort(_('most voted'), '-score', _("most voted questions"))),
         ), pagesizes=(15, 30, 50))
 
+class AnswerSort(pagination.SimpleSort):
+    def apply(self, objects):
+        if self.label == _('votes'):
+            return objects.order_by('-marked', self.order_by, 'added_at')
+        else:
+            return objects.order_by('-marked', self.order_by)
+
 class AnswerPaginatorContext(pagination.PaginatorContext):
     def __init__(self):
         super (AnswerPaginatorContext, self).__init__('ANSWER_LIST', sort_methods=(
-            (_('oldest'), pagination.SimpleSort(_('oldest answers'), 'added_at', _("oldest answers will be shown first"))),
-            (_('latest'), pagination.SimpleSort(_('newest answers'), '-added_at', _("newest answers will be shown first"))),
-            (_('votes'), pagination.SimpleSort(_('popular answers'), '-score', _("most voted answers will be shown first"))),
-        ), default_sort=_('votes'), pagesizes=(5, 10, 20))
-
-        self.sticky_sort = True
+            (_('oldest'), AnswerSort(_('oldest answers'), 'added_at', _("oldest answers will be shown first"))),
+            (_('newest'), AnswerSort(_('newest answers'), '-added_at', _("newest answers will be shown first"))),
+            (_('votes'), AnswerSort(_('popular answers'), '-score', _("most voted answers will be shown first"))),
+        ), default_sort=_('votes'), sticky_sort = True, pagesizes=(5, 10, 20))
     
 
 def feed(request):
@@ -277,27 +282,63 @@ def match_question_slug(slug):
 
     return None
 
+def answer_redirect(request, answer):
+    pc = AnswerPaginatorContext()
+
+    sort = pc.sort(request)
+
+    if sort == _('oldest'):
+        filter = Q(added_at__lt=answer.added_at)
+    elif sort == _('newest'):
+        filter = Q(added_at__gt=answer.added_at)
+    elif sort == _('votes'):
+        filter = Q(score__gt=answer.score) | Q(score=answer.score, added_at__lt=answer.added_at)
+    else:
+        raise Http404()
+        
+    count = answer.question.answers.filter(Q(marked=True) | filter).count()
+    pagesize = pc.pagesize(request)
+
+    page = count / pagesize
+    
+    if count % pagesize:
+        page += 1
+
+    return HttpResponsePermanentRedirect("%s?%s=%s#%s" % (
+        answer.question.get_absolute_url(), _('page'), page, answer.id))
+
 @decorators.render("question.html", 'questions', tabbed=False)
-def question(request, id, slug):
+def question(request, id, slug, answer=None):
     try:
         question = Question.objects.get(id=id)
     except:
         if slug:
             question = match_question_slug(slug)
             if question is not None:
-                return HttpResponsePermanentRedirect(question.get_absolute_url())
+                return HttpResponseRedirect(question.get_absolute_url())
 
         raise Http404()
 
     if question.nis.deleted and not request.user.can_view_deleted_post(question):
         raise Http404
 
+    if answer:
+        answer = get_object_or_404(Answer, id=answer)
+
+        if (question.nis.deleted and not request.user.can_view_deleted_post(question)) or answer.question != question:
+            raise Http404
+
+        if answer.marked:
+            return HttpResponsePermanentRedirect(question.get_absolute_url())
+
+        return answer_redirect(request, answer)
+
     if request.POST:
         answer_form = AnswerForm(question, request.POST)
     else:
         answer_form = AnswerForm(question)
 
-    answers = request.user.get_visible_answers(question).order_by("-marked")
+    answers = request.user.get_visible_answers(question)
 
     update_question_view_times(request, question)