From 722c155c17ea79336357f71aeb0cb8a69b816c0f Mon Sep 17 00:00:00 2001 From: Xeniac Date: Mon, 30 Apr 2018 11:12:20 +0200 Subject: [PATCH] Personal data is only needed for a club membership. --- src/membership/forms.py | 43 ++++++------------ .../locale/de/LC_MESSAGES/django.mo | Bin 10104 -> 9990 bytes .../locale/de/LC_MESSAGES/django.po | 40 ++++++++-------- .../templates/membership/register_form.html | 16 ++----- 4 files changed, 39 insertions(+), 60 deletions(-) diff --git a/src/membership/forms.py b/src/membership/forms.py index 2214837..62a0658 100644 --- a/src/membership/forms.py +++ b/src/membership/forms.py @@ -25,6 +25,10 @@ class MembershipForm(forms.ModelForm): help_text=_('Input format: yyyy-mm-dd') ) email = forms.EmailField(label=_('Email'), required=True) + required_membership_fields = ( + 'first_name', 'last_name', 'birthday', + 'street_name', 'post_code', 'city' + ) class Meta: """get the user model dyamicly""" @@ -35,36 +39,15 @@ class MembershipForm(forms.ModelForm): 'post_code', 'city' ) - def clean_birthday(self): - """If the user wants to be a member the birthday field is mandatory.""" - if self.cleaned_data['membership'] \ - and not self.cleaned_data['birthday']: - raise forms.ValidationError(_('For your membership, we need this. \ - Please fill out this field yet.')) - return self.cleaned_data['birthday'] + def clean(self): + cleaned_data = super().clean() + errormsg = _('For your membership, we need this. \ + Please fill out this field yet.') + membership = cleaned_data.get('membership') + for fieldname in self.required_membership_fields: + if membership and not cleaned_data.get(fieldname): + self.add_error(fieldname, errormsg) - def clean_street_name(self): - """If the user wants to be a member the address is mandatory.""" - if self.cleaned_data['membership'] \ - and not self.cleaned_data['street_name']: - raise forms.ValidationError(_('For your membership, we need this. \ - Please fill out this field yet.')) - return self.cleaned_data['street_name'] - - def clean_post_code(self): - """If the user wants to be a member the address is mandatory.""" - if self.cleaned_data['membership'] \ - and not self.cleaned_data['post_code']: - raise forms.ValidationError(_('For your membership, we need this. \ - Please fill out this field yet.')) - return self.cleaned_data['post_code'] - - def clean_city(self): - """If the user wants to be a member the address is mandatory.""" - if self.cleaned_data['membership'] and not self.cleaned_data['city']: - raise forms.ValidationError(_('For your membership, we need this. \ - Please fill out this field yet.')) - return self.cleaned_data['city'] class RegistrationForm(MembershipForm): @@ -122,7 +105,7 @@ class RegistrationForm(MembershipForm): def save(self, commit=True): """ Create the new User, set him/her inactive, create an acitivation request for the user and send him/her an activation email. - + :param commit: commit the SQL and send the email if True :return: the created User Object """ diff --git a/src/membership/locale/de/LC_MESSAGES/django.mo b/src/membership/locale/de/LC_MESSAGES/django.mo index fa1d36d4c70d2696fbbaf4624fa4613eadf9cd01..3b4bdf7070fcd3bffaae1b88f0e3eef5352ec08d 100644 GIT binary patch delta 1768 zcmXZbc}P@I6vy#vYBi2aHI3zNX@i=kR-zS#(NNMd6or9B(A1PeYN573(Fm6QRT>kK zD40bBffZP^LPZ-Dl?MGGZGS`?VTBcy1oizj55&*C_q}({x#!;bxfjPU--40Yh~u!?nz!Rf=3VH)ljy`B$e*zjRL>_wn}CU6 zkjjDrWmpfgF`0P*?#HF5LZ9PA{DdkXA|_Oz6II}3)O&f>yc89;93AMvvFJq=+z=B8 z?X-mj6>=EWi4G)(IgSc^4psS8bm2W~{WB`zZ%jlRZByqm`8NS)U>YvQ1l){CxDOSt zJHS8*dvFJ9^%x#b##g9I8OA}@VJhnA4&pUzM?N(b zY_ABbQFrGUs*oO3p;s&)VlMg33kLeo#?4TR(@-B~BgbfFq5`iL^I1|IU zx&ITQI^ss%>Izic-B^f0%)&dEiJvh89clDmtu5w9s)TN=#3k5*r?3q3(?gxvih6$s zs&jj-c`K^WPSgfYqdIvW756Htuv@6YK4L4H09RQRwIRo1F5p`1NBwLHxOutQfD5r3 zbyOcwg$$ws4{IJ~Ogh`5IVO*wYWZSZfGcqYcHmt6g6ddc8ee-Dl%XE9qAGlh+TmMN z0bfx8Q(U2+p$qewPerY-Mt!#db;i4_c@Wi^)2O4mgev@YXx(A@87RPOROKIB#}h|I z#8z+frq}ptyuP~nD*x8tM0-#0u05l-B)ZKuqNq6d!08%MG^=;Wxhm3G;j8ocYwK(L a4V$X{z76&3tJZC72$m-e^!k(I?Ee7Q;I&2o delta 1731 zcmXZbPe@cz6vy#9>SQ+lOS96^%+gW(Q%PDRrG}+hG+KiQ3R>pIFcHHbLWYRc60VPP==_efgR<*ccP* zFeU_(FdQ>56!UNemRR${7|FaA?bwV_c*>ezz-7#Punhaqjq@0Txddfs4xrXmh8nL4 zVo<{a1w3ZGXux>p&Df3I7=g|(W7c8@Dv>?ND^r0=v<|iIgf(wN#qGpsyo4)p2$lH# zFt5MjXFO0M)2NQTLk0eX3j7t7`5$y)Fm2NFO{fibU>uerf2N*)Yw#o{Vi(5ZC_3>e zD&9vg18ul~<@g;tu$ZWM_zKmk5XxPSkw^|E7L{Nss=^#p0eRNE0Cl8$QHhn|YCMju zcotL8`;);s1_|_~5O-n~wqY91U_RRDmjV`|64{SQ=)rT?iYzt1Fc2N|yAC6;2wQL$ z`cMgd!$S5q5v%+gl_SNQ)2IzPQLVm$s_+4-!U@#IPf+VVBQ=;sWN-5qiEW~|aq5r@ z)#|ON^#!PS`!I<8jhlfkl?NZ&jA_I~%;Es2cz%$V8*m*5s7tvUTd)Robg%It&LW?h zKFZ6*JE%J{he~7tmFRCv2Nxle{Y^XrwYC)Xfd_S|YEU0GSP!Rb4z;0I8+I|!nf0LpUPo0nh;-8o;{Z;ej;J=lf18_70XtCv zZ=hN|f-1m=s(2EW&|6fZA5fiIN}&I`mEqh#eUO6cL@p|U0#x7<)W#L4fc2Pzt@wXK zRA+psTm2jrcM-GE&TlXcw_`FM!Hsx6k^Za2!#vQ2KCHqC?7-Od{$Ir)sxxz_^G2v^|KkmOq|COj3(_s ztVShLiwfMRxy_iXNO#RGf~w_@uox%NgJCSr$D^o@c?TG{8H`ykY%YI>g{TTEQ3=$c z0`_1K_Ty%}X+3|5s%#o{#tYWmp6c&RJnE>@QHgK!KaVy=3>5efD)Xw;+i}4`z3Gn8 l-Zzfq@nOgNkbvwhy%%F#0ogg@V='\n" "Language-Team: Kasu \n" "Language: de\n" @@ -44,7 +44,6 @@ msgid "Groups" msgstr "Gruppen" #: admin.py:72 models.py:163 models.py:216 -#: templates/membership/register_form.html:32 msgid "Membership" msgstr "Mitgliedschaft" @@ -68,38 +67,32 @@ msgstr "Eingabeformat: tt.mm.jjjj" msgid "Email" msgstr "E-Mail" -#: forms.py:42 forms.py:50 forms.py:58 +#: forms.py:44 msgid "" "For your membership, we need this. Please fill out this field " "yet." msgstr "Diese Angabe wird für eine Mitgliedschaft benötigt, bitte ausfüllen." -#: forms.py:65 -msgid "" -"For your membership, we need this. Please fill out this field " -"yet." -msgstr "Diese Angabe wird für eine Mitgliedschaft benötigt, bitte ausfüllen." - -#: forms.py:78 +#: forms.py:61 msgid "password" msgstr "Passwort" -#: forms.py:80 +#: forms.py:63 msgid "password (again)" msgstr "Passwort (wiederholen)" -#: forms.py:102 +#: forms.py:85 msgid "This username is already taken. Please choose another." msgstr "" "Diesen Benutzername ist schon vergeben. Bitte einen anderen auswählen." -#: forms.py:109 +#: forms.py:92 msgid "" "This email address is already in use. Please supply a different " "email address." msgstr "Die E-Mail Adresse wird schon verwendet. Bitte eine andere angeben." -#: forms.py:119 +#: forms.py:102 msgid "The two password fields didn't match." msgstr "Die beiden Passwörter passen nicht." @@ -333,7 +326,6 @@ msgstr "" "eingesehen werden." #: templates/membership/membership_detail.html:45 -#: templates/membership/register_form.html:20 msgid "name" msgstr "Name" @@ -390,16 +382,21 @@ msgstr "" "Nach dem du deine Daten eingegeben hast, wirst du eine E-Mail zur Bestätigung bekommen.\n" "Bitte klicke auf den Link in dieser E-Mail zur Verifizierung, erst dann ist die Anmeldung möglich." -#: templates/membership/register_form.html:26 +#: templates/membership/register_form.html:20 #: templates/registration/login.html:41 msgid "login" msgstr "Anmelden" -#: templates/membership/register_form.html:39 +#: templates/membership/register_form.html:26 +#| msgid "Membership" +msgid "Club membership" +msgstr "Vereinsmitgliedschaft" + +#: templates/membership/register_form.html:33 msgid "reset" msgstr "Zurücksetzen" -#: templates/membership/register_form.html:41 +#: templates/membership/register_form.html:35 #: templates/registration/login.html:35 msgid "register" msgstr "Registrieren" @@ -577,6 +574,11 @@ msgstr "Benutzerprofil erfolgreich geändert." msgid "No Membership found matching the query" msgstr "Kein Mitglied gefunden welche der Anfrage entspricht" +#~ msgid "" +#~ "For your membership, we need this. Please fill out this field " +#~ "yet." +#~ msgstr "Diese Angabe wird für eine Mitgliedschaft benötigt, bitte ausfüllen." + #~ msgid "Given Name" #~ msgstr "Vorname" diff --git a/src/membership/templates/membership/register_form.html b/src/membership/templates/membership/register_form.html index 59519bf..eadc98e 100644 --- a/src/membership/templates/membership/register_form.html +++ b/src/membership/templates/membership/register_form.html @@ -16,21 +16,15 @@ {% block maincontent %}
{% csrf_token %} -
- {% trans "name"%} - {% get_fieldset "gender, first_name, last_name, username" from form as form1 %} - {% with form1 as form %}{% include "form.html" %}{% endwith %} -
-
{% trans "login"%} - {% get_fieldset "email, password1, password2, recaptcha" from form as form1 %} + {% get_fieldset "username, email, password1, password2, recaptcha" from form as form1 %} {% with form1 as form %}{% include "form.html" %}{% endwith %}
-
- {% trans "Membership"%} - {% get_fieldset "membership, birthday, street_name, post_code, city, telephone, website" from form as form2 %} +
+ {% trans "Club membership"%} + {% get_fieldset "membership, first_name, last_name, gender, birthday, street_name, post_code, city, telephone, website" from form as form2 %} {% with form2 as form %}{% include "form.html" %}{% endwith %}
@@ -42,4 +36,4 @@

-{% endblock %} \ No newline at end of file +{% endblock %}