Skip to content

Bugfixes and refactor for LogWarning/Error/Note#359

Open
LindyHopperGT wants to merge 3 commits intoMothCocoon:5.xfrom
LindyHopperGT:FlowRefactorsAndBugfixes
Open

Bugfixes and refactor for LogWarning/Error/Note#359
LindyHopperGT wants to merge 3 commits intoMothCocoon:5.xfrom
LindyHopperGT:FlowRefactorsAndBugfixes

Conversation

@LindyHopperGT
Copy link
Contributor

Bug Fix - CancelAndWarnForUnflushedDeferredTriggers()

  • Null-guard ToNode and FromNode before dereferencing in UE_LOG. Prevents crash during abnormal termination when nodes are already destroyed.

Bug Fix - TryFindActorOwner()

  • Now correctly returns the Owner when it IS already an AActor, not just when it's a component. Fulfills the documented contract.

Refactor - LogError/LogWarning/LogNote

  • Extracted shared LogRuntimeMessage() helper. Three identical copy-pasted functions → one implementation with severity parameter.

## Core Type System
- FFlowPinConnectionPolicy: Flexible pin connectivity policy framework
- FFlowPinTypeMatchPolicy: Per-type matching rules with EFlowPinTypeMatchRules flags
- Standard policy presets: VeryRelaxed, Relaxed, Strict, VeryStrict

### Graph Schema & Pin Connectivity
- UFlowGraphSchema::ArePinTypesCompatible(): Policy-driven compatibility checking

### Flow Asset & Runtime
- FFlowPinConnectionPolicy integration in UFlowAsset (was in UFlowGraphSchema, editor only, and not easily configurable for projects)
- UFlowAsset::GetFlowPinConnectionPolicy(): Runtime policy access for instances

### Predicates & Data Validation
- Now supports all FFlowDataPinValue types
- Generic equality template: TryCheckResolvedValuesEqual<TFlowPinType>()
- Validation with operator/type compatibility checks

### Settings & Configuration
- FInstancedStruct-based policy selection in UFlowSettings
- Per-asset-subclass policy override capability
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant