From 2ab207ccd9c1e90d15f67945846d5bbf2974549d Mon Sep 17 00:00:00 2001
From: Suherdy Yacob <suherdy.yacob@mapan.co.id>
Date: Thu, 8 Jan 2026 13:08:48 +0700
Subject: [PATCH] fix the revaluation  bugs

---
 __manifest__.py                               |   2 +-
 .../purchase_bill_sync_wizard.cpython-312.pyc | Bin 11347 -> 13070 bytes
 wizard/purchase_bill_sync_wizard.py           |  37 +++++++++++++++++-
 3 files changed, 37 insertions(+), 2 deletions(-)

diff --git a/__manifest__.py b/__manifest__.py
index 6489dce..0943e39 100644
--- a/__manifest__.py
+++ b/__manifest__.py
@@ -8,7 +8,7 @@
         1. Find discrepancies between Vendor Bills and linked Purchase Orders within a date range.
         2. Sync selected Vendor Bills to update the Purchase Orders.
     """,
-    'author': 'Antigravity',
+    'author': 'Suherdy Yacob',
     'depends': ['purchase', 'account'],
     'data': [
         'security/ir.model.access.csv',
diff --git a/wizard/__pycache__/purchase_bill_sync_wizard.cpython-312.pyc b/wizard/__pycache__/purchase_bill_sync_wizard.cpython-312.pyc
index 58ff1a63d3ec9dde7f55a1a989bf53b37ee32ad0..fa9b811950bc3c3a59981a4fe057ec2558bf7db3 100644
GIT binary patch
delta 2320
zcmZ`)TTC0-8J;t~;b2T08*Br<aKFSjz=o{k6qAO;feqnekV8UX7&2os25d7PLm=bQ
zmp*KTR#8u-Zd=;E>_fD>mD+kY-89=x(w9c6+RQB49c#2wv(id^*#~bzwJ+PtIVLux
zYRBXMKmYgt=ld^n#@?HGBWnIrRn;*9T@QLG?ngtvHh<d$_Thvn34HL8$p#wXCni<%
zh1HvTugU@{hCpN0Hh2+&e>Pt*j02z1R=5Ej)uZHu89r@mf#0uoY4menz_+Wf8TE5H
zg2-w`E$r3Ua^B-V)o8T*AK*t;r%^li8{{&JX85_zmfN>pC$-sHu@;Wib{$(M?f`yM
ztc^#Y#bCV%mM;((;Se%|3q5*?794pL!`4<M`XZgRBmXP}6*kkm$KL@5f$=#=>#UHn
z8sKmBHXCCR?HnjM0U;9c5YBC=bfWk;h1a?hj1?8QV7FC7HH?j^VXW{YTbDPgiD^nZ
zvkuY0e<GYcNJ0>zBR+`t&eoP9N;>NlolLPa{+?*b)`@kZ6IYzA7abRfxwp}(L1-`Y
zG{hH5>q9L%QD=+r$%!5xqPjs4yhg0cx)Rl5z4$*jZ0QiK3npw`Ephgcr?e7X<?39B
z;K+y~j2%JIQs|);U3kbxhF6C$JIaJw`agAGuXE>JEdl@Hu*30M8(6|)s&p)O5JZ~1
zL+<QjPfh$VPiYljEj>AX)-Af@NxZSzS+3ku!rJS~m{;^5Qq4|K^cMAb(eiTk^`aL&
z(Hc#o=)v09gq}rnqWQo=XwhqL64pxTMN65r8UDP+4j<T4x)#yR)WaUf5NJSU`^=2%
zXBx4EaWftUh&Fs=Ofi7Y31INYi(*b8?&cpjyDJ-ab?7veVO<m2(OjI$zic*3(HkR~
zrd>m}wR9*!GyKr$tZpqA(k8aW(=VrDJaI?6^S+i4t5B!<owr0&JX3N_AmE;sK=N-|
zf@xt|g^f~MOk0WVKd^QYyh9#DFda-M)5V<R0OQ-$G2OeGDWcO4zf<?m?M_lP(CaKu
z$5>Sx<)X>yPM{uVxHP@E!cufbNK)zb71f*;=mbOa43*-!n=Bth#3~r*)TmWFyGnBj
zE)k>FgiRF$d7PAg6^olVs`@ZsonFbXs*z{u6-wY%S=Eqxz5a&)d<(i=zXCjb>hgg#
z_>HUax-G>gQ!FoRQa6)nfm&sSr6i-8)-rTL;1G&x;%Uro+TesG6qFFyTY{=hXP6}4
zTh6V|Bv&bpQFVMWlVEuK*1!fg1=^wLerub*h8e07>;{z<UUsIM56eYK(yH~)M*UGj
zQlM8<*CAN!3BeSanZR?48G%)u|Lx8sIHA~ys#~EqSzfIX=w%C=ppFJYk7ohZ_FYdC
z{D~)Mz(0%WZ3FzPr-7tN_zzD9IY;K&z4rk~<vwqG1#I^ZDyK)}(<67=^WCSF?g6=b
zK<N(4-C-%heca7SiOkmokpx5JR|l#<5Fzps`2t&)$U+4%7y?RgS`JQ2GdCnEs!(Ye
z-7{%9DBNAm`}-CDdD(wnnx2vT=lA^A6#u;JpWpK@DE_GIk8Vp#w-DuRFiaw~1V+ex
zf;jaWS<n;bzy<K|*8N-3FeMKzD1$M1FqVI1@R9XV?2qxsaj9=I-*@={h%2F4IW((;
z=H<}5)JGKzS`PHQAhhQ|@7=^#CZg{g_-y#{lQCtKmPcu6acysu-y2R#ecKoF{lias
zKkgsRN3Kg}rt;w=sv`|TxZf$Hq%-5sh9;jArdL_;LF;D&uRZBkF3-!C=cR=-iX0&)
zNHm}k@+#hdRnb~FuoiGPK^nKll2P(GGNNQYf@U-IERvM?jjy-M$zJBXigO01Q2$e4
z4k+Q69F8gBv>Z+=;Y~TbiKuRa2|WKPFiA>*@n?a-hh6u(9{7~Ns2muT#wdx7D)hQc
zucO#4Fpf)EL<(7BXeX-Z>?HXta8;U3$pLpt3S7+xu1J#+Ws;H69YFV$f)n>!aObSe
zpMo}8lLDTB7mS8*cjVpPUnG8<_|)xxep@?7T22*=G-*6l7}n*gTdxDM0_2w4j6nY$
zeuLhJTkTg!D}c*w&G5_iH!9J8KEdC^;sf}-4$gcL=#D)fBU2<&j4t<0$1bS3kKMn+
z;sF*PV)1+U%dWF^5Akvui!c_COTrBxd2$0hhFd3B^hNX`{PLs&JcT-6lP{tL$KH%6
gu0Gk-w@+YeKUGlyE`c}CDD^#Zeb3(sY!!+B4N7^MyZ`_I

delta 931
zcmY*VTS!z<6g~Ss=H)z`yz{)?8E3rMSdNbgCFntgJp?0rU|Fe&N6X1H)ClQZBqH)d
zB-lSjBPv-*N>ukF3j+TF3nHQieFS~<FJ^u8qI*Y1>Ks`6?6ucfXYVuG`R17Wh0En2
z#%kZt+NISzbnkS+cWiL>LJ}9862$SX^P{+8AyQ0#<nn@TH^f~-@O2-)aMxN7LeefR
zwV>Zq&q;Zxl&LuEQFyU=56*a+>|(Qz5KUkLR`HS^$-l{4`9772>CJfE<FoV4l2LXu
zH+~i+eYxN`hr=NscKCgcvk5}_x$7{}i1+1yuhH!A|CMgSHn9MQJXVY5Xa4jF1e@fZ
zg;kXn$WmTw!TS6)F318|&q)@{#%boIaJH6^KA8pad_WSk5DS<GhZbgDP3|dPO-(*4
z2x{Sh=3F1lyXGw8r|>mPA@fsj2wzEIyecPPozZ&IRWQ!8LMl-o1tyu^_)E?Mzfmi*
zjdLu-WD4et_*;_l0vWNBr(o<gLD;w_=LD{-;50{r8LOt?DPO=E3K{*R>+oez5hE;;
zZq0s6tK#tTBLRUhmoch1Vqwm7Tds0I90>}<VCw`z!&Dw<&y7S2b0hu3DGX-0RRxXE
zC@rF~E}-#@NQ*Q4VN$B%yz)m+M5h2!xLEW6(s(IW23<H4i=i5Cfhb;yPvd-Xs}MD8
zFs4Rvms(>zY9#$s#38ki(*VcS1UCr!bM*;8y?(i56X>r>-$F?fR}ax`Ur%x*lK2oQ
zn&`RF^Da`p+|Lh!w_?ezfxTjBhoCQRJPzC!qL(D?V3{zi`zEovd_T8Duq0WGcgsif
z#!Yt0gj=|yqRYJpM8|RiSI-&Yq@X{p$bfI!EZ;WajtO^7n87nuTf+Cu;}H`MnJ|}w
zp9G&(U4%KTPIX7K?ElIiwyo!S>4ol&?R`&iAr&v{=fN@7_+Cl9R#Gd(%vRD?8|;O#
Rn)h<$Yq|0pF|+Jje*txk?!y28

diff --git a/wizard/purchase_bill_sync_wizard.py b/wizard/purchase_bill_sync_wizard.py
index f500857..1bc3c0a 100644
--- a/wizard/purchase_bill_sync_wizard.py
+++ b/wizard/purchase_bill_sync_wizard.py
@@ -53,7 +53,6 @@ class PurchaseBillSyncWizard(models.TransientModel):
                 if line.product_uom_id and po_line.product_uom and line.product_uom_id != po_line.product_uom:
                     bill_price_in_po_currency = line.product_uom_id._compute_price(bill_price_in_po_currency, po_line.product_uom)
 
-                # Check Price
                 if float_compare(bill_price_in_po_currency, po_line.price_unit, precision_digits=2) != 0:
                     discrepancies.append(f"Product {line.product_id.name}: Price {bill_price_in_po_currency:.2f} != {po_line.price_unit:.2f}")
                     
@@ -163,6 +162,42 @@ class PurchaseBillSyncWizard(models.TransientModel):
                                                new_date = stock_move.date + timedelta(seconds=1)
                                                self._cr.execute("UPDATE stock_valuation_layer SET create_date = %s WHERE id = %s", (new_date, svl.id))
                                           
+                                          # AVCO/FIFO Logic: Update Standard Price and Distribute Value to Layers
+                                          product = stock_move.product_id
+                                          if product.categ_id.property_cost_method in ['average', 'fifo'] and product.quantity_svl > 0:
+                                              # 1. Update Standard Price
+                                              # We use disable_auto_svl to prevent Odoo from creating an extra SVL for this price change
+                                              # because we already created the specific adjustment SVL above.
+                                              new_std_price = product.standard_price + (diff / product.quantity_svl)
+                                              product.with_context(disable_auto_svl=True).sudo().write({'standard_price': new_std_price})
+                                              
+                                              # 2. Distribute Value to Remaining Layers (Crucial for correct COGS later)
+                                              remaining_svls = self.env['stock.valuation.layer'].search([
+                                                  ('product_id', '=', product.id),
+                                                  ('remaining_qty', '>', 0),
+                                                  ('company_id', '=', stock_move.company_id.id),
+                                              ])
+                                              
+                                              if remaining_svls:
+                                                  remaining_qty_total = sum(remaining_svls.mapped('remaining_qty'))
+                                                  if remaining_qty_total > 0:
+                                                      remaining_value_to_distribute = diff
+                                                      remaining_value_unit_cost = remaining_value_to_distribute / remaining_qty_total
+                                                      
+                                                      for layer in remaining_svls:
+                                                          if float_compare(layer.remaining_qty, remaining_qty_total, precision_rounding=product.uom_id.rounding) >= 0:
+                                                              taken_remaining_value = remaining_value_to_distribute
+                                                          else:
+                                                              taken_remaining_value = remaining_value_unit_cost * layer.remaining_qty
+                                                              
+                                                          # Rounding
+                                                          taken_remaining_value = stock_move.company_id.currency_id.round(taken_remaining_value)
+                                                          
+                                                          layer.sudo().write({'remaining_value': layer.remaining_value + taken_remaining_value})
+                                                          
+                                                          remaining_value_to_distribute -= taken_remaining_value
+                                                          remaining_qty_total -= layer.remaining_qty
+
                                           # Handle Accounting Entry if Automated
                                           if stock_move.product_id.categ_id.property_valuation == 'real_time':
                                               accounts = stock_move.product_id.product_tmpl_id.get_product_accounts()