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

Bug: di surat kematian, drop-down konfirmasi yang meninggal keliru menampilkan nama untuk kolom alamat #956

Closed
eddieridwan opened this Issue Apr 25, 2018 · 5 comments

Comments

2 participants
@eddieridwan
Collaborator

eddieridwan commented Apr 25, 2018

Dilaporkan @balongbesuk:

untitled23

alamat pemohon surat menampilkan nama,

@egodasa

This comment has been minimized.

Contributor

egodasa commented Apr 26, 2018

Saya rasa penyebab alamat menampilkan nama karena ini : https://github.com/OpenSID/OpenSID/blob/master/donjo-app/models/Penduduk_model.php#L1096

Namun pada query https://github.com/OpenSID/OpenSID/blob/master/donjo-app/models/Penduduk_model.php#L1091 tidak menghasilkan alamat sehingga tidak ada alamat yang ditampilkan.

Dan sebenarnya di baris itu juga, kita bisa menggunakan concat untuk memformat output dari sql tanpa perlu looping lagi kedata yang sudah kita dapatkan tersebut.

@egodasa

This comment has been minimized.

Contributor

egodasa commented Apr 26, 2018

Kira-kira seperti ini menurut Saya seharusnya https://gist.github.com/egodasa/d288770ea6ae0115213c4feff8d8018f Namun diquery asli tersebut yaitu tabel tweb_penduduk, hanya ada kolom "alamat_sebelumnya" dan "alamat_sekarang". Dan di Penduduk_model, diperlukan kolom "alamat".

Dan diskrip yang Saya buat tersebut, alamat saya ganti sementar dengan nik karena kolom alamat tidak ada di tabel tweb_indonesia.

@eddieridwan

This comment has been minimized.

Collaborator

eddieridwan commented Apr 26, 2018

@egodasa : lihat contoh di surat lain, misalnya surat_ket_penduduk. Sebaiknya diseragamkan saja dengan surat yang lain. Di situ 'alamat' menampilkan RW, RW dan Dusun berdasarkan KK penduduk tersebut.

@egodasa

This comment has been minimized.

Contributor

egodasa commented Apr 26, 2018

bug_956
Seperti ini bukan Mas?
Querynya Saya ambil dari Surat_model.php dengan penambahan kolom "alamat" yang merupakan gabungan rt,rw dan dusun.

egodasa added a commit to egodasa/OpenSID that referenced this issue Apr 26, 2018

eddieridwan added a commit that referenced this issue Apr 28, 2018

@eddieridwan eddieridwan moved this from DIPRIORITASKAN to SUDAH DI MASTER in Rilis yang sedang dikerjakan Apr 28, 2018

@eddieridwan

This comment has been minimized.

Collaborator

eddieridwan commented Apr 28, 2018

Sudah dicommit ke master melalui PR #960

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment