added conditional to check user in finalize_report()
This commit is contained in:
		
							parent
							
								
									f8d08ae7fd
								
							
						
					
					
						commit
						dce18a5e28
					
				
					 1 changed files with 4 additions and 0 deletions
				
			
		|  | @ -241,6 +241,10 @@ def finalize_report(request, report_pk): | ||||||
|     :param report_pk: report ID |     :param report_pk: report ID | ||||||
|     :return: JSON response containing user message |     :return: JSON response containing user message | ||||||
|     """ |     """ | ||||||
|  |     # Check that the user owns the report | ||||||
|  |     if not user_owns_report(user=request.user, report=report_pk): | ||||||
|  |         return JsonResponse({"message": "Current user does not own the specified report."}, status=401) | ||||||
|  | 
 | ||||||
|     r = Report.objects.get(id=report_pk) |     r = Report.objects.get(id=report_pk) | ||||||
|     if r.submitted: |     if r.submitted: | ||||||
|         return JsonResponse({"message": "Cannot submit a report that has already been submitted."}, status=409) |         return JsonResponse({"message": "Cannot submit a report that has already been submitted."}, status=409) | ||||||
|  |  | ||||||
		Loading…
	
	Add table
		
		Reference in a new issue
	
	 Rupika
						Rupika