跳转至内容
  • 版块
  • 标签
  • 热门
  • 用户
  • 群组
皮肤
  • Light
  • Cerulean
  • Cosmo
  • Flatly
  • Journal
  • Litera
  • Lumen
  • Lux
  • Materia
  • Minty
  • Morph
  • Pulse
  • Sandstone
  • Simplex
  • Sketchy
  • Spacelab
  • United
  • Yeti
  • Zephyr
  • Dark
  • Cyborg
  • Darkly
  • Quartz
  • Slate
  • Solar
  • Superhero
  • Vapor

  • 默认(Flatly)
  • 不使用皮肤
折叠

Odoo 中文社区

  1. 主页
  2. 版块
  3. Odoo 新手求助
  4. 按库存采购的货品,设定了供应商和再订货规则,运行排程会不断重复产生询价单,何故

按库存采购的货品,设定了供应商和再订货规则,运行排程会不断重复产生询价单,何故

已定时 已固定 已锁定 已移动 Odoo 新手求助
6 帖子 4 发布者 3.7k 浏览
  • 从旧到新
  • 从新到旧
  • 最多赞同
登录后回复
此主题已被删除。只有拥有主题管理权限的用户可以查看。
  • S 离线
    S 离线
    suhuyy
    写于 最后由 编辑
    #1

    按库存采购的货品,设定了供应商和再订货规则,运行排程会不断重复产生询价单,何故
    货品资料:
    [img [检测到链接无效,已移除] /img]
    按库存采购的材料,设置了补货供应商:
    [img [检测到链接无效,已移除] /img]
    再订购规则:
    [img [检测到链接无效,已移除] /img]
    货品当前库存为0;
    第一次运行仓库-计划-运行排程:
    产生询价单然后确认订单如下:
    [img [检测到链接无效,已移除] /img]
    当我再次运行仓库-计划-运行排程时:
    需求没有任何增加,结果又产生询价单了,这次数量变成了50,我再次确认订单;
    再运行,询价单数量变成了100,怎么会这样呢?
    按照我的理解:前面采购单已经确认,预测数量已经变成了25,再次运行 排程,因为25已经高于最小数量10,就不应该再产生询价单了,这是哪里错了呢,请各位前辈指点一下,非常感谢!

    简单就是美

    1 条回复 最后回复
    0
    • 卓忆卓 离线
      卓忆卓 离线
      卓忆
      写于 最后由 编辑
      #2

      你之前的询价单 确认了么?收货了么?
      如果 产品 库存数量 低于 你设的 再订购 规则 中的 最小数量,
      你 跑 mrp(排程) 就会重复生成 作为 草稿的 采购询价单 。

      如果 产品的 在手 库存 数量 大于 触发条件 ,应该就不会 生成 询价单了。

      恬淡

      1 条回复 最后回复
      0
      • KevinKongK 离线
        KevinKongK 离线
        KevinKong
        写于 最后由 编辑
        #3

        恭喜你,又碰到了odoo给你预留的一个坑位。如果我没估计错,你的仓库是[b]两步入库[/b](因为我是三步入库,第二次的运行结果是75并非50)。

        其实你的理解并没错,按道理讲你已经生成了25个的采购单,再次运行MRP运算不应该生成后续的询价单,但是理想很美好,现实很骨感。

        我们来看看这个错误是如何发生的:

        <br />for key in product_dict.keys():<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; ctx = context.copy()<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; ctx.update({&#039;location&#039;: ops_dict[key][0].location_id.id})<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; prod_qty = product_obj._product_available(cr, uid, [x.id for x in product_dict[key]],<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; context=ctx)<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; subtract_qty = orderpoint_obj.subtract_procurements_from_orderpoints(cr, uid, [x.id for x in ops_dict[key]], context=context)<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; for op in ops_dict[key]:<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; try:<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; prods = prod_qty[op.product_id.id]&#91;&#039;virtual_available&#039;]<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; if prods is None:<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; continue<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; if float_compare(prods, op.product_min_qty, precision_rounding=op.product_uom.rounding) &lt;= 0:<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; qty = max(op.product_min_qty, op.product_max_qty) - prods<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; reste = op.qty_multiple &gt; 0 and qty % op.qty_multiple or 0.0<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; if float_compare(reste, 0.0, precision_rounding=op.product_uom.rounding) &gt; 0:<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; qty += op.qty_multiple - reste<br /><br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; if float_compare(qty, 0.0, precision_rounding=op.product_uom.rounding) &lt; 0:<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; continue<br /><br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; qty -= subtract_qty[op.id]<br /><br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; qty_rounded = float_round(qty, precision_rounding=op.product_uom.rounding)<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; if qty_rounded &gt; 0:<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; proc_id = procurement_obj.create(cr, uid,<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;  self._prepare_orderpoint_procurement(cr, uid, op, qty_rounded, context=context),<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;  context=context)<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; tot_procs.append(proc_id)<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; if use_new_cursor:<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; cr.commit()<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; except OperationalError:<br />
        


        由以上代码可以看出,Odoo官方其实是已经将预先采购但并未实际入库的数量扣除了,所以这里你入没入库实际并不影响它的运算结果。

        但作怪的究竟是谁呢?经过一番print之后,找到始作俑者:

         qty -= subtract_qty[op.id]
        



        进而,按图索骥,找到最终的罪魁祸首——
        orderpoint对象的subtract_procurements_from_orderpoints方法:

        <br />def subtract_procurements_from_orderpoints(self, cr, uid, orderpoint_ids, context=None):<br />&nbsp; &nbsp; &nbsp; &nbsp; &#039;&#039;&#039;This function returns quantity of product that needs to be deducted from the orderpoint computed quantity because there&#039;s already a procurement created with aim to fulfill it.<br />&nbsp; &nbsp; &nbsp; &nbsp; &#039;&#039;&#039;<br /><br />&nbsp; &nbsp; &nbsp; &nbsp; cr.execute(&quot;&quot;&quot;select op.id, p.id, p.product_uom, p.product_qty, pt.uom_id, sm.product_qty from procurement_order as p left join stock_move as sm ON sm.procurement_id = p.id,<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; stock_warehouse_orderpoint op, product_product pp, product_template pt<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; WHERE p.orderpoint_id = op.id AND p.state not in (&#039;done&#039;, &#039;cancel&#039;) AND (sm.state IS NULL OR sm.state not in (&#039;draft&#039;))<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; AND pp.id = p.product_id AND pp.product_tmpl_id = pt.id<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; AND op.id IN %s<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; ORDER BY op.id, p.id<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &quot;&quot;&quot;, (tuple(orderpoint_ids),))<br />&nbsp; &nbsp; &nbsp; &nbsp; results = cr.fetchall()<br />&nbsp; &nbsp; &nbsp; &nbsp; current_proc = False<br />&nbsp; &nbsp; &nbsp; &nbsp; current_op = False<br />&nbsp; &nbsp; &nbsp; &nbsp; uom_obj = self.pool.get(&quot;product.uom&quot;)<br />&nbsp; &nbsp; &nbsp; &nbsp; op_qty = 0<br />&nbsp; &nbsp; &nbsp; &nbsp; res = dict.fromkeys(orderpoint_ids, 0.0)<br />&nbsp; &nbsp; &nbsp; &nbsp; for move_result in results:<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; op = move_result[0]<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; if current_op != op:<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; if current_op:<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; res[current_op] = op_qty<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; current_op = op<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; op_qty = 0<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; proc = move_result[1]<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; if proc != current_proc:<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; op_qty += uom_obj._compute_qty(cr, uid, move_result[2], move_result[3], move_result[4], round=False)<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; current_proc = proc<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; if move_result[5]: #If a move is associated (is move qty)<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; op_qty -= move_result[5]<br />&nbsp; &nbsp; &nbsp; &nbsp; if current_op:<br />&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; res[current_op] = op_qty<br />&nbsp; &nbsp; &nbsp; &nbsp; return res<br />
        



        这段代码的问题在什么地方呢?
        在于联表查询stock_move的时候并没有判断move的源库位和目的库位。
        针对你这里的问题,按我估计,你是两步入库,第一步供应商到input 数量25,第二步 input到stock 25。根据上面的代码,第一次循环走前2个if 到第三个if的时候
        相互抵消,op_qty数量为0。第二次循环 不走前两个if 直接第三个if 结果为-25.
        回到_procure_orderpoint_confirm 方法 ,因此会创建一个 (25 -= -25 )50的询价单。

        第三次运行MRP运算,你确认了50的询价单 ,因而符合的stock move有4个 分别为25,25,50,50.
        因此 会创建一个(25-=-75) 100的询价单。

        依次可类推,第四次运算的结果应该为 200.

        WTF!!(ノಠ益ಠ)ノ彡┻━┻!!

        多大一坑啊!!!!

        好了,认识了这个坑就该知道怎么修补这个坑了吧~~


        什么?不知道如何修补?

















        ...........................










        那还玩什么odoo啊。。。。。。。。


        【原创,禁止转载】

        --------------------------------------------------更新----------------------------------------------------

        细心的同学可能已经发现了 按上面说的

        prods = prod_qty[op.product_id.id]&#91;&#039;virtual_available&#039;]
        


        当第一次的25入库以后,预计数量应该为25才对,但从结果上看,显然预计数量仍然为0 。

        oh,no!又是灵异事件么?

        经查,odoo关于预测数量的算法是这样的,预测数量 = 入库数量 - 出库数量 (当前库位)

        而楼主的问题,在于 预测数量始终为0 才导致的 每运行一次都出一次询价单。。。。。。

        所以,饶了一圈又回到了原点。

        解决楼主问题的方法非常简单,不要把订购点设置到input库位上!!!!!!!!!!!!

        为什么?

        两次入库的前提下,每确定一张采购单,系统会自动生成 供应商到 input的 move 和 input 到 stock的move

        这一进一出,预测数量不为0 才怪! 同理, 三步入库把 订购点设置到qc库位的,也是傻*一个。

        ╮(╯▽╰)╭

        1 条回复 最后回复
        0
        • S 离线
          S 离线
          suhuyy
          写于 最后由 编辑
          #4

          [quote author=KevinKong link=topic=17612.msg32268#msg32268 date=1463560230]
          如果我没估计错,你的仓库是[b]两步入库[/b](因为我是三步入库,第二次的运行结果是75并非50)。



          WTF!!(ノಠ益ಠ)ノ彡┻━┻!!

          多大一坑啊!!!!

          好了,认识了这个坑就该知道怎么修补这个坑了吧~~


          什么?不知道如何修补?

















          ...........................










          那还玩什么odoo啊。。。。。。。。


          【原创,禁止转载】

          --------------------------------------------------更新----------------------------------------------------





          解决楼主问题的方法非常简单,[b]不要把订购点设置到input库位上[/b]!!!!!!!!!!!!



          ╮(╯▽╰)╭
          [/quote]

          兄台真神人也,我的系统设置被你一一言中,我的仓库真的真的是[b]两步入库[/b],更神奇的是,我的产品订购点刚好就设置在input库位上,小生佩服的五体投地,非常非常非常感谢,[b]高手在民间[/b]呀!!!

          简单就是美

          1 条回复 最后回复
          0
          • wjfonhandW 离线
            wjfonhandW 离线
            wjfonhand
            写于 最后由 编辑
            #5

            再订购规则本就应该设置在库存库位上吧

            小孔可不是民间的,科班出身,知名公司就职,odoo资深哪

            GoodERP -- Odoo China fork

            1 条回复 最后回复
            0

            • 登录

            • 没有帐号? 注册

            • 登录或注册以进行搜索。
            • 第一个帖子
              最后一个帖子
            0
            • 版块
            • 标签
            • 热门
            • 用户
            • 群组