update documentation

2026-09-17 20:06:25 +07:00
9 changed files with 74 additions and 243 deletions
--- a/.gitignore
+++ b/.gitignore
@@ -10,6 +10,5 @@ __pycache__/
 # Terraform / backup dll
 *.bak
 *.log
-backups/

 .github/
--- a/README.md
+++ b/README.md
@@ -30,7 +30,7 @@ saat-app/

 ## Cara menjalankan
 ```bash
-cd /home/ardhi/workspace/erp/odoo-app/saat-app
+cd /home/ardhi/Application/saat-app
 docker compose config      # validasi
 docker compose up -d       # start
 ```
@@ -38,6 +38,70 @@ docker compose up -d       # start
 Akses UI: http://localhost:18069 — login `admin` dengan password
 `ODOO_ADMIN_PASSWORD` di `.env`.

+## Troubleshooting
+
+### `PermissionError: /var/lib/odoo/sessions`
+
+Odoo 17 berjalan sebagai user dengan UID/GID `101`. Perbaiki ownership
+named volume Odoo tanpa menghapus database atau filestore:
+
+```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 bahwa user Odoo dapat menulis session:
+
+```bash
+docker exec --user odoo "$cid" sh -c \
+  'touch /var/lib/odoo/sessions/.write-check && \
+   rm /var/lib/odoo/sessions/.write-check'
+```
+
+Gunakan satu volume utama berikut pada `docker-compose.yml`; jangan membuat
+mount terpisah untuk `/var/lib/odoo/session` (singular):
+
+```yaml
+volumes:
+  - saat_odoo_filestore:/var/lib/odoo
+```
+
+### `column res_lang.short_time_format does not exist`
+
+Pastikan versi image Odoo sama dengan versi database. Database proyek ini
+adalah Odoo 17, jadi gunakan:
+
+```yaml
+image: odoo:17.0
+```
+
+Jangan menjalankan database Odoo 17 dengan image Odoo 18 tanpa proses upgrade
+major-version yang resmi.
+
+### Tidak bisa login sebagai `admin`
+
+Jika log menunjukkan `Login failed` dan user `admin` aktif, reset password
+melalui ORM Odoo menggunakan nilai `ODOO_ADMIN_PASSWORD` dari `.env`:
+
+```bash
+set -a && source .env && set +a
+docker compose exec -T -e RESET_PASSWORD="$ODOO_ADMIN_PASSWORD" odoo \
+  odoo shell -d "$POSTGRES_DB" --db_host=db --db_port=5432 \
+  --db_user="$POSTGRES_USER" --db_password="$POSTGRES_PASSWORD" --no-http <<'PY'
+import os
+user = env['res.users'].search([('login', '=', 'admin')], limit=1)
+if not user.exists() or not user.active:
+    raise RuntimeError('Active admin user was not found')
+user.write({'password': os.environ['RESET_PASSWORD']})
+env.cr.commit()
+print('Admin password reset')
+PY
+```
+
+Setelah itu login dengan user `admin` dan password `ODOO_ADMIN_PASSWORD`.
+
 ## Status
 - ✅ Odoo 17 container jalan (port 18069), DB PostgreSQL 16 healthy
 - ✅ JSON-RPC aktif (uid `admin` verified) — endpoint yang akan dipakai Flutter
@@ -67,110 +131,6 @@ 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`.
--- a/addons/saat_invest/models/portofolio.py
+++ b/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([('state', '=', 'done')], order='tanggal, id')
+        trans = self.env['saat.transaksi'].search([], order='tanggal, id')
        per_saham = {}
        for t in trans:
            a = per_saham.setdefault(t.saham_id.id, {
--- a/addons/saat_invest/models/transaksi.py
+++ b/addons/saat_invest/models/transaksi.py
@@ -30,11 +30,6 @@ 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:
@@ -55,14 +50,6 @@ 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)."""
--- a/addons/saat_invest/views/transaksi_views.xml
+++ b/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" decoration-muted="state == 'cancel'">
+            <tree string="Transaksi Jual/Beli" create="true">
                <field name="tanggal"/>
                <field name="tipe"/>
                <field name="saham_id"/>
@@ -12,7 +12,6 @@
                <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>
@@ -22,11 +21,6 @@
        <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>
--- a/flutter/lib/home_screen.dart
+++ b/flutter/lib/home_screen.dart
@@ -304,44 +304,6 @@ 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>>(
@@ -365,76 +327,15 @@ 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: isCancelled
-                                  ? Colors.grey
-                                  : (buy ? Colors.green : Colors.red),
+                              backgroundColor:
+                                  buy ? Colors.green : Colors.red,
                              child: Text(buy ? 'B' : 'J',
                                  style: const TextStyle(color: Colors.white))),
-                          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),
-                                ),
-                            ],
-                          ),
+                          title: Text('${t.sahamTicker}  ·  ${t.volume} lbr'),
+                          subtitle: Text('${t.tanggal} @ ${_fmt(t.harga)} · fee ${_fmt(t.fee)}'),
+                          trailing: Text(_fmt(t.total)),
                        );
                      },
                    ),
--- a/flutter/lib/models/saham_transaksi.dart
+++ b/flutter/lib/models/saham_transaksi.dart
@@ -26,7 +26,6 @@ class Transaksi {
  final double harga;
  final double fee;
  final double total;
-  final String state; // done | cancel

  Transaksi({
    required this.id,
@@ -38,11 +37,9 @@ 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 = '';
@@ -61,7 +58,6 @@ 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',
    );
  }
 }
--- a/flutter/lib/services/stock_service.dart
+++ b/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', 'state'];
+    final fields = ['tanggal', 'tipe', 'saham_id', 'volume', 'harga', 'fee', 'total'];
    final res = await client.executeSearchReadPayload("saat.transaksi", fields);
    return (res as List).map((e) => Transaksi.fromOdoo(e as Map<String, dynamic>)).toList();
  }
@@ -81,10 +81,6 @@ 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]);
  }
--- a/flutter/test/widget_test.dart
+++ b/flutter/test/widget_test.dart
@@ -7,9 +7,7 @@ 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.pump();
-    await tester.pump();
-    await tester.pump(const Duration(seconds: 1));
+    await tester.pumpAndSettle();
    expect(find.text('Masuk'), findsOneWidget);
  });
 }