Consider clan leader as top pledge class.

This commit is contained in:
MobiusDevelopment
2020-02-25 18:34:30 +00:00
parent ba799c053a
commit 38a749f24f
17 changed files with 25 additions and 17 deletions

View File

@@ -48,6 +48,6 @@ public class ConditionPlayerPledgeClass extends Condition
{ {
return false; return false;
} }
return (_pledgeClass == -1) ? effector.getActingPlayer().isClanLeader() : (effector.getActingPlayer().getPledgeClass() >= _pledgeClass); return effector.getActingPlayer().isClanLeader() || (effector.getActingPlayer().getPledgeClass() >= _pledgeClass);
} }
} }

View File

@@ -48,6 +48,6 @@ public class ConditionPlayerPledgeClass extends Condition
{ {
return false; return false;
} }
return (_pledgeClass == -1) ? effector.getActingPlayer().isClanLeader() : (effector.getActingPlayer().getPledgeClass() >= _pledgeClass); return effector.getActingPlayer().isClanLeader() || (effector.getActingPlayer().getPledgeClass() >= _pledgeClass);
} }
} }

View File

@@ -48,6 +48,6 @@ public class ConditionPlayerPledgeClass extends Condition
{ {
return false; return false;
} }
return (_pledgeClass == -1) ? effector.getActingPlayer().isClanLeader() : (effector.getActingPlayer().getPledgeClass() >= _pledgeClass); return effector.getActingPlayer().isClanLeader() || (effector.getActingPlayer().getPledgeClass() >= _pledgeClass);
} }
} }

View File

@@ -48,6 +48,6 @@ public class ConditionPlayerPledgeClass extends Condition
{ {
return false; return false;
} }
return (_pledgeClass == -1) ? effector.getActingPlayer().isClanLeader() : (effector.getActingPlayer().getPledgeClass() >= _pledgeClass); return effector.getActingPlayer().isClanLeader() || (effector.getActingPlayer().getPledgeClass() >= _pledgeClass);
} }
} }

View File

@@ -48,6 +48,6 @@ public class ConditionPlayerPledgeClass extends Condition
{ {
return false; return false;
} }
return (_pledgeClass == -1) ? effector.getActingPlayer().isClanLeader() : (effector.getActingPlayer().getPledgeClass() >= _pledgeClass); return effector.getActingPlayer().isClanLeader() || (effector.getActingPlayer().getPledgeClass() >= _pledgeClass);
} }
} }

View File

@@ -48,6 +48,6 @@ public class ConditionPlayerPledgeClass extends Condition
{ {
return false; return false;
} }
return (_pledgeClass == -1) ? effector.getActingPlayer().isClanLeader() : (effector.getActingPlayer().getPledgeClass() >= _pledgeClass); return effector.getActingPlayer().isClanLeader() || (effector.getActingPlayer().getPledgeClass() >= _pledgeClass);
} }
} }

View File

@@ -48,6 +48,6 @@ public class ConditionPlayerPledgeClass extends Condition
{ {
return false; return false;
} }
return (_pledgeClass == -1) ? effector.getActingPlayer().isClanLeader() : (effector.getActingPlayer().getPledgeClass() >= _pledgeClass); return effector.getActingPlayer().isClanLeader() || (effector.getActingPlayer().getPledgeClass() >= _pledgeClass);
} }
} }

View File

@@ -48,6 +48,6 @@ public class ConditionPlayerPledgeClass extends Condition
{ {
return false; return false;
} }
return (_pledgeClass == -1) ? effector.getActingPlayer().isClanLeader() : (effector.getActingPlayer().getPledgeClass() >= _pledgeClass); return effector.getActingPlayer().isClanLeader() || (effector.getActingPlayer().getPledgeClass() >= _pledgeClass);
} }
} }

View File

@@ -44,6 +44,10 @@ public class ConditionPlayerPledgeClass extends Condition
@Override @Override
public boolean testImpl(Creature effector, Creature effected, Skill skill, Item item) public boolean testImpl(Creature effector, Creature effected, Skill skill, Item item)
{ {
return (effector.getActingPlayer() != null) && (effector.getActingPlayer().getClan() != null) && ((_pledgeClass == -1) ? effector.getActingPlayer().isClanLeader() : (effector.getActingPlayer().getPledgeClass() >= _pledgeClass)); if ((effector.getActingPlayer() == null) || (effector.getActingPlayer().getClan() == null))
{
return false;
}
return effector.getActingPlayer().isClanLeader() || (effector.getActingPlayer().getPledgeClass() >= _pledgeClass);
} }
} }

View File

@@ -44,6 +44,10 @@ public class ConditionPlayerPledgeClass extends Condition
@Override @Override
public boolean testImpl(Creature effector, Creature effected, Skill skill, Item item) public boolean testImpl(Creature effector, Creature effected, Skill skill, Item item)
{ {
return (effector.getActingPlayer() != null) && (effector.getActingPlayer().getClan() != null) && ((_pledgeClass == -1) ? effector.getActingPlayer().isClanLeader() : (effector.getActingPlayer().getPledgeClass() >= _pledgeClass)); if ((effector.getActingPlayer() == null) || (effector.getActingPlayer().getClan() == null))
{
return false;
}
return effector.getActingPlayer().isClanLeader() || (effector.getActingPlayer().getPledgeClass() >= _pledgeClass);
} }
} }

View File

@@ -48,6 +48,6 @@ public class ConditionPlayerPledgeClass extends Condition
{ {
return false; return false;
} }
return (_pledgeClass == -1) ? effector.getActingPlayer().isClanLeader() : (effector.getActingPlayer().getPledgeClass() >= _pledgeClass); return effector.getActingPlayer().isClanLeader() || (effector.getActingPlayer().getPledgeClass() >= _pledgeClass);
} }
} }

View File

@@ -48,6 +48,6 @@ public class ConditionPlayerPledgeClass extends Condition
{ {
return false; return false;
} }
return (_pledgeClass == -1) ? effector.getActingPlayer().isClanLeader() : (effector.getActingPlayer().getPledgeClass() >= _pledgeClass); return effector.getActingPlayer().isClanLeader() || (effector.getActingPlayer().getPledgeClass() >= _pledgeClass);
} }
} }

View File

@@ -48,6 +48,6 @@ public class ConditionPlayerPledgeClass extends Condition
{ {
return false; return false;
} }
return (_pledgeClass == -1) ? effector.getActingPlayer().isClanLeader() : (effector.getActingPlayer().getPledgeClass() >= _pledgeClass); return effector.getActingPlayer().isClanLeader() || (effector.getActingPlayer().getPledgeClass() >= _pledgeClass);
} }
} }

View File

@@ -48,6 +48,6 @@ public class ConditionPlayerPledgeClass extends Condition
{ {
return false; return false;
} }
return (_pledgeClass == -1) ? effector.getActingPlayer().isClanLeader() : (effector.getActingPlayer().getPledgeClass() >= _pledgeClass); return effector.getActingPlayer().isClanLeader() || (effector.getActingPlayer().getPledgeClass() >= _pledgeClass);
} }
} }

View File

@@ -48,6 +48,6 @@ public class ConditionPlayerPledgeClass extends Condition
{ {
return false; return false;
} }
return (_pledgeClass == -1) ? effector.getActingPlayer().isClanLeader() : (effector.getActingPlayer().getPledgeClass() >= _pledgeClass); return effector.getActingPlayer().isClanLeader() || (effector.getActingPlayer().getPledgeClass() >= _pledgeClass);
} }
} }

View File

@@ -48,6 +48,6 @@ public class ConditionPlayerPledgeClass extends Condition
{ {
return false; return false;
} }
return (_pledgeClass == -1) ? effector.getActingPlayer().isClanLeader() : (effector.getActingPlayer().getPledgeClass() >= _pledgeClass); return effector.getActingPlayer().isClanLeader() || (effector.getActingPlayer().getPledgeClass() >= _pledgeClass);
} }
} }

View File

@@ -48,6 +48,6 @@ public class ConditionPlayerPledgeClass extends Condition
{ {
return false; return false;
} }
return (_pledgeClass == -1) ? effector.getActingPlayer().isClanLeader() : (effector.getActingPlayer().getPledgeClass() >= _pledgeClass); return effector.getActingPlayer().isClanLeader() || (effector.getActingPlayer().getPledgeClass() >= _pledgeClass);
} }
} }