Add select_related to Post queries

Remove unneeded compat.py.

CONTRIBUTING.md

@@ -98,7 +98,6 @@ Here is an example of these rules applied:
     from tagging.fields import TagField
 
     # fourth set of imports are local apps
-    from .compat import reverse
     from .fields import MarkupField
 
 

README.md

@@ -86,19 +86,19 @@ To install pinax-blog:
 Add `pinax.blog` to your `INSTALLED_APPS` setting:
 
 ```python
-INSTALLED_APPS = [
-    # other apps
-    "pinax.blog",
-]
+    INSTALLED_APPS = [
+        # other apps
+        "pinax.blog",
+    ]
 ```
 
 Add `pinax.blog.urls` to your project urlpatterns:
 
 ```python
-urlpatterns = [
-    # other urls
-    url(r"^blog/", include("pinax.blog.urls", namespace="pinax_blog")),
-]
+    urlpatterns = [
+        # other urls
+        url(r"^blog/", include("pinax.blog.urls", namespace="pinax_blog")),
+    ]
 ```
 
 Optionally, if you want `creole` support for a mark up choice:

pinax/blog/models.py

@@ -2,6 +2,7 @@
 from random import choice
 
 from django.db import models
+from django.urls import reverse
 from django.utils import timezone
 from django.utils.encoding import python_2_unicode_compatible
 from django.utils.html import strip_tags
@@ -10,7 +11,6 @@
 import pytz
 from pinax.images.models import ImageSet
 
-from .compat import reverse
 from .conf import settings
 from .hooks import hookset
 from .managers import PostManager

pinax/blog/tests/tests.py

@@ -6,9 +6,9 @@
 from django.core.exceptions import ValidationError
 from django.http.request import HttpRequest
 from django.test import TestCase
+from django.urls import reverse
 from django.utils.text import slugify
 
-from ..compat import reverse
 from ..context_processors import scoped
 from ..models import Blog, Post, Section
 

pinax/blog/views.py

@@ -6,6 +6,7 @@
 from django.http import Http404, HttpResponse, JsonResponse
 from django.shortcuts import get_object_or_404, redirect
 from django.template.loader import render_to_string
+from django.urls import reverse
 from django.views.decorators.http import require_POST
 from django.views.generic import (
     CreateView,
@@ -16,7 +17,6 @@
 )
 from django.views.generic.dates import DateDetailView
 
-from .compat import reverse
 from .conf import settings
 from .forms import PostForm
 from .hooks import hookset
@@ -58,7 +58,7 @@ def search(self, posts):
 
     def get_queryset(self):
         blog = hookset.get_blog(**self.kwargs)
-        qs = Post.objects.current().filter(blog=blog)
+        qs = Post.objects.current().filter(blog=blog).select_related("section", "blog")
         return self.search(qs)
 
 
@@ -165,7 +165,10 @@ def blog_feed(request, **kwargs):
     scoper_lookup = kwargs.get(settings.PINAX_BLOG_SCOPING_URL_VAR, None)
 
     blog = hookset.get_blog(**kwargs)
-    posts = Post.objects.published().filter(blog=blog).order_by("-updated")
+    posts = (Post.objects.published()
+             .filter(blog=blog)
+             .order_by("-updated")
+             .select_related("section", "blog"))
 
     blog_url_kwargs = {}
     if scoper_lookup is not None:

setup.py

@@ -97,11 +97,11 @@
     install_requires=[
         "django>=1.11",
         "django-appconf>=1.0.1",
+        "pinax-images>=3.0.0",
         "pytz>=2016.6.1",
         "Pillow>=3.0.0",
         "Markdown>=2.6.5",
         "Pygments>=2.0.2",
-        "pinax-images>=3.0.0",
     ],
     tests_require=[
         "pinax-theme-bootstrap>=7.4.0",