Fixed probable CharSummonTable NPE.

This commit is contained in:
MobiusDevelopment
2022-10-01 09:49:49 +00:00
parent 445aa48832
commit a1b8557c8f
24 changed files with 144 additions and 72 deletions

View File

@@ -207,9 +207,12 @@ public class CharSummonTable
if (player.hasServitors()) if (player.hasServitors())
{ {
final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null); final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null);
summon.setCurrentHp(curHp); if (summon != null)
summon.setCurrentMp(curMp); {
summon.setLifeTimeRemaining(time); summon.setCurrentHp(curHp);
summon.setCurrentMp(curMp);
summon.setLifeTimeRemaining(time);
}
} }
} }
} }

View File

@@ -207,9 +207,12 @@ public class CharSummonTable
if (player.hasServitors()) if (player.hasServitors())
{ {
final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null); final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null);
summon.setCurrentHp(curHp); if (summon != null)
summon.setCurrentMp(curMp); {
summon.setLifeTimeRemaining(time); summon.setCurrentHp(curHp);
summon.setCurrentMp(curMp);
summon.setLifeTimeRemaining(time);
}
} }
} }
} }

View File

@@ -207,9 +207,12 @@ public class CharSummonTable
if (player.hasServitors()) if (player.hasServitors())
{ {
final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null); final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null);
summon.setCurrentHp(curHp); if (summon != null)
summon.setCurrentMp(curMp); {
summon.setLifeTimeRemaining(time); summon.setCurrentHp(curHp);
summon.setCurrentMp(curMp);
summon.setLifeTimeRemaining(time);
}
} }
} }
} }

View File

@@ -207,9 +207,12 @@ public class CharSummonTable
if (player.hasServitors()) if (player.hasServitors())
{ {
final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null); final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null);
summon.setCurrentHp(curHp); if (summon != null)
summon.setCurrentMp(curMp); {
summon.setLifeTimeRemaining(time); summon.setCurrentHp(curHp);
summon.setCurrentMp(curMp);
summon.setLifeTimeRemaining(time);
}
} }
} }
} }

View File

@@ -207,9 +207,12 @@ public class CharSummonTable
if (player.hasServitors()) if (player.hasServitors())
{ {
final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null); final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null);
summon.setCurrentHp(curHp); if (summon != null)
summon.setCurrentMp(curMp); {
summon.setLifeTimeRemaining(time); summon.setCurrentHp(curHp);
summon.setCurrentMp(curMp);
summon.setLifeTimeRemaining(time);
}
} }
} }
} }

View File

@@ -207,9 +207,12 @@ public class CharSummonTable
if (player.hasServitors()) if (player.hasServitors())
{ {
final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null); final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null);
summon.setCurrentHp(curHp); if (summon != null)
summon.setCurrentMp(curMp); {
summon.setLifeTimeRemaining(time); summon.setCurrentHp(curHp);
summon.setCurrentMp(curMp);
summon.setLifeTimeRemaining(time);
}
} }
} }
} }

View File

@@ -207,9 +207,12 @@ public class CharSummonTable
if (player.hasServitors()) if (player.hasServitors())
{ {
final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null); final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null);
summon.setCurrentHp(curHp); if (summon != null)
summon.setCurrentMp(curMp); {
summon.setLifeTimeRemaining(time); summon.setCurrentHp(curHp);
summon.setCurrentMp(curMp);
summon.setLifeTimeRemaining(time);
}
} }
} }
} }

View File

@@ -207,9 +207,12 @@ public class CharSummonTable
if (player.hasServitors()) if (player.hasServitors())
{ {
final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null); final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null);
summon.setCurrentHp(curHp); if (summon != null)
summon.setCurrentMp(curMp); {
summon.setLifeTimeRemaining(time); summon.setCurrentHp(curHp);
summon.setCurrentMp(curMp);
summon.setLifeTimeRemaining(time);
}
} }
} }
} }

View File

@@ -207,9 +207,12 @@ public class CharSummonTable
if (player.hasServitors()) if (player.hasServitors())
{ {
final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null); final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null);
summon.setCurrentHp(curHp); if (summon != null)
summon.setCurrentMp(curMp); {
summon.setLifeTimeRemaining(time); summon.setCurrentHp(curHp);
summon.setCurrentMp(curMp);
summon.setLifeTimeRemaining(time);
}
} }
} }
} }

View File

@@ -207,9 +207,12 @@ public class CharSummonTable
if (player.hasServitors()) if (player.hasServitors())
{ {
final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null); final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null);
summon.setCurrentHp(curHp); if (summon != null)
summon.setCurrentMp(curMp); {
summon.setLifeTimeRemaining(time); summon.setCurrentHp(curHp);
summon.setCurrentMp(curMp);
summon.setLifeTimeRemaining(time);
}
} }
} }
} }

View File

@@ -207,9 +207,12 @@ public class CharSummonTable
if (player.hasServitors()) if (player.hasServitors())
{ {
final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null); final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null);
summon.setCurrentHp(curHp); if (summon != null)
summon.setCurrentMp(curMp); {
summon.setLifeTimeRemaining(time); summon.setCurrentHp(curHp);
summon.setCurrentMp(curMp);
summon.setLifeTimeRemaining(time);
}
} }
} }
} }

View File

@@ -207,9 +207,12 @@ public class CharSummonTable
if (player.hasServitors()) if (player.hasServitors())
{ {
final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null); final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null);
summon.setCurrentHp(curHp); if (summon != null)
summon.setCurrentMp(curMp); {
summon.setLifeTimeRemaining(time); summon.setCurrentHp(curHp);
summon.setCurrentMp(curMp);
summon.setLifeTimeRemaining(time);
}
} }
} }
} }

View File

@@ -207,9 +207,12 @@ public class CharSummonTable
if (player.hasServitors()) if (player.hasServitors())
{ {
final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null); final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null);
summon.setCurrentHp(curHp); if (summon != null)
summon.setCurrentMp(curMp); {
summon.setLifeTimeRemaining(time); summon.setCurrentHp(curHp);
summon.setCurrentMp(curMp);
summon.setLifeTimeRemaining(time);
}
} }
} }
} }

View File

@@ -207,9 +207,12 @@ public class CharSummonTable
if (player.hasServitors()) if (player.hasServitors())
{ {
final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null); final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null);
summon.setCurrentHp(curHp); if (summon != null)
summon.setCurrentMp(curMp); {
summon.setLifeTimeRemaining(time); summon.setCurrentHp(curHp);
summon.setCurrentMp(curMp);
summon.setLifeTimeRemaining(time);
}
} }
} }
} }

View File

@@ -207,9 +207,12 @@ public class CharSummonTable
if (player.hasServitors()) if (player.hasServitors())
{ {
final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null); final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null);
summon.setCurrentHp(curHp); if (summon != null)
summon.setCurrentMp(curMp); {
summon.setLifeTimeRemaining(time); summon.setCurrentHp(curHp);
summon.setCurrentMp(curMp);
summon.setLifeTimeRemaining(time);
}
} }
} }
} }

View File

@@ -207,9 +207,12 @@ public class CharSummonTable
if (player.hasServitors()) if (player.hasServitors())
{ {
final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null); final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null);
summon.setCurrentHp(curHp); if (summon != null)
summon.setCurrentMp(curMp); {
summon.setLifeTimeRemaining(time); summon.setCurrentHp(curHp);
summon.setCurrentMp(curMp);
summon.setLifeTimeRemaining(time);
}
} }
} }
} }

View File

@@ -207,9 +207,12 @@ public class CharSummonTable
if (player.hasServitors()) if (player.hasServitors())
{ {
final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null); final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null);
summon.setCurrentHp(curHp); if (summon != null)
summon.setCurrentMp(curMp); {
summon.setLifeTimeRemaining(time); summon.setCurrentHp(curHp);
summon.setCurrentMp(curMp);
summon.setLifeTimeRemaining(time);
}
} }
} }
} }

View File

@@ -207,9 +207,12 @@ public class CharSummonTable
if (player.hasServitors()) if (player.hasServitors())
{ {
final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null); final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null);
summon.setCurrentHp(curHp); if (summon != null)
summon.setCurrentMp(curMp); {
summon.setLifeTimeRemaining(time); summon.setCurrentHp(curHp);
summon.setCurrentMp(curMp);
summon.setLifeTimeRemaining(time);
}
} }
} }
} }

View File

@@ -207,9 +207,12 @@ public class CharSummonTable
if (player.hasServitors()) if (player.hasServitors())
{ {
final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null); final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null);
summon.setCurrentHp(curHp); if (summon != null)
summon.setCurrentMp(curMp); {
summon.setLifeTimeRemaining(time); summon.setCurrentHp(curHp);
summon.setCurrentMp(curMp);
summon.setLifeTimeRemaining(time);
}
} }
} }
} }

View File

@@ -207,9 +207,12 @@ public class CharSummonTable
if (player.hasServitors()) if (player.hasServitors())
{ {
final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null); final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null);
summon.setCurrentHp(curHp); if (summon != null)
summon.setCurrentMp(curMp); {
summon.setLifeTimeRemaining(time); summon.setCurrentHp(curHp);
summon.setCurrentMp(curMp);
summon.setLifeTimeRemaining(time);
}
} }
} }
} }

View File

@@ -207,9 +207,12 @@ public class CharSummonTable
if (player.hasServitors()) if (player.hasServitors())
{ {
final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null); final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null);
summon.setCurrentHp(curHp); if (summon != null)
summon.setCurrentMp(curMp); {
summon.setLifeTimeRemaining(time); summon.setCurrentHp(curHp);
summon.setCurrentMp(curMp);
summon.setLifeTimeRemaining(time);
}
} }
} }
} }

View File

@@ -202,9 +202,12 @@ public class CharSummonTable
if (player.hasServitors()) if (player.hasServitors())
{ {
final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null); final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null);
summon.setCurrentHp(curHp); if (summon != null)
summon.setCurrentMp(curMp); {
summon.setLifeTimeRemaining(time); summon.setCurrentHp(curHp);
summon.setCurrentMp(curMp);
summon.setLifeTimeRemaining(time);
}
} }
} }
} }

View File

@@ -202,9 +202,12 @@ public class CharSummonTable
if (player.hasServitors()) if (player.hasServitors())
{ {
final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null); final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null);
summon.setCurrentHp(curHp); if (summon != null)
summon.setCurrentMp(curMp); {
summon.setLifeTimeRemaining(time); summon.setCurrentHp(curHp);
summon.setCurrentMp(curMp);
summon.setLifeTimeRemaining(time);
}
} }
} }
} }

View File

@@ -202,9 +202,12 @@ public class CharSummonTable
if (player.hasServitors()) if (player.hasServitors())
{ {
final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null); final Servitor summon = player.getServitors().values().stream().map(s -> ((Servitor) s)).filter(s -> s.getReferenceSkill() == skillId).findAny().orElse(null);
summon.setCurrentHp(curHp); if (summon != null)
summon.setCurrentMp(curMp); {
summon.setLifeTimeRemaining(time); summon.setCurrentHp(curHp);
summon.setCurrentMp(curMp);
summon.setLifeTimeRemaining(time);
}
} }
} }
} }