[FIX] main_seller - adjust depends and temp function

[quentinDupont]

• FIX main_seller changing if partner changes in supplier info (supplierinfo.name is the partner)
• TMP ? add a function to calculate main seller if it's different from first seller id → je n'ai pas réussi à le faire en requête sql, j'ai l'impression que ce n'est pas possible. Et c'est bien plus simple qu'un script python donc j'ai pris le parti de faire ça :) Ça permet manuellement aux actis de remettre bien fournisseur principal

[codecov]

Codecov Report

Attention: Patch coverage is 45.45455% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 78.52%. Comparing base (7bbe5af) to head (21312e5).
Report is 33 commits behind head on 12.0.

Files	Patch %	Lines
product_main_seller/models/product_product.py	45.45%	6 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             12.0     #360   +/-   ##
=======================================
  Coverage   78.51%   78.52%           
=======================================
  Files         221      221           
  Lines        2486     2515   +29     
  Branches      440      450   +10     
=======================================
+ Hits         1952     1975   +23     
- Misses        502      507    +5     
- Partials       32       33    +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[legalsylvain]

• 👍 pour le correctif pour arrêter que de nouveaux cas se produisent.

• pas fan du design pour corriger le truc.

  • Potentiellement, des erreurs vont perdurer, si des gens corrigent pas.
  • comme diff_product_main_first_seller n'est pas searchable, même si les gens veulent "s'y atteler", il devront faire défiler tous leurs produits. (je crois que l'une des acti veut "corriger" des stats qui sont fausses).
  • ça introduit du code qui ne servira pas dans le futur puisque ça sert à corriger de la data erronée. je pense qu'on s'éloigne d'un partage OCA du coup.

je n'ai pas réussi à le faire en requête sql, j'ai l'impression que ce n'est pas possible

Je vais regarder ça !

[legalsylvain]

proposition alternative ici : #361

[quentinDupont]

trop bien !
Oui je suis d'accord avec tout ce que tu as dis :)
et le patch était là par mon manque de talent SQL et le fait de quand même trouver une soluc ;) mais trop bien bravo !