From 530f360c8cb03e8c16eefa380459722289585340 Mon Sep 17 00:00:00 2001 From: Jordi Ballester Alomar Date: Thu, 31 Jan 2019 13:51:29 +0100 Subject: [PATCH 01/12] [11.][ADD] stock_orderpoint_route --- stock_orderpoint_route/README.rst | 82 ++++ stock_orderpoint_route/__init__.py | 1 + stock_orderpoint_route/__manifest__.py | 20 + stock_orderpoint_route/models/__init__.py | 1 + .../models/stock_warehouse_orderpoint.py | 53 +++ .../readme/CONTRIBUTORS.rst | 2 + stock_orderpoint_route/readme/DESCRIPTION.rst | 3 + stock_orderpoint_route/readme/USAGE.rst | 2 + .../static/description/icon.png | Bin 0 -> 9455 bytes .../static/description/index.html | 406 ++++++++++++++++++ stock_orderpoint_route/tests/__init__.py | 1 + .../tests/test_stock_orderpoint_route.py | 154 +++++++ .../stock_warehouse_orderpoint_views.xml | 19 + 13 files changed, 744 insertions(+) create mode 100644 stock_orderpoint_route/README.rst create mode 100644 stock_orderpoint_route/__init__.py create mode 100644 stock_orderpoint_route/__manifest__.py create mode 100644 stock_orderpoint_route/models/__init__.py create mode 100644 stock_orderpoint_route/models/stock_warehouse_orderpoint.py create mode 100644 stock_orderpoint_route/readme/CONTRIBUTORS.rst create mode 100644 stock_orderpoint_route/readme/DESCRIPTION.rst create mode 100644 stock_orderpoint_route/readme/USAGE.rst create mode 100644 stock_orderpoint_route/static/description/icon.png create mode 100644 stock_orderpoint_route/static/description/index.html create mode 100644 stock_orderpoint_route/tests/__init__.py create mode 100644 stock_orderpoint_route/tests/test_stock_orderpoint_route.py create mode 100644 stock_orderpoint_route/views/stock_warehouse_orderpoint_views.xml diff --git a/stock_orderpoint_route/README.rst b/stock_orderpoint_route/README.rst new file mode 100644 index 000000000..5fb6cfbbe --- /dev/null +++ b/stock_orderpoint_route/README.rst @@ -0,0 +1,82 @@ +====================== +Stock Orderpoint Route +====================== + +.. !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! + !! This file is generated by oca-gen-addon-readme !! + !! changes will be overwritten. !! + !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! + +.. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png + :target: https://odoo-community.org/page/development-status + :alt: Beta +.. |badge2| image:: https://img.shields.io/badge/licence-LGPL--3-blue.png + :target: http://www.gnu.org/licenses/lgpl-3.0-standalone.html + :alt: License: LGPL-3 +.. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fstock--logistics--warehouse-lightgray.png?logo=github + :target: https://github.com/OCA/stock-logistics-warehouse/tree/11.0/stock_orderpoint_route + :alt: OCA/stock-logistics-warehouse +.. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png + :target: https://translation.odoo-community.org/projects/stock-logistics-warehouse-11-0/stock-logistics-warehouse-11-0-stock_orderpoint_route + :alt: Translate me on Weblate +.. |badge5| image:: https://img.shields.io/badge/runbot-Try%20me-875A7B.png + :target: https://runbot.odoo-community.org/runbot/153/11.0 + :alt: Try me on Runbot + +|badge1| |badge2| |badge3| |badge4| |badge5| + +This module allows to restrict a specific route to be used in the +reordering rules. This route will be used instead of the default determined by +default. + +**Table of contents** + +.. contents:: + :local: + +Usage +===== + +Go to the reordering rule and change the route. Only the routes applicable +for this location or locations above it can be selected. + +Bug Tracker +=========== + +Bugs are tracked on `GitHub Issues `_. +In case of trouble, please check there if your issue has already been reported. +If you spotted it first, help us smashing it by providing a detailed and welcomed +`feedback `_. + +Do not contact contributors directly about support or help with technical issues. + +Credits +======= + +Authors +~~~~~~~ + +* Eficent + +Contributors +~~~~~~~~~~~~ + +* Jordi Ballester Alomar +* Lois Rilo + +Maintainers +~~~~~~~~~~~ + +This module is maintained by the OCA. + +.. image:: https://odoo-community.org/logo.png + :alt: Odoo Community Association + :target: https://odoo-community.org + +OCA, or the Odoo Community Association, is a nonprofit organization whose +mission is to support the collaborative development of Odoo features and +promote its widespread use. + +This module is part of the `OCA/stock-logistics-warehouse `_ project on GitHub. + +You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute. diff --git a/stock_orderpoint_route/__init__.py b/stock_orderpoint_route/__init__.py new file mode 100644 index 000000000..0650744f6 --- /dev/null +++ b/stock_orderpoint_route/__init__.py @@ -0,0 +1 @@ +from . import models diff --git a/stock_orderpoint_route/__manifest__.py b/stock_orderpoint_route/__manifest__.py new file mode 100644 index 000000000..dc18e2937 --- /dev/null +++ b/stock_orderpoint_route/__manifest__.py @@ -0,0 +1,20 @@ +# Copyright 2019 Eficent Business and IT Consulting Services, S.L. +# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html). +{ + "name": "Stock Orderpoint Route", + "summary": "Allows to force a route to be used when procuring from " + "orderpoints", + "version": "11.0.1.0.0", + "license": "LGPL-3", + "website": "https://github.com/stock-logistics-warehouse", + "author": "Eficent, " + "Odoo Community Association (OCA)", + "category": "Warehouse Management", + "depends": [ + "stock", + ], + "data": [ + "views/stock_warehouse_orderpoint_views.xml", + ], + "installable": True, +} diff --git a/stock_orderpoint_route/models/__init__.py b/stock_orderpoint_route/models/__init__.py new file mode 100644 index 000000000..5024fb8c7 --- /dev/null +++ b/stock_orderpoint_route/models/__init__.py @@ -0,0 +1 @@ +from . import stock_warehouse_orderpoint diff --git a/stock_orderpoint_route/models/stock_warehouse_orderpoint.py b/stock_orderpoint_route/models/stock_warehouse_orderpoint.py new file mode 100644 index 000000000..bbcaaa800 --- /dev/null +++ b/stock_orderpoint_route/models/stock_warehouse_orderpoint.py @@ -0,0 +1,53 @@ +# Copyright 2019 Eficent Business and IT Consulting Services, S.L. +# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html). + +from odoo import api, fields, models + + +class StockWarehouseOrderpoint(models.Model): + _inherit = 'stock.warehouse.orderpoint' + + route_ids = fields.Many2many( + 'stock.location.route', string='Allowed routes', + compute='_compute_route_ids', + ) + route_id = fields.Many2one('stock.location.route', string='Route', + domain="[('id', 'in', route_ids)]", + ondelete='restrict') + + @api.depends('product_id', 'warehouse_id', 'location_id') + def _compute_route_ids(self): + route_obj = self.env['stock.location.route'] + for wh in self.mapped('warehouse_id'): + wh_routes = route_obj.search( + [('warehouse_ids', '=', wh.id)]) + for record in self.filtered(lambda r: r.warehouse_id == wh): + routes = route_obj + if record.product_id: + routes += record.product_id.mapped( + 'route_ids' + ) | record.product_id.mapped( + 'categ_id' + ).mapped('total_route_ids') + if record.warehouse_id: + routes |= wh_routes + parents = record.get_parents().ids + record.route_ids = routes.filtered(lambda r: any( + p.location_id.id in parents for p in r.pull_ids)) + + def get_parents(self): + location = self.location_id + result = location + while location.location_id: + location = location.location_id + result |= location + return result + + def _prepare_procurement_values(self, product_qty, date=False, + group=False): + + res = super(StockWarehouseOrderpoint, + self)._prepare_procurement_values(product_qty, date=date, + group=group) + res['route_ids'] = self.route_id + return res diff --git a/stock_orderpoint_route/readme/CONTRIBUTORS.rst b/stock_orderpoint_route/readme/CONTRIBUTORS.rst new file mode 100644 index 000000000..daeadd46d --- /dev/null +++ b/stock_orderpoint_route/readme/CONTRIBUTORS.rst @@ -0,0 +1,2 @@ +* Jordi Ballester Alomar +* Lois Rilo diff --git a/stock_orderpoint_route/readme/DESCRIPTION.rst b/stock_orderpoint_route/readme/DESCRIPTION.rst new file mode 100644 index 000000000..b7b4b208b --- /dev/null +++ b/stock_orderpoint_route/readme/DESCRIPTION.rst @@ -0,0 +1,3 @@ +This module allows to restrict a specific route to be used in the +reordering rules. This route will be used instead of the default determined by +default. \ No newline at end of file diff --git a/stock_orderpoint_route/readme/USAGE.rst b/stock_orderpoint_route/readme/USAGE.rst new file mode 100644 index 000000000..eb30b124e --- /dev/null +++ b/stock_orderpoint_route/readme/USAGE.rst @@ -0,0 +1,2 @@ +Go to the reordering rule and change the route. Only the routes applicable +for this location or locations above it can be selected. diff --git a/stock_orderpoint_route/static/description/icon.png b/stock_orderpoint_route/static/description/icon.png new file mode 100644 index 0000000000000000000000000000000000000000..3a0328b516c4980e8e44cdb63fd945757ddd132d GIT binary patch literal 9455 zcmW++2RxMjAAjx~&dlBk9S+%}OXg)AGE&Cb*&}d0jUxM@u(PQx^-s)697TX`ehR4?GS^qbkof1cslKgkU)h65qZ9Oc=ml_0temigYLJfnz{IDzUf>bGs4N!v3=Z3jMq&A#7%rM5eQ#dc?k~! zVpnB`o+K7|Al`Q_U;eD$B zfJtP*jH`siUq~{KE)`jP2|#TUEFGRryE2`i0**z#*^6~AI|YzIWy$Cu#CSLW3q=GA z6`?GZymC;dCPk~rBS%eCb`5OLr;RUZ;D`}um=H)BfVIq%7VhiMr)_#G0N#zrNH|__ zc+blN2UAB0=617@>_u;MPHN;P;N#YoE=)R#i$k_`UAA>WWCcEVMh~L_ zj--gtp&|K1#58Yz*AHCTMziU1Jzt_jG0I@qAOHsk$2}yTmVkBp_eHuY$A9)>P6o~I z%aQ?!(GqeQ-Y+b0I(m9pwgi(IIZZzsbMv+9w{PFtd_<_(LA~0H(xz{=FhLB@(1&qHA5EJw1>>=%q2f&^X>IQ{!GJ4e9U z&KlB)z(84HmNgm2hg2C0>WM{E(DdPr+EeU_N@57;PC2&DmGFW_9kP&%?X4}+xWi)( z;)z%wI5>D4a*5XwD)P--sPkoY(a~WBw;E~AW`Yue4kFa^LM3X`8x|}ZUeMnqr}>kH zG%WWW>3ml$Yez?i%)2pbKPI7?5o?hydokgQyZsNEr{a|mLdt;X2TX(#B1j35xPnPW z*bMSSOauW>o;*=kO8ojw91VX!qoOQb)zHJ!odWB}d+*K?#sY_jqPdg{Sm2HdYzdEx zOGVPhVRTGPtv0o}RfVP;Nd(|CB)I;*t&QO8h zFfekr30S!-LHmV_Su-W+rEwYXJ^;6&3|L$mMC8*bQptyOo9;>Qb9Q9`ySe3%V$A*9 zeKEe+b0{#KWGp$F+tga)0RtI)nhMa-K@JS}2krK~n8vJ=Ngm?R!9G<~RyuU0d?nz# z-5EK$o(!F?hmX*2Yt6+coY`6jGbb7tF#6nHA zuKk=GGJ;ZwON1iAfG$E#Y7MnZVmrY|j0eVI(DN_MNFJmyZ|;w4tf@=CCDZ#5N_0K= z$;R~bbk?}TpfDjfB&aiQ$VA}s?P}xPERJG{kxk5~R`iRS(SK5d+Xs9swCozZISbnS zk!)I0>t=A<-^z(cmSFz3=jZ23u13X><0b)P)^1T_))Kr`e!-pb#q&J*Q`p+B6la%C zuVl&0duN<;uOsB3%T9Fp8t{ED108<+W(nOZd?gDnfNBC3>M8WE61$So|P zVvqH0SNtDTcsUdzaMDpT=Ty0pDHHNL@Z0w$Y`XO z2M-_r1S+GaH%pz#Uy0*w$Vdl=X=rQXEzO}d6J^R6zjM1u&c9vYLvLp?W7w(?np9x1 zE_0JSAJCPB%i7p*Wvg)pn5T`8k3-uR?*NT|J`eS#_#54p>!p(mLDvmc-3o0mX*mp_ zN*AeS<>#^-{S%W<*mz^!X$w_2dHWpcJ6^j64qFBft-o}o_Vx80o0>}Du;>kLts;$8 zC`7q$QI(dKYG`Wa8#wl@V4jVWBRGQ@1dr-hstpQL)Tl+aqVpGpbSfN>5i&QMXfiZ> zaA?T1VGe?rpQ@;+pkrVdd{klI&jVS@I5_iz!=UMpTsa~mBga?1r}aRBm1WS;TT*s0f0lY=JBl66Upy)-k4J}lh=P^8(SXk~0xW=T9v*B|gzIhN z>qsO7dFd~mgxAy4V?&)=5ieYq?zi?ZEoj)&2o)RLy=@hbCRcfT5jigwtQGE{L*8<@Yd{zg;CsL5mvzfDY}P-wos_6PfprFVaeqNE%h zKZhLtcQld;ZD+>=nqN~>GvROfueSzJD&BE*}XfU|H&(FssBqY=hPCt`d zH?@s2>I(|;fcW&YM6#V#!kUIP8$Nkdh0A(bEVj``-AAyYgwY~jB zT|I7Bf@%;7aL7Wf4dZ%VqF$eiaC38OV6oy3Z#TER2G+fOCd9Iaoy6aLYbPTN{XRPz z;U!V|vBf%H!}52L2gH_+j;`bTcQRXB+y9onc^wLm5wi3-Be}U>k_u>2Eg$=k!(l@I zcCg+flakT2Nej3i0yn+g+}%NYb?ta;R?(g5SnwsQ49U8Wng8d|{B+lyRcEDvR3+`O{zfmrmvFrL6acVP%yG98X zo&+VBg@px@i)%o?dG(`T;n*$S5*rnyiR#=wW}}GsAcfyQpE|>a{=$Hjg=-*_K;UtD z#z-)AXwSRY?OPefw^iI+ z)AXz#PfEjlwTes|_{sB?4(O@fg0AJ^g8gP}ex9Ucf*@_^J(s_5jJV}c)s$`Myn|Kd z$6>}#q^n{4vN@+Os$m7KV+`}c%4)4pv@06af4-x5#wj!KKb%caK{A&Y#Rfs z-po?Dcb1({W=6FKIUirH&(yg=*6aLCekcKwyfK^JN5{wcA3nhO(o}SK#!CINhI`-I z1)6&n7O&ZmyFMuNwvEic#IiOAwNkR=u5it{B9n2sAJV5pNhar=j5`*N!Na;c7g!l$ z3aYBqUkqqTJ=Re-;)s!EOeij=7SQZ3Hq}ZRds%IM*PtM$wV z@;rlc*NRK7i3y5BETSKuumEN`Xu_8GP1Ri=OKQ$@I^ko8>H6)4rjiG5{VBM>B|%`&&s^)jS|-_95&yc=GqjNo{zFkw%%HHhS~e=s zD#sfS+-?*t|J!+ozP6KvtOl!R)@@-z24}`9{QaVLD^9VCSR2b`b!KC#o;Ki<+wXB6 zx3&O0LOWcg4&rv4QG0)4yb}7BFSEg~=IR5#ZRj8kg}dS7_V&^%#Do==#`u zpy6{ox?jWuR(;pg+f@mT>#HGWHAJRRDDDv~@(IDw&R>9643kK#HN`!1vBJHnC+RM&yIh8{gG2q zA%e*U3|N0XSRa~oX-3EAneep)@{h2vvd3Xvy$7og(sayr@95+e6~Xvi1tUqnIxoIH zVWo*OwYElb#uyW{Imam6f2rGbjR!Y3`#gPqkv57dB6K^wRGxc9B(t|aYDGS=m$&S!NmCtrMMaUg(c zc2qC=2Z`EEFMW-me5B)24AqF*bV5Dr-M5ig(l-WPS%CgaPzs6p_gnCIvTJ=Y<6!gT zVt@AfYCzjjsMEGi=rDQHo0yc;HqoRNnNFeWZgcm?f;cp(6CNylj36DoL(?TS7eU#+ z7&mfr#y))+CJOXQKUMZ7QIdS9@#-}7y2K1{8)cCt0~-X0O!O?Qx#E4Og+;A2SjalQ zs7r?qn0H044=sDN$SRG$arw~n=+T_DNdSrarmu)V6@|?1-ZB#hRn`uilTGPJ@fqEy zGt(f0B+^JDP&f=r{#Y_wi#AVDf-y!RIXU^0jXsFpf>=Ji*TeqSY!H~AMbJdCGLhC) zn7Rx+sXw6uYj;WRYrLd^5IZq@6JI1C^YkgnedZEYy<&4(z%Q$5yv#Boo{AH8n$a zhb4Y3PWdr269&?V%uI$xMcUrMzl=;w<_nm*qr=c3Rl@i5wWB;e-`t7D&c-mcQl7x! zZWB`UGcw=Y2=}~wzrfLx=uet<;m3~=8I~ZRuzvMQUQdr+yTV|ATf1Uuomr__nDf=X zZ3WYJtHp_ri(}SQAPjv+Y+0=fH4krOP@S&=zZ-t1jW1o@}z;xk8 z(Nz1co&El^HK^NrhVHa-_;&88vTU>_J33=%{if;BEY*J#1n59=07jrGQ#IP>@u#3A z;!q+E1Rj3ZJ+!4bq9F8PXJ@yMgZL;>&gYA0%_Kbi8?S=XGM~dnQZQ!yBSgcZhY96H zrWnU;k)qy`rX&&xlDyA%(a1Hhi5CWkmg(`Gb%m(HKi-7Z!LKGRP_B8@`7&hdDy5n= z`OIxqxiVfX@OX1p(mQu>0Ai*v_cTMiw4qRt3~NBvr9oBy0)r>w3p~V0SCm=An6@3n)>@z!|o-$HvDK z|3D2ZMJkLE5loMKl6R^ez@Zz%S$&mbeoqH5`Bb){Ei21q&VP)hWS2tjShfFtGE+$z zzCR$P#uktu+#!w)cX!lWN1XU%K-r=s{|j?)Akf@q#3b#{6cZCuJ~gCxuMXRmI$nGtnH+-h z+GEi!*X=AP<|fG`1>MBdTb?28JYc=fGvAi2I<$B(rs$;eoJCyR6_bc~p!XR@O-+sD z=eH`-ye})I5ic1eL~TDmtfJ|8`0VJ*Yr=hNCd)G1p2MMz4C3^Mj?7;!w|Ly%JqmuW zlIEW^Ft%z?*|fpXda>Jr^1noFZEwFgVV%|*XhH@acv8rdGxeEX{M$(vG{Zw+x(ei@ zmfXb22}8-?Fi`vo-YVrTH*C?a8%M=Hv9MqVH7H^J$KsD?>!SFZ;ZsvnHr_gn=7acz z#W?0eCdVhVMWN12VV^$>WlQ?f;P^{(&pYTops|btm6aj>_Uz+hqpGwB)vWp0Cf5y< zft8-je~nn?W11plq}N)4A{l8I7$!ks_x$PXW-2XaRFswX_BnF{R#6YIwMhAgd5F9X zGmwdadS6(a^fjHtXg8=l?Rc0Sm%hk6E9!5cLVloEy4eh(=FwgP`)~I^5~pBEWo+F6 zSf2ncyMurJN91#cJTy_u8Y}@%!bq1RkGC~-bV@SXRd4F{R-*V`bS+6;W5vZ(&+I<9$;-V|eNfLa5n-6% z2(}&uGRF;p92eS*sE*oR$@pexaqr*meB)VhmIg@h{uzkk$9~qh#cHhw#>O%)b@+(| z^IQgqzuj~Sk(J;swEM-3TrJAPCq9k^^^`q{IItKBRXYe}e0Tdr=Huf7da3$l4PdpwWDop%^}n;dD#K4s#DYA8SHZ z&1!riV4W4R7R#C))JH1~axJ)RYnM$$lIR%6fIVA@zV{XVyx}C+a-Dt8Y9M)^KU0+H zR4IUb2CJ{Hg>CuaXtD50jB(_Tcx=Z$^WYu2u5kubqmwp%drJ6 z?Fo40g!Qd<-l=TQxqHEOuPX0;^z7iX?Ke^a%XT<13TA^5`4Xcw6D@Ur&VT&CUe0d} z1GjOVF1^L@>O)l@?bD~$wzgf(nxX1OGD8fEV?TdJcZc2KoUe|oP1#=$$7ee|xbY)A zDZq+cuTpc(fFdj^=!;{k03C69lMQ(|>uhRfRu%+!k&YOi-3|1QKB z z?n?eq1XP>p-IM$Z^C;2L3itnbJZAip*Zo0aw2bs8@(s^~*8T9go!%dHcAz2lM;`yp zD=7&xjFV$S&5uDaiScyD?B-i1ze`+CoRtz`Wn+Zl&#s4&}MO{@N!ufrzjG$B79)Y2d3tBk&)TxUTw@QS0TEL_?njX|@vq?Uz(nBFK5Pq7*xj#u*R&i|?7+6# z+|r_n#SW&LXhtheZdah{ZVoqwyT{D>MC3nkFF#N)xLi{p7J1jXlmVeb;cP5?e(=f# zuT7fvjSbjS781v?7{)-X3*?>tq?)Yd)~|1{BDS(pqC zC}~H#WXlkUW*H5CDOo<)#x7%RY)A;ShGhI5s*#cRDA8YgqG(HeKDx+#(ZQ?386dv! zlXCO)w91~Vw4AmOcATuV653fa9R$fyK8ul%rG z-wfS zihugoZyr38Im?Zuh6@RcF~t1anQu7>#lPpb#}4cOA!EM11`%f*07RqOVkmX{p~KJ9 z^zP;K#|)$`^Rb{rnHGH{~>1(fawV0*Z#)}M`m8-?ZJV<+e}s9wE# z)l&az?w^5{)`S(%MRzxdNqrs1n*-=jS^_jqE*5XDrA0+VE`5^*p3CuM<&dZEeCjoz zR;uu_H9ZPZV|fQq`Cyw4nscrVwi!fE6ciMmX$!_hN7uF;jjKG)d2@aC4ropY)8etW=xJvni)8eHi`H$%#zn^WJ5NLc-rqk|u&&4Z6fD_m&JfSI1Bvb?b<*n&sfl0^t z=HnmRl`XrFvMKB%9}>PaA`m-fK6a0(8=qPkWS5bb4=v?XcWi&hRY?O5HdulRi4?fN zlsJ*N-0Qw+Yic@s0(2uy%F@ib;GjXt01Fmx5XbRo6+n|pP(&nodMoap^z{~q ziEeaUT@Mxe3vJSfI6?uLND(CNr=#^W<1b}jzW58bIfyWTDle$mmS(|x-0|2UlX+9k zQ^EX7Nw}?EzVoBfT(-LT|=9N@^hcn-_p&sqG z&*oVs2JSU+N4ZD`FhCAWaS;>|wH2G*Id|?pa#@>tyxX`+4HyIArWDvVrX)2WAOQff z0qyHu&-S@i^MS-+j--!pr4fPBj~_8({~e1bfcl0wI1kaoN>mJL6KUPQm5N7lB(ui1 zE-o%kq)&djzWJ}ob<-GfDlkB;F31j-VHKvQUGQ3sp`CwyGJk_i!y^sD0fqC@$9|jO zOqN!r!8-p==F@ZVP=U$qSpY(gQ0)59P1&t@y?5rvg<}E+GB}26NYPp4f2YFQrQtot5mn3wu_qprZ=>Ig-$ zbW26Ws~IgY>}^5w`vTB(G`PTZaDiGBo5o(tp)qli|NeV( z@H_=R8V39rt5J5YB2Ky?4eJJ#b`_iBe2ot~6%7mLt5t8Vwi^Jy7|jWXqa3amOIoRb zOr}WVFP--DsS`1WpN%~)t3R!arKF^Q$e12KEqU36AWwnCBICpH4XCsfnyrHr>$I$4 z!DpKX$OKLWarN7nv@!uIA+~RNO)l$$w}p(;b>mx8pwYvu;dD_unryX_NhT8*Tj>BTrTTL&!?O+%Rv;b?B??gSzdp?6Uug9{ zd@V08Z$BdI?fpoCS$)t4mg4rT8Q_I}h`0d-vYZ^|dOB*Q^S|xqTV*vIg?@fVFSmMpaw0qtTRbx} z({Pg?#{2`sc9)M5N$*N|4;^t$+QP?#mov zGVC@I*lBVrOU-%2y!7%)fAKjpEFsgQc4{amtiHb95KQEwvf<(3T<9-Zm$xIew#P22 zc2Ix|App^>v6(3L_MCU0d3W##AB0M~3D00EWoKZqsJYT(#@w$Y_H7G22M~ApVFTRHMI_3be)Lkn#0F*V8Pq zc}`Cjy$bE;FJ6H7p=0y#R>`}-m4(0F>%@P|?7fx{=R^uFdISRnZ2W_xQhD{YuR3t< z{6yxu=4~JkeA;|(J6_nv#>Nvs&FuLA&PW^he@t(UwFFE8)|a!R{`E`K`i^ZnyE4$k z;(749Ix|oi$c3QbEJ3b~D_kQsPz~fIUKym($a_7dJ?o+40*OLl^{=&oq$<#Q(yyrp z{J-FAniyAw9tPbe&IhQ|a`DqFTVQGQ&Gq3!C2==4x{6EJwiPZ8zub-iXoUtkJiG{} zPaR&}_fn8_z~(=;5lD-aPWD3z8PZS@AaUiomF!G8I}Mf>e~0g#BelA-5#`cj;O5>N Xviia!U7SGha1wx#SCgwmn*{w2TRX*I literal 0 HcmV?d00001 diff --git a/stock_orderpoint_route/static/description/index.html b/stock_orderpoint_route/static/description/index.html new file mode 100644 index 000000000..69e16318c --- /dev/null +++ b/stock_orderpoint_route/static/description/index.html @@ -0,0 +1,406 @@ + + + + + + +Stock Orderpoint Route + + + +
+

Stock Orderpoint Route

+ + +

Beta License: LGPL-3 OCA/stock-logistics-warehouse Translate me on Weblate Try me on Runbot

+

This module allows to restrict a specific route to be used in the +reordering rules. This route will be used instead of the default determined by +default.

+

Table of contents

+ +
+

Usage

+

Go to the reordering rule and change the route. Only the routes applicable +for this location or locations above it can be selected.

+
+
+

Bug Tracker

+

Bugs are tracked on GitHub Issues. +In case of trouble, please check there if your issue has already been reported. +If you spotted it first, help us smashing it by providing a detailed and welcomed +feedback.

+

Do not contact contributors directly about support or help with technical issues.

+
+
+

Credits

+
+

Authors

+
    +
  • Eficent
    • +
+
+
+

Contributors

+ +
+
+

Maintainers

+

This module is maintained by the OCA.

+Odoo Community Association +

OCA, or the Odoo Community Association, is a nonprofit organization whose +mission is to support the collaborative development of Odoo features and +promote its widespread use.

+

This module is part of the OCA/stock-logistics-warehouse project on GitHub.

+

You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.

+
+
+
+ + diff --git a/stock_orderpoint_route/tests/__init__.py b/stock_orderpoint_route/tests/__init__.py new file mode 100644 index 000000000..c2355dd9a --- /dev/null +++ b/stock_orderpoint_route/tests/__init__.py @@ -0,0 +1 @@ +from . import test_stock_orderpoint_route diff --git a/stock_orderpoint_route/tests/test_stock_orderpoint_route.py b/stock_orderpoint_route/tests/test_stock_orderpoint_route.py new file mode 100644 index 000000000..398cd3a65 --- /dev/null +++ b/stock_orderpoint_route/tests/test_stock_orderpoint_route.py @@ -0,0 +1,154 @@ +# Copyright 2017 Eficent Business and IT Consulting Services S.L. +# License LGPL-3.0 or later (http://www.gnu.org/licenses/lgpl-3.0). + +from odoo.tests import common + + +class TestStockOrderpointRoute(common.TransactionCase): + def setUp(self): + super(TestStockOrderpointRoute, self).setUp() + + # common models + self.orderpoint_model = self.env[ + 'stock.warehouse.orderpoint'] + self.procurement_group_model = self.env['procurement.group'] + # refs + self.stock_manager_group = \ + self.env.ref('stock.group_stock_manager') + self.stock_multi_locations_group_group = \ + self.env.ref('stock.group_stock_multi_locations') + self.main_company = self.env.ref('base.main_company') + self.warehouse = self.env.ref('stock.warehouse0') + self.categ_unit = self.env.ref('product.product_uom_categ_unit') + self.virtual_loc = self.env.ref('stock.stock_location_customers') + + # common data + self.stock_manager = self._create_user( + 'stock_manager', + [self.stock_manager_group.id, + self.stock_multi_locations_group_group.id], + [self.main_company.id]) + self.product = self._create_product('SH', 'Shoes', False) + + self.ressuply_loc = self.env['stock.location'].create({ + 'name': 'Ressuply', + 'location_id': self.warehouse.view_location_id.id, + }) + + self.ressuply_loc2 = self.env['stock.location'].create({ + 'name': 'Ressuply2', + 'location_id': self.warehouse.view_location_id.id, + }) + + self.route = self.env['stock.location.route'].create({ + 'name': 'Transfer', + 'product_categ_selectable': False, + 'product_selectable': True, + 'company_id': self.main_company.id, + 'sequence': 10, + }) + self.route2 = self.env['stock.location.route'].create({ + 'name': 'Transfer', + 'product_categ_selectable': False, + 'product_selectable': True, + 'company_id': self.main_company.id, + 'sequence': 10, + }) + + self.uom_dozen = self.env['product.uom'].create({ + 'name': 'Test-DozenA', + 'category_id': self.categ_unit.id, + 'factor_inv': 12, + 'uom_type': 'bigger', + 'rounding': 0.001}) + + self.env['procurement.rule'].create({ + 'name': 'Transfer', + 'route_id': self.route.id, + 'location_src_id': self.ressuply_loc.id, + 'location_id': self.warehouse.lot_stock_id.id, + 'action': 'move', + 'picking_type_id': self.warehouse.int_type_id.id, + 'procure_method': 'make_to_stock', + 'warehouse_id': self.warehouse.id, + 'company_id': self.main_company.id, + 'propagate': 'False', + }) + + self.env['procurement.rule'].create({ + 'name': 'Transfer 2', + 'route_id': self.route2.id, + 'location_src_id': self.ressuply_loc2.id, + 'location_id': self.warehouse.lot_stock_id.id, + 'action': 'move', + 'picking_type_id': self.warehouse.int_type_id.id, + 'procure_method': 'make_to_stock', + 'warehouse_id': self.warehouse.id, + 'company_id': self.main_company.id, + 'propagate': 'False', + }) + + def _create_user(self, name, group_ids, company_ids): + return self.env['res.users'].with_context( + {'no_reset_password': True}).create( + {'name': name, + 'password': 'demo', + 'login': name, + 'email': '@'.join([name, '@test.com']), + 'groups_id': [(6, 0, group_ids)], + 'company_ids': [(6, 0, company_ids)] + }) + + def _create_product(self, default_code, name, company_id, **vals): + return self.env['product.product'].create(dict( + name=name, + default_code=default_code, + uom_id=self.env.ref('product.product_uom_unit').id, + company_id=company_id, + type='product', + **vals + )) + + def test_orderpoint_route_01(self): + self.product.route_ids = [(6, 0, [self.route.id, self.route2.id])] + vals = { + 'product_id': self.product.id, + 'product_min_qty': 10.0, + 'product_max_qty': 100.0, + 'company_id': self.main_company.id, + 'warehouse_id': self.warehouse.id, + 'location_id': self.warehouse.lot_stock_id.id, + } + + orderpoint = self.orderpoint_model.sudo( + self.stock_manager).create(vals) + self.assertIn(self.route, orderpoint.route_ids) + self.assertIn(self.route2, orderpoint.route_ids) + orderpoint.route_id = self.route.id + self.procurement_group_model.run_scheduler() + move = self.env['stock.move'].search( + [('product_id', '=', self.product.id), + ('location_id', '=', self.ressuply_loc.id)], limit=1) + self.assertEqual(len(move), 1) + + def test_orderpoint_route_02(self): + self.product.route_ids = [(6, 0, [self.route.id, self.route2.id])] + vals = { + 'product_id': self.product.id, + 'product_min_qty': 10.0, + 'product_max_qty': 100.0, + 'company_id': self.main_company.id, + 'warehouse_id': self.warehouse.id, + 'location_id': self.warehouse.lot_stock_id.id, + } + + orderpoint = self.orderpoint_model.sudo( + self.stock_manager).create(vals) + self.assertIn(self.route, orderpoint.route_ids) + self.assertIn(self.route2, orderpoint.route_ids) + orderpoint.route_id = self.route2.id + self.procurement_group_model.run_scheduler() + move = self.env['stock.move'].search( + [('product_id', '=', self.product.id), + ('location_id', '=', self.ressuply_loc2.id)], limit=1) + self.assertEqual(len(move), 1) diff --git a/stock_orderpoint_route/views/stock_warehouse_orderpoint_views.xml b/stock_orderpoint_route/views/stock_warehouse_orderpoint_views.xml new file mode 100644 index 000000000..565c421d7 --- /dev/null +++ b/stock_orderpoint_route/views/stock_warehouse_orderpoint_views.xml @@ -0,0 +1,19 @@ + + + + + + stock.warehouse.orderpoint.form + stock.warehouse.orderpoint + + + + + + + + + + From b2756120ae64d37cf3bdb6b6bd659352a1f2602f Mon Sep 17 00:00:00 2001 From: Guewen Baconnier Date: Wed, 4 Dec 2019 17:01:29 +0100 Subject: [PATCH 02/12] Migrate stock_orderpoint_route to 12.0 --- stock_orderpoint_route/__manifest__.py | 5 ++-- .../models/stock_warehouse_orderpoint.py | 27 +++++++++++-------- .../readme/CONTRIBUTORS.rst | 1 + .../tests/test_stock_orderpoint_route.py | 16 +++++------ 4 files changed, 28 insertions(+), 21 deletions(-) diff --git a/stock_orderpoint_route/__manifest__.py b/stock_orderpoint_route/__manifest__.py index dc18e2937..cc05df1a7 100644 --- a/stock_orderpoint_route/__manifest__.py +++ b/stock_orderpoint_route/__manifest__.py @@ -4,12 +4,13 @@ "name": "Stock Orderpoint Route", "summary": "Allows to force a route to be used when procuring from " "orderpoints", - "version": "11.0.1.0.0", + "version": "12.0.1.0.0", "license": "LGPL-3", "website": "https://github.com/stock-logistics-warehouse", "author": "Eficent, " + "Camptocamp, " "Odoo Community Association (OCA)", - "category": "Warehouse Management", + "category": "Warehouse", "depends": [ "stock", ], diff --git a/stock_orderpoint_route/models/stock_warehouse_orderpoint.py b/stock_orderpoint_route/models/stock_warehouse_orderpoint.py index bbcaaa800..11bd6221b 100644 --- a/stock_orderpoint_route/models/stock_warehouse_orderpoint.py +++ b/stock_orderpoint_route/models/stock_warehouse_orderpoint.py @@ -15,14 +15,14 @@ class StockWarehouseOrderpoint(models.Model): domain="[('id', 'in', route_ids)]", ondelete='restrict') - @api.depends('product_id', 'warehouse_id', 'location_id') + @api.depends('product_id', 'warehouse_id', + 'warehouse_id.route_ids', 'location_id') def _compute_route_ids(self): route_obj = self.env['stock.location.route'] for wh in self.mapped('warehouse_id'): - wh_routes = route_obj.search( - [('warehouse_ids', '=', wh.id)]) + wh_routes = wh.route_ids for record in self.filtered(lambda r: r.warehouse_id == wh): - routes = route_obj + routes = route_obj.browse() if record.product_id: routes += record.product_id.mapped( 'route_ids' @@ -31,9 +31,15 @@ class StockWarehouseOrderpoint(models.Model): ).mapped('total_route_ids') if record.warehouse_id: routes |= wh_routes - parents = record.get_parents().ids - record.route_ids = routes.filtered(lambda r: any( - p.location_id.id in parents for p in r.pull_ids)) + parents = record.get_parents() + record.route_ids = routes.filtered( + lambda route: any( + p.location_id in parents + for p in route.rule_ids.filtered( + lambda rule: rule.action in ('pull', 'pull_push') + ).mapped('location_src_id') + ) + ) def get_parents(self): location = self.location_id @@ -45,9 +51,8 @@ class StockWarehouseOrderpoint(models.Model): def _prepare_procurement_values(self, product_qty, date=False, group=False): - - res = super(StockWarehouseOrderpoint, - self)._prepare_procurement_values(product_qty, date=date, - group=group) + res = super()._prepare_procurement_values( + product_qty, date=date, group=group + ) res['route_ids'] = self.route_id return res diff --git a/stock_orderpoint_route/readme/CONTRIBUTORS.rst b/stock_orderpoint_route/readme/CONTRIBUTORS.rst index daeadd46d..e62cfba22 100644 --- a/stock_orderpoint_route/readme/CONTRIBUTORS.rst +++ b/stock_orderpoint_route/readme/CONTRIBUTORS.rst @@ -1,2 +1,3 @@ * Jordi Ballester Alomar * Lois Rilo +* Guewen Baconnier diff --git a/stock_orderpoint_route/tests/test_stock_orderpoint_route.py b/stock_orderpoint_route/tests/test_stock_orderpoint_route.py index 398cd3a65..202f5df4d 100644 --- a/stock_orderpoint_route/tests/test_stock_orderpoint_route.py +++ b/stock_orderpoint_route/tests/test_stock_orderpoint_route.py @@ -6,7 +6,7 @@ from odoo.tests import common class TestStockOrderpointRoute(common.TransactionCase): def setUp(self): - super(TestStockOrderpointRoute, self).setUp() + super().setUp() # common models self.orderpoint_model = self.env[ @@ -19,7 +19,7 @@ class TestStockOrderpointRoute(common.TransactionCase): self.env.ref('stock.group_stock_multi_locations') self.main_company = self.env.ref('base.main_company') self.warehouse = self.env.ref('stock.warehouse0') - self.categ_unit = self.env.ref('product.product_uom_categ_unit') + self.categ_unit = self.env.ref('uom.product_uom_categ_unit') self.virtual_loc = self.env.ref('stock.stock_location_customers') # common data @@ -55,19 +55,19 @@ class TestStockOrderpointRoute(common.TransactionCase): 'sequence': 10, }) - self.uom_dozen = self.env['product.uom'].create({ + self.uom_dozen = self.env['uom.uom'].create({ 'name': 'Test-DozenA', 'category_id': self.categ_unit.id, 'factor_inv': 12, 'uom_type': 'bigger', 'rounding': 0.001}) - self.env['procurement.rule'].create({ + self.env['stock.rule'].create({ 'name': 'Transfer', 'route_id': self.route.id, 'location_src_id': self.ressuply_loc.id, 'location_id': self.warehouse.lot_stock_id.id, - 'action': 'move', + 'action': 'pull', 'picking_type_id': self.warehouse.int_type_id.id, 'procure_method': 'make_to_stock', 'warehouse_id': self.warehouse.id, @@ -75,12 +75,12 @@ class TestStockOrderpointRoute(common.TransactionCase): 'propagate': 'False', }) - self.env['procurement.rule'].create({ + self.env['stock.rule'].create({ 'name': 'Transfer 2', 'route_id': self.route2.id, 'location_src_id': self.ressuply_loc2.id, 'location_id': self.warehouse.lot_stock_id.id, - 'action': 'move', + 'action': 'pull', 'picking_type_id': self.warehouse.int_type_id.id, 'procure_method': 'make_to_stock', 'warehouse_id': self.warehouse.id, @@ -103,7 +103,7 @@ class TestStockOrderpointRoute(common.TransactionCase): return self.env['product.product'].create(dict( name=name, default_code=default_code, - uom_id=self.env.ref('product.product_uom_unit').id, + uom_id=self.env.ref('uom.product_uom_unit').id, company_id=company_id, type='product', **vals From 68f5dd2b7c790e47b0f9f82a61d10d7e24b8b10d Mon Sep 17 00:00:00 2001 From: Guewen Baconnier Date: Thu, 5 Dec 2019 09:55:08 +0100 Subject: [PATCH 03/12] Use SavepointCase in tests To avoid executing the setup for each test --- .../tests/test_stock_orderpoint_route.py | 97 ++++++++++--------- 1 file changed, 50 insertions(+), 47 deletions(-) diff --git a/stock_orderpoint_route/tests/test_stock_orderpoint_route.py b/stock_orderpoint_route/tests/test_stock_orderpoint_route.py index 202f5df4d..3275f9d07 100644 --- a/stock_orderpoint_route/tests/test_stock_orderpoint_route.py +++ b/stock_orderpoint_route/tests/test_stock_orderpoint_route.py @@ -4,92 +4,94 @@ from odoo.tests import common -class TestStockOrderpointRoute(common.TransactionCase): - def setUp(self): - super().setUp() +class TestStockOrderpointRoute(common.SavepointCase): + @classmethod + def setUpClass(cls): + super().setUpClass() # common models - self.orderpoint_model = self.env[ + cls.orderpoint_model = cls.env[ 'stock.warehouse.orderpoint'] - self.procurement_group_model = self.env['procurement.group'] + cls.procurement_group_model = cls.env['procurement.group'] # refs - self.stock_manager_group = \ - self.env.ref('stock.group_stock_manager') - self.stock_multi_locations_group_group = \ - self.env.ref('stock.group_stock_multi_locations') - self.main_company = self.env.ref('base.main_company') - self.warehouse = self.env.ref('stock.warehouse0') - self.categ_unit = self.env.ref('uom.product_uom_categ_unit') - self.virtual_loc = self.env.ref('stock.stock_location_customers') + cls.stock_manager_group = \ + cls.env.ref('stock.group_stock_manager') + cls.stock_multi_locations_group_group = \ + cls.env.ref('stock.group_stock_multi_locations') + cls.main_company = cls.env.ref('base.main_company') + cls.warehouse = cls.env.ref('stock.warehouse0') + cls.categ_unit = cls.env.ref('uom.product_uom_categ_unit') + cls.virtual_loc = cls.env.ref('stock.stock_location_customers') # common data - self.stock_manager = self._create_user( + cls.stock_manager = cls._create_user( 'stock_manager', - [self.stock_manager_group.id, - self.stock_multi_locations_group_group.id], - [self.main_company.id]) - self.product = self._create_product('SH', 'Shoes', False) + [cls.stock_manager_group.id, + cls.stock_multi_locations_group_group.id], + [cls.main_company.id]) + cls.product = cls._create_product('SH', 'Shoes', False) - self.ressuply_loc = self.env['stock.location'].create({ + cls.ressuply_loc = cls.env['stock.location'].create({ 'name': 'Ressuply', - 'location_id': self.warehouse.view_location_id.id, + 'location_id': cls.warehouse.view_location_id.id, }) - self.ressuply_loc2 = self.env['stock.location'].create({ + cls.ressuply_loc2 = cls.env['stock.location'].create({ 'name': 'Ressuply2', - 'location_id': self.warehouse.view_location_id.id, + 'location_id': cls.warehouse.view_location_id.id, }) - self.route = self.env['stock.location.route'].create({ + cls.route = cls.env['stock.location.route'].create({ 'name': 'Transfer', 'product_categ_selectable': False, 'product_selectable': True, - 'company_id': self.main_company.id, + 'company_id': cls.main_company.id, 'sequence': 10, }) - self.route2 = self.env['stock.location.route'].create({ + cls.route2 = cls.env['stock.location.route'].create({ 'name': 'Transfer', 'product_categ_selectable': False, 'product_selectable': True, - 'company_id': self.main_company.id, + 'company_id': cls.main_company.id, 'sequence': 10, }) - self.uom_dozen = self.env['uom.uom'].create({ + cls.uom_dozen = cls.env['uom.uom'].create({ 'name': 'Test-DozenA', - 'category_id': self.categ_unit.id, + 'category_id': cls.categ_unit.id, 'factor_inv': 12, 'uom_type': 'bigger', 'rounding': 0.001}) - self.env['stock.rule'].create({ + cls.env['stock.rule'].create({ 'name': 'Transfer', - 'route_id': self.route.id, - 'location_src_id': self.ressuply_loc.id, - 'location_id': self.warehouse.lot_stock_id.id, + 'route_id': cls.route.id, + 'location_src_id': cls.ressuply_loc.id, + 'location_id': cls.warehouse.lot_stock_id.id, 'action': 'pull', - 'picking_type_id': self.warehouse.int_type_id.id, + 'picking_type_id': cls.warehouse.int_type_id.id, 'procure_method': 'make_to_stock', - 'warehouse_id': self.warehouse.id, - 'company_id': self.main_company.id, + 'warehouse_id': cls.warehouse.id, + 'company_id': cls.main_company.id, 'propagate': 'False', }) - self.env['stock.rule'].create({ + cls.env['stock.rule'].create({ 'name': 'Transfer 2', - 'route_id': self.route2.id, - 'location_src_id': self.ressuply_loc2.id, - 'location_id': self.warehouse.lot_stock_id.id, + 'route_id': cls.route2.id, + 'location_src_id': cls.ressuply_loc2.id, + 'location_id': cls.warehouse.lot_stock_id.id, 'action': 'pull', - 'picking_type_id': self.warehouse.int_type_id.id, + 'picking_type_id': cls.warehouse.int_type_id.id, 'procure_method': 'make_to_stock', - 'warehouse_id': self.warehouse.id, - 'company_id': self.main_company.id, + 'warehouse_id': cls.warehouse.id, + 'company_id': cls.main_company.id, 'propagate': 'False', }) - def _create_user(self, name, group_ids, company_ids): - return self.env['res.users'].with_context( + @classmethod + def _create_user(cls, name, group_ids, company_ids): + return cls.env['res.users'].with_context( {'no_reset_password': True}).create( {'name': name, 'password': 'demo', @@ -99,11 +101,12 @@ class TestStockOrderpointRoute(common.TransactionCase): 'company_ids': [(6, 0, company_ids)] }) - def _create_product(self, default_code, name, company_id, **vals): - return self.env['product.product'].create(dict( + @classmethod + def _create_product(cls, default_code, name, company_id, **vals): + return cls.env['product.product'].create(dict( name=name, default_code=default_code, - uom_id=self.env.ref('uom.product_uom_unit').id, + uom_id=cls.env.ref('uom.product_uom_unit').id, company_id=company_id, type='product', **vals From ba113edfc9aaafa6d0f274a9dd826fa8f35ccfad Mon Sep 17 00:00:00 2001 From: Guewen Baconnier Date: Thu, 19 Dec 2019 14:04:20 +0100 Subject: [PATCH 04/12] [IMP] stock_orderpoint_route: black, isort --- stock_orderpoint_route/__manifest__.py | 15 +- .../models/stock_warehouse_orderpoint.py | 40 ++- stock_orderpoint_route/readme/DESCRIPTION.rst | 2 +- .../tests/test_stock_orderpoint_route.py | 235 ++++++++++-------- 4 files changed, 151 insertions(+), 141 deletions(-) diff --git a/stock_orderpoint_route/__manifest__.py b/stock_orderpoint_route/__manifest__.py index cc05df1a7..b12b6dc38 100644 --- a/stock_orderpoint_route/__manifest__.py +++ b/stock_orderpoint_route/__manifest__.py @@ -2,20 +2,13 @@ # License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html). { "name": "Stock Orderpoint Route", - "summary": "Allows to force a route to be used when procuring from " - "orderpoints", + "summary": "Allows to force a route to be used when procuring from orderpoints", "version": "12.0.1.0.0", "license": "LGPL-3", "website": "https://github.com/stock-logistics-warehouse", - "author": "Eficent, " - "Camptocamp, " - "Odoo Community Association (OCA)", + "author": "Eficent, Camptocamp, Odoo Community Association (OCA)", "category": "Warehouse", - "depends": [ - "stock", - ], - "data": [ - "views/stock_warehouse_orderpoint_views.xml", - ], + "depends": ["stock"], + "data": ["views/stock_warehouse_orderpoint_views.xml"], "installable": True, } diff --git a/stock_orderpoint_route/models/stock_warehouse_orderpoint.py b/stock_orderpoint_route/models/stock_warehouse_orderpoint.py index 11bd6221b..82e855b56 100644 --- a/stock_orderpoint_route/models/stock_warehouse_orderpoint.py +++ b/stock_orderpoint_route/models/stock_warehouse_orderpoint.py @@ -5,30 +5,29 @@ from odoo import api, fields, models class StockWarehouseOrderpoint(models.Model): - _inherit = 'stock.warehouse.orderpoint' + _inherit = "stock.warehouse.orderpoint" route_ids = fields.Many2many( - 'stock.location.route', string='Allowed routes', - compute='_compute_route_ids', + "stock.location.route", string="Allowed routes", compute="_compute_route_ids" + ) + route_id = fields.Many2one( + "stock.location.route", + string="Route", + domain="[('id', 'in', route_ids)]", + ondelete="restrict", ) - route_id = fields.Many2one('stock.location.route', string='Route', - domain="[('id', 'in', route_ids)]", - ondelete='restrict') - @api.depends('product_id', 'warehouse_id', - 'warehouse_id.route_ids', 'location_id') + @api.depends("product_id", "warehouse_id", "warehouse_id.route_ids", "location_id") def _compute_route_ids(self): - route_obj = self.env['stock.location.route'] - for wh in self.mapped('warehouse_id'): + route_obj = self.env["stock.location.route"] + for wh in self.mapped("warehouse_id"): wh_routes = wh.route_ids for record in self.filtered(lambda r: r.warehouse_id == wh): routes = route_obj.browse() if record.product_id: routes += record.product_id.mapped( - 'route_ids' - ) | record.product_id.mapped( - 'categ_id' - ).mapped('total_route_ids') + "route_ids" + ) | record.product_id.mapped("categ_id").mapped("total_route_ids") if record.warehouse_id: routes |= wh_routes parents = record.get_parents() @@ -36,8 +35,8 @@ class StockWarehouseOrderpoint(models.Model): lambda route: any( p.location_id in parents for p in route.rule_ids.filtered( - lambda rule: rule.action in ('pull', 'pull_push') - ).mapped('location_src_id') + lambda rule: rule.action in ("pull", "pull_push") + ).mapped("location_src_id") ) ) @@ -49,10 +48,7 @@ class StockWarehouseOrderpoint(models.Model): result |= location return result - def _prepare_procurement_values(self, product_qty, date=False, - group=False): - res = super()._prepare_procurement_values( - product_qty, date=date, group=group - ) - res['route_ids'] = self.route_id + def _prepare_procurement_values(self, product_qty, date=False, group=False): + res = super()._prepare_procurement_values(product_qty, date=date, group=group) + res["route_ids"] = self.route_id return res diff --git a/stock_orderpoint_route/readme/DESCRIPTION.rst b/stock_orderpoint_route/readme/DESCRIPTION.rst index b7b4b208b..284647648 100644 --- a/stock_orderpoint_route/readme/DESCRIPTION.rst +++ b/stock_orderpoint_route/readme/DESCRIPTION.rst @@ -1,3 +1,3 @@ This module allows to restrict a specific route to be used in the reordering rules. This route will be used instead of the default determined by -default. \ No newline at end of file +default. diff --git a/stock_orderpoint_route/tests/test_stock_orderpoint_route.py b/stock_orderpoint_route/tests/test_stock_orderpoint_route.py index 3275f9d07..6a5589daa 100644 --- a/stock_orderpoint_route/tests/test_stock_orderpoint_route.py +++ b/stock_orderpoint_route/tests/test_stock_orderpoint_route.py @@ -10,148 +10,169 @@ class TestStockOrderpointRoute(common.SavepointCase): super().setUpClass() # common models - cls.orderpoint_model = cls.env[ - 'stock.warehouse.orderpoint'] - cls.procurement_group_model = cls.env['procurement.group'] + cls.orderpoint_model = cls.env["stock.warehouse.orderpoint"] + cls.procurement_group_model = cls.env["procurement.group"] # refs - cls.stock_manager_group = \ - cls.env.ref('stock.group_stock_manager') - cls.stock_multi_locations_group_group = \ - cls.env.ref('stock.group_stock_multi_locations') - cls.main_company = cls.env.ref('base.main_company') - cls.warehouse = cls.env.ref('stock.warehouse0') - cls.categ_unit = cls.env.ref('uom.product_uom_categ_unit') - cls.virtual_loc = cls.env.ref('stock.stock_location_customers') + cls.stock_manager_group = cls.env.ref("stock.group_stock_manager") + cls.stock_multi_locations_group_group = cls.env.ref( + "stock.group_stock_multi_locations" + ) + cls.main_company = cls.env.ref("base.main_company") + cls.warehouse = cls.env.ref("stock.warehouse0") + cls.categ_unit = cls.env.ref("uom.product_uom_categ_unit") + cls.virtual_loc = cls.env.ref("stock.stock_location_customers") # common data cls.stock_manager = cls._create_user( - 'stock_manager', - [cls.stock_manager_group.id, - cls.stock_multi_locations_group_group.id], - [cls.main_company.id]) - cls.product = cls._create_product('SH', 'Shoes', False) + "stock_manager", + [cls.stock_manager_group.id, cls.stock_multi_locations_group_group.id], + [cls.main_company.id], + ) + cls.product = cls._create_product("SH", "Shoes", False) - cls.ressuply_loc = cls.env['stock.location'].create({ - 'name': 'Ressuply', - 'location_id': cls.warehouse.view_location_id.id, - }) + cls.ressuply_loc = cls.env["stock.location"].create( + {"name": "Ressuply", "location_id": cls.warehouse.view_location_id.id} + ) - cls.ressuply_loc2 = cls.env['stock.location'].create({ - 'name': 'Ressuply2', - 'location_id': cls.warehouse.view_location_id.id, - }) + cls.ressuply_loc2 = cls.env["stock.location"].create( + {"name": "Ressuply2", "location_id": cls.warehouse.view_location_id.id} + ) - cls.route = cls.env['stock.location.route'].create({ - 'name': 'Transfer', - 'product_categ_selectable': False, - 'product_selectable': True, - 'company_id': cls.main_company.id, - 'sequence': 10, - }) - cls.route2 = cls.env['stock.location.route'].create({ - 'name': 'Transfer', - 'product_categ_selectable': False, - 'product_selectable': True, - 'company_id': cls.main_company.id, - 'sequence': 10, - }) + cls.route = cls.env["stock.location.route"].create( + { + "name": "Transfer", + "product_categ_selectable": False, + "product_selectable": True, + "company_id": cls.main_company.id, + "sequence": 10, + } + ) + cls.route2 = cls.env["stock.location.route"].create( + { + "name": "Transfer", + "product_categ_selectable": False, + "product_selectable": True, + "company_id": cls.main_company.id, + "sequence": 10, + } + ) - cls.uom_dozen = cls.env['uom.uom'].create({ - 'name': 'Test-DozenA', - 'category_id': cls.categ_unit.id, - 'factor_inv': 12, - 'uom_type': 'bigger', - 'rounding': 0.001}) + cls.uom_dozen = cls.env["uom.uom"].create( + { + "name": "Test-DozenA", + "category_id": cls.categ_unit.id, + "factor_inv": 12, + "uom_type": "bigger", + "rounding": 0.001, + } + ) - cls.env['stock.rule'].create({ - 'name': 'Transfer', - 'route_id': cls.route.id, - 'location_src_id': cls.ressuply_loc.id, - 'location_id': cls.warehouse.lot_stock_id.id, - 'action': 'pull', - 'picking_type_id': cls.warehouse.int_type_id.id, - 'procure_method': 'make_to_stock', - 'warehouse_id': cls.warehouse.id, - 'company_id': cls.main_company.id, - 'propagate': 'False', - }) + cls.env["stock.rule"].create( + { + "name": "Transfer", + "route_id": cls.route.id, + "location_src_id": cls.ressuply_loc.id, + "location_id": cls.warehouse.lot_stock_id.id, + "action": "pull", + "picking_type_id": cls.warehouse.int_type_id.id, + "procure_method": "make_to_stock", + "warehouse_id": cls.warehouse.id, + "company_id": cls.main_company.id, + "propagate": "False", + } + ) - cls.env['stock.rule'].create({ - 'name': 'Transfer 2', - 'route_id': cls.route2.id, - 'location_src_id': cls.ressuply_loc2.id, - 'location_id': cls.warehouse.lot_stock_id.id, - 'action': 'pull', - 'picking_type_id': cls.warehouse.int_type_id.id, - 'procure_method': 'make_to_stock', - 'warehouse_id': cls.warehouse.id, - 'company_id': cls.main_company.id, - 'propagate': 'False', - }) + cls.env["stock.rule"].create( + { + "name": "Transfer 2", + "route_id": cls.route2.id, + "location_src_id": cls.ressuply_loc2.id, + "location_id": cls.warehouse.lot_stock_id.id, + "action": "pull", + "picking_type_id": cls.warehouse.int_type_id.id, + "procure_method": "make_to_stock", + "warehouse_id": cls.warehouse.id, + "company_id": cls.main_company.id, + "propagate": "False", + } + ) @classmethod def _create_user(cls, name, group_ids, company_ids): - return cls.env['res.users'].with_context( - {'no_reset_password': True}).create( - {'name': name, - 'password': 'demo', - 'login': name, - 'email': '@'.join([name, '@test.com']), - 'groups_id': [(6, 0, group_ids)], - 'company_ids': [(6, 0, company_ids)] - }) + return ( + cls.env["res.users"] + .with_context({"no_reset_password": True}) + .create( + { + "name": name, + "password": "demo", + "login": name, + "email": "@".join([name, "@test.com"]), + "groups_id": [(6, 0, group_ids)], + "company_ids": [(6, 0, company_ids)], + } + ) + ) @classmethod def _create_product(cls, default_code, name, company_id, **vals): - return cls.env['product.product'].create(dict( - name=name, - default_code=default_code, - uom_id=cls.env.ref('uom.product_uom_unit').id, - company_id=company_id, - type='product', - **vals - )) + return cls.env["product.product"].create( + dict( + name=name, + default_code=default_code, + uom_id=cls.env.ref("uom.product_uom_unit").id, + company_id=company_id, + type="product", + **vals + ) + ) def test_orderpoint_route_01(self): self.product.route_ids = [(6, 0, [self.route.id, self.route2.id])] vals = { - 'product_id': self.product.id, - 'product_min_qty': 10.0, - 'product_max_qty': 100.0, - 'company_id': self.main_company.id, - 'warehouse_id': self.warehouse.id, - 'location_id': self.warehouse.lot_stock_id.id, + "product_id": self.product.id, + "product_min_qty": 10.0, + "product_max_qty": 100.0, + "company_id": self.main_company.id, + "warehouse_id": self.warehouse.id, + "location_id": self.warehouse.lot_stock_id.id, } - orderpoint = self.orderpoint_model.sudo( - self.stock_manager).create(vals) + orderpoint = self.orderpoint_model.sudo(self.stock_manager).create(vals) self.assertIn(self.route, orderpoint.route_ids) self.assertIn(self.route2, orderpoint.route_ids) orderpoint.route_id = self.route.id self.procurement_group_model.run_scheduler() - move = self.env['stock.move'].search( - [('product_id', '=', self.product.id), - ('location_id', '=', self.ressuply_loc.id)], limit=1) + move = self.env["stock.move"].search( + [ + ("product_id", "=", self.product.id), + ("location_id", "=", self.ressuply_loc.id), + ], + limit=1, + ) self.assertEqual(len(move), 1) def test_orderpoint_route_02(self): self.product.route_ids = [(6, 0, [self.route.id, self.route2.id])] vals = { - 'product_id': self.product.id, - 'product_min_qty': 10.0, - 'product_max_qty': 100.0, - 'company_id': self.main_company.id, - 'warehouse_id': self.warehouse.id, - 'location_id': self.warehouse.lot_stock_id.id, + "product_id": self.product.id, + "product_min_qty": 10.0, + "product_max_qty": 100.0, + "company_id": self.main_company.id, + "warehouse_id": self.warehouse.id, + "location_id": self.warehouse.lot_stock_id.id, } - orderpoint = self.orderpoint_model.sudo( - self.stock_manager).create(vals) + orderpoint = self.orderpoint_model.sudo(self.stock_manager).create(vals) self.assertIn(self.route, orderpoint.route_ids) self.assertIn(self.route2, orderpoint.route_ids) orderpoint.route_id = self.route2.id self.procurement_group_model.run_scheduler() - move = self.env['stock.move'].search( - [('product_id', '=', self.product.id), - ('location_id', '=', self.ressuply_loc2.id)], limit=1) + move = self.env["stock.move"].search( + [ + ("product_id", "=", self.product.id), + ("location_id", "=", self.ressuply_loc2.id), + ], + limit=1, + ) self.assertEqual(len(move), 1) From 6b6cf92d463cd9c38afd4ca9e4e5cec79a56a44e Mon Sep 17 00:00:00 2001 From: Guewen Baconnier Date: Thu, 19 Dec 2019 14:19:47 +0100 Subject: [PATCH 05/12] [MIG] stock_orderpoint_route: Migration to 13.0 --- stock_orderpoint_route/__manifest__.py | 2 +- stock_orderpoint_route/tests/test_stock_orderpoint_route.py | 2 -- .../views/stock_warehouse_orderpoint_views.xml | 2 +- 3 files changed, 2 insertions(+), 4 deletions(-) diff --git a/stock_orderpoint_route/__manifest__.py b/stock_orderpoint_route/__manifest__.py index b12b6dc38..c4e4224d7 100644 --- a/stock_orderpoint_route/__manifest__.py +++ b/stock_orderpoint_route/__manifest__.py @@ -3,7 +3,7 @@ { "name": "Stock Orderpoint Route", "summary": "Allows to force a route to be used when procuring from orderpoints", - "version": "12.0.1.0.0", + "version": "13.0.1.0.0", "license": "LGPL-3", "website": "https://github.com/stock-logistics-warehouse", "author": "Eficent, Camptocamp, Odoo Community Association (OCA)", diff --git a/stock_orderpoint_route/tests/test_stock_orderpoint_route.py b/stock_orderpoint_route/tests/test_stock_orderpoint_route.py index 6a5589daa..7989e0c52 100644 --- a/stock_orderpoint_route/tests/test_stock_orderpoint_route.py +++ b/stock_orderpoint_route/tests/test_stock_orderpoint_route.py @@ -78,7 +78,6 @@ class TestStockOrderpointRoute(common.SavepointCase): "procure_method": "make_to_stock", "warehouse_id": cls.warehouse.id, "company_id": cls.main_company.id, - "propagate": "False", } ) @@ -93,7 +92,6 @@ class TestStockOrderpointRoute(common.SavepointCase): "procure_method": "make_to_stock", "warehouse_id": cls.warehouse.id, "company_id": cls.main_company.id, - "propagate": "False", } ) diff --git a/stock_orderpoint_route/views/stock_warehouse_orderpoint_views.xml b/stock_orderpoint_route/views/stock_warehouse_orderpoint_views.xml index 565c421d7..fa4b92eda 100644 --- a/stock_orderpoint_route/views/stock_warehouse_orderpoint_views.xml +++ b/stock_orderpoint_route/views/stock_warehouse_orderpoint_views.xml @@ -8,7 +8,7 @@ stock.warehouse.orderpoint - + From 2d09392b1675bef6037909f4f782c05d0d470afe Mon Sep 17 00:00:00 2001 From: Guewen Baconnier Date: Tue, 17 Mar 2020 11:14:02 +0100 Subject: [PATCH 06/12] run pre-commit with new prettiers --- .../views/stock_warehouse_orderpoint_views.xml | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/stock_orderpoint_route/views/stock_warehouse_orderpoint_views.xml b/stock_orderpoint_route/views/stock_warehouse_orderpoint_views.xml index fa4b92eda..2d7cb9748 100644 --- a/stock_orderpoint_route/views/stock_warehouse_orderpoint_views.xml +++ b/stock_orderpoint_route/views/stock_warehouse_orderpoint_views.xml @@ -1,19 +1,20 @@ - + - stock.warehouse.orderpoint.form stock.warehouse.orderpoint - + - - + + - From a53fd653c3b68edd8936d1905bff3f9f8676bc70 Mon Sep 17 00:00:00 2001 From: Guewen Baconnier Date: Tue, 17 Mar 2020 11:44:58 +0100 Subject: [PATCH 07/12] Improve code after reviews * performance (less queries executed) * disable tracking in tests (faster) * use with_user instead of sudo * the double for loop in _compute_route_ids actually generates one more query as the simple one in this commit (thanks to the cache) * extract a method * the parent locations can be found using 'parent_of' which will use the parent_path under the hood (example: 1/7/8 will return locations 1, 7, 8) [UPD] Update stock_orderpoint_route.pot --- .../i18n/stock_orderpoint_route.pot | 29 +++++++++++ .../models/stock_warehouse_orderpoint.py | 49 +++++++++---------- .../tests/test_stock_orderpoint_route.py | 7 +-- 3 files changed, 57 insertions(+), 28 deletions(-) create mode 100644 stock_orderpoint_route/i18n/stock_orderpoint_route.pot diff --git a/stock_orderpoint_route/i18n/stock_orderpoint_route.pot b/stock_orderpoint_route/i18n/stock_orderpoint_route.pot new file mode 100644 index 000000000..4fadbbadd --- /dev/null +++ b/stock_orderpoint_route/i18n/stock_orderpoint_route.pot @@ -0,0 +1,29 @@ +# Translation of Odoo Server. +# This file contains the translation of the following modules: +# * stock_orderpoint_route +# +msgid "" +msgstr "" +"Project-Id-Version: Odoo Server 13.0\n" +"Report-Msgid-Bugs-To: \n" +"Last-Translator: \n" +"Language-Team: \n" +"MIME-Version: 1.0\n" +"Content-Type: text/plain; charset=UTF-8\n" +"Content-Transfer-Encoding: \n" +"Plural-Forms: \n" + +#. module: stock_orderpoint_route +#: model:ir.model.fields,field_description:stock_orderpoint_route.field_stock_warehouse_orderpoint__route_ids +msgid "Allowed routes" +msgstr "" + +#. module: stock_orderpoint_route +#: model:ir.model,name:stock_orderpoint_route.model_stock_warehouse_orderpoint +msgid "Minimum Inventory Rule" +msgstr "" + +#. module: stock_orderpoint_route +#: model:ir.model.fields,field_description:stock_orderpoint_route.field_stock_warehouse_orderpoint__route_id +msgid "Route" +msgstr "" diff --git a/stock_orderpoint_route/models/stock_warehouse_orderpoint.py b/stock_orderpoint_route/models/stock_warehouse_orderpoint.py index 82e855b56..ca857510a 100644 --- a/stock_orderpoint_route/models/stock_warehouse_orderpoint.py +++ b/stock_orderpoint_route/models/stock_warehouse_orderpoint.py @@ -20,33 +20,32 @@ class StockWarehouseOrderpoint(models.Model): @api.depends("product_id", "warehouse_id", "warehouse_id.route_ids", "location_id") def _compute_route_ids(self): route_obj = self.env["stock.location.route"] - for wh in self.mapped("warehouse_id"): - wh_routes = wh.route_ids - for record in self.filtered(lambda r: r.warehouse_id == wh): - routes = route_obj.browse() - if record.product_id: - routes += record.product_id.mapped( - "route_ids" - ) | record.product_id.mapped("categ_id").mapped("total_route_ids") - if record.warehouse_id: - routes |= wh_routes - parents = record.get_parents() - record.route_ids = routes.filtered( - lambda route: any( - p.location_id in parents - for p in route.rule_ids.filtered( - lambda rule: rule.action in ("pull", "pull_push") - ).mapped("location_src_id") - ) - ) + for record in self: + wh_routes = record.warehouse_id.route_ids + routes = route_obj.browse() + if record.product_id: + routes += record.product_id.mapped( + "route_ids" + ) | record.product_id.mapped("categ_id").mapped("total_route_ids") + if record.warehouse_id: + routes |= wh_routes + parents = record.get_parents() + record.route_ids = self._get_location_routes_of_parents(routes, parents) + + def _get_location_routes_of_parents(self, routes, parents): + return routes.filtered( + lambda route: any( + p.location_id in parents + for p in route.rule_ids.filtered( + lambda rule: rule.action in ("pull", "pull_push") + ).mapped("location_src_id") + ) + ) def get_parents(self): - location = self.location_id - result = location - while location.location_id: - location = location.location_id - result |= location - return result + return self.env["stock.location"].search( + [("id", "parent_of", self.location_id.id)] + ) def _prepare_procurement_values(self, product_qty, date=False, group=False): res = super()._prepare_procurement_values(product_qty, date=date, group=group) diff --git a/stock_orderpoint_route/tests/test_stock_orderpoint_route.py b/stock_orderpoint_route/tests/test_stock_orderpoint_route.py index 7989e0c52..f0bde822c 100644 --- a/stock_orderpoint_route/tests/test_stock_orderpoint_route.py +++ b/stock_orderpoint_route/tests/test_stock_orderpoint_route.py @@ -8,6 +8,7 @@ class TestStockOrderpointRoute(common.SavepointCase): @classmethod def setUpClass(cls): super().setUpClass() + cls.env = cls.env(context=dict(cls.env.context, tracking_disable=True)) # common models cls.orderpoint_model = cls.env["stock.warehouse.orderpoint"] @@ -99,7 +100,7 @@ class TestStockOrderpointRoute(common.SavepointCase): def _create_user(cls, name, group_ids, company_ids): return ( cls.env["res.users"] - .with_context({"no_reset_password": True}) + .with_context(no_reset_password=True) .create( { "name": name, @@ -136,7 +137,7 @@ class TestStockOrderpointRoute(common.SavepointCase): "location_id": self.warehouse.lot_stock_id.id, } - orderpoint = self.orderpoint_model.sudo(self.stock_manager).create(vals) + orderpoint = self.orderpoint_model.with_user(self.stock_manager).create(vals) self.assertIn(self.route, orderpoint.route_ids) self.assertIn(self.route2, orderpoint.route_ids) orderpoint.route_id = self.route.id @@ -161,7 +162,7 @@ class TestStockOrderpointRoute(common.SavepointCase): "location_id": self.warehouse.lot_stock_id.id, } - orderpoint = self.orderpoint_model.sudo(self.stock_manager).create(vals) + orderpoint = self.orderpoint_model.with_user(self.stock_manager).create(vals) self.assertIn(self.route, orderpoint.route_ids) self.assertIn(self.route2, orderpoint.route_ids) orderpoint.route_id = self.route2.id From 514a866d10df5f580f073a89a5cef89c673e594e Mon Sep 17 00:00:00 2001 From: OCA-git-bot Date: Mon, 23 Mar 2020 10:46:34 +0000 Subject: [PATCH 08/12] [UPD] README.rst --- stock_orderpoint_route/README.rst | 12 +++--- .../static/description/index.html | 38 +++++++++++++++---- 2 files changed, 38 insertions(+), 12 deletions(-) diff --git a/stock_orderpoint_route/README.rst b/stock_orderpoint_route/README.rst index 5fb6cfbbe..9e5bd015e 100644 --- a/stock_orderpoint_route/README.rst +++ b/stock_orderpoint_route/README.rst @@ -14,13 +14,13 @@ Stock Orderpoint Route :target: http://www.gnu.org/licenses/lgpl-3.0-standalone.html :alt: License: LGPL-3 .. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fstock--logistics--warehouse-lightgray.png?logo=github - :target: https://github.com/OCA/stock-logistics-warehouse/tree/11.0/stock_orderpoint_route + :target: https://github.com/OCA/stock-logistics-warehouse/tree/13.0/stock_orderpoint_route :alt: OCA/stock-logistics-warehouse .. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png - :target: https://translation.odoo-community.org/projects/stock-logistics-warehouse-11-0/stock-logistics-warehouse-11-0-stock_orderpoint_route + :target: https://translation.odoo-community.org/projects/stock-logistics-warehouse-13-0/stock-logistics-warehouse-13-0-stock_orderpoint_route :alt: Translate me on Weblate .. |badge5| image:: https://img.shields.io/badge/runbot-Try%20me-875A7B.png - :target: https://runbot.odoo-community.org/runbot/153/11.0 + :target: https://runbot.odoo-community.org/runbot/153/13.0 :alt: Try me on Runbot |badge1| |badge2| |badge3| |badge4| |badge5| @@ -46,7 +46,7 @@ Bug Tracker Bugs are tracked on `GitHub Issues `_. In case of trouble, please check there if your issue has already been reported. If you spotted it first, help us smashing it by providing a detailed and welcomed -`feedback `_. +`feedback `_. Do not contact contributors directly about support or help with technical issues. @@ -57,12 +57,14 @@ Authors ~~~~~~~ * Eficent +* Camptocamp Contributors ~~~~~~~~~~~~ * Jordi Ballester Alomar * Lois Rilo +* Guewen Baconnier Maintainers ~~~~~~~~~~~ @@ -77,6 +79,6 @@ OCA, or the Odoo Community Association, is a nonprofit organization whose mission is to support the collaborative development of Odoo features and promote its widespread use. -This module is part of the `OCA/stock-logistics-warehouse `_ project on GitHub. +This module is part of the `OCA/stock-logistics-warehouse `_ project on GitHub. You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute. diff --git a/stock_orderpoint_route/static/description/index.html b/stock_orderpoint_route/static/description/index.html index 69e16318c..d062da71a 100644 --- a/stock_orderpoint_route/static/description/index.html +++ b/stock_orderpoint_route/static/description/index.html @@ -3,13 +3,13 @@ - + Stock Orderpoint Route