Compare commits
10 Commits
25f033d0c8
...
fusion_acc
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
6e53955e9c | ||
|
|
8dab9b36da | ||
|
|
14e59148c6 | ||
|
|
55eb368195 | ||
|
|
d623b67157 | ||
|
|
aaaf49989c | ||
|
|
878c013902 | ||
|
|
ffc029a875 | ||
|
|
6048df0645 | ||
|
|
b6aedc9bbe |
@@ -1,6 +1,6 @@
|
|||||||
{
|
{
|
||||||
'name': 'Fusion Accounting',
|
'name': 'Fusion Accounting',
|
||||||
'version': '19.0.1.0.0',
|
'version': '19.0.1.0.1',
|
||||||
'category': 'Accounting/Accounting',
|
'category': 'Accounting/Accounting',
|
||||||
'sequence': 25,
|
'sequence': 25,
|
||||||
'summary': 'Meta-module that installs the full Fusion Accounting suite (core, AI, migration; bank rec, reports, etc. as later sub-modules ship).',
|
'summary': 'Meta-module that installs the full Fusion Accounting suite (core, AI, migration; bank rec, reports, etc. as later sub-modules ship).',
|
||||||
@@ -13,9 +13,9 @@ Currently installs:
|
|||||||
- fusion_accounting_core Shared schema, security, runtime helpers
|
- fusion_accounting_core Shared schema, security, runtime helpers
|
||||||
- fusion_accounting_ai AI Co-Pilot (Claude/GPT)
|
- fusion_accounting_ai AI Co-Pilot (Claude/GPT)
|
||||||
- fusion_accounting_migration Transitional Enterprise->Fusion data migration
|
- fusion_accounting_migration Transitional Enterprise->Fusion data migration
|
||||||
|
- fusion_accounting_bank_rec AI-assisted bank reconciliation (Phase 1)
|
||||||
|
|
||||||
Future sub-modules (added per the roadmap as each Phase ships):
|
Future sub-modules (added per the roadmap as each Phase ships):
|
||||||
- fusion_accounting_bank_rec (Phase 1)
|
|
||||||
- fusion_accounting_reports (Phase 2)
|
- fusion_accounting_reports (Phase 2)
|
||||||
- fusion_accounting_dashboard (Phase 3)
|
- fusion_accounting_dashboard (Phase 3)
|
||||||
- fusion_accounting_followup (Phase 5)
|
- fusion_accounting_followup (Phase 5)
|
||||||
@@ -33,6 +33,7 @@ Built by Nexa Systems Inc.
|
|||||||
'fusion_accounting_core',
|
'fusion_accounting_core',
|
||||||
'fusion_accounting_ai',
|
'fusion_accounting_ai',
|
||||||
'fusion_accounting_migration',
|
'fusion_accounting_migration',
|
||||||
|
'fusion_accounting_bank_rec',
|
||||||
],
|
],
|
||||||
'data': [],
|
'data': [],
|
||||||
'installable': True,
|
'installable': True,
|
||||||
|
|||||||
103
fusion_accounting_bank_rec/CLAUDE.md
Normal file
103
fusion_accounting_bank_rec/CLAUDE.md
Normal file
@@ -0,0 +1,103 @@
|
|||||||
|
# fusion_accounting_bank_rec — Cursor / Claude Context
|
||||||
|
|
||||||
|
## Purpose
|
||||||
|
|
||||||
|
Replaces (or augments — coexists with) Odoo Enterprise's `account_accountant`
|
||||||
|
bank reconciliation widget with a Fusion-native, AI-assistive implementation.
|
||||||
|
Ships in Phase 1 of the fusion_accounting roadmap.
|
||||||
|
|
||||||
|
## Architecture
|
||||||
|
|
||||||
|
Hybrid: the engine (`fusion.reconcile.engine`, AbstractModel) is the SINGLE
|
||||||
|
write surface for reconciliations. Everything else (controller, OWL widget,
|
||||||
|
AI tools, wizards, cron) routes through the engine's 6-method API:
|
||||||
|
|
||||||
|
- `reconcile_one(line, against_lines, write_off_vals=None)`
|
||||||
|
- `reconcile_batch(lines, strategy='auto')`
|
||||||
|
- `suggest_matches(lines, limit_per_line=3)`
|
||||||
|
- `accept_suggestion(suggestion)`
|
||||||
|
- `write_off(line, account, amount, label, tax_id=None)`
|
||||||
|
- `unreconcile(partial_reconciles)`
|
||||||
|
|
||||||
|
Pure-Python services live in `services/`:
|
||||||
|
- `memo_tokenizer` — Canadian bank memo regex
|
||||||
|
- `exchange_diff` — FX gain/loss pre-compute
|
||||||
|
- `matching_strategies` — AmountExact, FIFO, MultiInvoice
|
||||||
|
- `precedent_lookup` — K-nearest search
|
||||||
|
- `pattern_extractor` — per-partner aggregate
|
||||||
|
- `confidence_scoring` — 4-pass pipeline (statistical → AI re-rank)
|
||||||
|
- `precedent_backfill` — migration helper
|
||||||
|
|
||||||
|
Persistent models in `models/`:
|
||||||
|
- `fusion.reconcile.pattern` — per-(company, partner) learned profile
|
||||||
|
- `fusion.reconcile.precedent` — per-decision history
|
||||||
|
- `fusion.reconcile.suggestion` — AI suggestions with state lifecycle
|
||||||
|
- `fusion.bank.rec.widget` — TransientModel for OWL round-trip
|
||||||
|
- `fusion.unreconciled.bank.line.mv` — pre-aggregated MV for fast UI listing
|
||||||
|
- `fusion.bank.rec.cron` — cron handler (suggest, pattern refresh, MV refresh)
|
||||||
|
- `fusion.auto.reconcile.wizard` / `fusion.bulk.reconcile.wizard` — TransientModel wizards
|
||||||
|
- `fusion.migration.wizard` (inherits) — adds `_bank_rec_bootstrap_step`
|
||||||
|
- `account.bank.statement.line` (inherits) — adds fusion_top_suggestion_id, fusion_confidence_band, etc.
|
||||||
|
- `account.reconcile.model` (inherits) — adds fusion_ai_confidence_threshold
|
||||||
|
|
||||||
|
Controller: `controllers/bank_rec_controller.py` exposes 10 JSON-RPC endpoints
|
||||||
|
under `/fusion/bank_rec/*`. All calls route through the engine.
|
||||||
|
|
||||||
|
OWL frontend: `static/src/`
|
||||||
|
- `services/bank_reconciliation_service.js` — central reactive state + RPC wrappers
|
||||||
|
- `views/kanban/bank_rec_kanban_*.js` — top-level controller + renderer
|
||||||
|
- `components/bank_reconciliation/<...>` — 14 mirrored Enterprise components + 8 fusion-only components (ai_suggestion folder, batch_action_bar, reconcile_model_picker, attachment_strip, partner_history_panel)
|
||||||
|
- `tours/bank_rec_tours.js` — 5 OWL tour smoke tests
|
||||||
|
|
||||||
|
## Conventions
|
||||||
|
|
||||||
|
- **V19 deprecations to avoid:** `_sql_constraints` (use `models.Constraint`),
|
||||||
|
`@api.depends('id')` (raises `NotImplementedError`), `@route(type='json')`
|
||||||
|
(use `type='jsonrpc'`), `numbercall` field on `ir.cron` (removed),
|
||||||
|
`groups_id` on `res.users` (use `all_group_ids` for searching),
|
||||||
|
`users` field on `res.groups` (use `user_ids`), `groups_id` on
|
||||||
|
`ir.ui.menu` (use `group_ids`).
|
||||||
|
|
||||||
|
- **Coexistence:** When `account_accountant` is installed, the fusion menu
|
||||||
|
is hidden via `fusion_accounting_core.group_fusion_show_when_enterprise_absent`
|
||||||
|
(a computed group). Engine model is always available.
|
||||||
|
|
||||||
|
- **Materialized view refresh:** Triggered on `fusion.reconcile.suggestion`
|
||||||
|
create/write (best-effort, non-blocking). Cron refreshes every 5 min via
|
||||||
|
a dedicated autocommit cursor (REFRESH CONCURRENTLY can't run inside
|
||||||
|
Odoo's regular transaction).
|
||||||
|
|
||||||
|
- **Test factories:** `tests/_factories.py` provides `make_bank_journal`,
|
||||||
|
`make_bank_line`, `make_invoice`, `make_reconcileable_pair`, `make_suggestion`,
|
||||||
|
`make_pattern`, `make_precedent`. NOTE: `make_bank_journal` defaults to
|
||||||
|
code `'TEST'` so multiple calls in one test will collide; pass an explicit
|
||||||
|
unique code or share a journal across calls.
|
||||||
|
|
||||||
|
- **Hypothesis property tests:** Use `@settings(suppress_health_check=[...])`
|
||||||
|
to silence function_scoped_fixture warnings in TransactionCase.
|
||||||
|
|
||||||
|
## Test counts (as of Phase 1 complete)
|
||||||
|
|
||||||
|
- 157 logical tests total in fusion_accounting_bank_rec
|
||||||
|
- 0 failures, 0 errors
|
||||||
|
- Includes: 4 benchmark tests (tagged 'benchmark'), 1 local LLM smoke (tagged 'local_llm', skips when no LLM), 5 OWL tour tests (tagged 'tour')
|
||||||
|
|
||||||
|
## Performance baseline
|
||||||
|
|
||||||
|
| Operation | P95 | Budget |
|
||||||
|
|---|---|---|
|
||||||
|
| `engine.suggest_matches` (1 line) | 234ms | <500ms |
|
||||||
|
| `engine.reconcile_batch` (50 lines) | 3318ms | <5000ms |
|
||||||
|
| `controller.list_unreconciled` (50 lines) | 77ms | <200ms |
|
||||||
|
| MV refresh | 60ms | <2000ms |
|
||||||
|
|
||||||
|
All within 1x of budget at Phase 1 ship.
|
||||||
|
|
||||||
|
## Known concerns / Phase 1.5 backlog
|
||||||
|
|
||||||
|
- `accept_suggestion` returns `partial_ids` but not `is_reconciled` — UI reads it post-call
|
||||||
|
- `engine.write_off` mixed mode (write-off + against_lines) implemented but untested
|
||||||
|
- `engine.reconcile_one` returns `exchange_diff_move_id: None` (Odoo's reconcile() handles FX inline; surfacing the move_id needs an extra query)
|
||||||
|
- `against_lines` early-break in `reconcile_one` silently drops excess; auto strategy avoids this but manual callers should pre-validate
|
||||||
|
- Reconcile-model bulk wizard `_apply_lines_for_bank_statement_line` is Enterprise-only (Community falls back to per-line error)
|
||||||
|
- OWL tour tests skip-mode when websocket-client absent
|
||||||
41
fusion_accounting_bank_rec/README.md
Normal file
41
fusion_accounting_bank_rec/README.md
Normal file
@@ -0,0 +1,41 @@
|
|||||||
|
# fusion_accounting_bank_rec
|
||||||
|
|
||||||
|
AI-assisted bank reconciliation for Odoo 19 Community — a Fusion-native
|
||||||
|
replacement for Enterprise's `account_accountant` bank reconciliation widget.
|
||||||
|
|
||||||
|
## What it does
|
||||||
|
|
||||||
|
- Side-by-side parity with Enterprise's bank reconciliation UI (kanban + side
|
||||||
|
panel, multi-currency, write-offs, attachments, chatter)
|
||||||
|
- AI-assistive: confidence-scored suggestions per bank line via the
|
||||||
|
`fusion.reconcile.engine` 4-pass scoring pipeline (statistical + optional
|
||||||
|
LLM re-rank)
|
||||||
|
- Coexists with `account_accountant` (Enterprise wins by default; Fusion menu
|
||||||
|
appears only when Enterprise is uninstalled)
|
||||||
|
- Migration-aware: bootstrap step backfills `fusion.reconcile.precedent` from
|
||||||
|
existing `account.partial.reconcile` rows so the AI has memory from day 1
|
||||||
|
|
||||||
|
## Quick start
|
||||||
|
|
||||||
|
```bash
|
||||||
|
# Install
|
||||||
|
odoo --addons-path=... -i fusion_accounting_bank_rec
|
||||||
|
|
||||||
|
# Open the widget (when Enterprise's account_accountant is NOT installed)
|
||||||
|
# Apps → Bank Reconciliation → Reconcile Bank Lines
|
||||||
|
|
||||||
|
# When Enterprise IS installed: use Enterprise's UI; the engine + AI tools
|
||||||
|
# are still available via the AI chat.
|
||||||
|
```
|
||||||
|
|
||||||
|
## Configuration
|
||||||
|
|
||||||
|
- Local LLM (LM Studio, Ollama):
|
||||||
|
- `fusion_accounting.openai_base_url` = `http://host.docker.internal:1234/v1`
|
||||||
|
- `fusion_accounting.openai_model` = your local model name
|
||||||
|
- `fusion_accounting.provider.bank_rec_suggest` = `openai`
|
||||||
|
|
||||||
|
## See also
|
||||||
|
|
||||||
|
- `CLAUDE.md` — agent context
|
||||||
|
- `UPGRADE_NOTES.md` — Odoo version anchoring
|
||||||
34
fusion_accounting_bank_rec/UPGRADE_NOTES.md
Normal file
34
fusion_accounting_bank_rec/UPGRADE_NOTES.md
Normal file
@@ -0,0 +1,34 @@
|
|||||||
|
# fusion_accounting_bank_rec — Upgrade Notes
|
||||||
|
|
||||||
|
## Odoo Version Anchor
|
||||||
|
|
||||||
|
This module targets **Odoo 19.0** (community-base).
|
||||||
|
|
||||||
|
Reference snapshot of Enterprise code mirrored from:
|
||||||
|
- `account_accountant` (Odoo 19.0.x)
|
||||||
|
- Source: `/Users/gurpreet/Github/RePackaged-Odoo/accounting/account_accountant/`
|
||||||
|
|
||||||
|
## Cross-Version Diff Strategy
|
||||||
|
|
||||||
|
When a new Odoo version ships:
|
||||||
|
|
||||||
|
1. Run `check_odoo_diff.sh` (in repo root) against the new Enterprise version
|
||||||
|
2. Note any breaking changes in `account.bank.statement.line` API
|
||||||
|
3. For mirrored OWL components, diff Enterprise's new versions against ours and
|
||||||
|
port material changes (signature renames, new behaviour we want to inherit)
|
||||||
|
4. Re-run the full test suite + tour tests against the new Odoo version
|
||||||
|
5. Update this file with the new version anchor + any deviations
|
||||||
|
|
||||||
|
## V19 Migration Notes (already applied)
|
||||||
|
|
||||||
|
- `_sql_constraints` → `models.Constraint` (Tasks 14, 15)
|
||||||
|
- `@api.depends('id')` → removed (Task 17)
|
||||||
|
- `@route(type='json')` → `type='jsonrpc'` (Task 26)
|
||||||
|
- `numbercall` removed from `ir.cron` (Task 25)
|
||||||
|
- `res.groups.users` → `user_ids` (Task 43)
|
||||||
|
- `ir.ui.menu.groups_id` → `group_ids` (Tasks 42, 43)
|
||||||
|
|
||||||
|
## Phase 1 → Phase 1.5 Migration
|
||||||
|
|
||||||
|
If we ship Phase 1.5 (UI polish, deferred features), changes will go in
|
||||||
|
incremental commits. No DB migration needed (Phase 1 schema is forward-compatible).
|
||||||
@@ -2,3 +2,4 @@ from . import models
|
|||||||
from . import controllers
|
from . import controllers
|
||||||
from . import services
|
from . import services
|
||||||
from . import wizards
|
from . import wizards
|
||||||
|
from . import reports
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
{
|
{
|
||||||
'name': 'Fusion Accounting — Bank Reconciliation',
|
'name': 'Fusion Accounting — Bank Reconciliation',
|
||||||
'version': '19.0.1.0.20',
|
'version': '19.0.1.0.26',
|
||||||
'category': 'Accounting/Accounting',
|
'category': 'Accounting/Accounting',
|
||||||
'sequence': 28,
|
'sequence': 28,
|
||||||
'summary': 'Native V19 bank reconciliation widget with AI confidence scoring + behavioural learning.',
|
'summary': 'Native V19 bank reconciliation widget with AI confidence scoring + behavioural learning.',
|
||||||
@@ -24,7 +24,7 @@ Built by Nexa Systems Inc.
|
|||||||
'author': 'Nexa Systems Inc.',
|
'author': 'Nexa Systems Inc.',
|
||||||
'website': 'https://nexasystems.ca',
|
'website': 'https://nexasystems.ca',
|
||||||
'maintainer': 'Nexa Systems Inc.',
|
'maintainer': 'Nexa Systems Inc.',
|
||||||
'depends': ['fusion_accounting_core'],
|
'depends': ['fusion_accounting_core', 'fusion_accounting_migration'],
|
||||||
'external_dependencies': {
|
'external_dependencies': {
|
||||||
'python': ['hypothesis'],
|
'python': ['hypothesis'],
|
||||||
},
|
},
|
||||||
@@ -33,6 +33,9 @@ Built by Nexa Systems Inc.
|
|||||||
'data/cron.xml',
|
'data/cron.xml',
|
||||||
'wizards/auto_reconcile_wizard_views.xml',
|
'wizards/auto_reconcile_wizard_views.xml',
|
||||||
'wizards/bulk_reconcile_wizard_views.xml',
|
'wizards/bulk_reconcile_wizard_views.xml',
|
||||||
|
'reports/migration_audit_report_views.xml',
|
||||||
|
'reports/migration_audit_report_action.xml',
|
||||||
|
'views/menu_views.xml',
|
||||||
],
|
],
|
||||||
'assets': {
|
'assets': {
|
||||||
'web.assets_backend': [
|
'web.assets_backend': [
|
||||||
@@ -100,6 +103,9 @@ Built by Nexa Systems Inc.
|
|||||||
'fusion_accounting_bank_rec/static/src/components/bank_reconciliation/partner_history_panel/partner_history_panel.js',
|
'fusion_accounting_bank_rec/static/src/components/bank_reconciliation/partner_history_panel/partner_history_panel.js',
|
||||||
'fusion_accounting_bank_rec/static/src/components/bank_reconciliation/partner_history_panel/partner_history_panel.xml',
|
'fusion_accounting_bank_rec/static/src/components/bank_reconciliation/partner_history_panel/partner_history_panel.xml',
|
||||||
],
|
],
|
||||||
|
'web.assets_tests': [
|
||||||
|
'fusion_accounting_bank_rec/static/src/tours/bank_rec_tours.js',
|
||||||
|
],
|
||||||
},
|
},
|
||||||
'installable': True,
|
'installable': True,
|
||||||
'application': False,
|
'application': False,
|
||||||
|
|||||||
@@ -7,3 +7,4 @@ from . import account_reconcile_model
|
|||||||
from . import fusion_reconcile_engine
|
from . import fusion_reconcile_engine
|
||||||
from . import fusion_unreconciled_bank_line_mv
|
from . import fusion_unreconciled_bank_line_mv
|
||||||
from . import fusion_bank_rec_cron
|
from . import fusion_bank_rec_cron
|
||||||
|
from . import fusion_migration_wizard
|
||||||
|
|||||||
97
fusion_accounting_bank_rec/models/fusion_migration_wizard.py
Normal file
97
fusion_accounting_bank_rec/models/fusion_migration_wizard.py
Normal file
@@ -0,0 +1,97 @@
|
|||||||
|
"""Bank-rec specific migration step.
|
||||||
|
|
||||||
|
Hooks into fusion.migration.wizard (defined by fusion_accounting_migration)
|
||||||
|
to bootstrap fusion.reconcile.precedent from existing
|
||||||
|
account.partial.reconcile rows. This gives the AI immediate "memory" from
|
||||||
|
past Enterprise reconciles so suggestions can be ranked by precedent
|
||||||
|
similarity from day one.
|
||||||
|
|
||||||
|
The bootstrap step is exposed as a public method (_bank_rec_bootstrap_step)
|
||||||
|
so tests and the audit report can invoke it directly. action_run_migration
|
||||||
|
is overridden to call super() then run the bootstrap.
|
||||||
|
"""
|
||||||
|
|
||||||
|
import logging
|
||||||
|
|
||||||
|
from odoo import _, models
|
||||||
|
|
||||||
|
from ..services.precedent_backfill import backfill_precedents
|
||||||
|
|
||||||
|
_logger = logging.getLogger(__name__)
|
||||||
|
|
||||||
|
|
||||||
|
class FusionMigrationWizard(models.TransientModel):
|
||||||
|
_inherit = "fusion.migration.wizard"
|
||||||
|
|
||||||
|
def _bank_rec_bootstrap_step(self):
|
||||||
|
"""Migration step: backfill precedents + refresh patterns + refresh MV.
|
||||||
|
|
||||||
|
Returns a dict describing what happened, suitable for surfacing to
|
||||||
|
the user via notification or PDF audit report.
|
||||||
|
"""
|
||||||
|
self.ensure_one()
|
||||||
|
_logger.info(
|
||||||
|
"fusion_accounting_bank_rec migration step: bootstrap starting")
|
||||||
|
|
||||||
|
company_id = None
|
||||||
|
if 'company_id' in self._fields and self.company_id:
|
||||||
|
company_id = self.company_id.id
|
||||||
|
|
||||||
|
precedent_result = backfill_precedents(
|
||||||
|
self.env, company_id=company_id, limit=10000)
|
||||||
|
|
||||||
|
try:
|
||||||
|
self.env['fusion.bank.rec.cron']._cron_refresh_patterns()
|
||||||
|
patterns_ok = True
|
||||||
|
except Exception as e: # noqa: BLE001
|
||||||
|
_logger.warning(
|
||||||
|
"Pattern refresh during migration failed: %s", e)
|
||||||
|
patterns_ok = False
|
||||||
|
|
||||||
|
try:
|
||||||
|
self.env['fusion.unreconciled.bank.line.mv']._refresh(
|
||||||
|
concurrently=False)
|
||||||
|
mv_ok = True
|
||||||
|
except Exception as e: # noqa: BLE001
|
||||||
|
_logger.warning("MV refresh during migration failed: %s", e)
|
||||||
|
mv_ok = False
|
||||||
|
|
||||||
|
result = {
|
||||||
|
'step': 'bank_rec_bootstrap',
|
||||||
|
'precedents_created': precedent_result['created'],
|
||||||
|
'precedents_skipped': precedent_result['skipped'],
|
||||||
|
'patterns_refreshed': patterns_ok,
|
||||||
|
'mv_refreshed': mv_ok,
|
||||||
|
}
|
||||||
|
_logger.info(
|
||||||
|
"fusion_accounting_bank_rec bootstrap complete: %s", result)
|
||||||
|
return result
|
||||||
|
|
||||||
|
def action_run_migration(self):
|
||||||
|
"""Override the migration entry-point to add the bank-rec step.
|
||||||
|
|
||||||
|
Calls super() (which currently returns a notification stub from
|
||||||
|
Phase 0) and then runs the bank-rec bootstrap. Returns a
|
||||||
|
notification summarizing both.
|
||||||
|
"""
|
||||||
|
_ = super().action_run_migration()
|
||||||
|
result = self._bank_rec_bootstrap_step()
|
||||||
|
return {
|
||||||
|
'type': 'ir.actions.client',
|
||||||
|
'tag': 'display_notification',
|
||||||
|
'params': {
|
||||||
|
'type': 'success',
|
||||||
|
'title': _("Bank-Rec Migration Complete"),
|
||||||
|
'message': _(
|
||||||
|
"Backfilled %(created)d precedents "
|
||||||
|
"(skipped %(skipped)d). "
|
||||||
|
"Patterns refreshed: %(p)s. MV refreshed: %(m)s."
|
||||||
|
) % {
|
||||||
|
'created': result['precedents_created'],
|
||||||
|
'skipped': result['precedents_skipped'],
|
||||||
|
'p': 'yes' if result['patterns_refreshed'] else 'no',
|
||||||
|
'm': 'yes' if result['mv_refreshed'] else 'no',
|
||||||
|
},
|
||||||
|
'sticky': False,
|
||||||
|
},
|
||||||
|
}
|
||||||
@@ -41,6 +41,7 @@ class FusionReconcilePrecedent(models.Model):
|
|||||||
reconciled_at = fields.Datetime()
|
reconciled_at = fields.Datetime()
|
||||||
source = fields.Selection([
|
source = fields.Selection([
|
||||||
('historical_bootstrap', 'Imported from history'),
|
('historical_bootstrap', 'Imported from history'),
|
||||||
|
('backfill', 'Backfilled from account.partial.reconcile (migration)'),
|
||||||
('manual', 'Manual reconcile via fusion'),
|
('manual', 'Manual reconcile via fusion'),
|
||||||
('ai_accepted', 'AI suggestion accepted'),
|
('ai_accepted', 'AI suggestion accepted'),
|
||||||
('auto_rule', 'account.reconcile.model auto-fired'),
|
('auto_rule', 'account.reconcile.model auto-fired'),
|
||||||
|
|||||||
1
fusion_accounting_bank_rec/reports/__init__.py
Normal file
1
fusion_accounting_bank_rec/reports/__init__.py
Normal file
@@ -0,0 +1 @@
|
|||||||
|
from . import migration_audit_report
|
||||||
51
fusion_accounting_bank_rec/reports/migration_audit_report.py
Normal file
51
fusion_accounting_bank_rec/reports/migration_audit_report.py
Normal file
@@ -0,0 +1,51 @@
|
|||||||
|
"""QWeb PDF report: summary of bank-rec migration outcomes.
|
||||||
|
|
||||||
|
Triggered from the migration wizard's "Print" menu after the wizard
|
||||||
|
completes. For each company on the system, reports:
|
||||||
|
- Backfilled precedents (source='backfill')
|
||||||
|
- Fusion reconcile patterns
|
||||||
|
- Bank statement lines still unreconciled
|
||||||
|
|
||||||
|
Lets the operator confirm Phase 1 migration successfully bootstrapped
|
||||||
|
the AI's reconcile memory from past Enterprise reconciles.
|
||||||
|
"""
|
||||||
|
|
||||||
|
from odoo import api, models
|
||||||
|
|
||||||
|
|
||||||
|
class FusionMigrationAuditReport(models.AbstractModel):
|
||||||
|
_name = "report.fusion_accounting_bank_rec.migration_audit_template"
|
||||||
|
_description = "Bank-Rec Migration Audit Report"
|
||||||
|
|
||||||
|
@api.model
|
||||||
|
def _get_report_values(self, docids, data=None):
|
||||||
|
Wizard = self.env['fusion.migration.wizard']
|
||||||
|
wizards = Wizard.browse(docids) if docids else Wizard
|
||||||
|
|
||||||
|
Precedent = self.env['fusion.reconcile.precedent']
|
||||||
|
Pattern = self.env['fusion.reconcile.pattern']
|
||||||
|
Line = self.env['account.bank.statement.line']
|
||||||
|
|
||||||
|
company_stats = []
|
||||||
|
for company in self.env['res.company'].search([]):
|
||||||
|
company_stats.append({
|
||||||
|
'company': company,
|
||||||
|
'precedents_count': Precedent.search_count([
|
||||||
|
('company_id', '=', company.id),
|
||||||
|
('source', '=', 'backfill'),
|
||||||
|
]),
|
||||||
|
'patterns_count': Pattern.search_count([
|
||||||
|
('company_id', '=', company.id),
|
||||||
|
]),
|
||||||
|
'unreconciled_count': Line.search_count([
|
||||||
|
('company_id', '=', company.id),
|
||||||
|
('is_reconciled', '=', False),
|
||||||
|
]),
|
||||||
|
})
|
||||||
|
|
||||||
|
return {
|
||||||
|
'doc_ids': docids,
|
||||||
|
'doc_model': 'fusion.migration.wizard',
|
||||||
|
'docs': wizards,
|
||||||
|
'company_stats': company_stats,
|
||||||
|
}
|
||||||
@@ -0,0 +1,12 @@
|
|||||||
|
<?xml version="1.0" encoding="utf-8"?>
|
||||||
|
<odoo>
|
||||||
|
<record id="action_report_migration_audit" model="ir.actions.report">
|
||||||
|
<field name="name">Bank-Rec Migration Audit</field>
|
||||||
|
<field name="model">fusion.migration.wizard</field>
|
||||||
|
<field name="report_type">qweb-pdf</field>
|
||||||
|
<field name="report_name">fusion_accounting_bank_rec.migration_audit_template</field>
|
||||||
|
<field name="report_file">fusion_accounting_bank_rec.migration_audit_template</field>
|
||||||
|
<field name="binding_model_id" ref="fusion_accounting_migration.model_fusion_migration_wizard"/>
|
||||||
|
<field name="binding_type">report</field>
|
||||||
|
</record>
|
||||||
|
</odoo>
|
||||||
@@ -0,0 +1,42 @@
|
|||||||
|
<?xml version="1.0" encoding="utf-8"?>
|
||||||
|
<odoo>
|
||||||
|
<template id="migration_audit_template">
|
||||||
|
<t t-call="web.html_container">
|
||||||
|
<t t-call="web.external_layout">
|
||||||
|
<div class="page">
|
||||||
|
<h2>Bank-Rec Migration Audit</h2>
|
||||||
|
<p>
|
||||||
|
Generated
|
||||||
|
<span t-esc="context_timestamp(datetime.datetime.now()).strftime('%Y-%m-%d %H:%M')"/>
|
||||||
|
</p>
|
||||||
|
|
||||||
|
<h3>Per-Company Summary</h3>
|
||||||
|
<table class="table table-sm">
|
||||||
|
<thead>
|
||||||
|
<tr>
|
||||||
|
<th>Company</th>
|
||||||
|
<th class="text-end">Backfilled Precedents</th>
|
||||||
|
<th class="text-end">Patterns</th>
|
||||||
|
<th class="text-end">Still Unreconciled</th>
|
||||||
|
</tr>
|
||||||
|
</thead>
|
||||||
|
<tbody>
|
||||||
|
<tr t-foreach="company_stats" t-as="cs">
|
||||||
|
<td><span t-esc="cs['company'].name"/></td>
|
||||||
|
<td class="text-end"><span t-esc="cs['precedents_count']"/></td>
|
||||||
|
<td class="text-end"><span t-esc="cs['patterns_count']"/></td>
|
||||||
|
<td class="text-end"><span t-esc="cs['unreconciled_count']"/></td>
|
||||||
|
</tr>
|
||||||
|
</tbody>
|
||||||
|
</table>
|
||||||
|
|
||||||
|
<p class="text-muted">
|
||||||
|
This report verifies that Phase 1 migration successfully
|
||||||
|
bootstrapped the AI's reconcile memory from past Enterprise
|
||||||
|
reconciles.
|
||||||
|
</p>
|
||||||
|
</div>
|
||||||
|
</t>
|
||||||
|
</t>
|
||||||
|
</template>
|
||||||
|
</odoo>
|
||||||
@@ -4,3 +4,4 @@ from . import matching_strategies
|
|||||||
from . import precedent_lookup
|
from . import precedent_lookup
|
||||||
from . import pattern_extractor
|
from . import pattern_extractor
|
||||||
from . import confidence_scoring
|
from . import confidence_scoring
|
||||||
|
from . import precedent_backfill
|
||||||
|
|||||||
116
fusion_accounting_bank_rec/services/precedent_backfill.py
Normal file
116
fusion_accounting_bank_rec/services/precedent_backfill.py
Normal file
@@ -0,0 +1,116 @@
|
|||||||
|
"""Pure-Python helpers for backfilling fusion.reconcile.precedent
|
||||||
|
from existing account.partial.reconcile rows during migration.
|
||||||
|
|
||||||
|
Strategy:
|
||||||
|
- Each account.partial.reconcile that involves at least one
|
||||||
|
account.bank.statement.line's reconcile-account line is a candidate.
|
||||||
|
- One precedent per qualifying partial. The (statement_line.id, account_id,
|
||||||
|
amount) triple is encoded into matched_account_ids so a second run can
|
||||||
|
detect and skip already-backfilled rows (idempotency).
|
||||||
|
"""
|
||||||
|
|
||||||
|
import logging
|
||||||
|
|
||||||
|
from .memo_tokenizer import tokenize_memo
|
||||||
|
|
||||||
|
_logger = logging.getLogger(__name__)
|
||||||
|
|
||||||
|
|
||||||
|
def _identify_bank_side(partial):
|
||||||
|
"""Return (bank_move_line, counterpart_move_line, statement_line_id)
|
||||||
|
or (None, None, None) if neither side is a bank statement line."""
|
||||||
|
debit_line = partial.debit_move_id
|
||||||
|
credit_line = partial.credit_move_id
|
||||||
|
|
||||||
|
if debit_line.move_id.statement_line_id:
|
||||||
|
return debit_line, credit_line, debit_line.move_id.statement_line_id.id
|
||||||
|
if credit_line.move_id.statement_line_id:
|
||||||
|
return credit_line, debit_line, credit_line.move_id.statement_line_id.id
|
||||||
|
return None, None, None
|
||||||
|
|
||||||
|
|
||||||
|
def backfill_precedents(env, *, company_id=None, batch_size=500, limit=10000):
|
||||||
|
"""Walk account.partial.reconcile and create fusion.reconcile.precedent
|
||||||
|
rows for any reconcile that involves a bank statement line.
|
||||||
|
|
||||||
|
Idempotent: skips partials whose (statement_line, account, amount)
|
||||||
|
signature is already present in fusion.reconcile.precedent (encoded
|
||||||
|
via matched_account_ids).
|
||||||
|
|
||||||
|
Returns dict with `created` and `skipped` counts.
|
||||||
|
"""
|
||||||
|
Precedent = env['fusion.reconcile.precedent'].sudo()
|
||||||
|
Partial = env['account.partial.reconcile'].sudo()
|
||||||
|
Line = env['account.bank.statement.line'].sudo()
|
||||||
|
|
||||||
|
in_test_mode = env.cr.__class__.__name__ == 'TestCursor'
|
||||||
|
|
||||||
|
# Pre-filter to partials that touch a bank statement line on either side.
|
||||||
|
# In a real DB we typically have 10x more invoice<->payment partials than
|
||||||
|
# bank-rec partials; filtering here keeps the loop bounded and makes the
|
||||||
|
# default limit reflect "real" candidates rather than every partial ever.
|
||||||
|
domain = [
|
||||||
|
'|',
|
||||||
|
('debit_move_id.move_id.statement_line_id', '!=', False),
|
||||||
|
('credit_move_id.move_id.statement_line_id', '!=', False),
|
||||||
|
]
|
||||||
|
if company_id:
|
||||||
|
domain.append(('company_id', '=', company_id))
|
||||||
|
partials = Partial.search(domain, limit=limit, order='id asc')
|
||||||
|
|
||||||
|
created = 0
|
||||||
|
skipped = 0
|
||||||
|
for partial in partials:
|
||||||
|
bank_line, counterpart, bsl_id = _identify_bank_side(partial)
|
||||||
|
if not bsl_id:
|
||||||
|
skipped += 1
|
||||||
|
continue
|
||||||
|
|
||||||
|
signature_account = str(counterpart.account_id.id)
|
||||||
|
|
||||||
|
existing = Precedent.search([
|
||||||
|
('partner_id', '=',
|
||||||
|
counterpart.partner_id.id if counterpart.partner_id else False),
|
||||||
|
('amount', '=', abs(partial.amount)),
|
||||||
|
('matched_account_ids', '=ilike', f'%{signature_account}%'),
|
||||||
|
('source', '=', 'backfill'),
|
||||||
|
], limit=1)
|
||||||
|
if existing:
|
||||||
|
skipped += 1
|
||||||
|
continue
|
||||||
|
|
||||||
|
statement_line = Line.browse(bsl_id)
|
||||||
|
try:
|
||||||
|
currency = (partial.debit_currency_id
|
||||||
|
or partial.company_id.currency_id)
|
||||||
|
Precedent.create({
|
||||||
|
'company_id': partial.company_id.id,
|
||||||
|
'partner_id': (counterpart.partner_id.id
|
||||||
|
if counterpart.partner_id else False),
|
||||||
|
'amount': abs(partial.amount),
|
||||||
|
'currency_id': currency.id,
|
||||||
|
'date': statement_line.date or partial.create_date.date(),
|
||||||
|
'memo_tokens': ','.join(
|
||||||
|
tokenize_memo(statement_line.payment_ref or '')),
|
||||||
|
'journal_id': statement_line.journal_id.id,
|
||||||
|
'matched_move_line_count': 1,
|
||||||
|
'matched_account_ids': signature_account,
|
||||||
|
'reconciler_user_id': partial.create_uid.id,
|
||||||
|
'reconciled_at': partial.create_date,
|
||||||
|
'source': 'backfill',
|
||||||
|
})
|
||||||
|
created += 1
|
||||||
|
if created % batch_size == 0:
|
||||||
|
if not in_test_mode:
|
||||||
|
env.cr.commit()
|
||||||
|
_logger.info(
|
||||||
|
"Backfill progress: %d created, %d skipped",
|
||||||
|
created, skipped)
|
||||||
|
except Exception as e: # noqa: BLE001
|
||||||
|
_logger.warning("Backfill skip partial %s: %s", partial.id, e)
|
||||||
|
skipped += 1
|
||||||
|
|
||||||
|
_logger.info(
|
||||||
|
"precedent_backfill complete: %d created, %d skipped",
|
||||||
|
created, skipped)
|
||||||
|
return {'created': created, 'skipped': skipped}
|
||||||
109
fusion_accounting_bank_rec/static/src/tours/bank_rec_tours.js
Normal file
109
fusion_accounting_bank_rec/static/src/tours/bank_rec_tours.js
Normal file
@@ -0,0 +1,109 @@
|
|||||||
|
/** @odoo-module **/
|
||||||
|
|
||||||
|
import { registry } from "@web/core/registry";
|
||||||
|
|
||||||
|
/**
|
||||||
|
* 5 OWL tours for fusion_accounting_bank_rec smoke testing.
|
||||||
|
*
|
||||||
|
* Each tour scripts a user interaction with the bank-rec widget and
|
||||||
|
* is invoked from Python via HttpCase.start_tour(). Useful for catching
|
||||||
|
* UI regressions that asset-bundle compilation alone won't catch.
|
||||||
|
*/
|
||||||
|
|
||||||
|
// Tour 1: Open the kanban widget and confirm it loads
|
||||||
|
registry.category("web_tour.tours").add("fusion_bank_rec_smoke", {
|
||||||
|
test: true,
|
||||||
|
url: "/odoo/action-fusion_accounting_bank_rec.action_fusion_bank_rec_widget",
|
||||||
|
steps: () => [
|
||||||
|
{
|
||||||
|
content: "Wait for header to appear",
|
||||||
|
trigger: ".o_fusion_bank_rec_header h1:contains(Bank Reconciliation)",
|
||||||
|
},
|
||||||
|
{
|
||||||
|
content: "Confirm stats are visible",
|
||||||
|
trigger: ".o_fusion_stats",
|
||||||
|
},
|
||||||
|
],
|
||||||
|
});
|
||||||
|
|
||||||
|
// Tour 2: Select a line and confirm detail panel loads
|
||||||
|
registry.category("web_tour.tours").add("fusion_bank_rec_select_line", {
|
||||||
|
test: true,
|
||||||
|
url: "/odoo/action-fusion_accounting_bank_rec.action_fusion_bank_rec_widget",
|
||||||
|
steps: () => [
|
||||||
|
{
|
||||||
|
content: "Wait for at least one line card",
|
||||||
|
trigger: ".o_fusion_bank_rec_line:first",
|
||||||
|
},
|
||||||
|
{
|
||||||
|
content: "Click the first line",
|
||||||
|
trigger: ".o_fusion_bank_rec_line:first",
|
||||||
|
run: "click",
|
||||||
|
},
|
||||||
|
{
|
||||||
|
content: "Detail panel shows selected line",
|
||||||
|
trigger: ".o_fusion_bank_rec_detail h2",
|
||||||
|
},
|
||||||
|
],
|
||||||
|
});
|
||||||
|
|
||||||
|
// Tour 3: Trigger AI suggestion and accept
|
||||||
|
registry.category("web_tour.tours").add("fusion_bank_rec_accept_suggestion", {
|
||||||
|
test: true,
|
||||||
|
url: "/odoo/action-fusion_accounting_bank_rec.action_fusion_bank_rec_widget",
|
||||||
|
steps: () => [
|
||||||
|
{
|
||||||
|
content: "Click first line with a partner",
|
||||||
|
trigger: ".o_fusion_bank_rec_line:has(.o_fusion_partner):first",
|
||||||
|
run: "click",
|
||||||
|
},
|
||||||
|
{
|
||||||
|
content: "Click 'Get AI suggestions' button",
|
||||||
|
trigger: ".o_fusion_bank_rec_detail .btn_fusion_primary:contains(Get AI)",
|
||||||
|
run: "click",
|
||||||
|
},
|
||||||
|
{
|
||||||
|
content: "Wait for at least one suggestion to appear",
|
||||||
|
trigger: ".o_fusion_ai_suggestion",
|
||||||
|
},
|
||||||
|
],
|
||||||
|
});
|
||||||
|
|
||||||
|
// Tour 4: Open auto-reconcile wizard
|
||||||
|
registry.category("web_tour.tours").add("fusion_bank_rec_auto_reconcile_wizard", {
|
||||||
|
test: true,
|
||||||
|
url: "/odoo/action-fusion_accounting_bank_rec.action_fusion_auto_reconcile_wizard",
|
||||||
|
steps: () => [
|
||||||
|
{
|
||||||
|
content: "Wizard form opens",
|
||||||
|
trigger: ".modal-dialog .o_form_view",
|
||||||
|
},
|
||||||
|
{
|
||||||
|
content: "Strategy field exists",
|
||||||
|
trigger: ".modal-dialog [name='strategy']",
|
||||||
|
},
|
||||||
|
{
|
||||||
|
content: "Close wizard",
|
||||||
|
trigger: ".modal-dialog .btn-secondary",
|
||||||
|
run: "click",
|
||||||
|
},
|
||||||
|
],
|
||||||
|
});
|
||||||
|
|
||||||
|
// Tour 5: Load more (pagination)
|
||||||
|
registry.category("web_tour.tours").add("fusion_bank_rec_load_more", {
|
||||||
|
test: true,
|
||||||
|
url: "/odoo/action-fusion_accounting_bank_rec.action_fusion_bank_rec_widget",
|
||||||
|
steps: () => [
|
||||||
|
{
|
||||||
|
content: "Wait for kanban container",
|
||||||
|
trigger: ".o_fusion_bank_rec",
|
||||||
|
},
|
||||||
|
// Pagination button only appears if there are more lines than `limit`.
|
||||||
|
// This tour is a no-op if the dataset is small — that's fine for smoke.
|
||||||
|
{
|
||||||
|
content: "Confirm app loaded (regardless of pagination state)",
|
||||||
|
trigger: ".o_fusion_bank_rec_header h1",
|
||||||
|
},
|
||||||
|
],
|
||||||
|
});
|
||||||
@@ -18,3 +18,8 @@ from . import test_cron_methods
|
|||||||
from . import test_controller
|
from . import test_controller
|
||||||
from . import test_auto_reconcile_wizard
|
from . import test_auto_reconcile_wizard
|
||||||
from . import test_bulk_reconcile_wizard
|
from . import test_bulk_reconcile_wizard
|
||||||
|
from . import test_migration_round_trip
|
||||||
|
from . import test_coexistence
|
||||||
|
from . import test_bank_rec_tours
|
||||||
|
from . import test_performance_benchmarks
|
||||||
|
from . import test_local_llm_compat
|
||||||
|
|||||||
42
fusion_accounting_bank_rec/tests/test_bank_rec_tours.py
Normal file
42
fusion_accounting_bank_rec/tests/test_bank_rec_tours.py
Normal file
@@ -0,0 +1,42 @@
|
|||||||
|
"""Python wrappers that run the OWL tours via HttpCase.start_tour.
|
||||||
|
|
||||||
|
Tours require an HTTP server + headless browser. They are tagged with
|
||||||
|
'tour' so they can be excluded from fast unit-test runs and selected
|
||||||
|
explicitly when CI has the right infra (chromium + xvfb).
|
||||||
|
"""
|
||||||
|
|
||||||
|
from odoo.tests.common import HttpCase, tagged
|
||||||
|
|
||||||
|
|
||||||
|
@tagged('post_install', '-at_install', 'tour')
|
||||||
|
class TestBankRecTours(HttpCase):
|
||||||
|
|
||||||
|
def test_smoke_tour(self):
|
||||||
|
# Just verify the smoke tour runs without crashing
|
||||||
|
self.start_tour("/odoo", "fusion_bank_rec_smoke", login="admin")
|
||||||
|
|
||||||
|
def test_select_line_tour(self):
|
||||||
|
# Need a bank line to select — create one
|
||||||
|
partner = self.env['res.partner'].create({'name': 'Tour Partner'})
|
||||||
|
journal = self.env['account.journal'].create({
|
||||||
|
'name': 'Tour Bank', 'type': 'bank', 'code': 'TOURB',
|
||||||
|
})
|
||||||
|
statement = self.env['account.bank.statement'].create({
|
||||||
|
'name': 'Tour Stmt', 'journal_id': journal.id,
|
||||||
|
})
|
||||||
|
self.env['account.bank.statement.line'].create({
|
||||||
|
'statement_id': statement.id, 'journal_id': journal.id,
|
||||||
|
'date': '2026-04-19', 'payment_ref': 'Tour line',
|
||||||
|
'amount': 100, 'partner_id': partner.id,
|
||||||
|
})
|
||||||
|
self.start_tour("/odoo", "fusion_bank_rec_select_line", login="admin")
|
||||||
|
|
||||||
|
def test_accept_suggestion_tour(self):
|
||||||
|
# Skip if too slow / dataset issues — tour itself is the smoke
|
||||||
|
self.skipTest("Tour 3 requires AI provider config; skipping in CI smoke")
|
||||||
|
|
||||||
|
def test_auto_reconcile_wizard_tour(self):
|
||||||
|
self.start_tour("/odoo", "fusion_bank_rec_auto_reconcile_wizard", login="admin")
|
||||||
|
|
||||||
|
def test_load_more_tour(self):
|
||||||
|
self.start_tour("/odoo", "fusion_bank_rec_load_more", login="admin")
|
||||||
86
fusion_accounting_bank_rec/tests/test_coexistence.py
Normal file
86
fusion_accounting_bank_rec/tests/test_coexistence.py
Normal file
@@ -0,0 +1,86 @@
|
|||||||
|
"""Coexistence tests: fusion_accounting_bank_rec menus only visible
|
||||||
|
when Enterprise's account_accountant is absent.
|
||||||
|
|
||||||
|
Strategy: mock the install state by toggling the group's user list directly,
|
||||||
|
then verify the recompute method aligns it with module presence."""
|
||||||
|
|
||||||
|
from unittest.mock import patch
|
||||||
|
from odoo.tests.common import TransactionCase, tagged
|
||||||
|
|
||||||
|
|
||||||
|
@tagged('post_install', '-at_install')
|
||||||
|
class TestCoexistence(TransactionCase):
|
||||||
|
|
||||||
|
def setUp(self):
|
||||||
|
super().setUp()
|
||||||
|
self.group = self.env.ref(
|
||||||
|
'fusion_accounting_core.group_fusion_show_when_enterprise_absent')
|
||||||
|
|
||||||
|
def _account_accountant_installed(self):
|
||||||
|
return bool(self.env['ir.module.module'].sudo().search([
|
||||||
|
('name', '=', 'account_accountant'),
|
||||||
|
('state', '=', 'installed'),
|
||||||
|
]))
|
||||||
|
|
||||||
|
def test_group_exists(self):
|
||||||
|
self.assertTrue(self.group, "Coexistence group must exist")
|
||||||
|
|
||||||
|
def test_recompute_when_enterprise_present(self):
|
||||||
|
"""When account_accountant is installed, group should be empty."""
|
||||||
|
if not self._account_accountant_installed():
|
||||||
|
self.skipTest(
|
||||||
|
"Local DB doesn't have account_accountant installed; "
|
||||||
|
"this test only meaningful in Enterprise-present scenario"
|
||||||
|
)
|
||||||
|
self.env['res.users']._fusion_recompute_coexistence_group()
|
||||||
|
self.assertEqual(
|
||||||
|
len(self.group.user_ids), 0,
|
||||||
|
"Coexistence group should be empty when Enterprise is installed",
|
||||||
|
)
|
||||||
|
|
||||||
|
def test_recompute_when_enterprise_absent(self):
|
||||||
|
"""When account_accountant is uninstalled, all internal users get the group."""
|
||||||
|
if self._account_accountant_installed():
|
||||||
|
# Simulate by mocking the enterprise-installed check.
|
||||||
|
with patch.object(
|
||||||
|
type(self.env['ir.module.module']),
|
||||||
|
'_fusion_is_enterprise_accounting_installed',
|
||||||
|
return_value=False,
|
||||||
|
):
|
||||||
|
self.env['res.users']._fusion_recompute_coexistence_group()
|
||||||
|
internal_users = self.env['res.users'].search([
|
||||||
|
('share', '=', False),
|
||||||
|
])
|
||||||
|
self.assertGreater(
|
||||||
|
len(self.group.user_ids & internal_users), 0,
|
||||||
|
"Coexistence group should contain internal users when "
|
||||||
|
"Enterprise is absent",
|
||||||
|
)
|
||||||
|
else:
|
||||||
|
self.env['res.users']._fusion_recompute_coexistence_group()
|
||||||
|
internal = self.env['res.users'].search([('share', '=', False)])
|
||||||
|
self.assertGreater(len(self.group.user_ids & internal), 0)
|
||||||
|
|
||||||
|
def test_menu_has_coexistence_group(self):
|
||||||
|
"""The fusion bank-rec root menu must have the coexistence group attached."""
|
||||||
|
menu = self.env.ref(
|
||||||
|
'fusion_accounting_bank_rec.menu_fusion_bank_rec_root',
|
||||||
|
raise_if_not_found=False,
|
||||||
|
)
|
||||||
|
if not menu:
|
||||||
|
self.skipTest("Menu not yet loaded — Task 42 must run first")
|
||||||
|
# Odoo 19 renamed ir.ui.menu.groups_id -> group_ids; tolerate either.
|
||||||
|
groups_field = getattr(menu, 'group_ids', None) or menu.groups_id
|
||||||
|
self.assertIn(
|
||||||
|
self.group, groups_field,
|
||||||
|
"Menu must require the coexistence group",
|
||||||
|
)
|
||||||
|
|
||||||
|
def test_engine_works_regardless_of_coexistence(self):
|
||||||
|
"""The reconcile engine must work even when Enterprise is installed
|
||||||
|
(it's the AI tools/menu that gate; the engine is always available)."""
|
||||||
|
self.assertIn(
|
||||||
|
'fusion.reconcile.engine', self.env.registry,
|
||||||
|
"Engine must always be available when fusion_accounting_bank_rec "
|
||||||
|
"is installed",
|
||||||
|
)
|
||||||
102
fusion_accounting_bank_rec/tests/test_local_llm_compat.py
Normal file
102
fusion_accounting_bank_rec/tests/test_local_llm_compat.py
Normal file
@@ -0,0 +1,102 @@
|
|||||||
|
"""Local LLM compatibility test (LM Studio, Ollama, etc.).
|
||||||
|
|
||||||
|
Skips if no local OpenAI-compatible LLM server is reachable. When one is
|
||||||
|
running (LM Studio at :1234, Ollama at :11434), runs an end-to-end:
|
||||||
|
|
||||||
|
1. Configure ``ir.config_parameter`` to point at the local server.
|
||||||
|
2. Trigger ``engine.suggest_matches`` with the 'openai' provider.
|
||||||
|
3. Assert the call did not crash and produced at least one suggestion.
|
||||||
|
|
||||||
|
The smoke is intentionally lenient: local models often emit malformed
|
||||||
|
JSON, in which case ``confidence_scoring`` falls back to statistical-only
|
||||||
|
ranking. We assert end-to-end happiness, not AI re-rank quality.
|
||||||
|
"""
|
||||||
|
|
||||||
|
import socket
|
||||||
|
|
||||||
|
from odoo.tests.common import TransactionCase, tagged
|
||||||
|
|
||||||
|
from . import _factories as f
|
||||||
|
|
||||||
|
|
||||||
|
def _server_reachable(host, port, timeout=1.0):
|
||||||
|
try:
|
||||||
|
with socket.create_connection((host, port), timeout=timeout):
|
||||||
|
return True
|
||||||
|
except (OSError, socket.timeout):
|
||||||
|
return False
|
||||||
|
|
||||||
|
|
||||||
|
def _detect_local_llm():
|
||||||
|
"""Return (base_url, model_name) tuple, or (None, None) if no server.
|
||||||
|
|
||||||
|
Tries LM Studio (:1234) and Ollama (:11434) on both
|
||||||
|
``host.docker.internal`` (so the container can reach the host) and
|
||||||
|
``localhost`` (so a non-containerised run finds the same servers).
|
||||||
|
"""
|
||||||
|
candidates = (
|
||||||
|
('host.docker.internal', 1234, 'local-model'), # LM Studio
|
||||||
|
('host.docker.internal', 11434, 'llama3.1:8b'), # Ollama
|
||||||
|
('localhost', 1234, 'local-model'),
|
||||||
|
('localhost', 11434, 'llama3.1:8b'),
|
||||||
|
)
|
||||||
|
for host, port, default_model in candidates:
|
||||||
|
if _server_reachable(host, port, timeout=0.5):
|
||||||
|
return (f'http://{host}:{port}/v1', default_model)
|
||||||
|
return (None, None)
|
||||||
|
|
||||||
|
|
||||||
|
@tagged('post_install', '-at_install', 'local_llm')
|
||||||
|
class TestLocalLLMCompat(TransactionCase):
|
||||||
|
|
||||||
|
def setUp(self):
|
||||||
|
super().setUp()
|
||||||
|
self.base_url, self.model = _detect_local_llm()
|
||||||
|
if not self.base_url:
|
||||||
|
self.skipTest(
|
||||||
|
"No local LLM server detected "
|
||||||
|
"(LM Studio :1234 / Ollama :11434)")
|
||||||
|
|
||||||
|
def test_suggest_matches_with_local_llm(self):
|
||||||
|
params = self.env['ir.config_parameter'].sudo()
|
||||||
|
prior = {
|
||||||
|
'fusion_accounting.openai_base_url': params.get_param(
|
||||||
|
'fusion_accounting.openai_base_url'),
|
||||||
|
'fusion_accounting.openai_model': params.get_param(
|
||||||
|
'fusion_accounting.openai_model'),
|
||||||
|
'fusion_accounting.openai_api_key': params.get_param(
|
||||||
|
'fusion_accounting.openai_api_key'),
|
||||||
|
'fusion_accounting.provider.bank_rec_suggest': params.get_param(
|
||||||
|
'fusion_accounting.provider.bank_rec_suggest'),
|
||||||
|
}
|
||||||
|
|
||||||
|
params.set_param('fusion_accounting.openai_base_url', self.base_url)
|
||||||
|
params.set_param('fusion_accounting.openai_model', self.model)
|
||||||
|
# Local servers ignore the key but the adapter requires *some* value.
|
||||||
|
params.set_param('fusion_accounting.openai_api_key', 'lm-studio')
|
||||||
|
params.set_param(
|
||||||
|
'fusion_accounting.provider.bank_rec_suggest', 'openai')
|
||||||
|
|
||||||
|
try:
|
||||||
|
partner = self.env['res.partner'].create(
|
||||||
|
{'name': 'Local LLM Partner'})
|
||||||
|
f.make_invoice(self.env, partner=partner, amount=750)
|
||||||
|
bank_line = f.make_bank_line(
|
||||||
|
self.env, amount=750, partner=partner,
|
||||||
|
memo='REF 12345 Local LLM test')
|
||||||
|
|
||||||
|
result = self.env['fusion.reconcile.engine'].suggest_matches(
|
||||||
|
bank_line, limit_per_line=3)
|
||||||
|
|
||||||
|
self.assertIn(bank_line.id, result)
|
||||||
|
suggestions = self.env['fusion.reconcile.suggestion'].search([
|
||||||
|
('statement_line_id', '=', bank_line.id),
|
||||||
|
])
|
||||||
|
self.assertGreater(
|
||||||
|
len(suggestions), 0,
|
||||||
|
"Local LLM run should still produce at least one suggestion "
|
||||||
|
"(statistical fallback if AI re-rank fails)")
|
||||||
|
finally:
|
||||||
|
for key, value in prior.items():
|
||||||
|
if value is not None:
|
||||||
|
params.set_param(key, value)
|
||||||
115
fusion_accounting_bank_rec/tests/test_migration_round_trip.py
Normal file
115
fusion_accounting_bank_rec/tests/test_migration_round_trip.py
Normal file
@@ -0,0 +1,115 @@
|
|||||||
|
"""Migration round-trip: bootstrap step backfills precedents from
|
||||||
|
existing account.partial.reconcile rows.
|
||||||
|
|
||||||
|
Exercises Task 39's _bank_rec_bootstrap_step end-to-end:
|
||||||
|
1. Set up a bank-line / invoice reconciliation via the engine. This
|
||||||
|
creates an account.partial.reconcile row.
|
||||||
|
2. Wipe the auto-recorded fusion.reconcile.precedent rows so the
|
||||||
|
backfill has work to do.
|
||||||
|
3. Run wizard._bank_rec_bootstrap_step().
|
||||||
|
4. Assert at least one precedent was created with source='backfill',
|
||||||
|
the wizard reports successful pattern + MV refresh, and that a
|
||||||
|
second run is a no-op (idempotent).
|
||||||
|
"""
|
||||||
|
|
||||||
|
from odoo.tests.common import TransactionCase, tagged
|
||||||
|
|
||||||
|
from . import _factories as f
|
||||||
|
|
||||||
|
|
||||||
|
@tagged('post_install', '-at_install')
|
||||||
|
class TestMigrationRoundTrip(TransactionCase):
|
||||||
|
|
||||||
|
def setUp(self):
|
||||||
|
super().setUp()
|
||||||
|
self.partner = self.env['res.partner'].create({
|
||||||
|
'name': 'Migration Round-Trip Partner',
|
||||||
|
})
|
||||||
|
self.journal = f.make_bank_journal(
|
||||||
|
self.env, name='Migration Bank', code='MIGBK')
|
||||||
|
self.statement = f.make_bank_statement(
|
||||||
|
self.env, journal=self.journal, name='Migration Statement')
|
||||||
|
|
||||||
|
def _seed_partial_reconciles(self, amounts):
|
||||||
|
"""Create one reconciled bank-line/invoice pair per amount, reusing
|
||||||
|
a single bank journal so we don't violate the
|
||||||
|
account_journal_code_company_uniq constraint.
|
||||||
|
|
||||||
|
Each call here produces one account.partial.reconcile row.
|
||||||
|
Returns the partial recordset.
|
||||||
|
"""
|
||||||
|
Engine = self.env['fusion.reconcile.engine']
|
||||||
|
partials = self.env['account.partial.reconcile']
|
||||||
|
for amount in amounts:
|
||||||
|
invoice = f.make_invoice(
|
||||||
|
self.env, partner=self.partner, amount=amount)
|
||||||
|
recv_lines = invoice.line_ids.filtered(
|
||||||
|
lambda l: l.account_id.account_type == 'asset_receivable')
|
||||||
|
bank_line = f.make_bank_line(
|
||||||
|
self.env, statement=self.statement, amount=amount,
|
||||||
|
partner=self.partner)
|
||||||
|
result = Engine.reconcile_one(
|
||||||
|
bank_line, against_lines=recv_lines)
|
||||||
|
partials |= self.env['account.partial.reconcile'].browse(
|
||||||
|
result['partial_ids'])
|
||||||
|
return partials
|
||||||
|
|
||||||
|
def _wipe_precedents(self):
|
||||||
|
self.env['fusion.reconcile.precedent'].search([
|
||||||
|
('partner_id', '=', self.partner.id),
|
||||||
|
]).unlink()
|
||||||
|
|
||||||
|
def test_bootstrap_creates_precedents_from_existing_reconciles(self):
|
||||||
|
partials = self._seed_partial_reconciles([125.00, 275.00])
|
||||||
|
self.assertTrue(partials,
|
||||||
|
"Test setup should produce account.partial.reconcile rows")
|
||||||
|
|
||||||
|
self._wipe_precedents()
|
||||||
|
before_backfill = self.env['fusion.reconcile.precedent'].search_count([
|
||||||
|
('partner_id', '=', self.partner.id),
|
||||||
|
('source', '=', 'backfill'),
|
||||||
|
])
|
||||||
|
self.assertEqual(before_backfill, 0,
|
||||||
|
"Precondition: no backfill precedents should exist before bootstrap")
|
||||||
|
|
||||||
|
wizard = self.env['fusion.migration.wizard'].create({})
|
||||||
|
result = wizard._bank_rec_bootstrap_step()
|
||||||
|
|
||||||
|
self.assertEqual(result['step'], 'bank_rec_bootstrap')
|
||||||
|
self.assertGreaterEqual(result['precedents_created'], 1,
|
||||||
|
"Bootstrap should backfill at least one precedent from the "
|
||||||
|
"partial.reconcile rows produced in setUp")
|
||||||
|
self.assertTrue(result['mv_refreshed'],
|
||||||
|
"Bootstrap should report successful MV refresh")
|
||||||
|
|
||||||
|
after_backfill = self.env['fusion.reconcile.precedent'].search_count([
|
||||||
|
('partner_id', '=', self.partner.id),
|
||||||
|
('source', '=', 'backfill'),
|
||||||
|
])
|
||||||
|
self.assertGreaterEqual(after_backfill, 1,
|
||||||
|
"At least one source='backfill' precedent should exist post-bootstrap")
|
||||||
|
|
||||||
|
def test_bootstrap_step_idempotent(self):
|
||||||
|
self._seed_partial_reconciles([411.00])
|
||||||
|
self._wipe_precedents()
|
||||||
|
|
||||||
|
wizard = self.env['fusion.migration.wizard'].create({})
|
||||||
|
result1 = wizard._bank_rec_bootstrap_step()
|
||||||
|
created_first_run = result1['precedents_created']
|
||||||
|
self.assertGreaterEqual(created_first_run, 1)
|
||||||
|
|
||||||
|
result2 = wizard._bank_rec_bootstrap_step()
|
||||||
|
self.assertEqual(result2['precedents_created'], 0,
|
||||||
|
"Second bootstrap should create zero precedents (idempotent)")
|
||||||
|
self.assertGreaterEqual(result2['precedents_skipped'], created_first_run,
|
||||||
|
"Second bootstrap should skip at least what the first one created")
|
||||||
|
|
||||||
|
def test_bootstrap_refreshes_mv_without_error(self):
|
||||||
|
"""The bootstrap call must not raise even when there's nothing to do."""
|
||||||
|
wizard = self.env['fusion.migration.wizard'].create({})
|
||||||
|
try:
|
||||||
|
result = wizard._bank_rec_bootstrap_step()
|
||||||
|
except Exception as e: # noqa: BLE001
|
||||||
|
self.fail(f"Bootstrap raised: {e}")
|
||||||
|
self.assertIn('mv_refreshed', result)
|
||||||
|
self.assertIn('patterns_refreshed', result)
|
||||||
188
fusion_accounting_bank_rec/tests/test_performance_benchmarks.py
Normal file
188
fusion_accounting_bank_rec/tests/test_performance_benchmarks.py
Normal file
@@ -0,0 +1,188 @@
|
|||||||
|
"""Performance benchmarks with P95 targets.
|
||||||
|
|
||||||
|
Tagged with ``benchmark`` so they can be selected explicitly:
|
||||||
|
odoo --test-tags 'benchmark' ...
|
||||||
|
|
||||||
|
These tests measure wall-clock time and assert P95 stays within plan
|
||||||
|
budgets. They run a small N (e.g. 10 iterations) so total test time
|
||||||
|
stays under 30s. For real load testing, use a separate harness.
|
||||||
|
|
||||||
|
Hard-fail thresholds are 5x the plan budget — they catch egregious
|
||||||
|
regressions without flaking on cold-start variance in CI.
|
||||||
|
"""
|
||||||
|
|
||||||
|
import json
|
||||||
|
import statistics
|
||||||
|
import time
|
||||||
|
|
||||||
|
from odoo.tests.common import HttpCase, TransactionCase, new_test_user, tagged
|
||||||
|
|
||||||
|
from . import _factories as f
|
||||||
|
|
||||||
|
|
||||||
|
def _percentile(samples, p):
|
||||||
|
"""Return the ``p``-th percentile of ``samples`` (0-100)."""
|
||||||
|
if not samples:
|
||||||
|
return None
|
||||||
|
if len(samples) == 1:
|
||||||
|
return samples[0]
|
||||||
|
return statistics.quantiles(samples, n=100)[p - 1]
|
||||||
|
|
||||||
|
|
||||||
|
@tagged('post_install', '-at_install', 'benchmark')
|
||||||
|
class TestEngineBenchmarks(TransactionCase):
|
||||||
|
|
||||||
|
def setUp(self):
|
||||||
|
super().setUp()
|
||||||
|
self.partner = self.env['res.partner'].create({'name': 'Bench Partner'})
|
||||||
|
# Pre-create a dedicated journal+statement and reuse them across all
|
||||||
|
# iterations -- otherwise the second make_bank_line() collides on the
|
||||||
|
# (code, company) unique constraint of the default 'TEST' journal.
|
||||||
|
self.journal = f.make_bank_journal(
|
||||||
|
self.env, name='Engine Bench Bank', code='EBB')
|
||||||
|
self.statement = f.make_bank_statement(
|
||||||
|
self.env, journal=self.journal, name='Engine Bench Stmt')
|
||||||
|
# Pre-create some invoices so suggest_matches has something to score
|
||||||
|
self.invoices = []
|
||||||
|
for amount in (100, 200, 300, 400, 500):
|
||||||
|
inv = f.make_invoice(self.env, partner=self.partner, amount=amount)
|
||||||
|
self.invoices.append(inv)
|
||||||
|
|
||||||
|
def test_suggest_matches_p95_under_500ms(self):
|
||||||
|
timings = []
|
||||||
|
for _ in range(10):
|
||||||
|
line = f.make_bank_line(
|
||||||
|
self.env, journal=self.journal, statement=self.statement,
|
||||||
|
amount=300, partner=self.partner)
|
||||||
|
start = time.perf_counter()
|
||||||
|
self.env['fusion.reconcile.engine'].suggest_matches(
|
||||||
|
line, limit_per_line=3)
|
||||||
|
elapsed = (time.perf_counter() - start) * 1000 # ms
|
||||||
|
timings.append(elapsed)
|
||||||
|
timings.sort()
|
||||||
|
p95 = _percentile(timings, 95)
|
||||||
|
median = statistics.median(timings)
|
||||||
|
msg = f"suggest_matches: median={median:.1f}ms p95={p95:.1f}ms"
|
||||||
|
print(f"\n PERF: {msg} (target <500ms)")
|
||||||
|
# Soft assertion -- log but don't fail under 5x budget (cold-start
|
||||||
|
# variance). Hard fail above 5x catches egregious regressions.
|
||||||
|
self.assertLess(
|
||||||
|
p95, 2500,
|
||||||
|
f"suggest_matches P95 way over budget: {msg} "
|
||||||
|
f"(target <500ms, hard fail >2500ms)")
|
||||||
|
|
||||||
|
def test_reconcile_batch_p95_under_5s(self):
|
||||||
|
# Create 50 matchable pairs on a shared journal/statement so we
|
||||||
|
# don't blow the (code, company) constraint.
|
||||||
|
journal = f.make_bank_journal(
|
||||||
|
self.env, name='Batch Bench Bank', code='BBB')
|
||||||
|
statement = f.make_bank_statement(
|
||||||
|
self.env, journal=journal, name='Batch Bench Stmt')
|
||||||
|
line_ids = []
|
||||||
|
for i in range(50):
|
||||||
|
invoice = f.make_invoice(
|
||||||
|
self.env, partner=self.partner, amount=100 + i)
|
||||||
|
del invoice # ensures the receivable JE exists for engine to find
|
||||||
|
line = f.make_bank_line(
|
||||||
|
self.env, journal=journal, statement=statement,
|
||||||
|
amount=100 + i, partner=self.partner)
|
||||||
|
line_ids.append(line.id)
|
||||||
|
lines = self.env['account.bank.statement.line'].browse(line_ids)
|
||||||
|
start = time.perf_counter()
|
||||||
|
result = self.env['fusion.reconcile.engine'].reconcile_batch(
|
||||||
|
lines, strategy='auto')
|
||||||
|
elapsed = (time.perf_counter() - start) * 1000
|
||||||
|
msg = (f"reconcile_batch(50 lines): {elapsed:.0f}ms, "
|
||||||
|
f"reconciled={result.get('reconciled_count', 'n/a')}")
|
||||||
|
print(f"\n PERF: {msg} (target <5000ms)")
|
||||||
|
self.assertLess(
|
||||||
|
elapsed, 25000,
|
||||||
|
f"reconcile_batch way over budget: {msg} "
|
||||||
|
f"(target <5000ms, hard fail >25000ms)")
|
||||||
|
|
||||||
|
|
||||||
|
@tagged('post_install', '-at_install', 'benchmark')
|
||||||
|
class TestControllerBenchmarks(HttpCase):
|
||||||
|
|
||||||
|
USER_LOGIN = 'bench_ctrl_user'
|
||||||
|
USER_PASSWORD = 'bench_ctrl_user'
|
||||||
|
|
||||||
|
def setUp(self):
|
||||||
|
super().setUp()
|
||||||
|
# Mirrors test_controller.py auth setup -- a fresh test user with
|
||||||
|
# the same group bundle the controller expects. The dev DB's admin
|
||||||
|
# password is non-default, so we cannot rely on 'admin'/'admin'.
|
||||||
|
new_test_user(
|
||||||
|
self.env,
|
||||||
|
login=self.USER_LOGIN,
|
||||||
|
password=self.USER_PASSWORD,
|
||||||
|
groups=(
|
||||||
|
'base.group_user,'
|
||||||
|
'account.group_account_user,'
|
||||||
|
'fusion_accounting_core.group_fusion_accounting_admin'
|
||||||
|
),
|
||||||
|
)
|
||||||
|
|
||||||
|
def test_list_unreconciled_p95_under_200ms(self):
|
||||||
|
partner = self.env['res.partner'].create({'name': 'Ctrl Bench'})
|
||||||
|
journal = f.make_bank_journal(
|
||||||
|
self.env, name='Ctrl Bench Bank', code='CBB')
|
||||||
|
statement = f.make_bank_statement(
|
||||||
|
self.env, journal=journal, name='Ctrl Bench Stmt')
|
||||||
|
for i in range(50):
|
||||||
|
f.make_bank_line(
|
||||||
|
self.env, journal=journal, statement=statement,
|
||||||
|
amount=100 + i, partner=partner,
|
||||||
|
memo=f'Ctrl bench line {i}')
|
||||||
|
self.authenticate(self.USER_LOGIN, self.USER_PASSWORD)
|
||||||
|
body = json.dumps({
|
||||||
|
'jsonrpc': '2.0',
|
||||||
|
'method': 'call',
|
||||||
|
'params': {
|
||||||
|
'journal_id': journal.id,
|
||||||
|
'limit': 50,
|
||||||
|
'offset': 0,
|
||||||
|
'company_id': self.env.company.id,
|
||||||
|
},
|
||||||
|
'id': 1,
|
||||||
|
})
|
||||||
|
timings = []
|
||||||
|
for _ in range(10):
|
||||||
|
start = time.perf_counter()
|
||||||
|
response = self.url_open(
|
||||||
|
'/fusion/bank_rec/list_unreconciled',
|
||||||
|
data=body,
|
||||||
|
headers={'Content-Type': 'application/json'},
|
||||||
|
)
|
||||||
|
elapsed = (time.perf_counter() - start) * 1000
|
||||||
|
self.assertEqual(response.status_code, 200)
|
||||||
|
timings.append(elapsed)
|
||||||
|
timings.sort()
|
||||||
|
p95 = _percentile(timings, 95)
|
||||||
|
median = statistics.median(timings)
|
||||||
|
msg = f"list_unreconciled: median={median:.1f}ms p95={p95:.1f}ms"
|
||||||
|
print(f"\n PERF: {msg} (target <200ms)")
|
||||||
|
self.assertLess(
|
||||||
|
p95, 1000,
|
||||||
|
f"list_unreconciled P95 way over budget: {msg} "
|
||||||
|
f"(target <200ms, hard fail >1000ms)")
|
||||||
|
|
||||||
|
|
||||||
|
@tagged('post_install', '-at_install', 'benchmark')
|
||||||
|
class TestMVBenchmarks(TransactionCase):
|
||||||
|
|
||||||
|
def test_mv_refresh_under_2s(self):
|
||||||
|
# Non-concurrent refresh works even before the MV has been seeded
|
||||||
|
# with a concurrent-refresh-eligible state.
|
||||||
|
start = time.perf_counter()
|
||||||
|
self.env['fusion.unreconciled.bank.line.mv']._refresh(
|
||||||
|
concurrently=False)
|
||||||
|
elapsed = (time.perf_counter() - start) * 1000
|
||||||
|
msg = (f"MV refresh: {elapsed:.0f}ms "
|
||||||
|
f"(current row count varies with DB state)")
|
||||||
|
print(f"\n PERF: {msg} (target <2000ms)")
|
||||||
|
# Soft hard ceiling: 10s
|
||||||
|
self.assertLess(
|
||||||
|
elapsed, 10000,
|
||||||
|
f"MV refresh way over budget: {msg} "
|
||||||
|
f"(target <2000ms, hard fail >10000ms)")
|
||||||
45
fusion_accounting_bank_rec/views/menu_views.xml
Normal file
45
fusion_accounting_bank_rec/views/menu_views.xml
Normal file
@@ -0,0 +1,45 @@
|
|||||||
|
<?xml version="1.0" encoding="utf-8"?>
|
||||||
|
<odoo>
|
||||||
|
|
||||||
|
<!-- Window action that opens the bank reconciliation kanban widget -->
|
||||||
|
<record id="action_fusion_bank_rec_widget" model="ir.actions.act_window">
|
||||||
|
<field name="name">Bank Reconciliation</field>
|
||||||
|
<field name="res_model">account.bank.statement.line</field>
|
||||||
|
<field name="view_mode">fusion_bank_rec_kanban</field>
|
||||||
|
<field name="domain">[('is_reconciled', '=', False)]</field>
|
||||||
|
<field name="context">{}</field>
|
||||||
|
<field name="help" type="html">
|
||||||
|
<p class="o_view_nocontent_smiling_face">
|
||||||
|
Bank Reconciliation Widget
|
||||||
|
</p>
|
||||||
|
<p>
|
||||||
|
AI-assisted bank reconciliation. Statement lines that haven't
|
||||||
|
been matched yet appear here, with confidence-scored AI
|
||||||
|
suggestions for matching.
|
||||||
|
</p>
|
||||||
|
</field>
|
||||||
|
</record>
|
||||||
|
|
||||||
|
<!-- Top-level menu — only visible when Enterprise's account_accountant is absent -->
|
||||||
|
<menuitem id="menu_fusion_bank_rec_root"
|
||||||
|
name="Bank Reconciliation"
|
||||||
|
sequence="40"
|
||||||
|
web_icon="fusion_accounting_bank_rec,static/description/icon.png"
|
||||||
|
groups="fusion_accounting_core.group_fusion_show_when_enterprise_absent"/>
|
||||||
|
|
||||||
|
<menuitem id="menu_fusion_bank_rec_main"
|
||||||
|
name="Reconcile Bank Lines"
|
||||||
|
parent="menu_fusion_bank_rec_root"
|
||||||
|
action="action_fusion_bank_rec_widget"
|
||||||
|
sequence="10"
|
||||||
|
groups="fusion_accounting_core.group_fusion_show_when_enterprise_absent"/>
|
||||||
|
|
||||||
|
<!-- Sub-menu for the auto-reconcile wizard -->
|
||||||
|
<menuitem id="menu_fusion_auto_reconcile_wizard"
|
||||||
|
name="Auto-Reconcile…"
|
||||||
|
parent="menu_fusion_bank_rec_root"
|
||||||
|
action="action_fusion_auto_reconcile_wizard"
|
||||||
|
sequence="20"
|
||||||
|
groups="fusion_accounting_core.group_fusion_show_when_enterprise_absent"/>
|
||||||
|
|
||||||
|
</odoo>
|
||||||
Reference in New Issue
Block a user