arc/docs/INVOICE_CHECK_ANALYSIS.md

# بررسی ثبت چک در فاکتورها

## خلاصه
در فاکتورها، چک به صورت **غیرمستقیم** ثبت می‌شود. یعنی:
1. چک در بخش `payments` فاکتور ثبت می‌شود
2. یک سند دریافت/پرداخت مرتبط با فاکتور ایجاد می‌شود
3. چک در آن سند دریافت/پرداخت ثبت می‌شود

## جریان ثبت چک در فاکتور

### 1. ثبت فاکتور (`create_invoice`)
- خط 1025-1124: ثبت‌های حسابداری فاکتور (بدون چک)
- خط 1275-1380: اگر `payments` موجود باشد، برای هر چک:
  - خط 1322-1329: بررسی وجود چک و تطابق ارز
  - خط 1335-1347: ایجاد `account_line` برای چک
  - خط 1373: فراخوانی `create_receipt_payment` برای ایجاد سند دریافت/پرداخت

### 2. ایجاد سند دریافت/پرداخت مرتبط
- خط 1350: تعیین نوع سند بر اساس نوع فاکتور:
  - `INVOICE_SALES`, `INVOICE_PURCHASE_RETURN` → `receipt`
  - سایر انواع → `payment`
- خط 1373: فراخوانی `create_receipt_payment` با `account_lines` شامل چک

## سناریوهای مختلف

### سناریو 1: فاکتور فروش + چک دریافتی ✅
**جریان:**
1. چک دریافتی قبلاً ثبت شده (حساب 10403 بدهکار، حساب 10401 بستانکار)
2. فاکتور فروش ثبت می‌شود:
   - حساب 10401 (شخص) بدهکار می‌شود
   - حساب درآمد بستانکار می‌شود
3. سند دریافت ایجاد می‌شود:
   - چک در `account_lines` قرار می‌گیرد
   - `is_receipt = True`
   - حساب 10403 (اسناد دریافتنی) بستانکار می‌شود ✅
   - حساب 10401 (شخص) بستانکار می‌شود ✅
   - وضعیت چک به `DEPOSITED` یا `CLEARED` تغییر می‌کند ✅

**نتیجه:** ✅ صحیح - تداخلی وجود ندارد

---

### سناریو 2: فاکتور خرید + چک پرداختی ✅
**جریان:**
1. چک پرداختی قبلاً ثبت شده (حساب 20201 بدهکار، حساب 20202 بستانکار)
2. فاکتور خرید ثبت می‌شود:
   - حساب 20201 (شخص) بستانکار می‌شود
   - حساب هزینه بدهکار می‌شود
3. سند پرداخت ایجاد می‌شود:
   - چک در `account_lines` قرار می‌گیرد
   - `is_receipt = False`
   - حساب 20202 (اسناد پرداختنی) بدهکار می‌شود ✅
   - حساب 20201 (شخص) بدهکار می‌شود ✅
   - وضعیت چک به `CLEARED` تغییر می‌کند ✅

**نتیجه:** ✅ صحیح - تداخلی وجود ندارد

---

### سناریو 3: برگشت از فروش + چک دریافتی ✅
**جریان:**
1. چک دریافتی قبلاً ثبت شده
2. برگشت از فروش ثبت می‌شود:
   - حساب 10401 (شخص) بستانکار می‌شود
   - حساب برگشت از فروش بدهکار می‌شود
3. سند دریافت ایجاد می‌شود:
   - چک در `account_lines` قرار می‌گیرد
   - `is_receipt = True`
   - حساب 10403 (اسناد دریافتنی) بستانکار می‌شود ✅
   - حساب 10401 (شخص) بستانکار می‌شود ✅

**نتیجه:** ✅ صحیح - تداخلی وجود ندارد

---

### سناریو 4: برگشت از خرید + چک پرداختی ✅
**جریان:**
1. چک پرداختی قبلاً ثبت شده
2. برگشت از خرید ثبت می‌شود:
   - حساب 20201 (شخص) بدهکار می‌شود
   - حساب برگشت از خرید بستانکار می‌شود
3. سند پرداخت ایجاد می‌شود:
   - چک در `account_lines` قرار می‌گیرد
   - `is_receipt = False`
   - حساب 20202 (اسناد پرداختنی) بدهکار می‌شود ✅
   - حساب 20201 (شخص) بدهکار می‌شود ✅

**نتیجه:** ✅ صحیح - تداخلی وجود ندارد

---

## بررسی مشکلات احتمالی

### مشکل 1: بررسی نوع چک ❓
**موقعیت:** `invoice_service.py` خط 1322-1329
- فقط وجود چک و تطابق ارز بررسی می‌شود
- **نوع چک بررسی نمی‌شود!**
- آیا باید چک دریافتی فقط در فاکتور فروش استفاده شود؟
- آیا باید چک پرداختی فقط در فاکتور خرید استفاده شود؟

**مثال مشکل:**
- چک دریافتی در فاکتور خرید استفاده می‌شود
- `is_receipt = False` → سند پرداخت ایجاد می‌شود
- در `create_receipt_payment`: `ALLOWED_PAYMENT_CHECK_STATUSES = {TRANSFERRED_ISSUED}`
- چک دریافتی وضعیت `RECEIVED_ON_HAND` دارد
- **validation رد می‌شود!** ✅ (این مشکل نیست، سیستم جلوگیری می‌کند)

**اما:**
- اگر validation نباشد یا از validation عبور کند:
  - چک دریافتی در سند پرداخت استفاده می‌شود
  - حساب 20202 بدهکار می‌شود (اشتباه!)
  - باید حساب 10403 بستانکار می‌شد

### مشکل 2: بررسی وضعیت چک ❓
**موقعیت:** `invoice_service.py` خط 1322-1329
- فقط وجود چک بررسی می‌شود
- **وضعیت چک بررسی نمی‌شود!**
- آیا چک باید `RECEIVED_ON_HAND` باشد؟
- آیا چک باید `TRANSFERRED_ISSUED` باشد؟

**جواب:** 
- این بررسی در `create_receipt_payment` انجام می‌شود (خط 224)
- `ALLOWED_RECEIPT_CHECK_STATUSES = {RECEIVED_ON_HAND}`
- `ALLOWED_PAYMENT_CHECK_STATUSES = {TRANSFERRED_ISSUED}`
- ✅ پس مشکل نیست

### مشکل 3: تداخل با ثبت اولیه چک ❌
**موقعیت:** چک قبلاً در `create_check` ثبت شده
- خط 146: حساب 10403 بدهکار می‌شود (چک دریافتی)
- خط 147: حساب 10401 بستانکار می‌شود

**در فاکتور فروش:**
- خط 1035: حساب 10401 بدهکار می‌شود (فاکتور فروش)
- در سند دریافت مرتبط:
  - حساب 10403 بستانکار می‌شود (چک خارج شد) ✅
  - حساب 10401 بستانکار می‌شود (تسویه بدهی) ✅

**نتیجه:** ✅ تداخلی وجود ندارد - منطق صحیح است

---

## خلاصه مشکلات

### ✅ چیزهایی که درست هستند:
1. چک در سند دریافت/پرداخت مرتبط ثبت می‌شود (نه مستقیم در فاکتور)
2. منطق حسابداری در `create_receipt_payment` درست است
3. تغییر وضعیت چک انجام می‌شود
4. validation وضعیت چک در `create_receipt_payment` انجام می‌شود

### ⚠️ مشکلات احتمالی:

#### 1. عدم بررسی نوع چک در فاکتور (کم اهمیت)
**مشکل:** در `invoice_service.py` نوع چک بررسی نمی‌شود
**راه‌حل:** می‌توان validation اضافه کرد که:
- چک دریافتی فقط در فاکتور فروش/برگشت از فروش استفاده شود
- چک پرداختی فقط در فاکتور خرید/برگشت از خرید استفاده شود

**اما:** این validation فعلاً در `create_receipt_payment` انجام می‌شود (از طریق وضعیت چک)

#### 2. عدم validation مستقیم در فاکتور (کم اهمیت)
**مشکل:** اگر validation در `create_receipt_payment` شکست بخورد، خطا مبهم است
**راه‌حل:** می‌توان validation را در `create_invoice` هم اضافه کرد

---

## نتیجه‌گیری

**وضعیت کلی:** ✅ **خوب**

مشکلات اصلی که در اسناد دریافت/پرداخت و هزینه/درآمد وجود داشت، در فاکتورها وجود ندارد چون:
1. چک به صورت مستقیم در فاکتور ثبت نمی‌شود
2. منطق حسابداری از طریق `create_receipt_payment` انجام می‌شود که قبلاً اصلاح شده
3. تغییر وضعیت چک در `create_receipt_payment` انجام می‌شود

**پیشنهادات بهبود (اختیاری):**
1. اضافه کردن validation نوع چک در `create_invoice` برای خطای واضح‌تر
2. اضافه کردن validation وضعیت چک در `create_invoice` برای خطای واضح‌تر

اما این تغییرات **ضروری نیستند** چون validation در `create_receipt_payment` انجام می‌شود.