X-Git-Url: https://git.mdrn.pl/wolnelektury.git/blobdiff_plain/d96e1d2348113e3fc0c7961d83d278933d090a6d..297e48d3c1b26b83366b27cc4ba960c8471b634b:/src/pz/admin.py diff --git a/src/pz/admin.py b/src/pz/admin.py index 25c700b95..1b8838315 100644 --- a/src/pz/admin.py +++ b/src/pz/admin.py @@ -1,5 +1,7 @@ from django.contrib import admin +from django.contrib.admin.filters import FieldListFilter, SimpleListFilter from django.contrib import messages +from django.db.models import Q from django.shortcuts import get_object_or_404, redirect from django.urls import path, reverse from django.utils.safestring import mark_safe @@ -14,16 +16,75 @@ admin.site.register(models.Fundraiser) admin.site.register(models.Campaign) +# Backport from Django 3.1. +class EmptyFieldListFilter(FieldListFilter): + with_empty_str = False + + def __init__(self, field, request, params, model, model_admin, field_path): + self.lookup_kwarg = '%s__isempty' % field_path + self.lookup_val = params.get(self.lookup_kwarg) + super().__init__(field, request, params, model, model_admin, field_path) + + def queryset(self, request, queryset): + if self.lookup_kwarg not in self.used_parameters: + return queryset + if self.lookup_val not in ('0', '1'): + raise IncorrectLookupParameters + + lookup_condition = Q(**{'%s__isnull' % self.field_path: True}) + if self.with_empty_str: + lookup_condition |= Q(**{self.field_path: ''}) + if self.lookup_val == '1': + return queryset.filter(lookup_condition) + return queryset.exclude(lookup_condition) + + def expected_parameters(self): + return [self.lookup_kwarg] + + def choices(self, changelist): + for lookup, title in ( + (None, _('All')), + ('1', _('Empty')), + ('0', _('Not empty')), + ): + yield { + 'selected': self.lookup_val == lookup, + 'query_string': changelist.get_query_string({self.lookup_kwarg: lookup}), + 'display': title, + } + + +class PayedListFilter(SimpleListFilter): + title = 'pobrane' + parameter_name = 'payed' + def lookups(self, request, model_admin): + return ( + ('yes', 'tak'), + ('no', 'nie'), + ) + + def queryset(self, request, queryset): + if self.value() == 'yes': + return queryset.filter(payment__is_dd=True, payment__realised=True).distinct() + if self.value() == 'no': + return queryset.exclude(payment__is_dd=True, payment__realised=True).distinct() + + + class BankExportFeedbackLineInline(admin.TabularInline): model = models.BankExportFeedbackLine extra = 0 +class BankPaymentInline(admin.TabularInline): + model = models.Payment + extra = 0 @admin.register(models.DirectDebit) class DirectDebitAdmin(admin.ModelAdmin): list_display = [ 'payment_id', 'acquisition_date', 'iban_valid', + 'latest_status', 'bank_submission_date', 'bank_acceptance_date', 'amount', 'first_name', 'last_name', @@ -41,12 +102,16 @@ class DirectDebitAdmin(admin.ModelAdmin): 'agree_newsletter', 'fundraiser', 'campaign', - 'is_cancelled', + ('cancelled_at', EmptyFieldListFilter), + 'latest_status', 'needs_redo', 'optout', 'amount', 'sex', 'is_consumer', + ('fundraiser_commission', EmptyFieldListFilter), + ('fundraiser_bonus', EmptyFieldListFilter), + PayedListFilter, ] fieldsets = [ (None, { @@ -58,7 +123,7 @@ class DirectDebitAdmin(admin.ModelAdmin): ('postal_code', 'phone'), 'email', ('iban', 'iban_valid', 'iban_warning'), - 'payment_id', + ('payment_id', 'latest_status'), 'agree_contact', 'agree_fundraising', 'agree_newsletter', @@ -69,9 +134,10 @@ class DirectDebitAdmin(admin.ModelAdmin): ] }), (_('Processing'), {"fields": [ - ('is_cancelled', 'needs_redo', 'optout'), + ('cancelled_at', 'needs_redo', 'optout'), 'submission_date', 'fundraiser_commission', + 'fundraiser_bonus', 'fundraiser_bill', 'bank_submission_date', 'bank_acceptance_date', @@ -79,8 +145,8 @@ class DirectDebitAdmin(admin.ModelAdmin): ] }) ] - readonly_fields = ['agree_contact', 'iban_valid', 'iban_warning'] - inlines = [BankExportFeedbackLineInline] + readonly_fields = ['agree_contact', 'iban_valid', 'iban_warning', 'latest_status'] + inlines = [BankExportFeedbackLineInline, BankPaymentInline] def set_bank_submission(m, r, q): q.update(bank_submission_date=now()) @@ -114,7 +180,7 @@ class DirectDebitAdmin(admin.ModelAdmin): @admin.register(models.BankExportFeedback) class BankExportFeedbackAdmin(admin.ModelAdmin): - inlines = [BankExportFeedbackLineInline] + inlines = [BankExportFeedbackLineInline, BankPaymentInline] @@ -152,7 +218,11 @@ class BankOrderAdmin(admin.ModelAdmin): order = get_object_or_404( models.BankOrder, pk=pk) try: - return bank.bank_order(order.payment_date, order.debits.all()) + return bank.bank_order( + order.payment_date, + order.sent, + order.debits.all() + ) except Exception as e: messages.error(request, str(e)) return redirect('admin:pz_bankorder_change', pk)