Archives de la liste d​e​v​@p​a​h​e​k​o​.c​l​o​u​d​

Bug: pb de validation, mauvais champs validé sur le tarif lors de l'import d'une activité utilisateur

mail at matthieubessat.fr via dev

28/12/2023 12:23:28

Bonjour,

(je renvoie le message, car les mails signé avec PGP GPG ne sont pas
bien supporté par la mailing list)

J'ai un problème sur le endpoint `POST /api/services/subscriptions/import`,

En effet, paheko me dit que "Le tarif machin n'existe pas pour cette
activité", alors que le nom du tarif fourni est correct.

Donc je regarde dans le code, je remarque cette condition ici :

`src/include/lib/Paheko/Services/Services_User.php` ligne 147

```
if (strcasecmp($fee->label, $row->fee) === 0 && $fee->id === $id_service)
{
```

Je pense que l'intention de `$fee->id === $id_service` c'est
`$fee->id_service === $id_service`

Je propose ce patch pour corriger le bug.

```
Index: src/include/lib/Paheko/Services/Services_User.php
==================================================================
--- src/include/lib/Paheko/Services/Services_User.php
+++ src/include/lib/Paheko/Services/Services_User.php
@@ -142,11 +142,11 @@

                                 $id_fee = null;

                                 if (!empty($row->fee)) {
                                         foreach ($fees as $fee) {
-                                                if (strcasecmp($fee->label,
$row->fee) === 0 && $fee->id === $id_service) {
+                                                if (strcasecmp($fee->label,
$row->fee) === 0 && $fee->id_service === $id_service) {
                                                         $id_fee = $fee->id;
                                                         break;
                                                 }
                                         }
```

Merci de votre attention.

Librement,

Bug: pb de validation, mauvais champs validé sur le tarif lors de l'import d'une activité utilisateur

BohwaZ/Paheko

28/12/2023 12:42:01

Hello,

effectivement souci déjà signalé, le problème était différent, corrigé
ici je pense : https://fossil.kd2.org/paheko/info/3e35c7cb3653e418