From f0d39bb27b0520c9e1c9d9ec669f9a71180a37a8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E4=BE=9D=E7=91=AA=E8=B2=93?= <imacat@mail.imacat.idv.tw>
Date: Mon, 27 Feb 2023 18:59:42 +0800
Subject: [PATCH] Added the action button to convert a cash income or cash
 expense transaction to a transfer transaction.

---
 .../accounting/transaction/expense/detail.html    |  7 +++++++
 .../accounting/transaction/include/detail.html    |  1 +
 .../accounting/transaction/income/detail.html     |  7 +++++++
 src/accounting/transaction/template.py            | 15 +++++++++++++++
 src/accounting/transaction/views.py               |  6 ++++--
 5 files changed, 34 insertions(+), 2 deletions(-)

diff --git a/src/accounting/templates/accounting/transaction/expense/detail.html b/src/accounting/templates/accounting/transaction/expense/detail.html
index c172050..8ee3a8d 100644
--- a/src/accounting/templates/accounting/transaction/expense/detail.html
+++ b/src/accounting/templates/accounting/transaction/expense/detail.html
@@ -21,6 +21,13 @@ First written: 2023/2/26
 #}
 {% extends "accounting/transaction/include/detail.html" %}
 
+{% block to_transfer %}
+  <a class="btn btn-primary" href="{{ url_for("accounting.transaction.edit", txn=obj)|accounting_txn_to_transfer|accounting_inherit_next }}">
+    <i class="fa-solid fa-bars-staggered"></i>
+    {{ A_("To Transfer") }}
+  </a>
+{% endblock %}
+
 {% block transaction_currencies %}
   {% for currency in obj.currencies %}
     <div class="mb-3">
diff --git a/src/accounting/templates/accounting/transaction/include/detail.html b/src/accounting/templates/accounting/transaction/include/detail.html
index b5da3ea..55334df 100644
--- a/src/accounting/templates/accounting/transaction/include/detail.html
+++ b/src/accounting/templates/accounting/transaction/include/detail.html
@@ -41,6 +41,7 @@ First written: 2023/2/26
     {{ A_("Order") }}
   </a>
   {% if accounting_can_edit() %}
+    {% block to_transfer %}{% endblock %}
     <button class="btn btn-danger" type="button" data-bs-toggle="modal" data-bs-target="#accounting-delete-modal">
       <i class="fa-solid fa-trash"></i>
       {{ A_("Delete") }}
diff --git a/src/accounting/templates/accounting/transaction/income/detail.html b/src/accounting/templates/accounting/transaction/income/detail.html
index ae1c8cb..319821d 100644
--- a/src/accounting/templates/accounting/transaction/income/detail.html
+++ b/src/accounting/templates/accounting/transaction/income/detail.html
@@ -21,6 +21,13 @@ First written: 2023/2/26
 #}
 {% extends "accounting/transaction/include/detail.html" %}
 
+{% block to_transfer %}
+  <a class="btn btn-primary" href="{{ url_for("accounting.transaction.edit", txn=obj)|accounting_txn_to_transfer|accounting_inherit_next }}">
+    <i class="fa-solid fa-bars-staggered"></i>
+    {{ A_("To Transfer") }}
+  </a>
+{% endblock %}
+
 {% block transaction_currencies %}
   {% for currency in obj.currencies %}
     <div class="mb-3">
diff --git a/src/accounting/transaction/template.py b/src/accounting/transaction/template.py
index 505162d..42891c5 100644
--- a/src/accounting/transaction/template.py
+++ b/src/accounting/transaction/template.py
@@ -47,6 +47,21 @@ def with_type(uri: str) -> str:
     return urlunparse(parts)
 
 
+def to_transfer(uri: str) -> str:
+    """Adds the transfer transaction type to the URI.
+
+    :param uri: The URI.
+    :return: The result URL, with the transfer transaction type added.
+    """
+    uri_p: ParseResult = urlparse(uri)
+    params: list[tuple[str, str]] = parse_qsl(uri_p.query)
+    params = [x for x in params if x[0] != "as"]
+    params.append(("as", "transfer"))
+    parts: list[str] = list(uri_p)
+    parts[4] = urlencode(params)
+    return urlunparse(parts)
+
+
 def format_amount(value: Decimal | None) -> str:
     """Formats an amount for readability.
 
diff --git a/src/accounting/transaction/views.py b/src/accounting/transaction/views.py
index 377f44c..bda83cc 100644
--- a/src/accounting/transaction/views.py
+++ b/src/accounting/transaction/views.py
@@ -36,12 +36,14 @@ from accounting.utils.user import get_current_user_pk
 from .dispatcher import TransactionType, get_txn_type, TXN_TYPE_OBJ
 from .forms import sort_transactions_in, TransactionReorderForm
 from .query import get_transaction_query
-from .template import with_type, format_amount, format_amount_input, \
-    format_date, text2html, currency_options, default_currency_code
+from .template import with_type, to_transfer, format_amount, \
+    format_amount_input, format_date, text2html, currency_options, \
+    default_currency_code
 
 bp: Blueprint = Blueprint("transaction", __name__)
 """The view blueprint for the transaction management."""
 bp.add_app_template_filter(with_type, "accounting_txn_with_type")
+bp.add_app_template_filter(to_transfer, "accounting_txn_to_transfer")
 bp.add_app_template_filter(format_amount, "accounting_txn_format_amount")
 bp.add_app_template_filter(format_amount_input,
                            "accounting_txn_format_amount_input")