Private
Public Access
1
0

passing tests

This commit is contained in:
Sander Roosendaal
2020-01-11 17:05:08 +01:00
parent ad9cab4392
commit 48a361a43e
2 changed files with 21 additions and 43 deletions

View File

@@ -20,12 +20,7 @@ def team_view(request,team_id=0,userid=0):
myteams, memberteams, otherteams = get_teams(request)
teams.remove_expired_invites()
try:
t = Team.objects.get(id=team_id)
except Team.DoesNotExist:
raise Http404("Team doesn't exist")
t = get_object_or_404(Team,pk=team_id)
#if r.rowerplan == 'basic' and t.manager.rower.rowerplan != 'coach':
# raise PermissionDenied("You need to be on a Paid Plan to see or join this team")
@@ -589,12 +584,10 @@ def rower_invitations_view(request,code=None,message='',successmessage=''):
})
return HttpResponseRedirect(url)
@login_required()
def team_edit_view(request,id=0):
try:
t = Team.objects.get(id=id)
except Team.DoesNotExist:
raise Http404("Team does not exist")
@permission_required('teams.change_team',fn=get_team_by_pk,raise_exception=True)
def team_edit_view(request, team_id=0):
t = get_object_or_404(Team,pk=team_id)
if request.method == 'POST':
teamcreateform = TeamForm(request.POST,instance=t)
@@ -614,7 +607,7 @@ def team_edit_view(request,id=0):
url = reverse(team_view,
kwargs={
'team_id':int(id),
'team_id':int(team_id),
}
)
@@ -632,11 +625,11 @@ def team_edit_view(request,id=0):
'name': 'Groups'
},
{
'url':reverse(team_view,kwargs={'team_id':id}),
'url':reverse(team_view,kwargs={'team_id':team_id}),
'name': t.name
},
{
'url':reverse(team_edit_view,kwargs={'id':id}),
'url':reverse(team_edit_view,kwargs={'team_id':team_id}),
'name': 'Edit'
}
]
@@ -714,15 +707,10 @@ def team_create_view(request):
})
#@login_required()
@permission_required('teams.delete_team',fn=get_team_by_pk)
@permission_required('teams.delete_team',fn=get_team_by_pk,raise_exception=True)
def team_deleteconfirm_view(request,team_id):
r = getrower(request.user)
try:
t = Team.objects.get(id=team_id)
except Team.DoesNotExist:
raise Http404("This team doesn't exist")
if t.manager != request.user:
raise PermissionDenied("You are not allowed to delete this team")
t = get_object_or_404(Team,pk=team_id)
myteams, memberteams, otherteams = get_teams(request)
@@ -750,15 +738,10 @@ def team_deleteconfirm_view(request,team_id):
'active':'nav-teams',
})
@login_required()
def team_delete_view(request,id):
@permission_required('teams.delete_team',fn=get_team_by_pk,raise_exception=True)
def team_delete_view(request,team_id):
r = getrower(request.user)
try:
t = Team.objects.get(id=id)
except Team.DoesNotExist:
raise Http404("This team doesn't exist")
if t.manager != request.user:
raise PermissionDenied("You are not allowed to delete this team")
t = get_object_or_404(Team,pk=team_id)
teams.remove_team(t.id)
@@ -766,15 +749,10 @@ def team_delete_view(request,id):
response = HttpResponseRedirect(url)
return response
@login_required()
def team_members_stats_view(request,id):
@permission_required('teams.change_team',fn=get_team_by_pk,raise_exception=True)
def team_members_stats_view(request,team_id):
r = getrower(request.user)
try:
t = Team.objects.get(id=id)
except Team.DoesNotExist:
raise Http404("This team doesn't exist")
if t.manager != request.user:
raise PermissionDenied("You are not allowed to see this page")
t = get_object_or_404(Team,pk=team_id)
members = Rower.objects.filter(team=t).order_by("user__last_name","user__first_name")
@@ -788,11 +766,11 @@ def team_members_stats_view(request,id):
'name': 'Groups'
},
{
'url':reverse(team_view,kwargs={'team_id':id}),
'url':reverse(team_view,kwargs={'team_id':team_id}),
'name': t.name
},
{
'url':reverse(team_members_stats_view,kwargs={'id':id}),
'url':reverse(team_members_stats_view,kwargs={'team_id':team_id}),
'name': 'Members Stats'
}
]