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

[Bug] Tombol Paginasi? Admin Web Tidak tampil dengan benar #1411

Closed
balongbesuk opened this issue Jul 23, 2018 · 6 comments
Closed

[Bug] Tombol Paginasi? Admin Web Tidak tampil dengan benar #1411

balongbesuk opened this issue Jul 23, 2018 · 6 comments
Labels

Comments

@balongbesuk
Copy link
Contributor

Bagaimana alurnya sampai muncul masalah?

Admin Web > Artikel

Seperti apa yang diharapkan?

Fungsi Paginasi tertampil dengan benar

Apa yang terjadi?

saat ini menu paginasi pada admin web > artikel hanya memunculkan kotak kecil

Informasi tambahan

untitled1

Tanya Jawab
Versi OpenSID 2.x
Versi PHP
System operasi
@eddieridwan eddieridwan changed the title Bug? Tombol Paginasi? Admin Web Tidak tampil dengan benar [Bug] Tombol Paginasi? Admin Web Tidak tampil dengan benar Jul 24, 2018
@egodasa
Copy link
Contributor

egodasa commented Aug 6, 2018

@eddieridwan Apakah bug ini masih akan dikerjakan Mas? Karena sudah ada proyek memodernkan tampilan admin.

@eddieridwan
Copy link
Collaborator

@egodasa: masih. Tapi tunggu saya gabung dulu pekerjaan tampilan-admin ke branch master. Yaitu langsung kerjakan tampilan baru.

@eddieridwan
Copy link
Collaborator

@egodasa , saya sudah gabungkan pekerjaan tampilan-admin ke branch master. Jadi silakan kalau mau kerjakan.

@egodasa
Copy link
Contributor

egodasa commented Aug 6, 2018

@eddieridwan Saya menemukan kejanggalan dengan method xss_clean bawaan CI Mas. xss_clean tidak berjalan dengan baik jika data yang diterima adalah object. Jika data yang diterima array, maka tidak ada masalah.
Seperti yang dapat dilihat di https://github.com/OpenSID/OpenSID/blob/master/donjo-app/controllers/Web.php#L74 , $data['paging'] sebelum diproses xss_clean memiliki nilai object(Paging)#50 (12) { ["page"]=> int(1) ["per_page"]=> int(50) ["num_rows"]=> string(1) "1" ["num_page"]=> int(1) ["offset"]=> int(0) ["prev"]=> int(0) ["next"]=> int(0) ["start_link"]=> int(1) ["start"]=> int(1) ["end"]=> int(1) ["end_link"]=> int(1) ["suffix"]=> NULL }. Ternyata setelah diproses xss_clean, $data['paging'] berubah nilai menjadi "Object". Hal tersebut yang membuat issue ini muncul. Karena data paging menjadi salah.

Saya lihat di https://github.com/OpenSID/OpenSID/blob/master/donjo-app/controllers/Penduduk.php#L160 , tidak terdapat xss_clean. Jadi bagaimana Mas? Bug ini akan hilang jika https://github.com/OpenSID/OpenSID/blob/master/donjo-app/controllers/Web.php#L74 dihilangkan. Tapi data yang dientrikan user tidak akan diproses oleh xss_clean. Sama seperti di https://github.com/OpenSID/OpenSID/blob/master/donjo-app/controllers/Penduduk.php#L160.

@eddieridwan
Copy link
Collaborator

@egodasa mungkin cari solusi di mana xss_clean dilakukan secara selektif. Misalanya, pindahkan baris $data['paging'] = $this->web_artikel_model->paging($cat,$p,$o); sesudah baris xss_clean, sehingga $data['paging'] tidak terpengaruh.

eddieridwan added a commit that referenced this issue Aug 7, 2018
@eddieridwan
Copy link
Collaborator

Sudah dicommit ke master melalui PR #1561

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants