Corrige le chantier terrestre #20

Merged

nemunaire merged 1 commit from fix/chantier_terrestre into master

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

Nigel commented

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

Collaborator

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