From d288fefeccc84110b4e0654d8d11a26988f529dd Mon Sep 17 00:00:00 2001 From: "Pavel \"LEdoian\" Turinsky" Date: Wed, 23 Oct 2024 22:54:14 +0200 Subject: [PATCH] =?UTF-8?q?Zru=C5=A1en=C3=AD=20do=C4=8Dasn=C3=BDch=20odkaz?= =?UTF-8?q?=C5=AF=20do=20`seminar.models.*`?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Poznámky: - `seminar.models.base` je v pořádku (není to dočasný soubor) - dočasné importy v `seminar/models/*.py` jsou taky validní, protože odtamtud zmizí. Jde mi o to, aby náhodné věci buď používaly už-správnou aplikaci, nebo postaru `seminar.models` jako celek, aby během odstřelu nebylo potřeba všechny věci přepisovat. Ano, tohle je potenciálně kontroverzní commit. Ale je dočasný a mně poněkud rozbíjí workflow, když mi v náhodné okamžiky odmítne běžet náhodná část webu na kterou nesahám jen kvůli tomu, že importuje věci odněkud, odkud to nečekám. Alternativní řešení: dát správné importy (s dočasnými `seminar.models.*`) do správných `aplikace/models.py` už teď a importovat věci rovnou z výsledných modulů. To zajišťuje jak konzistenci, tak to, že při odstřelu se cesta změní na jednom očekávaném místě (resp. spíš se prostě smaže) a všechno ostatní bude pokračovat ve fungování. --- odevzdavatko/models.py | 2 +- odevzdavatko/testutils.py | 2 +- soustredeni/models.py | 2 +- soustredeni/testutils.py | 2 +- tvorba/admin.py | 2 +- 5 files changed, 5 insertions(+), 5 deletions(-) diff --git a/odevzdavatko/models.py b/odevzdavatko/models.py index 65ab9a37..ad41d4ec 100644 --- a/odevzdavatko/models.py +++ b/odevzdavatko/models.py @@ -9,7 +9,7 @@ from django.urls import reverse_lazy from django.utils import timezone from django.conf import settings -from seminar.models import tvorba as am +import seminar.models as am # tvorba from seminar.models import base as bm from odevzdavatko.utils import vzorecek_na_prepocet, inverze_vzorecku_na_prepocet diff --git a/odevzdavatko/testutils.py b/odevzdavatko/testutils.py index 1f382438..c1240ab4 100644 --- a/odevzdavatko/testutils.py +++ b/odevzdavatko/testutils.py @@ -1,7 +1,7 @@ import datetime import random -from seminar.models.odevzdavatko import Reseni, Hodnoceni +from odevzdavatko.models import Reseni, Hodnoceni def gen_reseni_ulohy(rnd, cisla, uloha, pocet_resitelu, poradi_cisla, resitele_cisla, resitele): diff --git a/soustredeni/models.py b/soustredeni/models.py index f71e6736..56db6974 100644 --- a/soustredeni/models.py +++ b/soustredeni/models.py @@ -10,7 +10,7 @@ from django.conf import settings from personalni.models import Resitel, Organizator from seminar.models.base import SeminarModelBase -from seminar.models import tvorba as am +import seminar.models as am # tvorba logger = logging.getLogger(__name__) diff --git a/soustredeni/testutils.py b/soustredeni/testutils.py index 52e81d1c..6591cbb3 100644 --- a/soustredeni/testutils.py +++ b/soustredeni/testutils.py @@ -6,7 +6,7 @@ from typing import Sequence import lorem from .models import Soustredeni, Konfera -import seminar.models.tvorba as am +import seminar.models as am # tvorba import personalni.models as pm logger = logging.getLogger(__name__) diff --git a/tvorba/admin.py b/tvorba/admin.py index e6c2c64b..01880d5b 100644 --- a/tvorba/admin.py +++ b/tvorba/admin.py @@ -9,7 +9,7 @@ from django.utils.safestring import mark_safe import soustredeni.models -from seminar.models.tvorba import Rocnik, ZmrazenaVysledkovka, Deadline, Uloha, Problem, Tema, Clanek, Cislo +from seminar.models import Rocnik, ZmrazenaVysledkovka, Deadline, Uloha, Problem, Tema, Clanek, Cislo # tvorba admin.site.register(Rocnik) admin.site.register(ZmrazenaVysledkovka)