Skip to content
Projects
Groups
Snippets
Help
This project
Loading...
Sign in / Register
Toggle navigation
D
django-rest-framework
Overview
Overview
Details
Activity
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Board
Labels
Milestones
Merge Requests
0
Merge Requests
0
CI / CD
CI / CD
Pipelines
Jobs
Schedules
Charts
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Jobs
Commits
Issue Boards
Open sidebar
edx
django-rest-framework
Commits
5cc0a3c2
Commit
5cc0a3c2
authored
Apr 20, 2015
by
Kevin Brown
Browse files
Options
Browse Files
Download
Plain Diff
Merge pull request #2836 from ryangallen/master
only allow integer field to accept integers and strings, fixes #2835
parents
ecb37f51
1f10a39d
Hide whitespace changes
Inline
Side-by-side
Showing
2 changed files
with
6 additions
and
3 deletions
+6
-3
rest_framework/fields.py
+1
-1
tests/test_fields.py
+5
-2
No files found.
rest_framework/fields.py
View file @
5cc0a3c2
...
...
@@ -682,7 +682,7 @@ class IntegerField(Field):
self
.
fail
(
'max_string_length'
)
try
:
data
=
int
(
data
)
data
=
int
(
re
.
compile
(
r'\.0*\s*$'
)
.
sub
(
''
,
str
(
data
))
)
except
(
ValueError
,
TypeError
):
self
.
fail
(
'invalid'
)
return
data
...
...
tests/test_fields.py
View file @
5cc0a3c2
...
...
@@ -549,10 +549,13 @@ class TestIntegerField(FieldValues):
1
:
1
,
0
:
0
,
1.0
:
1
,
0.0
:
0
0.0
:
0
,
'1.0'
:
1
}
invalid_inputs
=
{
'abc'
:
[
'A valid integer is required.'
]
0.5
:
[
'A valid integer is required.'
],
'abc'
:
[
'A valid integer is required.'
],
'0.5'
:
[
'A valid integer is required.'
]
}
outputs
=
{
'1'
:
1
,
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment