Add implementations in ExternalPowerSystems for chargeItem and requestEnergyFromArmor

[coderbot16]

Looks like I forgot these in the previous PR. Convenience functions in line with the other implementations provided in ExternalPowerSystems.

[modmuss50]

Is it worth checking ExternalPowerManager.isPoweredItem before charging the item?

Would be quite easy to add and may save some time / prevent issues if not checked in the ExternalPowerManager

[coderbot16]

Seems easy enough, I can add that. Would avoid potential bugs in other ExternalPowerManager implementations.

[coderbot16]

Alright, I added checks to dischargeItem and chargeItem. I didn't add any checks to requestEnergyFromArmor since the manager is responsible for finding and discharging the items itself, and ExternalPowerSystems doesn't know what items the implementation will be drawing energy from.

[coderbot16]

Also, I'll keep the isPoweredItem checks in the ExternalPowerManager implementations since it is possible to access them without going through ExternalPowerSystems methods.

[modmuss50]

Logic looks good, how ever this can be done in a cleaner way using filter

Here is an example:

public static void chargeItem(TilePowerAcceptor tilePowerAcceptor, ItemStack stack){
    externalPowerHandlerList.stream()
        .filter(externalPowerManager -> externalPowerManager.isPoweredItem(stack))
        .forEach(externalPowerManager -> externalPowerManager.chargeItem(tilePowerAcceptor, stack));
}

[coderbot16]

Ah yes, I probably should have done that since I'm already used to the Rust iterator APIs :P - will fix

[coderbot16]

Alright, the checks now properly utilize Stream::filter.

[modmuss50]

Looks good, thanks