删除时信用卡验证表单错误

Question

我不确定错误来自哪里，如果模型，表单或视图，很可能是表单。我让编辑表单正常工作，也添加了一张新的信用卡，但是当我尝试删除信用卡对象时，出现此错误：

Traceback:
File "/Users/Arturo/anaconda3/envs/SoftEngVEnv/lib/python3.7/site-packages/django/core/handlers/exception.py" in inner
  34.             response = get_response(request)
File "/Users/Arturo/anaconda3/envs/SoftEngVEnv/lib/python3.7/site-packages/django/core/handlers/base.py" in _get_response
  126.                 response = self.process_exception_by_middleware(e, request)
File "/Users/Arturo/anaconda3/envs/SoftEngVEnv/lib/python3.7/site-packages/django/core/handlers/base.py" in _get_response
  124.                 response = wrapped_callback(request, *callback_args, **callback_kwargs)
File "/Users/Arturo/anaconda3/envs/SoftEngVEnv/lib/python3.7/site-packages/django/contrib/auth/decorators.py" in _wrapped_view
  21.                 return view_func(request, *args, **kwargs)
File "/Users/Arturo/Documents/Arturo/Knowledge_center/Computer_Science/Projects/GitHub/Software Engineering 1/softwareEngineering_Group2/bookStore/users/views.py" in creditCardChange
  185.         if user_CreditCardForm.is_valid():
File "/Users/Arturo/anaconda3/envs/SoftEngVEnv/lib/python3.7/site-packages/django/forms/forms.py" in is_valid
  185.         return self.is_bound and not self.errors
File "/Users/Arturo/anaconda3/envs/SoftEngVEnv/lib/python3.7/site-packages/django/forms/forms.py" in errors
  180.             self.full_clean()
File "/Users/Arturo/anaconda3/envs/SoftEngVEnv/lib/python3.7/site-packages/django/forms/forms.py" in full_clean
  382.         self._clean_form()
File "/Users/Arturo/anaconda3/envs/SoftEngVEnv/lib/python3.7/site-packages/django/forms/forms.py" in _clean_form
  409.             cleaned_data = self.clean()
File "/Users/Arturo/Documents/Arturo/Knowledge_center/Computer_Science/Projects/GitHub/Software Engineering 1/softwareEngineering_Group2/bookStore/users/forms.py" in clean
  121.         number = self.cleaned_data['number']
Exception Type: KeyError at /settings/billing/creditcard/6/
Exception Value: 'number'

这是我的 forms.py

class CreditCardForm(ModelForm):
    def __init__(self, *args, **kwargs):
        super(CreditCardForm, self).__init__(*args, **kwargs)
        self.fields['name'].widget.attrs['placeholder'] = 'My Discover'
        self.fields['number'] = forms.CharField(widget=forms.TextInput(
            attrs={'id': 'creditcard-number'}))
        self.fields['number'].widget.attrs[
            'placeholder'] = '123456789'
        self.fields['expdate_month'] = ChoiceField(choices=MONTHS)
        self.fields['expdate_year'] = ChoiceField(choices=YEARS)
        self.fields['securitycode'].widget.attrs[
            'placeholder'] = '123'
    class Meta:
        model = CreditCard
        fields = [
            'name', 'number', 'expdate_month', 'expdate_year', 'securitycode'
        ]
    def clean(self):
        # errors
        self.error_messages = []
        # Card number block
        number = self.cleaned_data['number']
        visa_pattern = r'^4[0-9]{12}(?:[0-9]{3})?$'
        mastercard_pattern = r'^(?:5[1-5][0-9]{2}|222[1-9]|22[3-9][0-9]|2[3-6][0-9]{2}|27[01][0-9]|2720)[0-9]{12}$'
        americanexpress_pattern = r'^3[47][0-9]{13}$'
        discover_pattern = r'^6(?:011|5[0-9]{2})[0-9]{12}$'
        patterns_list = [
            discover_pattern,
            visa_pattern,
            mastercard_pattern,
        ]
        pattern_string = '|'.join(patterns_list)
        pattern1 = re.compile(pattern_string)  # 3 digits of security code
        pattern2 = re.compile(
            americanexpress_pattern)  # four digits security code
        if pattern1.match(str(number)):
            security_code_pattern = re.compile(r'^[0-9]{3}$')  # 3
        elif pattern2.match(str(number)):
            security_code_pattern = re.compile(r'^[0-9]{4}$')  # 4
        else:
            security_code_pattern = None
        if not pattern1.match(str(number)) and not pattern2.match(str(number)):
            self.error_messages.append('Credit card number not valid')
            # self._errors['number'] = 'Please enter a valid credit card number'
        # Expiration date block
        month = int(self.cleaned_data['expdate_month'])
        year = int(self.cleaned_data['expdate_year'])
        expdate = datetime.datetime(year, month, 1)  # first day of the month
        today = datetime.datetime.today()
        if expdate < today:
            self.error_messages.append('Card has expired')
        # Security code block
        security_code = self.cleaned_data['securitycode']
        # if not security_code_pattern created or does not match the work the errors
        if not security_code_pattern or not security_code_pattern.match(
                str(security_code)):
            self.error_messages.append('Invalid security code')
            # self._errors[
            #     'securitycode'] = 'Please verify the credit card security code'
        self.error_message = ''
        if len(self.error_messages):
            self.error_message = ' & '.join(self.error_messages)
            raise forms.ValidationError(' & '.join(self.error_messages))
        return self.cleaned_data

class DeleteCreditCardConfirmation(forms.Form):
    pass

views.py

@login_required
def creditCardChange(request, creditcard_slug):
    # Gets name of the credit card based on id
    currentCreditCard = CreditCard.objects.all().get(pk=creditcard_slug)
    if request.method == 'POST':
        user_CreditCardForm = CreditCardForm(request.POST, instance=currentCreditCard)
        if user_CreditCardForm.is_valid():
            user_CreditCardForm.save()
            messages.success(request, f'Your address has been updated successfully')
            return HttpResponseRedirect(request.path_info)
        elif DeleteCreditCardConfirmation():
            currentCreditCard.delete()
            return redirect('settings:billing-settings')
        else:
            messages.warning(
                request, f'There were some errors updating you credit card.')
    else:
        user_CreditCardForm = CreditCardForm(instance=currentCreditCard)
    context = {
        'creditcard_slug': creditcard_slug,
        'user_CreditCardForm': user_CreditCardForm,
    }
    return render(request, 'users/creditCardChange.html', context)

.html

   <form method="POST" enctype="multipart/form-data">
            {% csrf_token %}
            {{user_CreditCardForm |crispy}}
            <div class="form-group">
                <button class="btn btn-outline-info" type="submit">Save changes</button>
            </div>
        </form>
        <form method="POST" enctype="multipart/form-data">
            {% csrf_token %}
            {{DeleteCreditCardConfirmation |crispy}}
            <div class="form-group">
                <button class="btn btn-danger" type="submit">Delete</button>
            </div>
        </form>

它告诉我我有一行错误，上面写着：number = self.cleaned_data['number'].如何修复此错误？仅当我删除信用卡对象时，才会发生这种情况。

谢谢

Answer 1

您可以通过向

按钮添加 name 属性来尝试了解提交的表单：

<button class="btn btn-outline-info" type="submit" name="save_changes">Save changes</button>
...
<button class="btn btn-danger" type="submit" name="delete">Delete</button>

那么在您看来：

if request.method == 'POST': 
    if 'save_changes' in request.POST: # handle editing form
        user_CreditCardForm = CreditCardForm(request.POST, instance=currentCreditCard)
        if user_CreditCardForm.is_valid():
        ...
    elif 'delete' in request.POST: # handle deleting
        ...

这样，当您删除时，您将无法if user_CreditCardForm.is_valid():导致问题的原因。