lajayuhniyarsyah / ERP-Supra

ERP Supra with OpenERP
0 stars 11 forks source link

Sale Order Check Shipping and Invoicing #45

Open lajayuhniyarsyah opened 8 years ago

lajayuhniyarsyah commented 8 years ago

@Mohammadrezafah :

Bikin method dimana untuk mengecek dan update state Sale Order

lajayuhniyarsyah commented 8 years ago

untuk cek invoice berikut relasi nya

Image invoice relation so

Mohammadrezafah commented 8 years ago

@lajayuhniyarsyah : mas jay apakah seperti ini ?

689022db0845613360c6baf0312a31d397e64438

lajayuhniyarsyah commented 8 years ago

@Mohammadrezafah : kalo di commit dikasih nomor issue nya aja di message commit nya

contoh : git commit -m "Messege , issue #45

Mohammadrezafah commented 8 years ago

@lajayuhniyarsyah : okey lain kali seperti itu

lajayuhniyarsyah commented 8 years ago

@Mohammadrezafah : kalau dilihat dari script nya flow nya gini :

cek invoice,, ada invoice nya ga kalau invoice nya sudah ada yang paid maka True

lalu di bawah di cek kalau invoice sudah True maka cek shipped qty nya apa sudah semua,, kalau sudah semua shipped maka sale order "done"

hal tersebut salah dan dapat menimbulkan bug seperti ini:

Ketika so sudah di bayar DP (Belum pelunasan) lalu DN sudah shipped semua maka dari flow di atas SO akan "DONE"

Yang perlu digarisbawahi

SO done jika :

Mohammadrezafah commented 8 years ago

@lajayuhniyarsyah : mas jay apakah seperti ini ? b6bfb1353ccf46e044d4f6bfa07e31736cc32add

lajayuhniyarsyah commented 8 years ago

https://github.com/lajayuhniyarsyah/ERP-Supra/commit/b6bfb1353ccf46e044d4f6bfa07e31736cc32add#commitcomment-16108063

lajayuhniyarsyah commented 8 years ago

Oh iya ga lihat atasnya

if sale_order.picking_ids:
            #menghitung total amount di invoice
            for hitung in sale_order.picking_ids:
                amount_total+=hitung.invoice_id.amount_total

Itu berarti di hitung semua baik invoice cancel atau draft ??

Mohammadrezafah commented 8 years ago

iya begitu, harus nya jangan ya ?? klo gt saya kasih kondisi lagi

lajayuhniyarsyah commented 8 years ago

Ok kasih kondisi yang padi aja di hitungnya