From 97d3321e89c8d4434927bdbc308db1ccffa99d3b Mon Sep 17 00:00:00 2001 From: Jon Dufresne Date: Wed, 15 May 2019 05:31:42 -0700 Subject: [PATCH] Changed tuple choices to list in docs. --- .../contributing/writing-code/coding-style.txt | 10 +++++----- docs/ref/contrib/admin/actions.txt | 4 ++-- docs/ref/forms/widgets.txt | 12 ++++++------ docs/ref/models/conditional-expressions.txt | 4 ++-- docs/ref/models/fields.txt | 12 ++++++------ docs/topics/db/managers.txt | 4 ++-- docs/topics/db/models.txt | 4 ++-- docs/topics/forms/modelforms.txt | 4 ++-- 8 files changed, 27 insertions(+), 27 deletions(-) diff --git a/docs/internals/contributing/writing-code/coding-style.txt b/docs/internals/contributing/writing-code/coding-style.txt index ab7cc0baa9..f9952b3661 100644 --- a/docs/internals/contributing/writing-code/coding-style.txt +++ b/docs/internals/contributing/writing-code/coding-style.txt @@ -258,17 +258,17 @@ Model style * ``def get_absolute_url()`` * Any custom methods -* If ``choices`` is defined for a given model field, define each choice as - a tuple of tuples, with an all-uppercase name as a class attribute on the - model. Example:: +* If ``choices`` is defined for a given model field, define each choice as a + list of tuples, with an all-uppercase name as a class attribute on the model. + Example:: class MyModel(models.Model): DIRECTION_UP = 'U' DIRECTION_DOWN = 'D' - DIRECTION_CHOICES = ( + DIRECTION_CHOICES = [ (DIRECTION_UP, 'Up'), (DIRECTION_DOWN, 'Down'), - ) + ] Use of ``django.conf.settings`` =============================== diff --git a/docs/ref/contrib/admin/actions.txt b/docs/ref/contrib/admin/actions.txt index f0c8be7728..3956ecb93c 100644 --- a/docs/ref/contrib/admin/actions.txt +++ b/docs/ref/contrib/admin/actions.txt @@ -47,11 +47,11 @@ simple news application with an ``Article`` model:: from django.db import models - STATUS_CHOICES = ( + STATUS_CHOICES = [ ('d', 'Draft'), ('p', 'Published'), ('w', 'Withdrawn'), - ) + ] class Article(models.Model): title = models.CharField(max_length=100) diff --git a/docs/ref/forms/widgets.txt b/docs/ref/forms/widgets.txt index 40f9e52f65..0e701babeb 100644 --- a/docs/ref/forms/widgets.txt +++ b/docs/ref/forms/widgets.txt @@ -57,12 +57,12 @@ widget on the field. In the following example, the from django import forms - BIRTH_YEAR_CHOICES = ('1980', '1981', '1982') - FAVORITE_COLORS_CHOICES = ( + BIRTH_YEAR_CHOICES = ['1980', '1981', '1982'] + FAVORITE_COLORS_CHOICES = [ ('blue', 'Blue'), ('green', 'Green'), ('black', 'Black'), - ) + ] class SimpleForm(forms.Form): birth_year = forms.DateField(widget=forms.SelectDateWidget(years=BIRTH_YEAR_CHOICES)) @@ -90,14 +90,14 @@ changing :attr:`ChoiceField.choices` will update :attr:`Select.choices`. For example:: >>> from django import forms - >>> CHOICES = (('1', 'First',), ('2', 'Second',)) + >>> CHOICES = [('1', 'First'), ('2', 'Second')] >>> choice_field = forms.ChoiceField(widget=forms.RadioSelect, choices=CHOICES) >>> choice_field.choices [('1', 'First'), ('2', 'Second')] >>> choice_field.widget.choices [('1', 'First'), ('2', 'Second')] - >>> choice_field.widget.choices = () - >>> choice_field.choices = (('1', 'First and only',),) + >>> choice_field.widget.choices = [] + >>> choice_field.choices = [('1', 'First and only')] >>> choice_field.widget.choices [('1', 'First and only')] diff --git a/docs/ref/models/conditional-expressions.txt b/docs/ref/models/conditional-expressions.txt index 80146917d7..f9e681f667 100644 --- a/docs/ref/models/conditional-expressions.txt +++ b/docs/ref/models/conditional-expressions.txt @@ -21,11 +21,11 @@ We'll be using the following model in the subsequent examples:: REGULAR = 'R' GOLD = 'G' PLATINUM = 'P' - ACCOUNT_TYPE_CHOICES = ( + ACCOUNT_TYPE_CHOICES = [ (REGULAR, 'Regular'), (GOLD, 'Gold'), (PLATINUM, 'Platinum'), - ) + ] name = models.CharField(max_length=50) registered_on = models.DateField() account_type = models.CharField( diff --git a/docs/ref/models/fields.txt b/docs/ref/models/fields.txt index fbcba2bd66..ff650851d7 100644 --- a/docs/ref/models/fields.txt +++ b/docs/ref/models/fields.txt @@ -89,12 +89,12 @@ these choices instead of the standard text field. The first element in each tuple is the actual value to be set on the model, and the second element is the human-readable name. For example:: - YEAR_IN_SCHOOL_CHOICES = ( + YEAR_IN_SCHOOL_CHOICES = [ ('FR', 'Freshman'), ('SO', 'Sophomore'), ('JR', 'Junior'), ('SR', 'Senior'), - ) + ] Generally, it's best to define choices inside a model class, and to define a suitably-named constant for each value:: @@ -106,12 +106,12 @@ define a suitably-named constant for each value:: SOPHOMORE = 'SO' JUNIOR = 'JR' SENIOR = 'SR' - YEAR_IN_SCHOOL_CHOICES = ( + YEAR_IN_SCHOOL_CHOICES = [ (FRESHMAN, 'Freshman'), (SOPHOMORE, 'Sophomore'), (JUNIOR, 'Junior'), (SENIOR, 'Senior'), - ) + ] year_in_school = models.CharField( max_length=2, choices=YEAR_IN_SCHOOL_CHOICES, @@ -130,7 +130,7 @@ will work anywhere that the ``Student`` model has been imported). You can also collect your available choices into named groups that can be used for organizational purposes:: - MEDIA_CHOICES = ( + MEDIA_CHOICES = [ ('Audio', ( ('vinyl', 'Vinyl'), ('cd', 'CD'), @@ -142,7 +142,7 @@ be used for organizational purposes:: ) ), ('unknown', 'Unknown'), - ) + ] The first element in each tuple is the name to apply to the group. The second element is an iterable of 2-tuples, with each 2-tuple containing diff --git a/docs/topics/db/managers.txt b/docs/topics/db/managers.txt index 6c9e6a4163..a08dddd028 100644 --- a/docs/topics/db/managers.txt +++ b/docs/topics/db/managers.txt @@ -161,7 +161,7 @@ For example:: class Person(models.Model): first_name = models.CharField(max_length=50) last_name = models.CharField(max_length=50) - role = models.CharField(max_length=1, choices=(('A', _('Author')), ('E', _('Editor')))) + role = models.CharField(max_length=1, choices=[('A', _('Author')), ('E', _('Editor'))]) people = models.Manager() authors = AuthorManager() editors = EditorManager() @@ -261,7 +261,7 @@ custom ``QuerySet`` if you also implement them on the ``Manager``:: class Person(models.Model): first_name = models.CharField(max_length=50) last_name = models.CharField(max_length=50) - role = models.CharField(max_length=1, choices=(('A', _('Author')), ('E', _('Editor')))) + role = models.CharField(max_length=1, choices=[('A', _('Author')), ('E', _('Editor'))]) people = PersonManager() This example allows you to call both ``authors()`` and ``editors()`` directly from diff --git a/docs/topics/db/models.txt b/docs/topics/db/models.txt index f895db89fa..6964876a3e 100644 --- a/docs/topics/db/models.txt +++ b/docs/topics/db/models.txt @@ -161,13 +161,13 @@ ones: A choices list looks like this:: - YEAR_IN_SCHOOL_CHOICES = ( + YEAR_IN_SCHOOL_CHOICES = [ ('FR', 'Freshman'), ('SO', 'Sophomore'), ('JR', 'Junior'), ('SR', 'Senior'), ('GR', 'Graduate'), - ) + ] The first element in each tuple is the value that will be stored in the database. The second element is displayed by the field's form widget. diff --git a/docs/topics/forms/modelforms.txt b/docs/topics/forms/modelforms.txt index 0bd7833967..fe2e231a36 100644 --- a/docs/topics/forms/modelforms.txt +++ b/docs/topics/forms/modelforms.txt @@ -165,11 +165,11 @@ Consider this set of models:: from django.db import models from django.forms import ModelForm - TITLE_CHOICES = ( + TITLE_CHOICES = [ ('MR', 'Mr.'), ('MRS', 'Mrs.'), ('MS', 'Ms.'), - ) + ] class Author(models.Model): name = models.CharField(max_length=100)