Skip to content

Fix default data pada IdentitasSeeder - #1271

Merged
analisopendesa merged 2 commits into
rilis-devfrom
fix/identitas_seeder
Aug 29, 2026
Merged

Fix default data pada IdentitasSeeder#1271
analisopendesa merged 2 commits into
rilis-devfrom
fix/identitas_seeder

Conversation

@pandigresik

Copy link
Copy Markdown
Contributor

Pull Request: Fix default data pada IdentitasSeeder

Description

Perbaikan data default pada database/seeders/IdentitasSeeder.php yang sebelumnya menggunakan data milik Kota Bima (Simatik, kode 52.72). Data spesifik ini membuat bingung pengguna karena seolah-olah identitas aplikasi sudah lengkap padahal belum tentu sesuai dengan kabupaten yang menggunakannya.

Changes made:

  1. [Bug Fix]: Mengganti nama_aplikasi dari 'Simatik' menjadi 'OpenKab'
  2. [Bug Fix]: Mengganti nama_kabupaten dari 'KOTA BIMA' menjadi 'Belum Ditentukan'
  3. [Bug Fix]: Mengganti kode_kabupaten dari '52.72' menjadi '00.00'
  4. [Bug Fix]: Mengganti nama_provinsi dari 'Nusa Tenggara Barat' menjadi 'Belum Ditentukan'
  5. [Bug Fix]: Mengganti kode_provinsi dari '52' menjadi '00'
  6. [Bug Fix]: Mengganti sebutan_kab dari 'Kota' menjadi 'Kabupaten'
  7. [Bug Fix]: Mengganti sebutan_desa dari 'Kelurahan' menjadi 'Desa'
  8. [Bug Fix]: Menghapus return statement pada method run() (seharusnya void)

Reason for change:

  • Konsistensi: Seeder seharusnya menyediakan data default generik, bukan data spesifik daerah tertentu
  • Menghindari kebingungan: Pengguna kabupaten lain (seperti sikades.bimakab.go.id) menjadi bingung karena data identitas sudah terisi dengan data Kota Bima
  • Ketepatan: Data placeholder 'Belum Ditentukan' dan '00.00' lebih jelas sebagai penanda bahwa data belum diisi

Impact of change:

Clarity: Pengguna baru langsung tahu bahwa data identitas belum diisi
Consistency: Seeder tidak lagi memihak pada daerah tertentu
Correctness: Mengembalikan return type ke void sesuai konvensi Laravel seeder

Related Issue

#1270

Steps to Reproduce

Before fix (problem):

  1. Jalankan fresh install atau fresh database migration
  2. Jalankan seeder php artisan db:seed
  3. Cek tabel identitas
  4. ❌ Data identitas terisi dengan data Kota Bima (Simatik, kode 52.72)
  5. ❌ Pengguna kabupaten lain bingung karena identitas seolah sudah lengkap

After fix (solution):

  1. Jalankan fresh install atau fresh database migration
  2. Jalankan seeder php artisan db:seed
  3. Cek tabel identitas
  4. ✅ Data identitas terisi dengan data default generik (Belum Ditentukan, 00.00)
  5. ✅ Pengguna tahu bahwa data identitas perlu diisi manual

Checklist

Technical Details

Technical Explanation

Perubahan dilakukan pada file database/seeders/IdentitasSeeder.php. Data array $defaultData diubah dari data hardcoded Kota Bima menjadi data placeholder generik. Selain itu, return statement dihapus karena method run() pada Seeder seharusnya return void.

// Before
$defaultData = [
    'nama_aplikasi' => 'Simatik',
    'nama_kabupaten' => 'KOTA BIMA',
    'kode_kabupaten' => '52.72',
    'nama_provinsi' => 'Nusa Tenggara Barat',
    'kode_provinsi' => '52',
    'sebutan_kab' => 'Kota'
];
return Identitas::create($defaultData);

// After
$defaultData = [
    'nama_aplikasi' => 'OpenKab',
    'nama_kabupaten' => 'Belum Ditentukan',
    'kode_kabupaten' => '00.00',
    'nama_provinsi' => 'Belum Ditentukan',
    'kode_provinsi' => '00',
    'sebutan_kab' => 'Kabupaten'
];
Identitas::create($defaultData);

Dependencies added

  • No new dependencies

Testing

Manual Testing

  • Jalankan fresh migration
  • Jalankan seeder php artisan db:seed
  • Verifikasi data identitas di tabel identitas adalah data generik
  • Akses aplikasi dan pastikan tidak ada error

Video

simplescreenrecorder-2026-08-27_05.49.14.mp4
simplescreenrecorder-2026-08-27_05.53.48.mp4

None

Migration Guide

Not required

References


Additional notes: Perubahan ini bersifat backward-compatible. Pengguna yang sudah memiliki data identitas tidak akan terpengaruh karena seeder hanya berjalan pada fresh install.

@github-actions

Copy link
Copy Markdown

🔄 AI PR Review sedang antri di server...

Proses review akan segera dimulai di background — hasil akan muncul sebagai komentar setelah selesai.
Powered by CrewAI · PR #1271

@analisopendesa
analisopendesa merged commit 828be02 into rilis-dev Aug 29, 2026
@analisopendesa
analisopendesa deleted the fix/identitas_seeder branch August 29, 2026 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants