fix #139
This commit is contained in:
parent
bdc6c35b48
commit
f348566214
|
@ -10,6 +10,9 @@ from django.contrib import messages
|
||||||
from django.http import Http404, HttpResponseRedirect
|
from django.http import Http404, HttpResponseRedirect
|
||||||
from django.views.generic.detail import SingleObjectMixin
|
from django.views.generic.detail import SingleObjectMixin
|
||||||
from django.core.urlresolvers import reverse_lazy
|
from django.core.urlresolvers import reverse_lazy
|
||||||
|
|
||||||
|
from profiles.models import Profile
|
||||||
|
|
||||||
import logging
|
import logging
|
||||||
logger = logging.getLogger("bornhack.%s" % __name__)
|
logger = logging.getLogger("bornhack.%s" % __name__)
|
||||||
|
|
||||||
|
@ -27,7 +30,6 @@ class EnsureTeamResponsibleMixin(SingleObjectMixin):
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|
||||||
|
|
||||||
class TeamListView(CampViewMixin, ListView):
|
class TeamListView(CampViewMixin, ListView):
|
||||||
template_name = "team_list.html"
|
template_name = "team_list.html"
|
||||||
model = Team
|
model = Team
|
||||||
|
@ -55,6 +57,13 @@ class TeamJoinView(LoginRequiredMixin, CampViewMixin, UpdateView):
|
||||||
fields = []
|
fields = []
|
||||||
|
|
||||||
def get(self, request, *args, **kwargs):
|
def get(self, request, *args, **kwargs):
|
||||||
|
if not Profile.objects.get(user=request.user).description:
|
||||||
|
messages.warning(
|
||||||
|
request,
|
||||||
|
"Please fill the description in your profile before joining a team"
|
||||||
|
)
|
||||||
|
return redirect('team_list', camp_slug=self.camp.slug)
|
||||||
|
|
||||||
if request.user in self.get_object().members.all():
|
if request.user in self.get_object().members.all():
|
||||||
messages.warning(request, "You are already a member of this team")
|
messages.warning(request, "You are already a member of this team")
|
||||||
return redirect('team_list', camp_slug=self.camp.slug)
|
return redirect('team_list', camp_slug=self.camp.slug)
|
||||||
|
|
Loading…
Reference in a new issue