Raise 404 for HostingOrder not belonging to user
This commit is contained in:
		
					parent
					
						
							
								6c81ffbdda
							
						
					
				
			
			
				commit
				
					
						beefeb62cb
					
				
			
		
					 1 changed files with 10 additions and 3 deletions
				
			
		|  | @ -670,9 +670,16 @@ class OrdersHostingDetailView(LoginRequiredMixin, | ||||||
|     permission_required = ['view_hostingorder'] |     permission_required = ['view_hostingorder'] | ||||||
|     model = HostingOrder |     model = HostingOrder | ||||||
| 
 | 
 | ||||||
|     def get_object(self): |     def get_object(self, queryset=None): | ||||||
|         return HostingOrder.objects.get( |         try: | ||||||
|             pk=self.kwargs.get('pk')) if self.kwargs.get('pk') else None |             hosting_order_obj = HostingOrder.objects.get( | ||||||
|  |                 pk=self.kwargs.get('pk') | ||||||
|  |             ) | ||||||
|  |         except HostingOrder.DoesNotExist: | ||||||
|  |             hosting_order_obj = None | ||||||
|  |         if not self.request.user.has_perm(hosting_order_obj): | ||||||
|  |             raise Http404 | ||||||
|  |         return hosting_order_obj | ||||||
| 
 | 
 | ||||||
|     def get_context_data(self, **kwargs): |     def get_context_data(self, **kwargs): | ||||||
|         # Get context |         # Get context | ||||||
|  |  | ||||||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue