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 +1 -2
1 geänderte Dateien mit 1 neuen und 2 gelöschten Zeilen
Statistiken anzeigen Patch-Datei herunterladen Vergleichs-Datei herunterladen Alle Dateien ausklappen Alle Dateien einklappen
Nur Änderungen aus Commit af956db60b werden angezeigt - Alle Commits anzeigen

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

@ -56,8 +56,7 @@ public class SchematicType {
continue;
SchematicType checktype = null;
String materialString = config.getString("Schematic.Material");
Material material = materialString != null ? SWItem.getMaterial(materialString) : SWItem.getMaterial("STONE_BUTTON");
Material material = SWItem.getMaterial(config.getString("Schematic.Material", "STONE_BUTTON"));
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");
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);