Changeset 3e372da for remit/vouchers

Timestamp:

Jul 25, 2010, 4:31:09 AM (15 years ago)

Author:

Alex Dehnert <adehnert@…>

Branches:

master, client

Children:

0e2f379

Parents:

dd6edfb

git-author:

Alex Dehnert <adehnert@…> (07/25/10 04:31:09)

git-committer:

Alex Dehnert <adehnert@…> (07/25/10 04:31:09)

Message:

Add bulk actions to request list (Trac: #4)

Location:

remit/vouchers

Files:

• models.py (modified) (1 diff)
• views.py (modified) (2 diffs)

remit/vouchers/models.py

@@ -250 +250 @@
     ),
 }
+
+class BulkRequestAction:
+    def __init__(self, name, label, action, perm_predicate=None, ):
+        self.name = name
+        self.label = label
+        self.action = action
+        if perm_predicate is None:
+            perm_predicate = lambda user: True
+        elif perm_predicate == True:
+            perm_predicate = lambda user: True
+        self.perm_predicate = perm_predicate
+    def can(self, user):
+        return self.perm_predicate(user)
+    def do(self, http_request, rr, ):
+        if self.can(http_request.user):
+            return self.action(http_request, rr, )
+        else:
+            return False, "permission denied"
+    def __str__(self):
+        return self.label
+    @classmethod
+    def filter_can_only(cls, actions, user):
+        return [ action for action in actions if action.can(user) ]
+def bulk_action_approve(http_request, rr):
+    approver = http_request.user
+    signatory_name = http_request.user.get_full_name()
+    if rr.voucher:
+        return False, "already approved"
+    else:
+        rr.approve(approver, signatory_name)
+        return True, "request approved"
+
+def bulk_action_email_factory(stock_email_obj):
+    assert stock_email_obj.context == 'request'
+    def inner(http_request, rr):
+        stock_email_obj.send_email_request(rr)
+        return True, "mail sent"
+    return inner
+def perm_checker(perm):
+    def predicate(user):
+        return user.has_perm(perm)
+    return predicate
+
+bulk_request_actions = []
+if settings.SIGNATORY_EMAIL:
+    bulk_request_actions.append(BulkRequestAction(
+        name='approve',
+        label='Approve Requests',
+        action=bulk_action_approve,
+        perm_predicate=perm_checker('vouchers.can_approve'),
+    ))
+for name, stockemail in stock_emails.items():
+    if stockemail.context == 'request':
+        bulk_request_actions.append(BulkRequestAction(
+            name='email/%s' % name,
+            label='Stock Email: %s' % stockemail.label,
+            action=bulk_action_email_factory(stockemail),
+            perm_predicate=perm_checker('vouchers.can_email'),
+        ))

remit/vouchers/views.py

@@ -354 +354 @@
 )
 
+def list_to_keys(lst):
+    dct = {}
+    for key in lst:
+        dct[key] = True
+    return dct
+
 @login_required
 def show_requests(http_request, ):
+    # BULK ACTIONS
+    actions = vouchers.models.BulkRequestAction.filter_can_only(
+        vouchers.models.bulk_request_actions,
+        http_request.user,
+    )
+    apply_action_message = None
+    apply_action_errors = []
+    if 'select' in http_request.REQUEST:
+        selected_rr_ids = [ int(item) for item in http_request.REQUEST.getlist('select') ]
+    else:
+        selected_rr_ids = []
+    if "apply-action" in http_request.POST:
+        action_name = http_request.POST['action']
+        if action_name == 'none':
+            apply_action_message = "No action selected."
+        else:
+            matching_actions = [ action for action in actions if action.name == action_name]
+            if(len(matching_actions) > 0):
+                action = matching_actions[0]
+                rrs = ReimbursementRequest.objects.filter(pk__in=selected_rr_ids)
+                for rr in rrs:
+                    success, msg = action.do(http_request, rr)
+                    if not success:
+                        apply_action_errors.append((rr, msg))
+                apply_action_message = '"%s" applied to %d request(s) (%d errors encountered)' % (action.label, len(rrs), len(apply_action_errors), )
+            else:
+                apply_action_message = "Unknown or forbidden action requested."
+
     # PERMISSION-BASED REQUEST FILTERING
     if http_request.user.has_perm('vouchers.can_list'):
@@ -398 +432 @@
         queryset=qs,
         extra_context={
+            'actions' : actions,
+            'selected_ids'  : list_to_keys(selected_rr_ids),
+            'action_message': apply_action_message,
+            'action_errors' : apply_action_errors,
             'useronly': useronly,
             'order'   : order,