Fix devices in terminal mispredicting power state

[Tayrtahn]

About the PR

Devices like vending machines and arcade games in the terminal no longer incorrectly state that they do not have power before opening.

Why / Balance

Fixes #38628.

Technical details

Spot the difference:

This PR changes the client version to also return true from that conditional.

Yes, I also think it's silly/bad to have two seemingly-identical static classes that need to be kept in sync - there really should just be a single class in Shared. Unfortunately, the client and server have different ApcPowerReceiverComponent definitions and TryGetComponent doesn't work with inheritance (so it can't be used with SharedApcPowerReceiverComponent). Try building and running with the reverted commit in this PR if you want to see the problem yourself. This should be dealt with in the future, but for now, this PR just puts the client and server classes back in sync.

Requirements

• I have read and am following the Pull Request and Changelog Guidelines.
• I have added media to this PR or it does not require an ingame showcase.

[perryprog]

Technically you can do this very normal and very okay way :^)

public static bool IsPowered(this EntitySystem system, EntityUid uid, IEntityManager entManager, SharedApcPowerReceiverComponent? receiver = null)
{
    var comp = entManager.ComponentFactory.GetRegistration("ApcPowerReceiver");
    IComponent? teehee = null;
    if (receiver == null && !entManager.TryGetComponent(uid, comp, out teehee))
        return true;
    receiver = teehee as SharedApcPowerReceiverComponent;
    return receiver?.Powered ?? true;
}

[Tayrtahn]

Technically you can do this very normal and very okay way :^)

public static bool IsPowered(this EntitySystem system, EntityUid uid, IEntityManager entManager, SharedApcPowerReceiverComponent? receiver = null)
{
    var comp = entManager.ComponentFactory.GetRegistration("ApcPowerReceiver");
    IComponent? teehee = null;
    if (receiver == null && !entManager.TryGetComponent(uid, comp, out teehee))
        return true;
    receiver = teehee as SharedApcPowerReceiverComponent;
    return receiver?.Powered ?? true;
}

That's very clever and I kinda hate it!

[beck-thompson]

Yeah agree with what you said they should be combined but this is clearly a mistake!