Corrige le chantier terrestre #20

Nigel · 2021-02-17T19:41:32Z

Nigel commented

2021-02-17 19:41:32 +00:00

Dans https://gitea.nemunai.re/halo-battle/game/pulls/8, j'ai rajouté une petite différence : les humains et les covies n'ont plus la même liste de bâtiment de défense. Cela m'a forcé a mettre à jour une des constantes du code ici : 9a14ddcd8d. Je pensais avoir fait le tour, mais que neni, cette variable était appellée sous une autre forme : ${$calc.'VAR'} où $calc vaut terrestre (c'est pour améliorer la généralité du code que ça a été fait).

Du coup, le code tentait de mettre à jour la BDD en prenant une donnée qui n'existe pas, ce qui donnait du SQL qui ressemble a SET = 2 et ben.. ça rate. Mais une partie du code était exécutée avant, comme la mise à jour des points (d'où le bug de moftcu).

Cette PR corrige ce soucis en prenant la bonne valeure là où il faut.

Dans https://gitea.nemunai.re/halo-battle/game/pulls/8, j'ai rajouté une petite différence : les humains et les covies n'ont plus la même liste de bâtiment de défense. Cela m'a forcé a mettre à jour une des constantes du code ici : https://gitea.nemunai.re/halo-battle/game/commit/9a14ddcd8d80daae1749a4e65bab777dcc706479. Je pensais avoir fait le tour, mais que neni, cette variable était appellée sous une autre forme : `${$calc.'VAR'}` où `$calc` vaut `terrestre` (c'est pour améliorer la généralité du code que ça a été fait). Du coup, le code tentait de mettre à jour la BDD en prenant une donnée qui n'existe pas, ce qui donnait du SQL qui ressemble a `SET = 2` et ben.. ça rate. Mais une partie du code était exécutée avant, comme la mise à jour des points (d'où le bug de moftcu). Cette PR corrige ce soucis en prenant la bonne valeure là où il faut.

Nigel added 1 commit 2021-02-17 19:41:34 +00:00

Corrige le chantier terrestre

continuous-integration/drone/pr Build is passing

Details

continuous-integration/drone/push Build is passing

Details

6d6aac90be

nemunaire merged commit 28b0ceeee8 into master

2021-02-18 21:53:52 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: halo-battle/game#20