Skip to content

Commit

Permalink
The API FUnityVersionControlProvider::UsesFileRevisions() is still br…
Browse files Browse the repository at this point in the history
…oken as of UE5.3 Preview 1

so let's stick to the default value for now
  • Loading branch information
SRombautsU committed Sep 8, 2023
1 parent 685f7bb commit acf58e1
Showing 1 changed file with 3 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -517,19 +517,10 @@ bool FPlasticSourceControlProvider::UsesCheckout() const

bool FPlasticSourceControlProvider::UsesFileRevisions() const
{
// This API introduced in UE5.1 was broken (preventing the user to use the source control context menu for checkin,
// as well as selecting what files to submit in the global Submit Content window)
// but is going to be fixed for UE5.3 through the use of the new following UsesSnapshots() API
#if ENGINE_MAJOR_VERSION == 5 && ENGINE_MINOR_VERSION >= 3

// Only a Partial/Gluon workspace can sync/update files individually, operating on revisions (can use the context menu)
// while a regular workspace can only update all the files as a whole, operating at the changeset level (requires the global menu)
return IsPartialWorkspace();

#else
// This API introduced in UE5.1 is still broken as of UE5.3
// (preventing the user to use the source control context menu for checkin if returning false)
// return IsPartialWorkspace();
return true;

#endif
}

bool FPlasticSourceControlProvider::UsesSnapshots() const
Expand Down

0 comments on commit acf58e1

Please sign in to comment.