Dialogue Plugin

[CodeSpartan]

@iSpam

I have two bools, one visible for me to check whether I want this to be said once, and another that is set to true in the RunEventsFor, that way the next time the IsConditionsMet is run, the function will return false.

bool UDialogueUserWidget::IsConditionsMetForNode_Implementation(FDialogueNode Node)
{
UE_LOG(LogTemp, Warning, TEXT("No repeat for node at %d is %s"), &Node, Node.noRepeat ? TEXT("True") : TEXT("False"));
if (Node.noRepeat)
{
UE_LOG(LogTemp, Warning, TEXT("No repeat is true, returning false in UDialogueUserWidget::IsConditionsMet"));
return false;
}

for (UDialogueConditions* Condition : Node.Conditions)
{
if (IsValid(Condition))
{
if (!Condition->IsConditionMet(GetOwningPlayer(), NPCActor))
{
return false;
}
}
}
return true;
}

void UDialogueUserWidget::RunEventsForNode_Implementation(FDialogueNode Node)
{
if (Node.sayOnce)
{

UE_LOG(LogTemp, Warning, TEXT("sayOnce true in UDialogueUserWidget::RunEventsForNode"));
Node.noRepeat = true;
}

for (UDialogueEvents* Event : Node.Events)
{
if (IsValid(Event))
{
Event->RecieveEventTriggered(GetOwningPlayer(), NPCActor);
}
}
}

The line if statement in RunEventsFor is working as intended, but because the nodes are being passed in by value, the change isn’t being propagated to the actual data asset.
I have tried, for these two functions only, passing the node in by ref, but I’m still getting different nodes. I can’t make the node a pointer parameter since FDialogueNode is a struct.
void UDialogueUserWidget::RunEventsForNode_Implementation(UPARAM(ref) FDialogueNode& Node) is how I attempted to pass by reference, which I found from Rama

I’m just wondering if you could think of a workaround that won’t break the rest of your code.

I see.

Data assets are your game files. The only reason you can modify them at runtime in the editor is because the editor makes no distinction between whether you’re doing it through a Bluetility or in PIE. But you won’t be able to modify data assets on runtime in a cooked build. This is because they’re your game files. It’s like irrevocably modifying a mesh of a dinosaur into a bird because you’ve reached a certain checkpoint in game. But what happens if you load a prior savegame or you’ve started a new game? Your file will effectively be corrupt, because you’ll see a bird. That’s why it’s not possible.

Like I said, if you want to mark your nodes as read, you need to make an array of ints and add their indexes. This way you know you’ve visited the nodes.