Bernard Siebens
148f3a24c9
The post-composer screen was cramped and text-only, and its "audience" picker exposed the same "which team(s)" choice the create-event screen already dropped -- hard-lock teams to the coach's active team instead (hidden field, required=True so a tampered empty submission can't silently become club-wide), matching how CoachCreateEventView.teams already works. Photos reuse management's own NewsPhotoUploadForm/ NewsPhoto machinery, first upload becomes the main picture. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
82 lines
4.3 KiB
Python
82 lines
4.3 KiB
Python
from django import forms
|
|
from django.utils.translation import gettext_lazy as _
|
|
|
|
from members.models import Member
|
|
from teams.models import Position, TeamMembership
|
|
|
|
#: Shared by every text-ish field below -- mobile has no equivalent of
|
|
#: management/controlpanel's templatetags/field.html (which builds widget
|
|
#: classes at render time for their daisyUI-shaped inputs), so this app's one
|
|
#: form so far just bakes its own classes straight into the widget.
|
|
_INPUT_CLASSES = "h-11 w-full rounded-lg border border-stroke bg-paper px-3 text-[15px] text-ink placeholder:text-dim focus:border-ink focus:outline-none"
|
|
|
|
#: Same look as _INPUT_CLASSES, sized for a multi-line body instead of a
|
|
#: single-line value -- a fixed height (not h-11) plus vertical padding a
|
|
#: fixed-height input doesn't need, and resize-y so a longer post isn't
|
|
#: stuck scrolling inside a short box.
|
|
_TEXTAREA_CLASSES = "h-40 w-full resize-y rounded-lg border border-stroke bg-paper px-3 py-2.5 text-[15px] text-ink placeholder:text-dim focus:border-ink focus:outline-none"
|
|
|
|
|
|
class MemberProfileForm(forms.ModelForm):
|
|
"""M6 -- "Edit personal info" (design_handoff_rosterchief_platform/README.md).
|
|
|
|
Covers exactly the fields ``members.models.Member`` actually has. The
|
|
design mock also shows a "National register no.", an "Address", an
|
|
"Allergies / notes" field and two "Consent" toggles -- none of those have
|
|
a backing field on ``Member`` (see EditProfileView's own docstring), so
|
|
they're simply not part of this form rather than being invented here.
|
|
|
|
Same field list and date widget as ``management.forms.MemberForm`` (the
|
|
staff-side equivalent editing the same model) -- diverging widget
|
|
conventions across the platform for identical fields would be its own bug.
|
|
"""
|
|
|
|
class Meta:
|
|
model = Member
|
|
fields = ["first_name", "last_name", "date_of_birth", "email", "phone", "emergency_phone"]
|
|
# Same date widget as management.forms.MemberForm. phone/emergency_phone
|
|
# deliberately keep django-phonenumber-field's own RegionalPhoneNumberWidget
|
|
# (national-format display, region-aware parsing) rather than being
|
|
# swapped for a plain TextInput here -- __init__ below only adds a CSS
|
|
# class to whatever widget each field already has, never replaces it.
|
|
widgets = {"date_of_birth": forms.DateInput(attrs={"type": "date"})}
|
|
|
|
def __init__(self, *args, **kwargs):
|
|
super().__init__(*args, **kwargs)
|
|
for field in self.fields.values():
|
|
field.widget.attrs["class"] = _INPUT_CLASSES
|
|
|
|
|
|
class CoachRosterEditForm(forms.ModelForm):
|
|
"""Coach-mode roster row edit -- mobile/coach/roster_member.html's player
|
|
detail sheet. Same fields management.forms.TeamMembershipForm edits,
|
|
minus ``member`` (fixed by the URL here, never reassigned from this
|
|
screen) -- this used to be a desktop-only action; see CoachSquadView's
|
|
own docstring for why that changed."""
|
|
|
|
class Meta:
|
|
model = TeamMembership
|
|
fields = ["position", "jersey_number", "is_captain", "is_alternate_captain"]
|
|
|
|
def __init__(self, *args, club=None, team=None, season=None, **kwargs):
|
|
super().__init__(*args, **kwargs)
|
|
self.team = team
|
|
self.season = season
|
|
self.fields["position"].queryset = Position.objects.filter(club=club, staff_position=False)
|
|
self.fields["position"].required = True
|
|
self.fields["position"].widget.attrs["class"] = _INPUT_CLASSES
|
|
self.fields["jersey_number"].widget.attrs["class"] = _INPUT_CLASSES
|
|
self.fields["is_captain"].widget.attrs["class"] = "h-5 w-5 shrink-0 accent-ink"
|
|
self.fields["is_alternate_captain"].widget.attrs["class"] = "h-5 w-5 shrink-0 accent-ink"
|
|
|
|
def clean(self):
|
|
# Same jersey-clash check as TeamMembershipForm.clean -- team/season aren't
|
|
# form fields, so Django's automatic validate_unique() can't catch this itself.
|
|
cleaned = super().clean()
|
|
jersey_number = cleaned.get("jersey_number")
|
|
if jersey_number is not None and self.team is not None and self.season is not None:
|
|
clash = TeamMembership.objects.filter(team=self.team, season=self.season, jersey_number=jersey_number).exclude(pk=self.instance.pk).exists()
|
|
if clash:
|
|
self.add_error("jersey_number", _("Another player on this team already has this jersey number this season."))
|
|
return cleaned
|