Code

Opened 7 years ago

Closed 7 years ago

#4203 closed (fixed)

Add "Default is False" information to model-api.txt

Reported by: david@… Owned by: jacob
Component: Documentation Version: master
Severity: Keywords: docs, model-api
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

On page http://www.djangoproject.com/documentation/model-api/

I recommend insertion of text in << >>

...

blank

If True, the field is allowed to be blank. <<Default is False.>>

...

David

Attachments (1)

model-api.txt (463 bytes) - added by david@… 7 years ago.
required patch

Download all attachments as: .zip

Change History (8)

comment:1 Changed 7 years ago by SmileyChris

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Summary changed from Default value of blank is false to For `blank`, add "Default is `False`."
  • Triage Stage changed from Unreviewed to Accepted

Hi David, thanks for the report. If you're up to it, I'd ask that you modify your model-api.txt file, create a diff and attach that - this is the best way to get tickets in fast ;)

comment:2 Changed 7 years ago by SmileyChris

  • Summary changed from For `blank`, add "Default is `False`." to For blank, add "Default is False."

comment:3 Changed 7 years ago by david@…

Happy to help but my django installation does not include a model-api.txt file. What am I missing?

comment:4 Changed 7 years ago by SmileyChris

If you're using SVN, make sure you check out from trunk, not trunk/django. There is a docs folder located in the trunk root.

comment:5 Changed 7 years ago by david@…

Index: model-api.txt
===================================================================
--- model-api.txt (revision 5148)
+++ model-api.txt (working copy)
@@ -459,7 +459,7 @@

blank
~


-If True, the field is allowed to be blank.
+If True, the field is allowed to be blank. Default is False.

Note that this is different than null. null is purely
database-related, whereas blank is validation-related. If a field has

Changed 7 years ago by david@…

required patch

comment:6 Changed 7 years ago by Simon G. <dev@…>

  • Has patch set
  • Keywords docs, model-api added
  • Summary changed from For blank, add "Default is False." to Add "Default is False" information to model-api.txt
  • Triage Stage changed from Accepted to Ready for checkin

Looks good, thank you

comment:7 Changed 7 years ago by mtredinnick

  • Resolution set to fixed
  • Status changed from new to closed

(In [5166]) Fixed #4203 -- Noted that "blank" defaults to False in models. Thanks,
david@….

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
as The resolution will be set. Next status will be 'closed'
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.