Give stores the ability to check for owner only (#26573)

adds a check if the store belongs to the user
This commit is contained in:
keronshb
2024-03-30 19:25:36 -04:00
committed by GitHub
parent c9e19445b4
commit b8363cd82a
3 changed files with 26 additions and 0 deletions

View File

@@ -78,6 +78,13 @@ public sealed partial class StoreComponent : Component
[ViewVariables, DataField]
public bool RefundAllowed;
/// <summary>
/// Checks if store can be opened by the account owner only.
/// Not meant to be used with uplinks.
/// </summary>
[ViewVariables(VVAccess.ReadWrite), DataField]
public bool OwnerOnly;
/// <summary>
/// The map the store was originally from, used to block refunds if the map is changed
/// </summary>

View File

@@ -10,6 +10,7 @@ using JetBrains.Annotations;
using Robust.Server.GameObjects;
using Robust.Shared.Prototypes;
using System.Linq;
using Robust.Shared.Utility;
namespace Content.Server.Store.Systems;
@@ -26,6 +27,7 @@ public sealed partial class StoreSystem : EntitySystem
{
base.Initialize();
SubscribeLocalEvent<StoreComponent, ActivatableUIOpenAttemptEvent>(OnStoreOpenAttempt);
SubscribeLocalEvent<CurrencyComponent, AfterInteractEvent>(OnAfterInteract);
SubscribeLocalEvent<StoreComponent, BeforeActivatableUIOpenEvent>(BeforeActivatableUiOpen);
@@ -65,6 +67,21 @@ public sealed partial class StoreSystem : EntitySystem
RaiseLocalEvent(uid, ref ev, true);
}
private void OnStoreOpenAttempt(EntityUid uid, StoreComponent component, ActivatableUIOpenAttemptEvent args)
{
if (!component.OwnerOnly)
return;
component.AccountOwner ??= args.User;
DebugTools.Assert(component.AccountOwner != null);
if (component.AccountOwner == args.User)
return;
_popup.PopupEntity(Loc.GetString("store-not-account-owner", ("store", uid)), uid, args.User);
args.Cancel();
}
private void OnAfterInteract(EntityUid uid, CurrencyComponent component, AfterInteractEvent args)
{
if (args.Handled || !args.CanReach)

View File

@@ -6,3 +6,5 @@ store-ui-traitor-flavor = Copyright (C) NT -30643
store-ui-traitor-warning = Operatives must lock their uplinks after use to avoid detection.
store-withdraw-button-ui = Withdraw {$currency}
store-not-account-owner = This {$store} is not bound to you!