add cancel transaction

.gitignore

@@ -10,5 +10,6 @@ __pycache__/
 # Terraform / backup dll
 *.bak
 *.log
+backups/
 
 .github/

README.md

@@ -30,13 +30,13 @@ saat-app/
 
 ## Cara menjalankan
 ```bash
-cd /home/ardhi/Application/saat-app
+cd /home/ardhi/workspace/erp/odoo-app/saat-app
 docker compose config      # validasi
 docker compose up -d       # start
 ```
 
-Akses UI: http://localhost:18069 — login `admin` / `admin`
-(`ADMIN_PASSWORD` untuk Database Management di `.env`).
+Akses UI: http://localhost:18069 — login `admin` dengan password
+`ODOO_ADMIN_PASSWORD` di `.env`.
 
 ## Status
 - ✅ Odoo 17 container jalan (port 18069), DB PostgreSQL 16 healthy
@@ -67,6 +67,110 @@ docker restart saat-odoo
 > Catatan Odoo: `<list>` view di Odoo 17 kadang bermasalah, pakai `<tree>`
 > (kompatibel lintas versi) seperti yang sudah dilakukan.
 
+## Troubleshooting PermissionError sessions
+
+Jika muncul:
+
+```text
+PermissionError: [Errno 13] Permission denied: '/var/lib/odoo/sessions'
+```
+
+Odoo 17 berjalan sebagai user `odoo` dengan UID/GID `101`. Perbaiki
+ownership seluruh volume data Odoo dari container yang sedang berjalan:
+
+```bash
+cid=$(docker compose ps -q odoo)
+docker exec --user root "$cid" sh -c \
+  'chown -R 101:101 /var/lib/odoo && \
+   chmod 755 /var/lib/odoo && chmod 700 /var/lib/odoo/sessions'
+```
+
+Verifikasi write access sebagai user Odoo, lalu restart dan cek log:
+
+```bash
+docker exec --user odoo "$cid" sh -c \
+  'touch /var/lib/odoo/sessions/.write-check && \
+   rm /var/lib/odoo/sessions/.write-check'
+docker compose restart odoo
+docker compose logs --since=30s odoo
+```
+
+Gunakan hanya satu mount volume utama berikut pada Compose:
+
+```yaml
+volumes:
+  - saat_odoo_filestore:/var/lib/odoo
+```
+
+Jangan menambahkan mount terpisah ke `/var/lib/odoo/session` (singular).
+Path yang dipakai Odoo adalah `/var/lib/odoo/sessions` (plural). Jangan
+menggunakan `chmod 777`; cukup berikan ownership kepada user Odoo.
+
+## Rencana migrasi Odoo 17 ke Odoo 18
+
+Migrasi major version harus mengubah schema database. Mengganti image secara
+langsung dari `odoo:17.0` ke `odoo:18.0` **bukan** prosedur migrasi dan dapat
+menyebabkan error seperti `column res_lang.short_time_format does not exist`.
+Compose harus tetap menggunakan `odoo:17.0` sampai database selesai dimigrasi.
+
+### 1. Persiapan dan backup
+
+Pastikan `.env` terisi dan jangan commit file tersebut. Hentikan penulisan data,
+lalu buat backup database dan filestore:
+
+```bash
+set -a && source .env && set +a
+backup_dir="backups/odoo17-$(date +%Y%m%d-%H%M%S)"
+mkdir -p "$backup_dir"
+docker compose exec -T db pg_dump -U "$POSTGRES_USER" -d "$POSTGRES_DB" -Fc \
+  > "$backup_dir/database.dump"
+volume=$(docker inspect saat-odoo --format '{{range .Mounts}}{{if eq .Destination "/var/lib/odoo"}}{{.Name}}{{end}}{{end}}')
+docker run --rm -v "$volume:/source:ro" -v "$PWD/$backup_dir:/backup" alpine \
+  sh -c 'tar czf /backup/filestore.tar.gz -C /source .'
+```
+
+Validasi backup dengan `pg_restore --list` dari image PostgreSQL 16 dan pastikan
+arsip filestore berisi direktori `filestore/`.
+
+### 2. Migrasi pada salinan
+
+Jangan menguji migrasi pada volume produksi. Restore `database.dump` dan
+`filestore.tar.gz` ke environment staging yang terpisah, lalu gunakan salah
+satu jalur resmi berikut:
+
+- Odoo Upgrade Service untuk database Enterprise.
+- OpenUpgrade dengan branch yang sesuai untuk migrasi Community 17 ke 18.
+
+Migrasikan database terlebih dahulu. Sesuaikan custom addon `saat_invest`
+dengan API, view, security, dan dependency Odoo 18, lalu install/upgrade addon
+di database staging hasil migrasi.
+
+### 3. Validasi staging
+
+Uji login, menu saham, create/update/delete transaksi, kalkulasi portofolio,
+cron harga pasar, attachment/filestore, dan seluruh endpoint JSON-RPC yang
+dipakai Flutter. Periksa log untuk `ERROR`, `Traceback`, `UndefinedColumn`, dan
+`PermissionError`.
+
+### 4. Cutover produksi
+
+Setelah staging lulus, hentikan Odoo 17, buat backup final, restore atau jalankan
+migrasi pada database produksi sesuai prosedur tool migrasi, kemudian ubah:
+
+```yaml
+image: odoo:18.0
+```
+
+Pastikan ownership volume sesuai user Odoo 18 sebelum start, jalankan
+`docker compose up -d`, dan ulangi seluruh validasi staging.
+
+### 5. Rollback
+
+Jika migrasi gagal, jangan menjalankan Odoo 17 pada database yang sudah sebagian
+dimigrasi. Hentikan stack, restore database dan filestore dari backup Odoo 17
+ke volume/database terpisah, ubah image kembali ke `odoo:17.0`, lalu validasi
+aplikasi sebelum cutover ulang.
+
 ## API untuk Flutter (via JSON-RPC `execute_kw`)
 Semua melalui `POST /jsonrpc` dengan `{service:'object', method:'execute_kw'}`.
 Login: `common.authenticate(db, user, pass)` → `uid`.

addons/saat_invest/models/portofolio.py

@@ -38,7 +38,7 @@ class SaatPortofolio(models.Model):
     def action_recalculate(self):
         """Rekalkulasi semua posisi portofolio dari transaksi. Return dict ringkas."""
         self.env['saat.portofolio'].search([]).unlink()
-        trans = self.env['saat.transaksi'].search([], order='tanggal, id')
+        trans = self.env['saat.transaksi'].search([('state', '=', 'done')], order='tanggal, id')
         per_saham = {}
         for t in trans:
             a = per_saham.setdefault(t.saham_id.id, {

addons/saat_invest/models/transaksi.py

@@ -30,6 +30,11 @@ class SaatTransaksi(models.Model):
         # IDR
         return self.env.ref('base.IDR', raise_if_not_found=False) or self.env.company.currency_id
 
+    state = fields.Selection([
+        ('done', 'Selesai'),
+        ('cancel', 'Dibatalkan'),
+    ], string="Status", default='done', required=True, copy=False)
+
     @api.depends('tanggal', 'tipe', 'saham_id')
     def _compute_name(self):
         for r in self:
@@ -50,6 +55,14 @@ class SaatTransaksi(models.Model):
             if r.harga < 0:
                 raise models.ValidationError("Harga tidak boleh negatif.")
 
+    def action_cancel(self):
+        self.write({'state': 'cancel'})
+        return True
+
+    def action_set_to_done(self):
+        self.write({'state': 'done'})
+        return True
+
     # --- auto-sync portofolio: setiap transaksi berubah, portofolio diperbarui ---
     def _resync_portofolio(self):
         """Rekalkulasi portofolio dari seluruh transaksi (idempoten)."""

addons/saat_invest/views/transaksi_views.xml

@@ -4,7 +4,7 @@
         <field name="name">saat.transaksi.list</field>
         <field name="model">saat.transaksi</field>
         <field name="arch" type="xml">
-            <tree string="Transaksi Jual/Beli" create="true">
+            <tree string="Transaksi Jual/Beli" create="true" decoration-muted="state == 'cancel'">
                 <field name="tanggal"/>
                 <field name="tipe"/>
                 <field name="saham_id"/>
@@ -12,6 +12,7 @@
                 <field name="harga"/>
                 <field name="fee"/>
                 <field name="total"/>
+                <field name="state" widget="badge" decoration-success="state == 'done'" decoration-danger="state == 'cancel'"/>
             </tree>
         </field>
     </record>
@@ -21,6 +22,11 @@
         <field name="model">saat.transaksi</field>
         <field name="arch" type="xml">
             <form string="Transaksi">
+                <header>
+                    <button name="action_cancel" string="Batalkan Transaksi" type="object" class="btn-danger" invisible="state == 'cancel'"/>
+                    <button name="action_set_to_done" string="Set Selesai" type="object" invisible="state != 'cancel'"/>
+                    <field name="state" widget="statusbar" statusbar_visible="done,cancel"/>
+                </header>
                 <sheet>
                     <group>
                         <group>

flutter/lib/home_screen.dart

@@ -304,6 +304,44 @@ class _TransaksiTabState extends State<_TransaksiTab> {
 
   void _reload() => setState(() => _fut = widget.service.listTransaksi());
 
+  Future<void> _cancelTransaksi(Transaksi t) async {
+    final confirm = await showDialog<bool>(
+      context: context,
+      builder: (ctx) => AlertDialog(
+        title: const Text('Batalkan Transaksi'),
+        content: Text(
+            'Apakah Anda yakin ingin membatalkan transaksi ${t.sahamTicker} (${t.volume} lembar)?'),
+        actions: [
+          TextButton(
+            onPressed: () => Navigator.of(ctx).pop(false),
+            child: const Text('Batal'),
+          ),
+          FilledButton(
+            style: FilledButton.styleFrom(backgroundColor: Colors.red),
+            onPressed: () => Navigator.of(ctx).pop(true),
+            child: const Text('Ya, Batalkan'),
+          ),
+        ],
+      ),
+    );
+
+    if (confirm == true) {
+      try {
+        await widget.service.cancelTransaksi(t.id);
+        if (!mounted) return;
+        ScaffoldMessenger.of(context).showSnackBar(
+          const SnackBar(content: Text('Transaksi berhasil dibatalkan')),
+        );
+        _reload();
+      } catch (e) {
+        if (!mounted) return;
+        ScaffoldMessenger.of(context).showSnackBar(
+          SnackBar(content: Text('Gagal membatalkan transaksi: $e')),
+        );
+      }
+    }
+  }
+
   @override
   Widget build(BuildContext context) {
     return FutureBuilder<List<Transaksi>>(
@@ -327,15 +365,76 @@ class _TransaksiTabState extends State<_TransaksiTab> {
                       itemBuilder: (c, i) {
                         final t = list[i];
                         final buy = t.isBuy;
+                        final isCancelled = t.isCancelled;
                         return ListTile(
                           leading: CircleAvatar(
-                              backgroundColor:
-                                  buy ? Colors.green : Colors.red,
+                              backgroundColor: isCancelled
+                                  ? Colors.grey
+                                  : (buy ? Colors.green : Colors.red),
                               child: Text(buy ? 'B' : 'J',
                                   style: const TextStyle(color: Colors.white))),
-                          title: Text('${t.sahamTicker}  ·  ${t.volume} lbr'),
-                          subtitle: Text('${t.tanggal} @ ${_fmt(t.harga)} · fee ${_fmt(t.fee)}'),
-                          trailing: Text(_fmt(t.total)),
+                          title: Row(
+                            children: [
+                              Text(
+                                '${t.sahamTicker}  ·  ${t.volume} lbr',
+                                style: TextStyle(
+                                  decoration: isCancelled
+                                      ? TextDecoration.lineThrough
+                                      : null,
+                                  color: isCancelled ? Colors.grey : null,
+                                ),
+                              ),
+                              if (isCancelled) ...[
+                                const SizedBox(width: 8),
+                                Container(
+                                  padding: const EdgeInsets.symmetric(
+                                      horizontal: 6, vertical: 2),
+                                  decoration: BoxDecoration(
+                                    color: Colors.red.shade100,
+                                    borderRadius: BorderRadius.circular(4),
+                                  ),
+                                  child: Text(
+                                    'Dibatalkan',
+                                    style: TextStyle(
+                                      fontSize: 10,
+                                      color: Colors.red.shade900,
+                                      fontWeight: FontWeight.bold,
+                                    ),
+                                  ),
+                                ),
+                              ],
+                            ],
+                          ),
+                          subtitle: Text(
+                            '${t.tanggal} @ ${_fmt(t.harga)} · fee ${_fmt(t.fee)}',
+                            style: TextStyle(
+                              decoration: isCancelled
+                                  ? TextDecoration.lineThrough
+                                  : null,
+                              color: isCancelled ? Colors.grey : null,
+                            ),
+                          ),
+                          trailing: Row(
+                            mainAxisSize: MainAxisSize.min,
+                            children: [
+                              Text(
+                                _fmt(t.total),
+                                style: TextStyle(
+                                  decoration: isCancelled
+                                      ? TextDecoration.lineThrough
+                                      : null,
+                                  color: isCancelled ? Colors.grey : null,
+                                ),
+                              ),
+                              if (!isCancelled)
+                                IconButton(
+                                  icon: const Icon(Icons.cancel_outlined,
+                                      color: Colors.redAccent, size: 20),
+                                  tooltip: 'Batalkan transaksi',
+                                  onPressed: () => _cancelTransaksi(t),
+                                ),
+                            ],
+                          ),
                         );
                       },
                     ),

flutter/lib/models/saham_transaksi.dart

@@ -26,6 +26,7 @@ class Transaksi {
   final double harga;
   final double fee;
   final double total;
+  final String state; // done | cancel
 
   Transaksi({
     required this.id,
@@ -37,9 +38,11 @@ class Transaksi {
     required this.harga,
     required this.fee,
     required this.total,
+    this.state = 'done',
   });
 
   bool get isBuy => tipe == 'buy';
+  bool get isCancelled => state == 'cancel';
 
   factory Transaksi.fromOdoo(Map<String, dynamic> m) {
     String ticker = '';
@@ -58,6 +61,7 @@ class Transaksi {
       harga: (m['harga'] as num?)?.toDouble() ?? 0,
       fee: (m['fee'] as num?)?.toDouble() ?? 0,
       total: (m['total'] as num?)?.toDouble() ?? 0,
+      state: (m['state'] as String?) ?? 'done',
     );
   }
 }

flutter/lib/services/stock_service.dart

@@ -56,7 +56,7 @@ class StockService {
 
   // ---- Transaksi ----
   Future<List<Transaksi>> listTransaksi() async {
-    final fields = ['tanggal', 'tipe', 'saham_id', 'volume', 'harga', 'fee', 'total'];
+    final fields = ['tanggal', 'tipe', 'saham_id', 'volume', 'harga', 'fee', 'total', 'state'];
     final res = await client.executeSearchReadPayload("saat.transaksi", fields);
     return (res as List).map((e) => Transaksi.fromOdoo(e as Map<String, dynamic>)).toList();
   }
@@ -81,6 +81,10 @@ class StockService {
     return (res as num).toInt();
   }
 
+  Future<void> cancelTransaksi(int id) async {
+    await client.executeKwd('saat.transaksi', 'action_cancel', args: [[id]]);
+  }
+
   Future<void> hapusTransaksi(List<int> ids) async {
     await client.executeKwd('saat.transaksi', 'unlink', args: [ids]);
   }

flutter/test/widget_test.dart

@@ -7,7 +7,9 @@ import 'package:saat_invest_mobile/main.dart';
 void main() {
   testWidgets('App boot shows Saat Invest login', (tester) async {
     await tester.pumpWidget(const SaatInvestApp());
-    await tester.pumpAndSettle();
+    await tester.pump();
+    await tester.pump();
+    await tester.pump(const Duration(seconds: 1));
     expect(find.text('Masuk'), findsOneWidget);
   });
 }