Сохранение данных из выпадающего списка в модель

select, choices

from django.db import models

# Create your models here.
class Test(models.Model):
    METRIC = 'metric'
    IMPERIAL = 'imperial'
    ATHER = ''
    UNIT = (
        (METRIC, 'kg'),
        (IMPERIAL, 'lb'),
        (ATHER, 'none')
    )
    name = models.CharField(max_length=25)
    temperature_unit = models.CharField(max_length=8,
                                        choices=UNIT,
                                        default=METRIC)

    def __str__(self):
        return self.name

    class Meta:
        verbose_name_plural = 'tests'

from django.forms import ModelForm, TextInput
from .models import Test

class TestForm(ModelForm):
    class Meta:
        model = Test
        fields = ['name', 'unit']
        widgets = {'name': TextInput(attrs={'class': 'input', 'placeholder': 'Name'})}

<form method="POST">
    {% csrf_token %}
        <div class="field has-addons">
            <div class="control is-expanded">
                {{ form.name }}
            </div>
            <select>
                {% for temp in form.tunit %}
                    <option value=""{{ temp.data.value }}>{{ temp.data.label }}</option>
                {% endfor %}
             </select>
             <div class="control">
                <button type="submit" class="button is-info">
                    Add Unit
                </button>
            </div>
        </div>
</form>

import requests
from django.shortcuts import render
from .models import Test
from .forms import TestForm

def index(request):
    url = 'http://api.site.org/data/2.5/?q={}&units={}&id=1111&appid=111111111'

    if request.method == 'POST':
        # Здесь request.POST нет данных unit только name
        # Вылетает ошибка
        form = TestForm(request.POST) 
        form.save()

    tests = Test.objects.all()

    test_data = []

    for test in tests:
        # Преобразование данных из metric в kg итд.
        r = requests.get(url.format(test, test.unit)).json()
        res = [data for (t_type, data) in Test.UNIT if t_type == test.unit]

        test_w = {
            'name': test.name,
            'unit': res[0],
            'temp': r['main']['temp'],
            'description': r['data'][0]['description'],
            'icon': r['data'][0]['icon'],
        }

        test_data.append(test_w)

    context = {'test_data': test_data, 'form': form}

    return render(request, 'module/module.html', context)

Рекомендуем хостинг TIMEWEB

Стабильный хостинг, на котором располагается социальная сеть EVILEG. Для проектов на Django рекомендуем VDS хостинг.

Добрый день!

Так у вас в модели поде unit объявлено как temperature_unit

Вы имя перепутали

Так нужно

from django.forms import ModelForm, TextInput
from .models import Test

class TestForm(ModelForm):
    class Meta:
        model = Test
        fields = ['name', 'temperature_unit']
        widgets = {'name': TextInput(attrs={'class': 'input', 'placeholder': 'Name'})}

Ну и так далее везде поменяйте unit на temperature_unit

Я уже поправил. Скопировал данные из реального и тестово проектов. Ошибка вышла. Извините. :(

Но вопрос остался

Посмотрел документацию, делаю правильно, но не работает.

я так понимаю, у вас там в шаблоне пропущены id и name поля, поэтому и пропускаются значения, а они там должны быть.

<select id="id_temperature_unit" name="temperature_unit">
    {% for temp in form.temperature_unit %}
         <option value=""{{ temp.data.value }}>{{ temp.data.label }}</option>
    {% endfor %}
</select>

Вообще у Field есть методы для получения этих всех id и name, но я как-то не помню их, нужно в документации покопаться.

А так я использую django_bootstrap4, там эти моменты уже реализованы, так что всё автоматически работает. Не нужно всю эту вёрстку делать вручную.

Все исправил, все пересмотрел еще раз.
Понять не могу почему я в качестве request.POST при попытке сохранения данных, от формы получаю:

{'csrfmiddlewaretoken': 'BrD9OchYiCLJJAV3U8MX7NewnCO2uZ2EjmyfOIesySZp6dmcyoA1GRwnwReSch0T',
'name': 'Unit1'}

а не:

{'csrfmiddlewaretoken': 'BrD9OchYiCLJJAV3U8MX7NewnCO2uZ2EjmyfOIesySZp6dmcyoA1GRwnwReSch0T',
'name': 'Unit1', temperature_unit: 'kg'}

А как выглядит отрендеренная форма? В смысле html можете показать итоговый?
То есть открываете страницу с формой и через браузер посмотрите html и покажите, что там вышло.
Вот например у меня в одной из форм так получается.

<select name="type" id="id_type" class="form-control" title="">
    <option value="T" selected="">Руководство</option>
    <option value="N">Новости</option>
    <option value="S">Сниппет</option>
    <option value="O">Заметка</option>
</select>

После исправления:

<select id="id_temperature_unit" name="temperature_unit">
    {% for temp in form.temperature_unit %}
         <option value=""{{ temp.data.value }}>{{ temp.data.label }}</option>
    {% endfor %}
</select>

Получаю:

{'csrfmiddlewaretoken': ['EvCSJRprJbsn3Rte7AxtnpeXeNO3pIThmqxYJnmVZrG3quUnLQlxWtwOn2eT70Rw'], 'name': ['Lviv'], 'temperature_unit': ['']}

Только что-бы я не выбрал в списке значение temperature_unit не меняется.

Форма выглядит так:

Пытаюсь на сайт притулить индикатор погоды с выбором в каких унитах показывать температеру.

Нет, вы не поняли, покажите мне отрендеренный html из консоли браузера.

Вот отсюда

Все ясно:

И кстати да, я увидел вашу ошибку,

вот ваш код

<select id="id_temperature_unit" name="temperature_unit">
    {% for temp in form.temperature_unit %}
         <option value=""{{ temp.data.value }}>{{ temp.data.label }}</option>
    {% endfor %}
</select>

а вот как надо

<select id="id_temperature_unit" name="temperature_unit">
    {% for temp in form.temperature_unit %}
         <option value="{{ temp.data.value }}">{{ temp.data.label }}</option>
    {% endfor %}
</select>

Найдёте, где кавычки перепутали?

Нашел ошибку. Было:

option value=""{{ temperature.data.value }}>{{ temperature.data.label }}

Нужно:

option value="{{ temperature.data.value }}">{{ temperature.data.label }}

Оказывается все было правильно кроме кавычки :) Буду чаше заглядывать в Инспектор.

Огромное спасибо! :)

Я с консолью браузера при разработке на django вообще не расстаюсь. Иначе можно голову долго ломать.

Теперь так и буду делать :)

Если кому-то понадобится есть еще один вариант, который я думаю более верный:

models.py

from django.db import models

class TempUnit(models.Model):
    name = models.CharField(max_length=2)
    unit = models.CharField(max_length=8)

    def __str__(self):
        return self.name


class City(models.Model):
    name = models.CharField(max_length=25)
    temperature_unit = models.ForeignKey(TempUnit, on_delete=models.CASCADE)

    def __str__(self):
        return self.name

    class Meta:
        verbose_name_plural = 'cities'

views.py

def index(request):
    url = 'http://api.site.org/data/2.5/?q={}&units={}&id=1111&appid=111111111111'

    if request.method == 'POST':
        form = CityForm(request.POST)
        form.save()

    cities = City.objects.all()
    weather_data = []

    for city in cities:
        r = requests.get(url.format(city, city.temperature_unit)).json()

        city_weather = {
            'city': city.name,
            'temperature_unit': city.temperature_unit,
            'temperature': r['main']['temp'],
            'description': r['weather'][0]['description'],
            'icon': r['weather'][0]['icon'],
        }

        weather_data.append(city_weather)

    context = {'weather_data': weather_data, 'form': form}

    return render(request, 'data/data.html', context)

Тоесть просто создал новую модель и все скинул туда.

Если вы создадите вручную несколько TempUnit, а потом только установка этих TempUnit через выпадающий список, то да, будет даже лучше.

Однако переходите к CBV (Class Based View) вместо обычных вьюшек. Так лучше будет, в будущем оцените.

Спасибо.