Projet

Général

Profil

Development #43128

contrôle par le modèle d'un connecteur de son formulaire de création/édition

Ajouté par Frédéric Péters il y a presque 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
20 mai 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description


Fichiers


Demandes liées

Lié à Passerelle - Bug #20995: connecteur pastell casséFermé04 janvier 2018

Actions

Révisions associées

Révision f2c5518c (diff)
Ajouté par Frédéric Péters il y a presque 4 ans

general: add connector class method to get new/edit form class (#43128)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 4 ans

#2

Mis à jour par Nicolas Roche il y a presque 4 ans

Je constate une régression lors de l'ajout d'un connecteur Pastell : https://passerelle.dev.publik.love/manage/pastell/add

Internal Server Error: /manage/pastell/add
  File "/home/nroche/src/passerelle/passerelle/views.py", line 156, in init_stuff
    self.form_class = self.model.get_manager_form_class(exclude=self.exclude_fields)
TypeError: get_manager_form_class() got an unexpected keyword argument 'exclude'

Je joins un patch pour donner un exemple de correction (pour éviter de copier/coller ici).

#3

Mis à jour par Frédéric Péters il y a presque 4 ans

Je ne sais même pas comment tu arrives là le "connecteur" pastell est cassé de fond en comble.

J’ai tapé dans la branche un commit supplémentaire pour le corriger un peu mais la bonne solution est de dégager tout ça.

#4

Mis à jour par Frédéric Péters il y a presque 4 ans

  • Lié à Bug #20995: connecteur pastell cassé ajouté
#5

Mis à jour par Nicolas Roche il y a presque 4 ans

Je ne sais même pas comment tu arrives là le "connecteur" pastell est cassé de fond en comble.

C'est parce que pour #39650 j'ai eu ce même soucis lors de l'ajout.
Comme je l'intègre de la même façon que tu as intégré Pastell dans ce patch, j'ai testé sur Pastell (mais sans l'avoir activé) pour m'assurer que l'erreur énoncée ci-dessus ne venait pas de moi.

#6

Mis à jour par Frédéric Péters il y a presque 4 ans

Passons.

Commentaire sur la branche attachée ?

#7

Mis à jour par Nicolas Roche il y a presque 4 ans

Il n'y a plus de problème avec Pastel, mais je n'arrive pas encore à utiliser ce path pour #39650.

#8

Mis à jour par Frédéric Péters il y a presque 4 ans

Tu peux décrire tont problème ? (développer "je n'arrive pas à utiliser ce patch").

J'imagine que tu veux dire : ce patch s'applique mais je reste encore bloqué sur un aspect de #39650 qui me semble en rapport, qui est que j'aimerais faire ceci ... et que j'ai essayé cela ..., sans succès.

Mais peut-être pas ?

Je ne peux pas deviner.

#9

Mis à jour par Nicolas Roche il y a presque 4 ans

ce patch s'applique mais je reste encore bloqué sur un aspect de #39650 qui me semble en rapport

oui

Désolé, ne voulant pas polluer ici, j'ai mis ces informations dans https://dev.entrouvert.org/issues/39650#note-26 parce que cela me semble hors contexte ici.

#10

Mis à jour par Nicolas Roche il y a presque 4 ans

  • Statut changé de Solution proposée à Solution validée

(Ok, mon erreur était de penser que formulaires SMS s'intégreraient de la même façon que celui du connecteur Pastell.)

#11

Mis à jour par Frédéric Péters il y a presque 4 ans

Le connecteur pastell n'est pas activé, est à supprimer, certainement pas un modèle.

#12

Mis à jour par Frédéric Péters il y a presque 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit f2c5518c4aaa5668eeb585bd3829afd7cf36013a
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue May 19 20:30:12 2020 +0200

    general: add connector class method to get new/edit form class (#43128)
#13

Mis à jour par Frédéric Péters il y a presque 4 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF