laravolt / indonesia

Package Laravel yang berisi data Provinsi, Kabupaten/Kota, dan Kecamatan/Desa di seluruh Indonesia.
MIT License
581 stars 201 forks source link

Code districts & villages does not match with source #95

Open novrizal opened 2 years ago

novrizal commented 2 years ago

Halo,

Pada sumber ini, kecamatan Cilandak memiliki kode/id/nilai 3171030 https://github.com/edwardsamuel/Wilayah-Administratif-Indonesia/blob/master/csv/districts.csv

laravolt v0.29 = 3171030 (cocok) laravolt v0.30 = 317406 (tidak cocok), sampai dengan yang terbaru di v0.32

Test: v0.24, findProvince(31) => DKI Jakarta (saya menggunakan versi ini) v0.32, findProvince(31) => Maluku

Problem:

Suggestions:

Thank You

masadi commented 1 year ago

Karena kodenya pake find bukan pake where('code')

public function findProvince($provinceId, $with = null) { $with = (array) $with; if ($with) { $withVillages = array_search('villages', $with); if ($withVillages !== false) { unset($with[$withVillages]); $province = Models\Province::with($with)->find($provinceId); $province = $this->loadRelation($province, 'cities.districts.villages'); } else { $province = Models\Province::with($with)->find($provinceId); } return $province; } return Models\Province::find($provinceId); } yang dicari by code, tapi query by id ga nyambung result nya

ryanaryap commented 1 year ago

iya nih jadi susah make

uyab commented 1 year ago

Ini kalau mengubah param findProvince dari id menjadi kode bakal jadi major changes. Opsi yang terpikirkan:

  1. Tambahkan config primary_key, default id. Jika ingin mengubah agar findProvince mencari ke kolom code, maka diganti jadi code.

Ada feedback ga kalau seperti itu @masadi @ryanaryap @novrizal ?

masadi commented 1 year ago

@uyab boleh aja om kalo mau major changes. Tapi kalo saya bikin model baru,

Sesuai kebutuhan saya, jadi eloquent nya gak manggil fungsi dari laravolt... hehe

ryanaryap commented 1 year ago

boleh aja om

lakuapik commented 1 year ago

pak @uyab dan tim terima kasih sblmnya untuk package ini; saya kebetulan fork package ini di https://github.com/kodepandai/laravel-indonesia dan ambil data sumbernya dari repo ini. ditunggu update data terbarunya 💪