I tested addMagazineCargo, addItemCargo and addMagazineAmmoCargo, all are fixed. Tyvm!

next dev/prof

The submunition being spawned multiple times is actually normal.
Every shot has a local-only copy on every player. But they should be flagged as being local and not cause effects like that.

In T154359#2602895, @dedmen wrote:

We're switching from "the target that was last shot at" to "the target we are going to shoot at". I think that fits.

Works for me

getAttackTarget isn't well defined.
So it returning the "target the unit is told to attack" seems correct.
We're switching from "the target that was last shot at" to "the target we are going to shoot at". I think that fits.

next prof/dev

I love it. It's easy to play. I will win all these cookie clicker 2 missions.

next prof/dev

I think you need to read a bit more carefully.

@Bis! 9 years have passed, Instead of fixing it, you only change the status of this problem, 9 years later! Looool!
Mod makers fixed this themselves a long time ago!

The Grenade Window Breaker mod works for both hand and UGL grenades...

@Bis! 9 years have passed, Instead of fixing it, you only change the status of this problem, 9 years later! Looool!

false report - after extensive debugging turns out was likely viewdistance related

In T154359#2602361, @dedmen wrote:

The weapon the unit is using, knows what its told to shoot at.
We could say "If you are told to shoot something, that should be considered target"
That would be doable, if unit has no assigned target, check if its weapon is told to shoot at something and use that instead..

Repro mission is a bit meh, only updating every 0.5s
replacing with a onEachFrame makes it much easier to see

Ugh this is a mess.

No

Fixed next profiling/dev branch

Any chance to see removeMagazineTurret get the same treatment as addMagazineTurret did (i.e. getting a third optional parameter specifying the ammo count)?

Can you close https://feedback.bistudio.com/T122981 and https://feedback.bistudio.com/T80537, as they are related to the fix?
It's not really related to this, but https://feedback.bistudio.com/T79683 can be closed too.

Revision: 151791

Revision: 151791

Revision: 151791

Revision: 151791

Revision: 151791

Revision: 151791

Revision: 151791

Revision: 151791

Revision: 151791

Revision: 151791

Revision: 151791

Revision: 151791

Revision: 151791

Revision: 151791

EHs extended instead
Rev 151794

The game already has destroyable doors, they are glass doors. So I thought maybe allowing wooden doors to break wouldn't be such a big task.

I'm looking forward to the next profiling branch!

Next profiling and dev branch

This bug happened because the unit needs to be notified that their inventory contents have changed.
With the commands that get the unit as argument its easy, the unit is right there.
But the commands that get a container, first need to find which unit it belongs to, and there might not be any unit at all.

In T180137#2597902, @dedmen wrote:

You can already see in your chat display bottom left, who is transmitting right?

@dedmen thanks a lot for looking into it and sorry for the belated response!

Huh. Weird that we didn't find that before.
Thank you, the crashdump was enough to see what was going on. Fixed next profiling/dev branch.

next prof

I hope it will be fixed with next profiling branch update, v5.

difficulty setting seems very unlikely to happen. a scripting command may be more possible

This is essentially the bug we had before.
Where textures referenced in the display, would not be loaded at first, so you need to keep re-rendering the display.