Readd the powerboost for players and the proper maximum power per faction calculation.

This commit is contained in:
Olof Larsson 2013-04-25 07:53:21 +02:00
parent 8382d39409
commit 8070cc5579
4 changed files with 94 additions and 18 deletions

View File

@ -2,7 +2,9 @@ package com.massivecraft.factions.cmd;
import com.massivecraft.factions.Factions; import com.massivecraft.factions.Factions;
import com.massivecraft.factions.Perm; import com.massivecraft.factions.Perm;
import com.massivecraft.factions.cmd.arg.ARUPlayer;
import com.massivecraft.factions.cmd.arg.ARFaction; import com.massivecraft.factions.cmd.arg.ARFaction;
import com.massivecraft.factions.entity.UPlayer;
import com.massivecraft.factions.entity.Faction; import com.massivecraft.factions.entity.Faction;
import com.massivecraft.mcore.cmd.arg.ARDouble; import com.massivecraft.mcore.cmd.arg.ARDouble;
import com.massivecraft.mcore.cmd.req.ReqHasPerm; import com.massivecraft.mcore.cmd.req.ReqHasPerm;
@ -13,8 +15,9 @@ public class CmdFactionsPowerBoost extends FCommand
{ {
this.addAliases("powerboost"); this.addAliases("powerboost");
this.addRequiredArg("faction"); this.addRequiredArg("p|f|player|faction");
this.addRequiredArg("amount"); this.addRequiredArg("name");
this.addRequiredArg("#");
this.addRequirements(ReqHasPerm.get(Perm.POWERBOOST.node)); this.addRequirements(ReqHasPerm.get(Perm.POWERBOOST.node));
} }
@ -22,17 +25,42 @@ public class CmdFactionsPowerBoost extends FCommand
@Override @Override
public void perform() public void perform()
{ {
Faction faction = this.arg(0, ARFaction.get(usender)); String type = this.arg(0).toLowerCase();
if (faction == null) return; boolean doPlayer = true;
if (type.equals("f") || type.equals("faction"))
{
doPlayer = false;
}
else if (!type.equals("p") && !type.equals("player"))
{
msg("<b>You must specify \"p\" or \"player\" to target a player or \"f\" or \"faction\" to target a faction.");
msg("<b>ex. /f powerboost p SomePlayer 0.5 -or- /f powerboost f SomeFaction -5");
return;
}
Double amount = this.arg(1, ARDouble.get()); Double targetPower = this.arg(2, ARDouble.get());
if (amount == null) return; if (targetPower == null) return;
faction.setPowerBoost(amount); String target;
msg("<i>"+faction.getName()+" now has a power bonus/penalty of "+amount+" to min and max power levels."); if (doPlayer)
{
UPlayer targetPlayer = this.arg(1, ARUPlayer.getStartAny(sender));
if (targetPlayer == null) return;
// TODO: Inconsistent. Why is there no boolean to toggle this logging of? targetPlayer.setPowerBoost(targetPower);
Factions.get().log(usender.getName()+" has set the power bonus/penalty for "+faction.getName()+" to "+amount+"."); target = "Player \""+targetPlayer.getName()+"\"";
}
else
{
Faction targetFaction = this.arg(1, ARFaction.get(sender));
if (targetFaction == null) return;
targetFaction.setPowerBoost(targetPower);
target = "Faction \""+targetFaction.getName()+"\"";
}
msg("<i>"+target+" now has a power bonus/penalty of "+targetPower+" to min and max power levels.");
Factions.get().log(usender.getName()+" has set the power bonus/penalty for "+target+" to "+targetPower+".");
} }
} }

View File

@ -652,10 +652,14 @@ public class Faction extends Entity<Faction> implements EconomyParticipator
{ {
ret += uplayer.getPower(); ret += uplayer.getPower();
} }
ret += this.getPowerBoost();
ret = Math.min(ret, this.getPowerMax()); double factionPowerMax = UConf.get(this).factionPowerMax;
ret = Math.max(ret, 0); if (factionPowerMax > 0 && ret > factionPowerMax)
{
ret = factionPowerMax;
}
ret += this.getPowerBoost();
return ret; return ret;
} }
@ -663,7 +667,22 @@ public class Faction extends Entity<Faction> implements EconomyParticipator
public double getPowerMax() public double getPowerMax()
{ {
if (this.getFlag(FFlag.INFPOWER)) return 999999; if (this.getFlag(FFlag.INFPOWER)) return 999999;
return UConf.get(this).factionPowerMax + this.getPowerBoost();
double ret = 0;
for (UPlayer uplayer : this.getUPlayers())
{
ret += uplayer.getPowerMax();
}
double factionPowerMax = UConf.get(this).factionPowerMax;
if (factionPowerMax > 0 && ret > factionPowerMax)
{
ret = factionPowerMax;
}
ret += this.getPowerBoost();
return ret;
} }
public int getPowerRounded() public int getPowerRounded()

View File

@ -44,6 +44,7 @@ public class UPlayer extends SenderEntity<UPlayer> implements EconomyParticipato
this.setFactionId(that.factionId); this.setFactionId(that.factionId);
this.setRole(that.role); this.setRole(that.role);
this.setTitle(that.title); this.setTitle(that.title);
this.powerBoost = that.powerBoost;
this.setPower(that.power); this.setPower(that.power);
return this; return this;
@ -86,6 +87,11 @@ public class UPlayer extends SenderEntity<UPlayer> implements EconomyParticipato
// Null means the player has no title. // Null means the player has no title.
private String title = null; private String title = null;
// Player usually do not have a powerboost. It defaults to 0.
// The powerBoost is a custom increase/decrease to default and maximum power.
// Note that player powerBoost and faction powerBoost are very similar.
private Double powerBoost = null;
// Each player has an individual power level. // Each player has an individual power level.
// The power level for online players is occasionally updated by a recurring task and the power should stay the same for offline players. // The power level for online players is occasionally updated by a recurring task and the power should stay the same for offline players.
// For that reason the value is to be considered correct when you pick it. Do not call the power update method. // For that reason the value is to be considered correct when you pick it. Do not call the power update method.
@ -263,6 +269,29 @@ public class UPlayer extends SenderEntity<UPlayer> implements EconomyParticipato
this.changed(); this.changed();
} }
// -------------------------------------------- //
// FIELD: powerBoost
// -------------------------------------------- //
public double getPowerBoost()
{
Double ret = this.powerBoost;
if (ret == null) ret = 0D;
return ret;
}
public void setPowerBoost(Double powerBoost)
{
if (powerBoost == null || powerBoost == 0) powerBoost = null;
this.powerBoost = powerBoost;
this.changed();
}
public boolean hasPowerBoost()
{
return this.getPowerBoost() != 0D;
}
// -------------------------------------------- // // -------------------------------------------- //
// FIELD: power // FIELD: power
// -------------------------------------------- // // -------------------------------------------- //

View File

@ -25,13 +25,13 @@ public class PowerMixinDefault implements PowerMixin
@Override @Override
public double getMax(UPlayer uplayer) public double getMax(UPlayer uplayer)
{ {
return UConf.get(uplayer).powerMax; return UConf.get(uplayer).powerMax + uplayer.getPowerBoost();
} }
@Override @Override
public double getMin(UPlayer uplayer) public double getMin(UPlayer uplayer)
{ {
return UConf.get(uplayer).powerMin; return UConf.get(uplayer).powerMin + uplayer.getPowerBoost();
} }
@Override @Override