Replaced the get_absolute_url() method in the account data model with the get_success_url() method in the account form model, and added the current application to the arguments to enable multiple instances of the application in the accounting application.

This commit is contained in:
依瑪貓 2020-08-17 22:27:54 +08:00
parent 64769440a7
commit b3e18d547d
2 changed files with 5 additions and 6 deletions

View File

@ -18,14 +18,12 @@
"""The data models of the accounting application.
"""
import datetime
from typing import Dict, List, Optional
from dirtyfields import DirtyFieldsMixin
from django.conf import settings
from django.db import models, transaction
from django.db.models import Q, Max
from django.urls import reverse
from mia_core.utils import get_multi_lingual_attr, set_multi_lingual_attr, \
new_pk
@ -83,10 +81,6 @@ class Account(DirtyFieldsMixin, models.Model):
super().save(force_insert=force_insert, force_update=force_update,
using=using, update_fields=update_fields)
def get_absolute_url(self):
"""REturns the """
return reverse("accounting:accounts.detail", args=(self,))
class Meta:
db_table = "accounting_accounts"

View File

@ -979,6 +979,11 @@ class AccountFormView(FormView):
"""Returns the current object, or None on a create form."""
return self.kwargs.get("account")
def get_success_url(self) -> str:
"""Returns the URL on success."""
return reverse("accounting:accounts.detail", args=(self.get_object(),),
current_app=self.request.resolver_match.namespace)
@require_POST
@login_required