fix: usr: the bomb using was not recorded correctly - the old data will be corrupted #6
This commit is contained in:
@@ -610,8 +610,13 @@ readonly class TopicManager implements TopicManagerInterface
|
|||||||
|
|
||||||
$playedGame->setRedPoints($redPoints);
|
$playedGame->setRedPoints($redPoints);
|
||||||
$playedGame->setBluePoints($bluePoints);
|
$playedGame->setBluePoints($bluePoints);
|
||||||
$playedGame->setRedExplodedBomb((bool)$event['bomb'] && 'red' === $player ? true : null);
|
if ((bool)$event['bomb']) {
|
||||||
$playedGame->setBlueExplodedBomb((bool)$event['bomb'] && 'blue' === $player ? true : null);
|
if ('red' === $player) {
|
||||||
|
$playedGame->setRedExplodedBomb(true);
|
||||||
|
} elseif ('blue' === $player) {
|
||||||
|
$playedGame->setBlueExplodedBomb(true);
|
||||||
|
}
|
||||||
|
}
|
||||||
$playedGame->setUpdated(new DateTime());
|
$playedGame->setUpdated(new DateTime());
|
||||||
|
|
||||||
/** Bonus data is already persisted in calculateBonuses, but we ensure it's up to date */
|
/** Bonus data is already persisted in calculateBonuses, but we ensure it's up to date */
|
||||||
|
|||||||
@@ -127,7 +127,6 @@
|
|||||||
{% endif %}
|
{% endif %}
|
||||||
{% set pointDiff = (redPts|default(0) - bluePts|default(0))|abs %}
|
{% set pointDiff = (redPts|default(0) - bluePts|default(0))|abs %}
|
||||||
{% set winnerName = redPts|default(0) > bluePts|default(0) ? redName : (bluePts|default(0) > redPts|default(0) ? blueName : null) %}
|
{% set winnerName = redPts|default(0) > bluePts|default(0) ? redName : (bluePts|default(0) > redPts|default(0) ? blueName : null) %}
|
||||||
|
|
||||||
<div class="bshare-details">
|
<div class="bshare-details">
|
||||||
{% if resign %}
|
{% if resign %}
|
||||||
<div class="bshare-detail">
|
<div class="bshare-detail">
|
||||||
@@ -166,8 +165,6 @@
|
|||||||
</div>
|
</div>
|
||||||
{% endif %}
|
{% endif %}
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
{# Bonus Stats Section #}
|
|
||||||
{% set hasRedStats = redBonusStats is not empty and (redBonusStats.blindHits or redBonusStats.chainBest or redBonusStats.edgeMines or redBonusStats.lastMineHits or redBonusStats.biggestReveal) %}
|
{% set hasRedStats = redBonusStats is not empty and (redBonusStats.blindHits or redBonusStats.chainBest or redBonusStats.edgeMines or redBonusStats.lastMineHits or redBonusStats.biggestReveal) %}
|
||||||
{% set hasBlueStats = blueBonusStats is not empty and (blueBonusStats.blindHits or blueBonusStats.chainBest or blueBonusStats.edgeMines or blueBonusStats.lastMineHits or blueBonusStats.biggestReveal) %}
|
{% set hasBlueStats = blueBonusStats is not empty and (blueBonusStats.blindHits or blueBonusStats.chainBest or blueBonusStats.edgeMines or blueBonusStats.lastMineHits or blueBonusStats.biggestReveal) %}
|
||||||
{% if redBonusPoints > 0 or blueBonusPoints > 0 or hasRedStats or hasBlueStats %}
|
{% if redBonusPoints > 0 or blueBonusPoints > 0 or hasRedStats or hasBlueStats %}
|
||||||
@@ -220,8 +217,6 @@
|
|||||||
{% endif %}
|
{% endif %}
|
||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
{# Blue Bonus #}
|
|
||||||
<div class="bshare-bonus__player bshare-bonus__player--blue">
|
<div class="bshare-bonus__player bshare-bonus__player--blue">
|
||||||
<div class="bshare-bonus__header">
|
<div class="bshare-bonus__header">
|
||||||
<span class="bshare-bonus__points">{{ blueBonusPoints|number_format(1, '.', '') }}</span>
|
<span class="bshare-bonus__points">{{ blueBonusPoints|number_format(1, '.', '') }}</span>
|
||||||
|
|||||||
Reference in New Issue
Block a user