View Issue Details

IDProjectCategoryView StatusLast Update
0002333Infinitode 2[All Projects] Infinitode 2public2024-04-25 17:59
ReporterLdream Assigned Totherainycat  
PriorityhighSeveritymajorReproducibilityalways
Status closedResolutionwon't fix 
PlatformDesktopOSWindowsOS VersionAny
Product Version1.9.0 (Season 3) 
Target VersionFixed in Version 
Summary0002333: Release multiple thunders ability will cause them aim to same target
Description

In version 1.8, if you release the thunder ability multiple times, the thunders will be released one after another until the number is exhausted.
In version 1.9, if you release the thunder ability multiple times, the thunders will be released together, and the thunders will hit a target at the same time.
e.g.: you release 2 thunder during a game, 2 thunders will fall on one enemy, even though one thunder can kill an enemy.
Note: i don't know whether it is a feature or a bug, so i report it.

Steps To Reproduce

Release multiple times thunder will trigger this situtation.

TagsNo tags attached.

Activities

Ldream

Ldream

2024-04-25 15:59

reporter   ~0002226

This bug still exists.



15.png (209,741 bytes)
15.png (209,741 bytes)
16.png (198,235 bytes)
16.png (198,235 bytes)
therainycat

therainycat

2024-04-25 17:59

administrator   ~0002240

It is complicated
1.9.0 introduces a damage queue to prevent deep recursion and fix some bugs
Previously all damage has been issued immediately, so for example in 1.8.10 when Thunder hit an enemy and killed it, the next instance of Thunder ability no longer sees the enemy cause he's dead.
In 1.9.0 the damage gets enqueued and will only be applied to the enemy later in the same frame, meaning those two Thunders damaged the enemy "exactly at the same time" and then the damage manager has split their damage equally to kill the enemy.

So, architecture-wise, it works properly and while it was convenient to chain Thunders this way, I'll have to leave it as it is, because it does not conflict with the ability's description and returning the old behavior will complicate the code / make it more dirty and (probably) buggy.

Issue History

Date Modified Username Field Change
2024-04-24 20:47 Ldream New Issue
2024-04-25 15:59 Ldream File Added: 15.png
2024-04-25 15:59 Ldream File Added: 16.png
2024-04-25 15:59 Ldream Note Added: 0002226
2024-04-25 17:59 therainycat Note Added: 0002240
2024-04-25 17:59 therainycat Assigned To => therainycat
2024-04-25 17:59 therainycat Status new => closed
2024-04-25 17:59 therainycat Resolution open => won't fix