Produits Entr'ouvert » w.c.s.

Sujet du commit, plutôt préfixer par commands: ...

Import de Role inutile.

s/is_already_configurated/is_empty/

On n'utilise pas file().

Utiliser le init_tenant_publisher de TenantCommand plutôt que dupliquer ce code.

(je ne suis pas certain d'avoir bien compris le dernier point)

En effet :

+ from . import TenantCommand

- class Command(BaseCommand):
+ class Command(TenantCommand):
...
+     publisher = self.init_tenant_publisher(domain)
...

Ah je comprends enfin l'affaire, désolé, il faut d'abord #34405, c'est là-dedans qu'arrive la classe TenantCommand.

ok j'ai poussé une version qui utilise TenantCommand (nécessite #34405).

Mais la branche en question ne le reprend pas ? Et jenkins passe par l'absence de tests.

+        parser.add_argument('--if-empty', action='store_true', default=False,
+                            help='Import only if wcs is empty')

Plutôt "if site is empty", ou "if tenant is empty".

+        if not os.path.exists(filename):
+            raise CommandError('unknown file: %s' % filename)

Quand un fichier est manquant on écrira plus volontiers "missing file".

J'ai adapté la branche avec les corrections demandées.

Moi ça me semble OK, mais je préfère que ceux qui ont commencé à gérer (Christophe ou Frédéric) pose le "solution validée" ici.

Un seul truc, dont j'ai déjà discuté avec Nicolas :

        if if_empty and not is_empty:
            return

Ici j'aurais bien vu un raise CommandError pour dire que l'import n'a pas eu lieu. Mais je comprends qu'on préfère avoir le même résultat que pour Combo & co, où rien ne se passe dans ce cas... Peut-être faire un print en cas de verbosity>0. Bon c'est vraiment mineur comme truc, et comme c'est déjà tout silencieux dans Combo, on peut sans doute pousser ça.