**Refactored StagedTransactionModel logic and template conditions:**

- Simplified queryset filters and refactored helpers like `can_unbundle` and `ready_to_import` for better readability.
- Replaced `children_count` check with `children_mapping_done` to streamline logic.
- Updated template conditional to check for `receipt_uuid` instead of `has_receipt`.
- Added commented-out logic for potential future conditions in `data_import_job_txs_table.html`.

### **Summary**
Refactored transaction evaluation logic and templates for clarity and maintainability. Improved existing checks and prepared room for future enhancements in transaction handling.

Author: elarroba

django_ledger/models/data_import.py

@@ -643,7 +643,7 @@ def get_queryset(self) -> StagedTransactionModelQuerySet:
                                 & Q(receipt_type__isnull=True)
                                 & Q(vendor_model__isnull=True)
                                 & Q(customer_model__isnull=True)
-                                & Q(children_count=F('children_mapped_count'))
+                                & Q(children_mapping_done=True)
                                 & Q(total_amount_split__exact=F('amount'))
                                 & Q(transaction_model__isnull=True)
                             )
@@ -1318,9 +1318,7 @@ def has_mapped_receipt(self) -> bool:
         return False
 
     def can_unbundle(self) -> bool:
-        if any([
-            not self.is_single()
-        ]):
+        if any([not self.is_single()]):
             return True
         return False
 
@@ -1341,7 +1339,8 @@ def can_split(self) -> bool:
             [
                 self.is_single(),
                 self.is_parent_is_bundled_has_receipt(),
-                self.is_parent_is_bundled_no_receipt()
+                self.is_parent_is_bundled_no_receipt(),
+                # self.is_parent_not_bundled_no_receipt(),
             ]
         ):
             return True
@@ -1452,13 +1451,15 @@ def can_migrate(self, as_split: bool = False) -> bool:
         """
         ready_to_import = getattr(self, 'ready_to_import')
 
-        if not ready_to_import:
-            return False
-
         if ready_to_import:
             is_role_valid = self.is_role_mapping_valid(raise_exception=False)
-            if is_role_valid:
-                return True
+            if self.is_bundled():
+                return is_role_valid
+            else:
+                if all([self.has_children(), self.are_all_children_mapped()]):
+                    return True
+        else:
+            return False
 
         can_split_into_je = getattr(self, 'can_split_into_je')
         if can_split_into_je and as_split:
@@ -1472,10 +1473,7 @@ def can_migrate_receipt(self) -> bool:
             if ready_to_import:
                 if self.is_transfer():
                     return True
-                if any([
-                    self.is_single_has_receipt(),
-                    self.is_parent_is_bundled_has_receipt()
-                ]):
+                if any([self.is_single_has_receipt(), self.is_parent_is_bundled_has_receipt()]):
                     return True
         return False
 
@@ -1619,11 +1617,14 @@ def get_prospect_je_activity_try(
             The journal entry activity if successfully retrieved or updated; otherwise,
             returns the existing activity or None if no activity is present.
         """
-        if (
-            force_update
-            or all([self.is_children() and self.parent.has_activity(), not self.is_bundled(), not self.has_activity()])
-            or all([not self.has_activity(), getattr(self, 'ready_to_import')])
-        ):
+        if any([
+            force_update,
+            self.is_single(),
+            self.is_parent_is_bundled_no_receipt(),
+            self.is_parent_is_bundled_has_receipt(),
+            self.is_child_not_bundled_has_receipt(),
+            self.is_child_not_bundled_no_receipt(),
+        ]):
             role_set = self.get_import_role_set()
             if role_set is not None:
                 try:

django_ledger/templates/django_ledger/data_import/tags/data_import_job_txs_imported.html

@@ -65,7 +65,7 @@
                                 <div class="dropdown-content">
                                     <a href="{% url 'django_ledger:je-detail' entity_slug=import_job_model.entity_slug ledger_pk=imported_tx.transaction_model.journal_entry.ledger_id je_pk=imported_tx.transaction_model.journal_entry_id %}"
                                        class="dropdown-item">{% trans 'View JE' %}</a>
-                                    {% if imported_tx.has_receipt %}
+                                    {% if imported_tx.receipt_uuid %}
                                         <a href="{% url 'django_ledger:receipt-detail' entity_slug=import_job_model.entity_slug receipt_pk=imported_tx.receiptmodel.uuid %}"
                                            class="dropdown-item">{% trans 'View Receipt' %}</a>
                                     {% endif %}

django_ledger/templates/django_ledger/data_import/tags/data_import_job_txs_table.html

@@ -80,6 +80,7 @@
                         {{ txf.vendor_model }}
 {#                        {{ txf.instance.can_migrate }}#}
 {#                        {{ txf.instance.ready_to_import }}#}
+{#                        {{ txf.instance.can_have_activity }}#}
                     </td>
                     <td class="has-text-centered">{{ txf.tx_import }}</td>
                     <td class="has-text-centered">{{ txf.bundle_split }}</td>