Project

General

Profile

Development #43128

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

Added by Frédéric Péters 10 days ago. Updated 1 day ago.

Status:
Solution validée
Priority:
Normal
Target version:
-
Start date:
20 May 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

0001-general-add-connector-class-method-to-get-new-edit-f.patch View (5.43 KB) Frédéric Péters, 20 May 2020 09:32 PM

0001-adaptation-nicolas-pour-pouvoir-faire.patch View (3.13 KB) Nicolas Roche, 21 May 2020 11:21 AM


Related issues

Related to Passerelle - Bug #20995: connecteur pastell cassé Solution déployée 04 Jan 2018

History

#1 Updated by Frédéric Péters 10 days ago

#2 Updated by Nicolas Roche 9 days ago

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 Updated by Frédéric Péters 9 days ago

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 Updated by Frédéric Péters 9 days ago

  • Related to Bug #20995: connecteur pastell cassé added

#5 Updated by Nicolas Roche 8 days ago

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 Updated by Frédéric Péters 1 day ago

Passons.

Commentaire sur la branche attachée ?

#7 Updated by Nicolas Roche 1 day ago

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

#8 Updated by Frédéric Péters 1 day ago

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 Updated by Nicolas Roche 1 day ago

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 Updated by Nicolas Roche 1 day ago

  • Status changed from Solution proposée to 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 Updated by Frédéric Péters 1 day ago

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

Also available in: Atom PDF