Development #15314
Considérer les connecteurs activés par défaut
0%
Description
C'est nul de devoir taper des PASSERELLE_WHATEVER_ENABLED pour activer des connecteurs; si nul que le code de is_enabled se trouve redéfini dans des connecteurs pour simplement faire :
@classmethod
def is_enabled(cls):
return True
Il faudrait plutôt qu'on active les connecteurs par défaut, libre alors de taper un ..._ENABLED = False explicite si l'on veut. (et comme ça plus besoin de taper ces is_enabled dans les connecteurs).
Fichiers
Révisions associées
enable connectors by default (#15314)
remove useless reimplementations of get_verbose_name (#15314)
add default implementation of get_icon_class (#15314)
general: remove unnecessary get_icon_class from connectors (#15314)
general: enable connectors by default (bis) (#15314)
Revert "remove useless reimplementations of get_verbose_name (#15314)"
This reverts commit f35dbf487dd30423c44bd4ca49004dbaa87d1a96.
Historique
Mis à jour par Benjamin Dauvergne il y a environ 7 ans
- Fichier 0001-enable-connectors-by-default-15314.patch 0001-enable-connectors-by-default-15314.patch ajouté
- Fichier 0002-remove-useless-reimplementations-of-get_verbose_name.patch 0002-remove-useless-reimplementations-of-get_verbose_name.patch ajouté
- Patch proposed changé de Non à Oui
Au passage je vire aussi tous les get_verbose_name() en trop.
Mis à jour par Benjamin Dauvergne il y a environ 7 ans
À voir quand on pourra virer le check sur get_icon_class et ajouter une implémentation par défaut retournant 'ressources'.
Mis à jour par Frédéric Péters il y a environ 7 ans
À voir quand on pourra virer le check sur get_icon_class et ajouter une implémentation par défaut retournant 'ressources'.
On peut désormais (post #14022); à voir si tu veux aussi faire ça dans ce ticket, ou pas.
Mis à jour par Benjamin Dauvergne il y a environ 7 ans
Mis à jour par Frédéric Péters il y a environ 7 ans
Il y a le petit bout suivant qui peut être retiré :
# TODO: once the legacy connectors are ported, the test on
# get_icon_class won't be necessary anymore.
if not hasattr(cls, 'get_icon_class'):
return False
Mis à jour par Benjamin Dauvergne il y a environ 7 ans
- Fichier 0003-add-default-implementation-of-get_icon_class-15314.patch 0003-add-default-implementation-of-get_icon_class-15314.patch ajouté
diff --git a/passerelle/base/models.py b/passerelle/base/models.py
index b0cf8e1..a9d43ce 100644
--- a/passerelle/base/models.py
+++ b/passerelle/base/models.py
@@ -146,10 +146,6 @@ class BaseResource(models.Model):
@classmethod
def is_enabled(cls):
- # TODO: once the legacy connectors are ported, the test on
- # get_icon_class won't be necessary anymore.
- if not hasattr(cls, 'get_icon_class'):
- return False
return getattr(settings, 'PASSERELLE_APP_%s_ENABLED' % cls._meta.app_label.upper(), True)
@classmethod
Mis à jour par Frédéric Péters il y a presque 7 ans
- Fichier 0001-general-remove-unnecessary-get_icon_class-from-conne.patch 0001-general-remove-unnecessary-get_icon_class-from-conne.patch ajouté
- Fichier 0002-settings-remove-unnecessary-_ENABLED-flags-for-appli.patch 0002-settings-remove-unnecessary-_ENABLED-flags-for-appli.patch ajouté
- Statut changé de Nouveau à En cours
Ensuite retirer les get_icon_class et les _ENABLED devenu inutiles.
Avec ça, ce serait ack pour moi.
Mis à jour par Frédéric Péters il y a presque 7 ans
- Statut changé de En cours à Résolu (à déployer)
Patchs réalignés et poussés.
commit 86fd009a8a95c8cf332cc88b4d4d3ec752103395
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sat May 13 21:11:38 2017 +0200
general: remove unnecessary get_icon_class from connectors (#15314)
commit da6a680a7fa9162b0a7285810de981d95183260f
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Mon Mar 27 11:56:07 2017 +0200
add default implementation of get_icon_class (#15314)
commit f35dbf487dd30423c44bd4ca49004dbaa87d1a96
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Tue Mar 7 16:24:41 2017 +0100
remove useless reimplementations of get_verbose_name (#15314)
commit 794f715613ae9c16ea70dafa6cf0421dfd520197
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Tue Mar 7 16:24:33 2017 +0100
enable connectors by default (#15314)
commit 829fe1cd825b6a2241a17a2e19590b7c04adea1f
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sat May 13 21:15:46 2017 +0200
settings: remove unnecessary _ENABLED flags for applications (#15314)
Mis à jour par Frédéric Péters il y a presque 7 ans
- Statut changé de Résolu (à déployer) à En cours
- Assigné à mis à Frédéric Péters
Bon et il y a un truc qui a été loupé et je n'avais pas exécuté les tests en local avant :/
Mis à jour par Frédéric Péters il y a presque 7 ans
- Statut changé de En cours à Résolu (à déployer)
Poussé la correction, manquait un bout dans 0001-enable-connectors-by-default-15314.patch.
--- a/passerelle/urls_utils.py
+++ b/passerelle/urls_utils.py
@@ -58,7 +58,7 @@ def unless(test, message):
def app_enabled(app_label):
'''for enabling a view based on PASSERELLE_APP_<APP_LABEL>_ENABLED flag'''
def test():
- return getattr(settings, 'PASSERELLE_APP_%s_ENABLED' % app_label.upper(), False)
+ return getattr(settings, 'PASSERELLE_APP_%s_ENABLED' % app_label.upper(), True)
return unless(test, 'please enable %s' % app_label)
Mis à jour par Frédéric Péters il y a presque 7 ans
+ revert du 0002-remove-useless-reimplementations-of-get_verbose_name.patch qui ne fonctionne pas. (misère du patch, misère de la relecture).
Mis à jour par Frédéric Péters il y a presque 7 ans
+ suppression des .pyc qui restaient sur le jenkins.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Fermé
settings: remove unnecessary _ENABLED flags for applications (#15314)