Changeset 5391


Ignore:
Timestamp:
10/19/11 16:22:03 (7 months ago)
Author:
pmartin
Message:

See #2151 Last changes in refactor of exception Permission Denied

Location:
trunk/merengueproj/merengue
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • trunk/merengueproj/merengue/base/admin.py

    r5386 r5391  
    601601            if not self.has_change_permission(request, obj): 
    602602                raise PermissionDenied(content=obj, 
    603                                        user=request.user, 
    604                                        perm=perms_api.MANAGE_SITE_PERMISION) 
     603                                       user=request.user) 
    605604            checkbox_data['object_name'] = escape(obj) 
    606605            checkbox_data['object_id'] = obj.id 
    … …  
    657656        if not self.has_delete_permission(request, obj): 
    658657            raise PermissionDenied(content=obj, 
    659                                    user=request.user, 
    660                                    perm=perms_api.MANAGE_SITE_PERMISION) 
     658                                   user=request.user) 
    661659 
    662660        if obj is None: 
    … …  
    777775 
    778776    def change_state(self, request, queryset, state, confirm_msg, perm=None): 
    779         if perm and not perms_api.has_permission_in_queryset(queryset, request.user, perm, None) or not self.has_change_permission(request): 
    780             raise PermissionDenied(content=queryset, user=request.user, perm=perm or 'edit') 
     777        if perm: 
     778            perms_api.assert_has_permission_in_queryset(queryset, request.user, perm, None) 
     779        if not self.has_change_permission(request): 
     780            raise PermissionDenied(content=queryset, user=request.user) 
    781781        selected = request.POST.getlist(admin.ACTION_CHECKBOX_NAME) 
    782782        if selected: 
    … …  
    12261226        extra_context = self._update_extra_context(request, extra_context, parent_model_admin, parent_object) 
    12271227        if not self.has_change_permission(request, None): 
    1228             raise PermissionDenied(user=request.user, perm='edit') 
     1228            raise PermissionDenied(user=request.user) 
    12291229        contents = [{'name': unicode(i), 'url': i.get_admin_absolute_url()} for i in self.queryset(request)] 
    12301230        json_dict = simplejson.dumps({'contents': contents, 

  • trunk/merengueproj/merengue/multimedia/admin.py

    r5386 r5391  
    6464        selected = request.POST.getlist(admin.ACTION_CHECKBOX_NAME) 
    6565        if selected: 
    66             if not perms_api.has_permission_in_queryset(queryset, request.user, 'edit') or not self.has_change_permission(request): 
    67                     raise PermissionDenied(content=queryset, user=request.user, perm='edit') 
     66            perms_api.assert_has_permission_in_queryset(queryset, request.user, 'edit') 
     67            if not self.has_change_permission(request): 
     68                raise PermissionDenied(user=request.user, perm='edit') 
    6869            if request.POST.get('post'): 
    6970                multimedia = self.basecontent 
    … …  
    9697        selected = request.POST.getlist(admin.ACTION_CHECKBOX_NAME) 
    9798        if selected: 
    98             if not perms_api.has_permission_in_queryset(queryset, request.user, 'edit') or not self.has_change_permission(request): 
    99                     raise PermissionDenied(content=queryset, user=request.user, perm='edit') 
     99            perms_api.assert_has_permission_in_queryset(queryset, request.user, 'edit') 
     100            if not self.has_change_permission(request): 
     101                raise PermissionDenied(user=request.user, perm='edit') 
    100102            if request.POST.get('post'): 
    101103                multimedia = self.basecontent 
Note: See TracChangeset for help on using the changeset viewer.