Improve paper stamping experience (#17135)
This commit is contained in:
@@ -1,3 +1,4 @@
|
||||
using System.Linq;
|
||||
using Content.Server.Administration.Logs;
|
||||
using Content.Server.Popups;
|
||||
using Content.Server.UserInterface;
|
||||
@@ -90,7 +91,7 @@ namespace Content.Server.Paper
|
||||
|
||||
if (paperComp.StampedBy.Count > 0)
|
||||
{
|
||||
var commaSeparated = string.Join(", ", paperComp.StampedBy);
|
||||
var commaSeparated = string.Join(", ", paperComp.StampedBy.Select(s => Loc.GetString(s.StampedName)));
|
||||
args.PushMarkup(
|
||||
Loc.GetString(
|
||||
"paper-component-examine-detail-stamped-by", ("paper", uid), ("stamps", commaSeparated))
|
||||
@@ -114,13 +115,15 @@ namespace Content.Server.Paper
|
||||
}
|
||||
|
||||
// If a stamp, attempt to stamp paper
|
||||
if (TryComp<StampComponent>(args.Used, out var stampComp) && TryStamp(uid, stampComp.StampedName, stampComp.StampState, paperComp))
|
||||
if (TryComp<StampComponent>(args.Used, out var stampComp) && TryStamp(uid, GetStampInfo(stampComp), stampComp.StampState, paperComp))
|
||||
{
|
||||
// successfully stamped, play popup
|
||||
var stampPaperOtherMessage = Loc.GetString("paper-component-action-stamp-paper-other", ("user", Identity.Entity(args.User, EntityManager)), ("target", Identity.Entity(args.Target, EntityManager)), ("stamp", args.Used));
|
||||
_popupSystem.PopupEntity(stampPaperOtherMessage, args.User, Filter.PvsExcept(args.User, entityManager: EntityManager), true);
|
||||
var stampPaperOtherMessage = Loc.GetString("paper-component-action-stamp-paper-other",
|
||||
("user", args.User), ("target", args.Target), ("stamp", args.Used));
|
||||
|
||||
var stampPaperSelfMessage = Loc.GetString("paper-component-action-stamp-paper-self", ("target", Identity.Entity(args.Target, EntityManager)), ("stamp", args.Used));
|
||||
_popupSystem.PopupEntity(stampPaperOtherMessage, args.User, Filter.PvsExcept(args.User, entityManager: EntityManager), true);
|
||||
var stampPaperSelfMessage = Loc.GetString("paper-component-action-stamp-paper-self",
|
||||
("target", args.Target), ("stamp", args.Used));
|
||||
_popupSystem.PopupEntity(stampPaperSelfMessage, args.User, args.User);
|
||||
|
||||
_audio.PlayPvs(stampComp.Sound, uid);
|
||||
@@ -129,6 +132,14 @@ namespace Content.Server.Paper
|
||||
}
|
||||
}
|
||||
|
||||
private StampDisplayInfo GetStampInfo(StampComponent stamp)
|
||||
{
|
||||
return new StampDisplayInfo {
|
||||
StampedName = stamp.StampedName,
|
||||
StampedColor = stamp.StampedColor
|
||||
};
|
||||
}
|
||||
|
||||
private void OnInputTextMessage(EntityUid uid, PaperComponent paperComp, PaperInputTextMessage args)
|
||||
{
|
||||
if (string.IsNullOrEmpty(args.Text))
|
||||
@@ -161,17 +172,19 @@ namespace Content.Server.Paper
|
||||
/// <summary>
|
||||
/// Accepts the name and state to be stamped onto the paper, returns true if successful.
|
||||
/// </summary>
|
||||
public bool TryStamp(EntityUid uid, string stampName, string stampState, PaperComponent? paperComp = null)
|
||||
public bool TryStamp(EntityUid uid, StampDisplayInfo stampInfo, string spriteStampState, PaperComponent? paperComp = null)
|
||||
{
|
||||
if (!Resolve(uid, ref paperComp))
|
||||
return false;
|
||||
|
||||
if (!paperComp.StampedBy.Contains(Loc.GetString(stampName)))
|
||||
if (!paperComp.StampedBy.Contains(stampInfo))
|
||||
{
|
||||
paperComp.StampedBy.Add(Loc.GetString(stampName));
|
||||
paperComp.StampedBy.Add(stampInfo);
|
||||
if (paperComp.StampState == null && TryComp<AppearanceComponent>(uid, out var appearance))
|
||||
{
|
||||
paperComp.StampState = stampState;
|
||||
paperComp.StampState = spriteStampState;
|
||||
// Would be nice to be able to display multiple sprites on the paper
|
||||
// but most of the existing images overlap
|
||||
_appearance.SetData(uid, PaperVisuals.Stamp, paperComp.StampState, appearance);
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user