aboutsummaryrefslogtreecommitdiffstats
path: root/docs/topics
diff options
context:
space:
mode:
authorAnler Hp2014-08-01 10:20:10 +0200
committerAnler Hp2014-08-01 10:20:10 +0200
commitebcc78d96cf6f4cd6e464cd6b8eccd83305900c2 (patch)
treebcc9c10997659c55d67d81ec897d32936a2e8a11 /docs/topics
parent18f2ca4b020f7a06eecd45a83473b0beb413f98e (diff)
downloaddjango-rest-framework-ebcc78d96cf6f4cd6e464cd6b8eccd83305900c2.tar.bz2
Leave status responsibility to parent class
Django's `HttpResponse` class checks for the `status` param when it's initialized, if it's `None` it uses the class attribute `status_code` and thanks to that we can do things like: ``` class BadRequest(HttpResponse): status_code = 400 ``` Now, that doesn't work when inheriting from rest-framework's `Response`: ``` class BadRequest(rest_framework.response.Response): status_code = 400 # Bad, it's always ignored ``` Because a default status of `200` is being specified in `rest_framework.response.Response`. I think is more Django-friendly to just leave that status default value to `None` and leave the responsibility of choosing its value to the parent class: `HttpResponse`.
Diffstat (limited to 'docs/topics')
0 files changed, 0 insertions, 0 deletions