Django 5.1 ja Python 3.12

.github/workflows/continuous-integration.yml

@@ -18,8 +18,8 @@ jobs:
           pip install 'black==24.10.0'
       - name: Check formatting
         run: |
-          black -t py33 --check .
-          black -t py33 --check web/**/*.py.example
+          black -t py312 --check .
+          black -t py312 --check web/**/*.py.example
   unit-tests-sqlite:
     runs-on: ubuntu-latest
     timeout-minutes: 15

Dockerfile

@@ -1,4 +1,4 @@
-FROM python:3.3
+FROM python:3.12
 
 WORKDIR /app/web
 
@@ -8,4 +8,4 @@ CMD ["python", "./manage.py", "runserver", "0.0.0.0:3000"]
 
 COPY . /app/
 COPY web/settings/docker.py.example /app/web/settings/docker.py
-RUN pip install --trusted-host pypi.python.org -r /app/requirements.txt
+RUN pip install -r /app/requirements.txt

README.md

@@ -17,20 +17,22 @@ Tämän ohjelman mukana pitäisi tulla kopio GPL-lisenssistä; jos näin ei ole,
 
 ### Paikallisen kehitysympäristön pystytys
 
+
+* Luo Kipalle virtuaaliympäristö jonnekkin: `python -m venv /tmp/venv`
+* Ota virtuaaliympäristö käyttöön: `source /tmp/venv/bin/activate`
+* Asenna riippuvuudet: `pip install -r requirements.txt`
 * Luo jonnekkin väliaikainen hakemisto tietokannalle: `mkdir /tmp/tietokanta`
 * Kopioi kehitystietokanta: `cp docs/initial.db /tmp/tietokanta/kipa.db`
 * `cp ./web/settings/local.py.example ./web/settings/local.py`
-* Muokkaa edellisessä luotuun asetustiedostoon tietokantatiedostolle polku `/db/kipa.db`
-* Rakenna kontti: `docker build -t kipa_dev -f dev.Dockerfile .`
-* Käynnistä kontti: `docker run -it --rm --volume ".:/app" --volume "/tmp/tietokanta:/db" -p 8000:8000 kipa_dev  bash`
-* Kontissa: `cd web`
-* Kontissa: `python manage.py runserver  0.0.0.0:8000`
+* Muokkaa edellisessä luotuun asetustiedostoon tietokantatiedostolle polku `/tmp/tietokanta/kipa.db`
+* `cd web`
+* `python manage.py migrate --fake-initial --noinput`
+* `python manage.py runserver`
 
 ### Yksikkötestien ajaminen
 
-* tarvittaessa edellä kuvattu kontin rakennus ja käynnistäminen
-* Kontissa: `cd web`
-* Kontissa: `python manage.py test`
+* `cd web`
+* `python manage.py test`
 
 ### E2E-testien ajaminen
 
@@ -46,4 +48,4 @@ Kipaan suunniteltua kirjautumista.
 
 ### Python-koodin formatointi
 
-Koodi noudattaa Black-autoformatterin vesion 24.10.0 mukaista tyyliä. Blackille annetaan lippu `--target-version py33`. Formatointi tarkastetaan osana CI-putkea.
+Koodi noudattaa Black-autoformatterin vesion 24.10.0 mukaista tyyliä. Blackille annetaan lippu `--target-version py312`. Formatointi tarkastetaan osana CI-putkea.

docs/manual.md

@@ -137,9 +137,9 @@ Vieraile tupa2.sf.net Imagen linkkiä varten.
 ### Muut käyttöjärjestelmät / itse muokattava asennus
 
 Kipan laskenta perustuu Pythonin-ohjelmointikielellä kirjoitettuun koodiin.
-Tällä hetkellä kehitysversiossa on käytössä Python 3.3.
+Tällä hetkellä kehitysversiossa on käytössä Python 3.12.
 
-Djangon versio 1.6 on testattu kehityspalvelimen kanssa toimivaksi.
+Djangon versio 5.1 on testattu kehityspalvelimen kanssa toimivaksi.
 
 Apache-asennus ei toimi kehitysversiossa.
 
@@ -922,15 +922,11 @@ Käyttää testidatana kaikkia fixtures/tests/ löytyviä XML-päätteisiä
 tietokantapohjia.
 
 ```
-python manage.py syncdb
+python manage.py makemigrations
+python manage.py migrate
 ```
-
-päivittää tietokantatiedoston "tupa.db" "models.py" päivitetyn määrittelyn perustella.
-
-VAROITUS: Mikäli tietokannan rakenne olennaisesti muuttuu, on
-mahdollista ettei se enään ole yhteensopiva jo olemassa olevan tietokannan kanssa.
-Näin tietokantaa voi olla vaikea enään muokata päivityksen jälkeen.
-On suositeltavaa varmuuskopioida ja nollata tietokanta aina ennen päivittämistä.
+Ovat komentoja, joilla saadaan päivitettyä tietokannan rakennetta, jos models.py on muuttunut.
+Djangon dokumentaatio opastaa niiden käytössä.
 
 ```
 python manage.py flush

requirements.txt

@@ -1,4 +1,3 @@
-django==1.6
-south
+django==5.1
 #mysql-python
 # git+https://github.com/PyMySQL/[email protected]#egg=MySQLdb

web/media/kipa.css

@@ -498,3 +498,6 @@ td.th_erotin {
 
 .capitalize {text-transform:capitalize;}
 
+.oneRowInlineForm {
+	display: flex;
+}

web/settings/__init__.py

@@ -42,9 +42,10 @@
 
 # Local time zone for this installation.
 TIME_ZONE = "Europe/Helsinki"
+USE_TZ = False
 
 # Language code for this installation.
-LANGUAGE_CODE = "fi-FI"
+LANGUAGE_CODE = "fi"
 
 SITE_ID = 1
 
@@ -75,27 +76,31 @@
 # Make this unique, and don't share it with anybody.
 SECRET_KEY = "shbtq($_^om(xep=5f97k2+ntb3!cqn+)%8r#s6udzqnhj$5p6"
 
-# List of callables that know how to import templates from various sources.
-TEMPLATE_LOADERS = (
-    "django.template.loaders.filesystem.Loader",
-    "django.template.loaders.app_directories.Loader",
-)
-
-MIDDLEWARE_CLASSES = (
+MIDDLEWARE = [
     "django.middleware.common.CommonMiddleware",
     "django.contrib.sessions.middleware.SessionMiddleware",
     "django.contrib.messages.middleware.MessageMiddleware",
     "django.contrib.auth.middleware.AuthenticationMiddleware",
-)
+]
 
 ROOT_URLCONF = "urls"
 
-TEMPLATE_DIRS = (
-    hakemisto + "/templates",
-    # Put strings here, like "/home/html/django_templates" or "C:/www/django/templates".
-    # Always use forward slashes, even on Windows.
-    # Don't forget to use absolute paths, not relative paths.
-)
+TEMPLATES = [
+    {
+        "BACKEND": "django.template.backends.django.DjangoTemplates",
+        "DIRS": [
+            hakemisto + "/templates",
+        ],
+        "APP_DIRS": True,
+        "OPTIONS": {
+            "context_processors": [
+                "django.template.context_processors.request",
+                "django.contrib.auth.context_processors.auth",
+                "django.contrib.messages.context_processors.messages",
+            ],
+        },
+    },
+]
 
 INSTALLED_APPS = [
     "django.contrib.auth",
@@ -114,6 +119,8 @@
 
 TEST_RUNNER = "tupa.tests.CustomTestRunner"
 
+DEFAULT_AUTO_FIELD = "django.db.models.AutoField"
+
 # Should we serve the media files through Python?
 SERVE_MEDIA = DEBUG
 

web/templates/tupa/base_riisuttu.html

@@ -6,6 +6,7 @@
 <link rel="stylesheet" type="text/css" href="/kipamedia/tabcontent.css" />
 <link rel="icon" type="image/x-icon" href="/kipamedia/favicon.ico" />
 {% block header %}  {% endblock %}
+<script type="text/javascript" language="javascript" src="/kipamedia/tooltip.js"></script>
 <script type="text/javascript" src="/kipamedia/tabcontent.js">
 /***********************************************
 * Tab Content script v2.2- © Dynamic Drive DHTML code library (www.dynamicdrive.com)

web/templates/tupa/forms/arviointi.html

@@ -7,7 +7,7 @@
 <tr>
 <td width=300>
 <p>Käytössa: <input type="checkbox" name="{{arvio.name}}" id="{{arvio.id}}" value="abs"
-{% ifequal arvio.value "abs" %}checked="checked" {% endifequal %}/></p>
+{% if arvio.value == "abs" %}checked="checked" {% endif %}/></p>
 <p><font color=red><b>
                         {{oikea.errors}}
 </b></font>

web/templates/tupa/forms/maksimi_suoritus.html

@@ -9,13 +9,13 @@
 </b></font>
 Parhaat pisteet saa:<br>
 <input type="radio" id="{{parhaan_haku.id}}_1" value="min" name="{{parhaan_haku.name}}"
-{% ifequal parhaan_haku.value "min" %}checked="checked" {% endifequal %}
+{% if parhaan_haku.value == "min" %}checked="checked" {% endif %}
 /> <label for="{{parhaan_haku.id}}_1">pienin</label><br>
 <input type="radio" id="{{parhaan_haku.id}}_2" value="max" name="{{parhaan_haku.name}}"
-{% ifequal parhaan_haku.value "max" %}checked="checked" {% endifequal %}
+{% if parhaan_haku.value == "max" %}checked="checked" {% endif %}
 /> <label for="{{parhaan_haku.id}}_2">suurin</label><br>
 <input type="radio" id="{{parhaan_haku.id}}_3" value="" name="{{parhaan_haku.name}}"
-{% ifequal parhaan_haku.value "" %}checked="checked" {% endifequal %}
+{% if parhaan_haku.value == "" %}checked="checked" {% endif %}
 />
 <label for="{{parhaan_haku.id}}_3">
 {%if vapaa %} kaava:

web/templates/tupa/forms/nolla_suoritus.html

@@ -18,9 +18,9 @@
                 <legend>{%if vapaa %} Vapaa: {%else%} Kiinteä {% endif %}</legend>
                 <p>
         <input type="radio" id="{{nollan_kerroin.id}}" value="1" name="{{nollan_kerroin.name}}"
-        {% ifequal nollan_kerroin.value "1" %}
+        {% if nollan_kerroin.value == "1" %}
         checked="checked"
-        {% endifequal %}
+        {% endif %}
         {% if not nollan_kerroin.value %}
         checked="checked"
         {% endif %} />
@@ -47,22 +47,22 @@
                         </b></font>
 
 <input type="radio" id="{{nollan_kerroin.id}}" value="1.5" name="{{nollan_kerroin.name}}"
-{% ifequal nollan_kerroin.value "1.5" %}checked="checked" {% endifequal %}/>
+{% if nollan_kerroin.value == "1.5" %}checked="checked" {% endif %}/>
 1.5 (pienin tulos saa parhaat pisteet)<br>
 <input type="radio" id="{{nollan_kerroin.id}}" value="0.5" name="{{nollan_kerroin.name}}"
-{% ifequal nollan_kerroin.value "0.5" %}checked="checked" {% endifequal %}/>
+{% if nollan_kerroin.value == "0.5" %}checked="checked" {% endif%}/>
 0.5 (suurin tulos saa parhaat pisteet)<br>
 <input type="radio" id="{{nollan_kerroin.id}}" value="m" name="{{nollan_kerroin.name}}"
-{% ifnotequal nollan_kerroin.value "1.5" %}
-{% ifnotequal nollan_kerroin.value "0.5" %}
-{% ifnotequal nollan_kerroin.value "1" %}
+{% if nollan_kerroin.value != "1.5" %}
+{% if nollan_kerroin.value != "0.5" %}
+{% if nollan_kerroin.value != "1" %}
 {% if nollan_kerroin.value %}
 
 checked="checked"
 {% endif %}
-{% endifnotequal %}
-{% endifnotequal %}
-{% endifnotequal %}/>
+{% endif %}
+{% endif %}
+{% endif %}/>
 
 muu:
 

web/templates/tupa/forms/osa_tehtava.html

@@ -23,14 +23,14 @@ <h1>{{ heading }}</h1>
         <input type="radio" name="{{osatehtava.id}}" value="{{sarake.tyyppi}}"
         onclick="
         {% for sa in taulukko%}
-                {%ifequal sa.tyyppi sarake.tyyppi %}
+                {%if sa.tyyppi == sarake.tyyppi %}
                         setVisibility('{{sa.id}}','inline')
                 {% else %}
                         setVisibility('{{sa.id}}','none')
-                {% endifequal %}
+                {% endif %}
         {% endfor %} "
 
-        {%ifequal sarake.tyyppi tyyppi %} checked='checked' {% endifequal %} />
+        {%if sarake.tyyppi == tyyppi %} checked='checked' {% endif %} />
         {{sarake.otsikko}} &nbsp; &nbsp;
 {% endfor %}
 {% for sarake in taulukko%}

web/templates/tupa/forms/puhdas_kaava.html

@@ -17,10 +17,10 @@
 
             <select name="{{maarite.tyyppi.name}}">
                 <option value="piste"
-                {% ifequal maarite.tyyppi.value "piste" %}selected="selected"{% endifequal %}
+                {% if maarite.tyyppi.value == "piste" %}selected="selected"{% endif %}
                  >piste</option>
                 <option value="aika"
-                {% ifequal maarite.tyyppi.value "aika" %}selected="selected"{% endifequal %}
+                {% if maarite.tyyppi.value == "aika" %}selected="selected"{% endif %}
                 >aika</option>
             </select>
         </td>

web/templates/tupa/forms/vapaa_kaava.html

@@ -13,10 +13,10 @@
 
             <select name="{{maarite.tyyppi.name}}">
                 <option value="piste"
-                {% ifequal maarite.tyyppi.value "piste" %}selected="selected"{% endifequal %}
+                {% if maarite.tyyppi.value == "piste" %}selected="selected"{% endif %}
                  >piste</option>
                 <option value="aika"
-                {% ifequal maarite.tyyppi.value "aika" %}selected="selected"{% endifequal %}
+                {% if maarite.tyyppi.value == "aika" %}selected="selected"{% endif %}
                 >aika</option>
             </select>
         </td>

web/templates/tupa/heijasta.html

@@ -114,13 +114,13 @@
                                <td align="right" style="padding-right:10px;"><em> {{ sarake.maksimipisteet }} </em></td>
                                 {% else %}
 
-                                {% ifnotequal forloop.counter 0 %}
-                                {% ifnotequal forloop.counter 1 %}
-                                {% ifnotequal forloop.counter 2 %}
+                                {% if forloop.counter != 0 %}
+                                {% if forloop.counter != 1 %}
+                                {% if forloop.counter != 2 %}
                                         <td align="right" style="padding-right:10px;"><em> {{ sarake.maksimipisteet }} </em></td>
-                                {% endifnotequal %}
-                                {% endifnotequal %}
-                                {% endifnotequal %}
+                                {% endif %}
+                                {% endif %}
+                                {% endif %}
                                 {% endif %}
 
                         {%endfor %}
@@ -137,15 +137,15 @@
                 {% else %}
                 {% for sarake in rivi %}
                      {% comment %} ylarivi {% endcomment %}
-                     {% ifequal forloop.counter 2 %}
+                     {% if forloop.counter == 2 %}
                      {% comment %} ekasarake {% endcomment %}
                         <td align="right" style="padding-right:10px;"> {{ sarake.nro }}</td> <td>{{sarake.nimi}}
                                                 {%if sarake.lippukunta %}<br>{%endif%}
                                                 {{sarake.lippukunta}}
                                                 {%if sarake.piiri %}<br>{%endif%}
                                                 {{sarake.piiri}}</td>
                      {% else %}
-                     {% ifequal forloop.counter 3 %}
+                     {% if forloop.counter == 3 %}
                         <td align="right" style="padding-right:10px;">{{sarake}}</td>
                      {% comment %} muut sarakkeet {% endcomment %}
                      {% else %}
@@ -154,8 +154,8 @@
                                         {{ sarake }}
                                 {% if forloop.first %} </b> {%endif%}
                         </td>
-                     {% endifequal %}
-                     {% endifequal %}
+                     {% endif %}
+                     {% endif %}
 
                 {% endfor %}
                      {% endif %}
@@ -169,7 +169,7 @@
                         {% for sarake in rivi %}
 
 
-                                {% ifequal forloop.counter 2 %}
+                                {% if forloop.counter == 2 %}
                                         {% comment %} ekasarake {% endcomment %}
                                         <td align="right" style="padding-right:10px;">{{ sarake.nro }}</td> <td>{{ sarake.nimi }}
                                                 {%if sarake.lippukunta %}<br>{%endif%}
@@ -184,7 +184,7 @@
                                         {{ sarake }}
                                         {% if forloop.first %} </b> {%endif%}
                                 </td>
-                                {% endifequal %}
+                                {% endif %}
                         {% endfor %}
                         </tr>
                         {%if forloop.counter|divisibleby:"5" %}