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

Добрый день!

Так у вас в модели поде 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) вместо обычных вьюшек. Так лучше будет, в будущем оцените.

Спасибо.