SchematicSelectorItemFix #203

xSpw · 2022-07-12T20:08:10+02:00

xSpw hat

2022-07-12 20:08:10 +02:00

kommentiert

Mitglied

Keine Beschreibung angegeben.

xSpw hat 2 Commits 2022-07-12 20:08:14 +02:00 hinzugefügt

SchematicSelector.renderItem can not be null

SteamWarCI Build successful

Details

a1fca372fc

SchematicSelector.renderItem null material replaced to Barrier

SteamWarCI Build successful

Details

568d21a50d

Chaoscaot hat 2022-07-12 20:58:44 +02:00 Änderungen angefragt

SpigotCore_Main/src/de/steamwar/inventory/SchematicSelector.java Veraltet

						
				@ -235,6 +235,10 @@ public class SchematicSelector {

				    private SWListInv.SWListEntry<SchematicNode> renderItem(SchematicNode node) {

				        Material m = SWItem.getMaterial(node.getItem());

				        if(m == null) {

Chaoscaot hat

2022-07-12 20:56:29 +02:00

kommentiert

Besitzer

Ist nicht der Fehler hier.

SWItem.getMaterial sollte ein Material oder STONE zurück geben. Also sollte da irgendwo der Bug liegen, hier ist nur das Symptom.

Ist nicht der Fehler hier. SWItem.getMaterial sollte ein Material oder STONE zurück geben. Also sollte da irgendwo der Bug liegen, hier ist nur das Symptom.

xSpw hat

2022-07-12 22:27:37 +02:00

kommentiert

Autor

Mitglied

Aber das kann doch garkein null ausgeben

Chaoscaot hat

2022-07-12 23:48:35 +02:00

kommentiert

Besitzer

Doch.

SWItem.getMaterial ruft einen FlatteningWrapper auf (Version Dependend Code). d.h. In der 1.14 aufwärts wird der FlatteningWrapper14 aufgerufen. Dieser hat eine HashMap welche einen Lookup für Legacy Namen hat.

Also wenn die Material.valueOf nen Fehler wirft, dann kommt der HashMap Lookup, welcher dann null ausgeben kann. Daher kommt das null.
Das war dann woll irgendein 1.19 Material welches es nicht in der 1.15 gibt.

Schau mal im SpigotCore_14 de.steamwar.core.FlatteningWrapper14:244.

Doch. SWItem.getMaterial ruft einen FlatteningWrapper auf (Version Dependend Code). d.h. In der 1.14 aufwärts wird der FlatteningWrapper14 aufgerufen. Dieser hat eine HashMap welche einen Lookup für Legacy Namen hat. Also wenn die Material.valueOf nen Fehler wirft, dann kommt der HashMap Lookup, welcher dann null ausgeben kann. Daher kommt das null. Das war dann woll irgendein 1.19 Material welches es nicht in der 1.15 gibt. Schau mal im SpigotCore_14 de.steamwar.core.FlatteningWrapper14:244.

SpigotCore_Main/src/de/steamwar/inventory/SchematicSelector.java Veraltet

						
				@ -236,2 +236,4 @@

				        Material m = SWItem.getMaterial(node.getItem());

				        if(m == null) {

				            m = Material.BARRIER;

Chaoscaot hat

2022-07-12 20:58:13 +02:00

kommentiert

Besitzer

Sollte auch STONE sein wie in der SWItem.getMaterial Methode, damit es konsistent bleibt.

xSpw hat 1 Commit 2022-07-12 22:08:12 +02:00 hinzugefügt

SWItem can not be null

SteamWarCI Build successful

Details

50a58afa6a

Lixfel hat

2022-07-14 21:56:04 +02:00

kommentiert

Besitzer

Wie Chaoscaot schon geschrieben hat wäre es wahrscheinlich besser, statt dieser Änderung direkt in der SWItem.getMaterial -Methode das Ergebnis des FlatteningWrappers auf null zu prüfen und dort ggf. das Material direkt zu ersetzen (bislang hatten wir da immer STONE als fallback, aber ich finde die BARRIER da eigentlich auch nicht schlecht. Die Änderung direkt in getMaterial würde auch Items, welche an anderer Stelle/über andere Methode ngeneriert werden dann korrekt "fixen", daher wäre das zu bevorzugen.