[WorldPartition] Extend WorldPartitionNavigationDataBuilder

Hi,

We would like to keep our generated navdata in a specific place (outliner folder, datalayer) and thus have thought of overriding the builder WorldPartitionNavigationDataBuilder.

Unfortunately, UWorldPartitionNavigationDataBuilder doesn’t export its functions (even the virtual ones).

One way we found was to make this patch:

`class UNREALED_API UWorldPartitionNavigationDataBuilder : public UWorldPartitionBuilder`

We have noticed that some Builders export some virtual functions but not all.

Is there a reason we should be warned about?

Thank you,

Tom

[Attachment Removed]

Hello!

The developer that built that commandlet probably didn’t think that licensees would want to extend it. Adding the API macro at that level will result in exporting all methods and should unblock you.

Depending on the changes, this might be a case where making a copy of the code and renaming the class might be more appropriate.

Regards,

Martin

[Attachment Removed]

Hi,

Thank you for the answer.

Our changes are not complicated and we would like to keep the base behavior (and benefit from the updates you would make :wink:).

However, do you plan to add yourself the missing “export keyword” like you did for “UWorldPartitionMiniMapBuilder” for instance? Because obviously, the fewer patches we have to maintain, the easier the integration of a new UE version is :smiling_face_with_sunglasses:

Regards,

Tom

[Attachment Removed]

Hello!

Which method would you need? We don’t expose full classes anymore as we were running into linker issues when the modules are compiled as DLLs.,

Martin

[Attachment Removed]

Hi,

At least, please, we would like to have access to:

  • PreRun
  • RunInternal
  • PostRun

Thanks,

Tom

[Attachment Removed]

Ok, those are the ones I was considering. I wasn’t sure if you needed SavePackages and DeletePackages.

[Attachment Removed]

I call SavePackages and DeletePackages, but I don’t override them.

[Attachment Removed]

Hello,

I just submitted CL50046464 that exports the methods you need.

Regards,

Martin

[Attachment Removed]

Hi,

Thank you :flexed_biceps:

Regards,

Tom

[Attachment Removed]