Skip to content

Commit

Permalink
Fix #193 & some more bugfixes
Browse files Browse the repository at this point in the history
The link @Herkarl sent here:
#192 (review)
was correct and the code snippet was almost correct
  • Loading branch information
foodelevator committed Jul 15, 2024
1 parent a7f81e1 commit 07a879d
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 5 deletions.
7 changes: 2 additions & 5 deletions invoices/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
from django.http import Http404, HttpResponseRedirect, HttpResponseBadRequest, HttpResponseForbidden, HttpResponseServerError, JsonResponse
from django.views.decorators.http import require_http_methods, require_GET, require_POST
from django.contrib.auth.decorators import login_required, user_passes_test
from django.contrib import messages
from datetime import date, datetime
from django.urls import reverse
from django.core.exceptions import ObjectDoesNotExist
Expand Down Expand Up @@ -33,11 +34,7 @@ def new_invoice(request):
messages.error(request, 'Du har angivit en icke-positiv summa i någon av fakturadelarna')
valid = False

if len(request.POST.getlist('amounts[]')) != len(request.POST.getlist('budgetLine[]')):
messages.error(request, 'Sluta fippla')
valid = False

if len(request.POST.getlist('budgetLine[]')) == 0:
if len(request.POST.getlist('budgetLines[]')) == 0:
messages.error(request, 'Du måste lägga till minst en del på kvittot')
valid = False

Expand Down
8 changes: 8 additions & 0 deletions templates/invoices/new.html
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,14 @@

{% block full_content %}
<div id="form">
{% if messages %}
<ul class="messages" style="margin: 20px 0 0;">
{% for message in messages %}
<li{% if message.tags %} class="{{ message.tags }}"{% endif %}>{{ message }}</li>
{% endfor %}
</ul>
{% endif %}

<form method="POST" action="" class="form-horizontal" enctype="multipart/form-data">
<p>Ladda endast upp en faktura i taget.</p>
<div class="form">
Expand Down

0 comments on commit 07a879d

Please sign in to comment.