Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Code districts & villages does not match with source #95

Open
novrizal opened this issue Sep 5, 2022 · 6 comments
Open

Code districts & villages does not match with source #95

novrizal opened this issue Sep 5, 2022 · 6 comments

Comments

@novrizal
Copy link

novrizal commented Sep 5, 2022

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
Copy link

masadi commented Mar 22, 2023

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
Copy link

iya nih jadi susah make

@uyab
Copy link
Contributor

uyab commented Jun 12, 2023

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
Copy link

masadi commented Jun 13, 2023

@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

@ryanaryap
Copy link

boleh aja om

@lakuapik
Copy link
Contributor

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 💪

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

No branches or pull requests

5 participants