laravolt/indonesia

Code districts & villages does not match with source

Opened this issue · 6 comments

novrizal commented

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:

  • Value ID pada versi lama, dengan value code pada versi baru, tidak sesuai, terutama jika mengacu ke source wilayah administratif.

Suggestions:

  • Update CSV districts & villages
  • Nama kolom saat melakukan findProvince (dan yang lain), bisa diarahkan ke kolom code via getRouteKeyName

Thank You

masadi commented

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

iya nih jadi susah make

uyab commented

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

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

  • App\Models\Provinsi
  • App\Models\Kabupaten
  • dst

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

  • 👍1
ryanaryap commented

boleh aja om

lakuapik commented

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 💪