Merge pull request #15358 from peppy/online-id-potentially-zero

Add a note about `OnlineID` potentially being zero in non-autoincrement cases
This commit is contained in:
Dan Balasescu
2021-11-01 15:54:08 +09:00
committed by GitHub

View File

@ -8,7 +8,7 @@ namespace osu.Game.Database
public interface IHasOnlineID<out T>
{
/// <summary>
/// The server-side ID representing this instance, if one exists. Any value 0 or less denotes a missing ID.
/// The server-side ID representing this instance, if one exists. Any value 0 or less denotes a missing ID (except in special cases where autoincrement is not used, like rulesets).
/// </summary>
/// <remarks>
/// Generally we use -1 when specifying "missing" in code, but values of 0 are also considered missing as the online source