Anomalie #1362
fermé
Modification d'une fiche adhérent impossible après un test spécifique.
100%
Description
Navigateur Firefox. Php 7.2.24
Test réalisé comme suit:
- Saisir une nouvelle fiche sans son adresse électronique,
- Valider la fiche. Galette demande de remplir le champ obligatoire "Courriel",
- Ajouter un parent ne débloque pas l'obligation de saisir une adresse électronique,
- Saisir une adresse électronique et valider.
Modifier la fiche déclenche une erreur. Les logs de Galette donnent :
[...] - 2019-11-25 09:45:24 - 3 - Galette error: Type: ArgumentCountError Message: Too few arguments to function Galette\Entity\Adherent::getSName(), 0 passed in /home/clients/e04180b8f6cbad0ee1dbac5df7b7630a/adherents/includes/routes/members.routes.php on line 850 and at least 2 expected File: /home/clients/e04180b8f6cbad0ee1dbac5df7b7630a/adherents/lib/Galette/Entity/Adherent.php Line: 764 .../...
Mis à jour par Johan Cwiklinski il y a plus de 4 ans
- Catégorie mis à Core
- Statut changé de Nouveau à In Progress
- Assigné à mis à Johan Cwiklinski
Tu pourrai tester ça ?
diff --git a/galette/includes/routes/members.routes.php b/galette/includes/routes/members.routes.php
index ea2bf824..106421f8 100644
--- a/galette/includes/routes/members.routes.php
+++ b/galette/includes/routes/members.routes.php
@@ -126,7 +126,7 @@ $app->get(
if ($member->hasParent()) {
if (!isset($members[$member->parent->id])) {
$members =
- [$member->parent->id => $member->parent->getSName()] +
+ [$member->parent->id => Adherent::getSName($this->zdb, $member->parent->id)] +
$members
;
}
@@ -848,7 +848,7 @@ $app->get(
if ($member->hasParent()) {
if (!isset($members[$member->parent->id])) {
$members =
- [$member->parent->id => $member->parent->getSName()] +
+ [$member->parent->id => Adherent::getSName($this->zdb, $member->parent->id)] +
$members
;
}
/pre>
Mis à jour par Roland Telle il y a plus de 4 ans
Johan Cwiklinski a écrit :
Tu pourrai tester ça ? [...]
L'erreur principale (impossible de modifier la fiche) est corrigée.
En revanche, la sélection d'un adhérent-parent ne désactive pas l'obligation de saisir une adresse électronique. Il faut donc saisir une adresse même bidon, puis la supprimer en modifiant la fiche.
Merci ;-)
Mis à jour par Johan Cwiklinski il y a plus de 4 ans
Ok, ça ressemble a un pbm de JavaScript. As tu ce pépin sur différents navigateurs ?
Pourrais tu regarder si une erreur apparaît dans les outils de dev du navigateur ?
Mis à jour par Roland Telle il y a plus de 4 ans
Johan Cwiklinski a écrit :
Ok, ça ressemble a un pbm de JavaScript. As tu ce pépin sur différents navigateurs ?
Firefox, Chrome et Edge. Pas testé sur d'autres navigateurs.
Pourrais tu regarder si une erreur apparaît dans les outils de dev du navigateur ?
Je vais essayer de voir ça.
Mis à jour par Roland Telle il y a plus de 4 ans
Johan Cwiklinski a écrit :
Ok, ça ressemble a un pbm de JavaScript. As tu ce pépin sur différents navigateurs ?
Pourrais tu regarder si une erreur apparaît dans les outils de dev du navigateur ?
Avec Firefox, j'obtiens ceci lorsque j'ajoute un adhérent :
Cette page utilise la propriété non standard « zoom ». Envisagez d’utiliser calc() dans les valeurs des propriétés pertinentes ou utilisez « transform » avec « transform-origin: 0 0 ». add Erreur dans les liens source : Error: request failed with status 404 URL de la ressource : https://adherents.aselp.fr/webroot/js/jquery/jquery-1.10.2.min.js URL du lien source : jquery-1.10.2.min.map Erreur dans les liens source : Error: NetworkError when attempting to fetch resource. URL de la ressource : moz-extension://14099a4a-ab80-4c6e-bb3c-43ac514393f3/lib/tp/browser-polyfill.min.js URL du lien source : browser-polyfill.min.js.map
Les deux dernières erreurs pointent vers une page explicative. Tu les veux ?
Mis à jour par Johan Cwiklinski il y a plus de 4 ans
Ok il ne semble pas y avoir d'erreur particulière, je ne vois pas trop d'où vie t ce souci. Je testerai voir ;)
Mis à jour par Johan Cwiklinski il y a plus de 4 ans
- Statut changé de In Progress à Résolu
- % réalisé changé de 0 à 100
Appliqué par commit 22e5ff26e2fa55fa3b22e2d65506deaf02463474.