Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#18614 closed Cleanup/optimization (fixed)

https://docs.djangoproject.com/en/1.4/topics/forms/ missing some sample code

Reported by: xarses@… Owned by: nobody
Component: Documentation Version: 1.4
Severity: Normal Keywords: from csrf missing imports
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

The first two examples should include example file names that the code might be inside, since we might be new to the whole model / view concept.

in https://docs.djangoproject.com/en/1.4/topics/forms/#using-a-form-in-a-view
example is missing import statements for HttpResponseRedirect and render_to_response
example's final return is missing context_instance=RequestContext(request) after the dictionary, or a note needs to be made about this when explaining {% csrf_token %}. import path should also be referenced.

Change History (6)

in reply to:  description ; comment:1 by Aymeric Augustin, 12 years ago

Triage Stage: UnreviewedAccepted
Type: BugCleanup/optimization

Replying to xarses@…:

The first two examples should include example file names that the code might be inside, since we might be new to the whole model / view concept.

We don't generally include file names in code samples, maybe because Django leaves a lot of flexibility on code layout. You're supposed to have gone through the tutorial first and be familiar with models, views and templates.


in https://docs.djangoproject.com/en/1.4/topics/forms/#using-a-form-in-a-view
example is missing import statements for HttpResponseRedirect and render_to_response
example's final return is missing context_instance=RequestContext(request) after the dictionary, or a note needs to be made about this when explaining {% csrf_token %}. import path should also be referenced.

Yes, we should include import paths, and switch to render while we're there.

in reply to:  1 ; comment:2 by xarses, 12 years ago

Replying to aaugustin:

Replying to xarses@…:

The first two examples should include example file names that the code might be inside, since we might be new to the whole model / view concept.

We don't generally include file names in code samples, maybe because Django leaves a lot of flexibility on code layout. You're supposed to have gone through the tutorial first and be familiar with models, views and templates.

Alright, doesn't matter that much, been reading some of the other db examples and they have been saying stuff along the lines of "your model ... {code block}.

Yes I've read the tutorial but still like to get a better idea of what this blob is supposed to represent.


in https://docs.djangoproject.com/en/1.4/topics/forms/#using-a-form-in-a-view
example is missing import statements for HttpResponseRedirect and render_to_response
example's final return is missing context_instance=RequestContext(request) after the dictionary, or a note needs to be made about this when explaining {% csrf_token %}. import path should also be referenced.

Yes, we should include import paths, and switch to render while we're there.

comment:4 by Jannis Leidel, 12 years ago

Triage Stage: AcceptedReady for checkin

comment:5 by Florian Apolloner <florian@…>, 12 years ago

Resolution: fixed
Status: newclosed

In [7d06f975fe445f0393455b3eb9ec17dbe04f2ec3]:

Fixed #18614 -- Added missing imports in code samples.

comment:6 by Kevin London <kevinlondon@…>, 12 years ago

In [c6d06a9453859c5520312454b6353498332ff838]:

[1.4.x] Fixed #18614 -- Added imports and changed render_to_response to render.

Note: See TracTickets for help on using tickets.
Back to Top