Ignore:
Timestamp:
Jul 24, 2010, 9:19:56 PM (15 years ago)
Author:
Alex Dehnert <adehnert@…>
Branches:
master, client
Children:
dd6edfb
Parents:
b1a259c
git-author:
Alex Dehnert <adehnert@…> (07/24/10 21:19:56)
git-committer:
Alex Dehnert <adehnert@…> (07/24/10 21:19:56)
Message:

Rename request to http_request in show_requests

vouchers.views.show_requests has this awkward ambiguity between
request=http_request and request=ReimbursementRequest?. By renaming
request to http_request, this ambiguity can be reduced.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • remit/vouchers/views.py

    rb1a259c rbef7191  
    355355
    356356@login_required
    357 def show_requests(request, ):
     357def show_requests(http_request, ):
    358358    # PERMISSION-BASED REQUEST FILTERING
    359     if request.user.has_perm('vouchers.can_list'):
     359    if http_request.user.has_perm('vouchers.can_list'):
    360360        qs = ReimbursementRequest.objects.all()
    361361        useronly = False
    362362    else:
    363         qs = ReimbursementRequest.objects.filter(get_related_requests_qobj(request.user))
     363        qs = ReimbursementRequest.objects.filter(get_related_requests_qobj(http_request.user))
    364364        useronly = True
    365365
    366366    # SORTING
    367     if 'order' in request.REQUEST:
    368         order_row = [row for row in request_list_orders if row[0] == request.REQUEST['order']]
     367    if 'order' in http_request.REQUEST:
     368        order_row = [row for row in request_list_orders if row[0] == http_request.REQUEST['order']]
    369369        if order_row:
    370370            order, label, cols = order_row[0]
     
    376376
    377377    # DISCRETIONARY REQUEST FILTERING
    378     if 'approval_status' in request.REQUEST:
    379         approval_status = request.REQUEST['approval_status']
     378    if 'approval_status' in http_request.REQUEST:
     379        approval_status = http_request.REQUEST['approval_status']
    380380    else:
    381381        approval_status = vouchers.models.APPROVAL_STATE_PENDING
     
    395395    # GENERATE THE REQUEST
    396396    return list_detail.object_list(
    397         request,
     397        http_request,
    398398        queryset=qs,
    399399        extra_context={
Note: See TracChangeset for help on using the changeset viewer.