@isi-ui7/bos7-shared 0.5.1 → 0.5.3

package/dist/form-types.d.ts

@@ -105,6 +105,23 @@ export type DetailRowsConfig = {
      * Forwarded to `<EditableTable lockExistingRows>`.
      */
     lockExistingRows?: boolean;
+    /**
+     * Menyembunyikan tombol "Tambah Baris" / aksi "Hapus" per baris.
+     *
+     * 🔴 Ada karena `maxRows` dan `lockExistingRows` KEDUANYA salah untuk tabel
+     * yang barisnya ditentukan master tapi NILAINYA diisi petugas (limit
+     * transaksi CDD): `maxRows` men-DISABLE tombolnya (ia tetap ada, dan
+     * kontrol yang ada-tapi-nol-pernah-bekerja mengajari petugas bahwa fiturnya
+     * rusak), `lockExistingRows` membuat baris termuat READ-ONLY seluruhnya.
+     *
+     * ⚠️ Keduanya diteruskan APA ADANYA ke `<EditableTable>` (0.2.4+). Prop yang
+     * dideklarasikan di sini tapi NOL diteruskan akan lolos tsc, lolos uji tipe,
+     * dan mencapai NOL — itu persis keadaan yang berkas ini perbaiki, dan
+     * `detail-rows-hide-buttons.test.tsx` menguji SAMPAI (tombol hilang di DOM),
+     * bukan ADA (prop hadir di objek config).
+     */
+    hideAddButton?: boolean;
+    hideDeleteAction?: boolean;
 };
 /**
  * Satu kolom pada tabel RINGKAS `detail-modal`.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@isi-ui7/bos7-shared",
-  "version": "0.5.1",
+  "version": "0.5.3",
   "description": "Shared auth7 and layout primitives for bos7 applications.",
   "license": "MIT",
   "repository": {
@@ -70,9 +70,9 @@
   "peerDependencies": {
     "@carbon/icons-react": "^11.71.0",
     "@carbon/react": "^1.97.0",
-    "@isi-ui7/corporate-themes": ">=0.2.19",
+    "@isi-ui7/corporate-themes": ">=0.2.21",
     "@isi-ui7/data-table": ">=0.2.5",
-    "@isi-ui7/editable-table": ">=0.2.3",
+    "@isi-ui7/editable-table": ">=0.2.4",
     "@isi-ui7/i18n": ">=0.2.4",
     "@isi-ui7/lookup-input": ">=0.2.4",
     "@isi-ui7/modal-manager": ">=0.2.0",
@@ -106,14 +106,14 @@
     "vite": "^5.0.0",
     "vite-plugin-dts": "^3.6.0",
     "vitest": "^1.0.0",
-    "@isi-ui7/corporate-themes": "0.2.19",
-    "@isi-ui7/i18n": "0.2.4",
     "@isi-ui7/data-table": "0.2.5",
+    "@isi-ui7/editable-table": "0.2.4",
+    "@isi-ui7/i18n": "0.2.4",
     "@isi-ui7/lookup-input": "0.2.4",
-    "@isi-ui7/modal-manager": "0.2.2",
-    "@isi-ui7/editable-table": "0.2.3",
     "@isi-ui7/realtime": "0.2.3",
-    "@isi-ui7/ui-shell": "0.2.8"
+    "@isi-ui7/ui-shell": "0.2.8",
+    "@isi-ui7/modal-manager": "0.2.2",
+    "@isi-ui7/corporate-themes": "0.2.21"
   },
   "dependencies": {
     "@carbon/icons-react": "^11.71.0",
@@ -127,8 +127,8 @@
     "jspdf": "^4.2.1",
     "jspdf-autotable": "^5.0.7",
     "@isi-ui7/data-table": "0.2.5",
-    "@isi-ui7/lookup-input": "0.2.4",
-    "@isi-ui7/editable-table": "0.2.3"
+    "@isi-ui7/editable-table": "0.2.4",
+    "@isi-ui7/lookup-input": "0.2.4"
   },
   "scripts": {
     "build": "vite build",

package/src/detail-rows-hide-buttons.test.tsx

@@ -0,0 +1,109 @@
+import { afterEach, describe, expect, it } from "vitest";
+import { cleanup, render, screen } from "@testing-library/react";
+import { SchemaFormRenderer } from "./form-renderer";
+import type { FormSection } from "./form-types";
+
+afterEach(cleanup);
+
+type D = Record<string, unknown>;
+
+// `hideAddButton` / `hideDeleteAction` menembus `detail-rows` sampai ke
+// `<EditableTable>`.
+//
+// 🔴 Kenapa berkas ini menguji DOM dan bukan objek config.
+//
+// Kedua prop mendarat di `@isi-ui7/editable-table` 0.2.4 dengan ujinya sendiri
+// (28 hijau, sabotase dua arah). Lalu NOL SEORANG PUN dapat memakainya: layar
+// nol merender `<EditableTable>` langsung — ia lewat `detail-rows` di paket
+// ini, dan paket ini nol mengenal kedua prop itu. Menyetelnya di skema
+// konsumen mencapai NOL, dan ia terlihat seperti berhasil: tsc hijau, uji
+// hijau, tombolnya tetap ada.
+//
+// ⚠️ Prop yang dideklarasikan di `DetailRowsConfig` tapi nol diteruskan di
+// `form-renderer.tsx` menghasilkan keadaan yang SAMA, satu lapis lebih tinggi
+// — dan uji yang mengasersi "propnya ada di config" akan HIJAU sepanjang
+// waktu. Karena itu asersinya: tombolnya HILANG DI DOM.
+//
+// ⚠️ Dan lengan SEBALIKNYA wajib: prop yang SELALU menyembunyikan lolos uji
+// yang cuma memeriksa `true`.
+
+const BARIS = [{ jenis: "Tarik Tunai", nominal: "1000000" }];
+
+function sectionDgn(detail: Record<string, unknown>): FormSection<D> {
+  return {
+    title: "Limit",
+    fields: [
+      {
+        key: "limits",
+        label: "Limit Transaksi",
+        type: "detail-rows",
+        span: 12,
+        detailRows: {
+          columns: [
+            { field: "jenis", header: "Jenis", columnType: { type: "text" } },
+            { field: "nominal", header: "Nominal", columnType: { type: "text" } },
+          ],
+          newRowFactory: () => ({ jenis: "", nominal: "" }),
+          ...detail,
+        },
+      },
+    ],
+  };
+}
+
+function renderDgn(detail: Record<string, unknown> = {}) {
+  return render(
+    <SchemaFormRenderer<D>
+      mode="edit"
+      value={{ limits: BARIS }}
+      sections={[sectionDgn(detail)]}
+      onChange={() => {}}
+    />,
+  );
+}
+
+describe("detail-rows meneruskan hideAddButton / hideDeleteAction ke EditableTable", () => {
+  it("BAWAAN (nol diberikan): KEDUA kontrol tampil", () => {
+    // 🔴 Kontrol positif untuk kedua uji di bawah. Tanpa lengan ini, sebuah
+    // perender yang menyembunyikan SEGALANYA lulus keduanya — dan begitu juga
+    // perender yang kebetulan nol merender tabelnya sama sekali.
+    renderDgn();
+    expect(screen.getByText("Tambah Baris")).toBeTruthy();
+    expect(screen.getAllByLabelText("Hapus").length).toBe(1);
+    // Dan barisnya memang ADA: kalau tabelnya kosong, "1 tombol Hapus" nol
+    // dapat terjadi, jadi asersi ini juga menjaga penyebutnya.
+    expect(screen.getByText("Tarik Tunai")).toBeTruthy();
+  });
+
+  it("hideAddButton: Tambah HILANG di DOM, Hapus TETAP", () => {
+    renderDgn({ hideAddButton: true });
+    expect(screen.queryByText("Tambah Baris")).toBeNull();
+    // Lengan yang membuktikan keduanya TERPISAH saat menyeberangi lapis ini —
+    // satu flag yang menyembunyikan dua hal lolos tanpa ini.
+    expect(screen.getAllByLabelText("Hapus").length).toBe(1);
+  });
+
+  it("hideDeleteAction: Hapus HILANG di DOM, Tambah TETAP", () => {
+    renderDgn({ hideDeleteAction: true });
+    expect(screen.queryAllByLabelText("Hapus").length).toBe(0);
+    expect(screen.getByText("Tambah Baris")).toBeTruthy();
+  });
+
+  it("false eksplisit ⇒ KEDUANYA tetap tampil", () => {
+    // Membedakan "diteruskan apa adanya" dari "diteruskan sebagai truthy".
+    // `hideAddButton={detail.hideAddButton ?? true}` lolos ketiga uji di atas.
+    renderDgn({ hideAddButton: false, hideDeleteAction: false });
+    expect(screen.getByText("Tambah Baris")).toBeTruthy();
+    expect(screen.getAllByLabelText("Hapus").length).toBe(1);
+  });
+
+  it("keduanya menyala: nol ada kontrol baris, TAPI selnya tetap tersunting", () => {
+    // Pembedaan yang paling mudah hilang: `readOnly` juga menyembunyikan
+    // kedua tombol — tapi ia MEMBEKUKAN selnya, dan itu cacat yang persis
+    // baru diperbaiki di layar limit transaksi.
+    renderDgn({ hideAddButton: true, hideDeleteAction: true });
+    expect(screen.queryByText("Tambah Baris")).toBeNull();
+    expect(screen.queryAllByLabelText("Hapus").length).toBe(0);
+    expect(screen.getByText("Tarik Tunai")).toBeTruthy();
+  });
+});

package/src/form-renderer.tsx

@@ -661,6 +661,8 @@ export function SchemaFormRenderer<TData extends Record<string, unknown>>({
                           maxRows={detail.maxRows}
                           readOnly={effectiveReadonly}
                           lockExistingRows={detail.lockExistingRows}
+                          hideAddButton={detail.hideAddButton}
+                          hideDeleteAction={detail.hideDeleteAction}
                           validateRow={detail.validateRow}
                           lookupCellRenderer={(args) => {
                             const ct = args.column.columnType;

package/src/form-types.ts

@@ -112,6 +112,24 @@ export type DetailRowsConfig = {
    * Forwarded to `<EditableTable lockExistingRows>`.
    */
   lockExistingRows?: boolean;
+
+  /**
+   * Menyembunyikan tombol "Tambah Baris" / aksi "Hapus" per baris.
+   *
+   * 🔴 Ada karena `maxRows` dan `lockExistingRows` KEDUANYA salah untuk tabel
+   * yang barisnya ditentukan master tapi NILAINYA diisi petugas (limit
+   * transaksi CDD): `maxRows` men-DISABLE tombolnya (ia tetap ada, dan
+   * kontrol yang ada-tapi-nol-pernah-bekerja mengajari petugas bahwa fiturnya
+   * rusak), `lockExistingRows` membuat baris termuat READ-ONLY seluruhnya.
+   *
+   * ⚠️ Keduanya diteruskan APA ADANYA ke `<EditableTable>` (0.2.4+). Prop yang
+   * dideklarasikan di sini tapi NOL diteruskan akan lolos tsc, lolos uji tipe,
+   * dan mencapai NOL — itu persis keadaan yang berkas ini perbaiki, dan
+   * `detail-rows-hide-buttons.test.tsx` menguji SAMPAI (tombol hilang di DOM),
+   * bukan ADA (prop hadir di objek config).
+   */
+  hideAddButton?: boolean;
+  hideDeleteAction?: boolean;
 };
 
 /**