Merge lp:~ricardokirkner/django-piston/django-1.11 into lp:django-piston

Proposed by Ricardo Kirkner
Status: Merged
Approved by: Ricardo Kirkner
Approved revision: 8
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: lp:~ricardokirkner/django-piston/django-1.11
Merge into: lp:django-piston
Diff against target: 144 lines (+38/-23)
3 files modified
demo/urls.py (+3/-4)
piston/emitters.py (+9/-6)
tox.ini (+26/-13)
To merge this branch: bzr merge lp:~ricardokirkner/django-piston/django-1.11
Reviewer Review Type Date Requested Status
Matias Bordese (community) Approve
Review via email: mp+322705@code.launchpad.net

Commit message

support django up to 1.11

To post a comment you must log in.
Revision history for this message
Matias Bordese (matiasb) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'demo/urls.py'
--- demo/urls.py 2015-04-24 22:17:32 +0000
+++ demo/urls.py 2017-04-18 17:58:53 +0000
@@ -1,11 +1,10 @@
1from django.conf.urls import patterns, include, url1from django.conf.urls import include, url
2from django.contrib import admin2from django.contrib import admin
33
4urlpatterns = patterns(4urlpatterns = [
5 '',
6 # Examples:5 # Examples:
7 # url(r'^$', 'demo.views.home', name='home'),6 # url(r'^$', 'demo.views.home', name='home'),
8 # url(r'^blog/', include('blog.urls')),7 # url(r'^blog/', include('blog.urls')),
98
10 url(r'^admin/', include(admin.site.urls)),9 url(r'^admin/', include(admin.site.urls)),
11)10]
1211
=== modified file 'piston/emitters.py'
--- piston/emitters.py 2015-04-27 15:30:47 +0000
+++ piston/emitters.py 2017-04-18 17:58:53 +0000
@@ -17,7 +17,10 @@
17from django.utils.xmlutils import SimplerXMLGenerator17from django.utils.xmlutils import SimplerXMLGenerator
18from django.utils.encoding import smart_unicode18from django.utils.encoding import smart_unicode
19from django.core.urlresolvers import NoReverseMatch19from django.core.urlresolvers import NoReverseMatch
20from django.core.serializers.json import DateTimeAwareJSONEncoder20try:
21 from django.core.serializers.json import DjangoJSONEncoder
22except ImportError:
23 from django.core.serializers.json import DateTimeAwareJSONEncoder as DjangoJSONEncoder
21from django.http import HttpResponse24from django.http import HttpResponse
22from django.core import serializers25from django.core import serializers
2326
@@ -86,7 +89,7 @@
86 ret = None89 ret = None
8790
88 # return anything we've already seen as a string only91 # return anything we've already seen as a string only
89 # this prevents infinite recursion in the case of recursive 92 # this prevents infinite recursion in the case of recursive
90 # relationships93 # relationships
9194
92 if thing in self.stack:95 if thing in self.stack:
@@ -160,8 +163,8 @@
160 # the nested models won't appear properly163 # the nested models won't appear properly
161 # Refs #157164 # Refs #157
162 if handler:165 if handler:
163 fields = getattr(handler, 'fields') 166 fields = getattr(handler, 'fields')
164 167
165 if not fields or hasattr(handler, 'fields'):168 if not fields or hasattr(handler, 'fields'):
166 """169 """
167 Fields was not specified, try to find teh correct170 Fields was not specified, try to find teh correct
@@ -177,7 +180,7 @@
177 if not get_fields:180 if not get_fields:
178 get_fields = set([ f.attname.replace("_id", "", 1)181 get_fields = set([ f.attname.replace("_id", "", 1)
179 for f in data._meta.fields + data._meta.virtual_fields])182 for f in data._meta.fields + data._meta.virtual_fields])
180 183
181 if hasattr(mapped, 'extra_fields'):184 if hasattr(mapped, 'extra_fields'):
182 get_fields.update(mapped.extra_fields)185 get_fields.update(mapped.extra_fields)
183186
@@ -391,7 +394,7 @@
391 """394 """
392 def render(self, request):395 def render(self, request):
393 cb = request.GET.get('callback', None)396 cb = request.GET.get('callback', None)
394 seria = json.dumps(self.construct(), cls=DateTimeAwareJSONEncoder, ensure_ascii=False, indent=4)397 seria = json.dumps(self.construct(), cls=DjangoJSONEncoder, ensure_ascii=False, indent=4)
395398
396 # Callback399 # Callback
397 if cb and is_valid_jsonp_callback_value(cb):400 if cb and is_valid_jsonp_callback_value(cb):
398401
=== modified file 'tox.ini'
--- tox.ini 2015-04-24 22:17:32 +0000
+++ tox.ini 2017-04-18 17:58:53 +0000
@@ -1,41 +1,54 @@
1[tox]1[tox]
2envlist =2envlist =
3 py2.7-django1.4, py2.7-django1.5, py2.7-django1.6, py2.7-django1.7, py2.7-django1.83 py27-django1.4, py27-django1.5, py27-django1.6, py27-django1.7, py27-django1.8, py27-django1.9, py27-django1.10, py27-django1.11
44
5[testenv]5[testenv]
6commands = python manage.py test piston6commands = python manage.py test piston
7deps=7deps=
8 mock8 mock
99
10[testenv:py2.7-django1.4]10[testenv:py27]
11basepython = python2.711basepython = python2.7
12
13[testenv:py27-django1.4]
12deps =14deps =
13 django >= 1.4, < 1.515 django >= 1.4, < 1.5
16 south==1.0
14 {[testenv]deps}17 {[testenv]deps}
15 south==1.0
1618
17[testenv:py2.7-django1.5]19[testenv:py27-django1.5]
18basepython = python2.7
19deps =20deps =
20 django >= 1.5, < 1.621 django >= 1.5, < 1.6
22 south==1.0
21 {[testenv]deps}23 {[testenv]deps}
22 south==1.0
2324
24[testenv:py2.7-django1.6]25[testenv:py27-django1.6]
25basepython = python2.7
26deps =26deps =
27 django >= 1.6, < 1.727 django >= 1.6, < 1.7
28 south==1.0
28 {[testenv]deps}29 {[testenv]deps}
29 south==1.0
3030
31[testenv:py2.7-django1.7]31[testenv:py27-django1.7]
32basepython = python2.7
33deps =32deps =
34 django >= 1.7, < 1.833 django >= 1.7, < 1.8
35 {[testenv]deps}34 {[testenv]deps}
3635
37[testenv:py2.7-django1.8]36[testenv:py27-django1.8]
38basepython = python2.7
39deps =37deps =
40 django >= 1.8, < 1.938 django >= 1.8, < 1.9
41 {[testenv]deps}39 {[testenv]deps}
40
41[testenv:py27-django1.9]
42deps =
43 django >= 1.9, < 1.10
44 {[testenv]deps}
45
46[testenv:py27-django1.10]
47deps =
48 django >= 1.10, < 1.11
49 {[testenv]deps}
50
51[testenv:py27-django1.11]
52deps =
53 django >= 1.11, < 1.12
54 {[testenv]deps}

Subscribers

People subscribed via source and target branches