Skip to content
Snippets Groups Projects
Commit 6b5c1d88 authored by Piotr Gawron's avatar Piotr Gawron
Browse files

unused argument removed

parent 060f61ba
No related branches found
No related tags found
1 merge request!336Pylint checks
......@@ -52,6 +52,7 @@ disable=
W0102, # dangerous-default-value
C0200, # consider-using-enumerate
W5104, # modelform-uses-exclude
E1003, # bad-super-call
[FORMAT]
# Maximum number of characters on a single line.
......
......@@ -11,10 +11,6 @@ from web.models import AppointmentType, Availability, FlyingTeam, Holiday, Item,
from web.models.constants import VISIT_SHOW_VISIT_NUMBER_FROM_ZERO
from web.templatetags.filters import display_visit_number
"""
Possible redundancy, but if need arises, contents of forms can be easily customized
"""
DATE_FORMAT_TIME = "%H:%M"
CURRENT_YEAR = datetime.datetime.now().year
YEAR_CHOICES = tuple(range(CURRENT_YEAR, CURRENT_YEAR - 120, -1))
......@@ -163,7 +159,7 @@ class AvailabilityAddForm(ModelForm):
worker = Worker.objects.get(id=self.cleaned_data["person"].id)
availabilities = worker.availability_set.all()
for availability in availabilities:
validate_availability_conflict(self, self.cleaned_data, availability)
validate_availability_conflict(self, availability)
class AvailabilityEditForm(ModelForm):
......@@ -190,10 +186,10 @@ class AvailabilityEditForm(ModelForm):
availabilities = worker.availability_set.all()
for availability in availabilities:
if availability.id != self.availability_id:
validate_availability_conflict(self, self.cleaned_data, availability)
validate_availability_conflict(self, availability)
def validate_availability_conflict(self, cleaned_data, availability):
def validate_availability_conflict(self, availability):
start_hour = self.cleaned_data.get("available_from", None)
end_hour = self.cleaned_data.get("available_till", None)
if availability.day_number == self.cleaned_data.get("day_number", None) and \
......@@ -246,7 +242,7 @@ class HolidayAddForm(ModelForm):
worker = Worker.objects.get(id=self.cleaned_data["person"].id)
availabilities = worker.availability_set.all()
for availability in availabilities:
validate_availability_conflict(self, self.cleaned_data, availability)
validate_availability_conflict(self, availability)
class RoomForm(ModelForm):
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment