The incorrect comparison caused us not to keep the objects referenced, which allowed them to be GC'd, which breaks the CookPackageSplitter contract and also triggered an error in FGeneratorPackage::ResetSaveState when we detected that deletion and demoted the package out of save.
Cooker: GeneratedPackages: do not demote generator or generated packages when private objects are garbage collected. Demote (and give an error) only when public objects are garbage collected. Normal packages that are not a generator and are not owned by a generator keep the same behavior of demoting if any object, even a private one, is garbage collected.
Cooker: When a generated package is demoted due to deleted objects, add feedback with the name of the object that was deleted.
#jira UE-197955
#rbx
[CL 28918477 by Matt Peters in 5.3 branch]
Sequencer: Fix property owner/container
PropertyContainer is what we look to for the setter function
PropertyOwner is the owner struct, which is where we look for permissions
The original fix was to find the property owner struct to determine DoesPropertyPassFilter. This broke finding setter functions for some properties.
Modified version of 27916475 to make this hotfix compatible.
#jira UE-198396
#rb andrew.rodham, ludovic.chabant
#lockdown aurel.cordonnier
[CL 28903124 by Max Chen in 5.3 branch]
PropertyContainer is what we look to for the setter function
PropertyOwner is the owner struct, which is where we look for permissions
The original fix was to find the property owner struct to determine DoesPropertyPassFilter. This broke finding setter functions for some properties.
Modified version of 27916475 to make this hotfix compatible.
#jira UE-198396
#rb andrew.rodham, ludovic.chabant
#lockdown aurel.cordonnier
[CL 28883250 by Max Chen in 5.3 branch]
#jira UE-196471
[RN] Bugfix, Foundation
#rb Zousar.Shaker
#[robomerge][bot5] -FNMain
#ushell-cherrypick of 28331505 by matt.peters
[CL 28878783 by Zousar Shaker in 5.3 branch]
- for example, virtual production projects want to have local exposure disabled by default.
#rb
#jira UE-195152
#ushell-cherrypick of 27786211 by tiago.costa
[CL 27964101 by tiago costa in 5.3 branch]
Transform channels are being negated twice (ty, ry/rz).
Script doesn't seem to pick-up the correct values for other controls (they match if looking at the curves but the script is warning about mismatch's and empty curves)
Fix from AnatoleM but out of office hours.
#rb max.chen
#jira UE-193225
#lockdown julien.marchand
[CL 27160909 by Mike Zyracki in 5.3 branch]
This fixes a crash moving keys after deleting marked frames because the selected marked frames is stale.
Note, when keys are deleted, we similarly clear the key selection.
#jira UE-193116
#rb andrew.rodham
#lockdown julien.marchand
[CL 27140577 by Max Chen in 5.3 branch]
#fix ensure per-curve drawing parameters are always reset - issue occured when deleting the last key on a curve, leaving the cached drawing params in place until getting user-focus or making another change to the curve
#jira UE-188647
#rb Thomas.Sarkanen, Mike.Zyracki
[CL 27139322 by Jurre deBaare in 5.3 branch]
Switched custom parsing to use FEditorClassUtils::GetImplementedInterfaceClassPathsFromAsset
#jira UE-192562
[CL 27070113 by Thomas Sarkanen in 5.3 branch]
Don't load the UViewportInteractionAssetContainer in the APivotTransformGizmo constructor, and don't early out of the constructor if the CDO flag is set. Instead, if no other material has been specified, let UGizmoHandleGroup::UpdateHandleColor load the default materials from the asset container just before it creates the MIDs.
#jira UE-191486
#rb ben.zeigler
#rnx
[CL 26970691 by zach brockway in 5.3 branch]
Accidentally merged change to 5.3... Backing out and resubmitting without the robomerge flag
Original CL Desc
-----------------------------------------------------------------
[BugFix] Made it so users can no longer reparent blueprints to classes managed by a different blueprint type (IE Actor to EditorUtilityActor)
#jira UE-159086
#rb dan.oconnor
[CL 26922220 by jordan hoffmann in 5.3 branch]
