MothCocoon / FlowGraph

Design-agnostic node system for scripting game’s flow in Unreal Engine

Home Page:https://discord.gg/Xmtr6GhbmW

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Wrong number of newly added output pin on Sequence node

F0bRE opened this issue · comments

commented

Found this bug on the 1.3 version for UE 5.0.

If you add new output pins to the Sequence node, delete one of the pins (except last one) and then try to add new pin, the newly added pin will have wrong number. It will duplicate the number of the last pin.

For example, we add pins 2 and 3 to the Sequence node:
image

Remove pin 2:
image

Add new pin:
image

When this Sequence will be executed, only one of identically named output pins will be triggered:
image

As far as I understood, problem is in function void UFlowGraphNode::AddInstancePin, in the row const FFlowPin PinName = FFlowPin(FString::FromInt(NumberedPinsAmount));.
It names new pin according to the number of output pins, with disregard of last pin's number.

So in my example, we have output pins 0, 1 and 3. There are three output pins, so the newly added will have PinName = 3, and we will have output pins 0, 1, 3 and 3.

Fixed. Now pin names below removed pin should be correctly updated :)