Update in command line don't respect dependances !

[Joël Grand-Guillaume @ camptocamp]

If I try to upgrade a module trought the client I don't get the same result than if I do it from command line --update=MY_MODULE !!

It seems like form the client, dependances are respected, but NOT from the --update= !!!!

This is really dangerous... I got lots of problems with that, cause the needed module are not updated...

For exemple :
-------------------
If hr_holiday depends on hr

Update hr with the client will update also hr_holiday, but not from --update=hr !!!!!!!!!!!!!!!!

Really I don't understand how something like this could hapen ! Dependances are really important, We must correct this as soon as possible !

[Fabien (Open ERP) (fp-tinyerp)]

dependencies are correct in the server. It's in the client that this is False and only a few people do upgrade through the client, that's why we did not noticed it. It's fixed in trunk.

[Joël Grand-Guillaume @ camptocamp (jgrandguillaume-c2c)]

Le 17 sept. 08 à 17:55, Fabien Pinckaers a écrit :

Salut,

Sur quelle version es-tu ?

Sur une stable 4.2

Je viens de tester sur la trunk et c'est l'inverse. Le --update=module est juste mais pas le client.

Si je fais --update=hr, il doit updater hr_timesheet mais pas base.

Oui, c'est exact !

Le truc c'est que l'upgrade doit avoir le comportement inverse de l'install. A l'install il doit vérifier les parents, à l'upgrade, il doit vérifier les fils.

Je suis d'accord !

Dans mon exemple :

J'update hr avec --update=hr => seul HR est mis à jour => ça ne va pas !

Depuis le client je fait upgrade hr, il me propose de mettre à jour hr_timesheet, hr_expense, hr_holiday, etc... => C'est correct, il veut mettre à jour les modules qui ont dans leur "depends" hr...

J'ai montré à Christophe le problème. C'est vraiment la base du tout ces update. Je travail énormément avec et je me suis tiré des balles hier... J'en suis venu à faire une liste de succession d'update pour obtenir le résultat souhaité.... ça vient stupide !

Est-ce que tu peux regarder ça avec Christophe au plus vite et me donner des nouvelles ?

[Joël Grand-Guillaume @ camptocamp (jgrandguillaume-c2c)]

You only fix the problem in the trunk branch as I heard... Please, fix it in stable version !!

[Joël Grand-Guillaume @ camptocamp (jgrandguillaume-c2c)]

I confirm this:

In stable release the update is only right from client side.

--update from the server does'nt respect dependencies !!!

We must correct this asap !

[Raphaël Valyi - http://www.akretion.com (rvalyi)]

Please, as you fix this bug,
 consider changing the wrong dependency message, see my bug report here:
https://bugs.launchpad.net/openerp/+bug/272101
Nothing critical this time, but trivial fix and would make things more clear when you can't install/remove a module.