author | Tero Marttila <terom@fixme.fi> |
Mon, 02 Sep 2013 03:50:34 +0300 | |
changeset 73 | 66b572c06b0a |
parent 72 | ea7a5a5ce7d4 |
child 74 | b8f1ae4247ad |
permissions | -rw-r--r-- |
73
66b572c06b0a
caching for shorturl redirect
Tero Marttila <terom@fixme.fi>
parents:
72
diff
changeset
|
1 |
from django.core.cache import cache |
2 | 2 |
from django.http import HttpResponse, HttpResponseRedirect, Http404 |
12 | 3 |
from django.shortcuts import render |
72
ea7a5a5ce7d4
add Last-Modified to urlitem responses (redirect and html)
Tero Marttila <terom@fixme.fi>
parents:
71
diff
changeset
|
4 |
from django.utils import timezone, http |
2 | 5 |
|
6 |
from qrurls.models import URL, URLItem |
|
7 |
||
72
ea7a5a5ce7d4
add Last-Modified to urlitem responses (redirect and html)
Tero Marttila <terom@fixme.fi>
parents:
71
diff
changeset
|
8 |
import calendar # timegm |
ea7a5a5ce7d4
add Last-Modified to urlitem responses (redirect and html)
Tero Marttila <terom@fixme.fi>
parents:
71
diff
changeset
|
9 |
|
54 | 10 |
""" |
11 |
Public frontend UI. |
|
12 |
""" |
|
13 |
||
71
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
14 |
class URLNotFound (Http404): |
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
15 |
pass |
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
16 |
|
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
17 |
class URLItemNotFound (Http404): |
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
18 |
pass |
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
19 |
|
72
ea7a5a5ce7d4
add Last-Modified to urlitem responses (redirect and html)
Tero Marttila <terom@fixme.fi>
parents:
71
diff
changeset
|
20 |
def http_datetime (dt) : |
ea7a5a5ce7d4
add Last-Modified to urlitem responses (redirect and html)
Tero Marttila <terom@fixme.fi>
parents:
71
diff
changeset
|
21 |
return http.http_date(calendar.timegm(dt.utctimetuple())) |
ea7a5a5ce7d4
add Last-Modified to urlitem responses (redirect and html)
Tero Marttila <terom@fixme.fi>
parents:
71
diff
changeset
|
22 |
|
71
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
23 |
def _get_url_item (shorturl=None, shorturl_id=None, item_id=None, related=()) : |
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
24 |
# JOIN against shorturl, urlimage |
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
25 |
url_item = URLItem.objects.select_related(*related) |
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
26 |
|
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
27 |
if shorturl: |
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
28 |
url_item = url_item.filter(shorturl__shorturl=shorturl) |
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
29 |
elif shorturl_id: |
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
30 |
shorturl_id = int(shorturl_id) |
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
31 |
url_item = url_item.filter(shorturl__id=shorturl_id) |
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
32 |
else: |
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
33 |
raise URLNotFound() |
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
34 |
|
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
35 |
# match for published items |
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
36 |
now = timezone.now() |
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
37 |
url_item = url_item.filter(published__lt=now).order_by('-published') |
73
66b572c06b0a
caching for shorturl redirect
Tero Marttila <terom@fixme.fi>
parents:
72
diff
changeset
|
38 |
|
66b572c06b0a
caching for shorturl redirect
Tero Marttila <terom@fixme.fi>
parents:
72
diff
changeset
|
39 |
if item_id : |
71
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
40 |
return url_item.get(id=int(item_id)) |
73
66b572c06b0a
caching for shorturl redirect
Tero Marttila <terom@fixme.fi>
parents:
72
diff
changeset
|
41 |
else : |
71
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
42 |
# most recent |
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
43 |
try: |
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
44 |
return url_item[0] |
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
45 |
except IndexError: |
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
46 |
raise URLItemNotFound() |
33
b714115da724
use seprate frontend view for url_images in url_items
Tero Marttila <terom@fixme.fi>
parents:
19
diff
changeset
|
47 |
|
12 | 48 |
def index (request) : |
49 |
urls = URL.objects.all() |
|
50 |
return render(request, 'qrurls/index.html', dict( |
|
51 |
urls = urls, |
|
52 |
)) |
|
53 |
||
71
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
54 |
def shorturl (request, **opts) : |
70
e8a8d490df2f
optimize shorturl() to use a single SELECT JOIN query
Tero Marttila <terom@fixme.fi>
parents:
54
diff
changeset
|
55 |
""" |
e8a8d490df2f
optimize shorturl() to use a single SELECT JOIN query
Tero Marttila <terom@fixme.fi>
parents:
54
diff
changeset
|
56 |
Primary frontend for redirecting based on current time. |
e8a8d490df2f
optimize shorturl() to use a single SELECT JOIN query
Tero Marttila <terom@fixme.fi>
parents:
54
diff
changeset
|
57 |
""" |
73
66b572c06b0a
caching for shorturl redirect
Tero Marttila <terom@fixme.fi>
parents:
72
diff
changeset
|
58 |
|
66b572c06b0a
caching for shorturl redirect
Tero Marttila <terom@fixme.fi>
parents:
72
diff
changeset
|
59 |
key = 'qrurls/urlfeed/{}/url'.format(opts) |
66b572c06b0a
caching for shorturl redirect
Tero Marttila <terom@fixme.fi>
parents:
72
diff
changeset
|
60 |
data = cache.get(key) |
66b572c06b0a
caching for shorturl redirect
Tero Marttila <terom@fixme.fi>
parents:
72
diff
changeset
|
61 |
|
66b572c06b0a
caching for shorturl redirect
Tero Marttila <terom@fixme.fi>
parents:
72
diff
changeset
|
62 |
if data : |
66b572c06b0a
caching for shorturl redirect
Tero Marttila <terom@fixme.fi>
parents:
72
diff
changeset
|
63 |
url, modified = data |
66b572c06b0a
caching for shorturl redirect
Tero Marttila <terom@fixme.fi>
parents:
72
diff
changeset
|
64 |
else: |
66b572c06b0a
caching for shorturl redirect
Tero Marttila <terom@fixme.fi>
parents:
72
diff
changeset
|
65 |
url_item = _get_url_item(**opts) |
66b572c06b0a
caching for shorturl redirect
Tero Marttila <terom@fixme.fi>
parents:
72
diff
changeset
|
66 |
modified = url_item.last_modified() |
66b572c06b0a
caching for shorturl redirect
Tero Marttila <terom@fixme.fi>
parents:
72
diff
changeset
|
67 |
url = url_item.get_absolute_url() |
71
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
68 |
|
73
66b572c06b0a
caching for shorturl redirect
Tero Marttila <terom@fixme.fi>
parents:
72
diff
changeset
|
69 |
cache.set(key, (url, modified)) # XXX: expiry |
33
b714115da724
use seprate frontend view for url_images in url_items
Tero Marttila <terom@fixme.fi>
parents:
19
diff
changeset
|
70 |
|
40 | 71 |
# redirect, either directly, or to image() |
73
66b572c06b0a
caching for shorturl redirect
Tero Marttila <terom@fixme.fi>
parents:
72
diff
changeset
|
72 |
response = HttpResponseRedirect(url) |
72
ea7a5a5ce7d4
add Last-Modified to urlitem responses (redirect and html)
Tero Marttila <terom@fixme.fi>
parents:
71
diff
changeset
|
73 |
|
ea7a5a5ce7d4
add Last-Modified to urlitem responses (redirect and html)
Tero Marttila <terom@fixme.fi>
parents:
71
diff
changeset
|
74 |
# set cache |
73
66b572c06b0a
caching for shorturl redirect
Tero Marttila <terom@fixme.fi>
parents:
72
diff
changeset
|
75 |
response['Last-Modified'] = http_datetime(modified) |
72
ea7a5a5ce7d4
add Last-Modified to urlitem responses (redirect and html)
Tero Marttila <terom@fixme.fi>
parents:
71
diff
changeset
|
76 |
return response |
2 | 77 |
|
71
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
78 |
def item (request, **opts) : |
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
79 |
""" |
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
80 |
Frontend for a specific item. |
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
81 |
""" |
42
e4f6ea904257
qrurls item view: only show published items
Tero Marttila <terom@fixme.fi>
parents:
40
diff
changeset
|
82 |
|
71
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
83 |
url_item = _get_url_item(related=('shorturl', 'image',), **opts) |
04f4f312d4ad
optimize item view to JOIN across models, using common _get_url_item for both views
Tero Marttila <terom@fixme.fi>
parents:
70
diff
changeset
|
84 |
|
43
3b1f1a928283
rename qrurls image view to item view
Tero Marttila <terom@fixme.fi>
parents:
42
diff
changeset
|
85 |
if url_item.url : |
72
ea7a5a5ce7d4
add Last-Modified to urlitem responses (redirect and html)
Tero Marttila <terom@fixme.fi>
parents:
71
diff
changeset
|
86 |
response = HttpResponseRedirect(url_item.url) |
43
3b1f1a928283
rename qrurls image view to item view
Tero Marttila <terom@fixme.fi>
parents:
42
diff
changeset
|
87 |
elif url_item.image : |
51
6f35a169ef01
Make URLImage title optional, use URLFeed title
Tero Marttila <terom@fixme.fi>
parents:
43
diff
changeset
|
88 |
title = url_item.image.title or url_item.shorturl.title |
6f35a169ef01
Make URLImage title optional, use URLFeed title
Tero Marttila <terom@fixme.fi>
parents:
43
diff
changeset
|
89 |
|
72
ea7a5a5ce7d4
add Last-Modified to urlitem responses (redirect and html)
Tero Marttila <terom@fixme.fi>
parents:
71
diff
changeset
|
90 |
response = render(request, 'qrurls/image.html', dict( |
51
6f35a169ef01
Make URLImage title optional, use URLFeed title
Tero Marttila <terom@fixme.fi>
parents:
43
diff
changeset
|
91 |
title = title, |
40 | 92 |
image = url_item.image, |
93 |
)) |
|
94 |
else : |
|
33
b714115da724
use seprate frontend view for url_images in url_items
Tero Marttila <terom@fixme.fi>
parents:
19
diff
changeset
|
95 |
raise Http404() |
72
ea7a5a5ce7d4
add Last-Modified to urlitem responses (redirect and html)
Tero Marttila <terom@fixme.fi>
parents:
71
diff
changeset
|
96 |
|
ea7a5a5ce7d4
add Last-Modified to urlitem responses (redirect and html)
Tero Marttila <terom@fixme.fi>
parents:
71
diff
changeset
|
97 |
# set cache |
ea7a5a5ce7d4
add Last-Modified to urlitem responses (redirect and html)
Tero Marttila <terom@fixme.fi>
parents:
71
diff
changeset
|
98 |
response['Last-Modified'] = http_datetime(url_item.last_modified()) |
ea7a5a5ce7d4
add Last-Modified to urlitem responses (redirect and html)
Tero Marttila <terom@fixme.fi>
parents:
71
diff
changeset
|
99 |
return response |
ea7a5a5ce7d4
add Last-Modified to urlitem responses (redirect and html)
Tero Marttila <terom@fixme.fi>
parents:
71
diff
changeset
|
100 |