iiuni / projektzapisy

System Zapisów na zajęcia w Instytucie Informatyki Uniwersytetu Wrocławskiego
https://zapisy.ii.uni.wroc.pl
30 stars 10 forks source link

1620 drobne porządki w importach #1630

Open cptkidd62 opened 8 months ago

cptkidd62 commented 8 months ago

Przejrzałam wszystkie pliki pod kątem innych importów zlokalizowanych niezgodnie regułami przy użyciu poleceń grep -r -A 10000 '^.*class ' | grep '.*import ', grep -r -A 10000 '^.*def ' | grep '.*import ', grep 'import ' $(grep -rL 'class') oraz grep 'import ' $(grep -rL 'def ') (uznając słowa kluczowe class i def za pewną granicę między importami z początku pliku a jego resztą) i nie wykazały one żadnych anomalii poza tymi wymienionymi w issue #1620. W plikach tych przeniosłam wszystkie importy na początek tychże plików, ponieważ nie znalazłam żadnego uzasadnienia, żeby musiały tam pozostać.

Dodatkowo, w pliku schedule/models/event.py zmieniłam zapytania Term.objects.filter(event=self) na self.term_set.all() w celu wyeliminowania importu Term w tym module.

Natomiast w aplikacji enrollment przeniosłam ChangedDay oraz Freeday do osobnego pliku specialdays.py, żeby uniknąć problemów z importem tych klas. Statyczne metody get_day_of_week i get_python_day_of_week z modułu Term dla porządku przeniosłam do pliku common/days_of_week.py, a metodę get_day_of_week z ChangedDay przemianowałam na get_official_day_of_week.

Na swoim miejscu pozostał import django.contrib.auth.models w pliku users/apps.py, który prawdopodobnie musi się znajdować wewnątrz funkcji ready.

Co do importów zakończonych komentarzem # noqa - wyszukałam wszystkie miejsca, w których występują i tam, gdzie nie było to dospecyfikowane, ograniczyłam ich działanie do reguły F401 (imported but unused), jako że jest to jedyna reguła, którą te linie łamią.