SteamWar/SpigotCore
SteamWar/SpigotCore
Archiviert
13
0
Fork 0
Code Issues 4 Pull-Requests 1 Releases Wiki Aktivität

Add optional Material to SchematicType #144

Zusammengeführt
Lixfel hat 5 Commits von SchemTypeMaterial nach master 2021-12-22 07:57:06 +01:00 zusammengeführt
Diskussion 0 Commits 5 Geänderte Dateien 1 +4 -4
1 geänderte Dateien mit 4 neuen und 4 gelöschten Zeilen
Statistiken anzeigen Patch-Datei herunterladen Vergleichs-Datei herunterladen Alle Dateien ausklappen Alle Dateien einklappen
Nur Änderungen aus Commit a63dbafd2b werden angezeigt - Alle Commits anzeigen

8
SpigotCore_Main/src/de/steamwar/sql/SchematicType.java
Datei anzeigen

@ -29,7 +29,7 @@ import java.util.*;
import java.util.stream.Collectors;
public class SchematicType {
public static final SchematicType Normal = new SchematicType("Normal", "", Type.NORMAL, null, null); //Has to stay publicly availible
public static final SchematicType Normal = new SchematicType("Normal", "", Type.NORMAL, null, SWItem.getMaterial("STONE_BUTTON")); //Has to stay publicly availible
private static final Map<String, SchematicType> fromDB;
private static final List<SchematicType> types;
@ -56,8 +56,8 @@ public class SchematicType {
continue;
SchematicType checktype = null;
String materialString = config.getString("material");
Material material = materialString != null ? SWItem.getMaterial(materialString) : null;
String materialString = config.getString("Schematic.Material");
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Lixfel hat 2021-12-19 08:27:01 +01:00 kommentiert
Veraltet
Review
Link kopieren

Bitte unbedingt ändern zu: config.getString("Schematic.Material");

Bitte unbedingt ändern zu: config.getString("Schematic.Material");
Material material = materialString != null ? SWItem.getMaterial(materialString) : SWItem.getMaterial("STONE_BUTTON");
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Lixfel hat 2021-12-19 08:27:39 +01:00 kommentiert
Veraltet
Review
Link kopieren

Warum Material null zulassen und nicht einfach auf den Steinknopf (bitte über SWItem) defaulten?

Warum Material null zulassen und nicht einfach auf den Steinknopf (bitte über SWItem) defaulten?
if(!config.getStringList("CheckQuestions").isEmpty()) {
checktype = new SchematicType("C" + type, "C" + shortcut, Type.CHECK_TYPE, null, material);
@ -86,7 +86,7 @@ public class SchematicType {
this.kuerzel = kuerzel;
this.type = type;
this.checkType = checkType;
this.material = material == null ? Material.STONE_BUTTON : material;
this.material = material;
}
public boolean isAssignable(){