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

MultiSchemTypes #403

Zusammengeführt
YoyoNow hat 4 Commits von MultiSchemTypes nach master 2023-12-05 15:21:23 +01:00 zusammengeführt
Diskussion 1 Commits 4 Geänderte Dateien 5 +5 -4
4 geänderte Dateien mit 5 neuen und 4 gelöschten Zeilen
Statistiken anzeigen Patch-Datei herunterladen Vergleichs-Datei herunterladen Alle Dateien ausklappen Alle Dateien einklappen
Nur Änderungen aus Commit c0665b1868 werden angezeigt - Alle Commits anzeigen

1
FightSystem_Core/src/config.yml
Datei anzeigen

@ -61,6 +61,7 @@ Schematic:
z: 0 z: 0
# The schematic type that can be chosen in this arena # The schematic type that can be chosen in this arena
Type: Normal # defaults to Normal if missing Type: Normal # defaults to Normal if missing
# The schematic types that are also allowed to be chosen in this arena
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Lixfel hat 2023-12-01 17:51:19 +01:00 kommentiert
Veraltet
Review
Link kopieren

Bitte wie bei den anderen Optionen in einen Kommentar in der Zeile darüber noch erklären, was diese Option bedeutet.

Bitte wie bei den anderen Optionen in einen Kommentar in der Zeile darüber noch erklären, was diese Option bedeutet.
SubTypes: [] # defaults to empty List SubTypes: [] # defaults to empty List
# Shortcut of the schematic type # Shortcut of the schematic type
Shortcut: "" # defaults to "" if missing Shortcut: "" # defaults to "" if missing

4
FightSystem_Core/src/de/steamwar/fightsystem/Config.java
Datei anzeigen

@ -178,8 +178,8 @@ public class Config {
int schemsizeY = config.getInt("Schematic.Size.y"); int schemsizeY = config.getInt("Schematic.Size.y");
int schemsizeZ = config.getInt("Schematic.Size.z"); int schemsizeZ = config.getInt("Schematic.Size.z");
RanksEnabled = !config.getStringList("Ranks").isEmpty(); RanksEnabled = !config.getStringList("Ranks").isEmpty();
SchematicType = de.steamwar.sql.SchematicType.fromDB(Objects.requireNonNull(config.getString("Schematic.Type", "normal"))); SchematicType = de.steamwar.sql.SchematicType.fromDB(Objects.requireNonNull(config.getString("Schematic.Type", "normal").toLowerCase()));
SubTypes = config.getStringList("Schematic.SubTypes").stream().map(de.steamwar.sql.SchematicType::fromDB).collect(Collectors.toList()); SubTypes = config.getStringList("Schematic.SubTypes").stream().map(String::toLowerCase).map(de.steamwar.sql.SchematicType::fromDB).collect(Collectors.toList());
Lixfel markierte diese Unterhaltung als gelöst Veraltet
Lixfel hat 2023-12-01 17:52:08 +01:00 kommentiert
Veraltet
Review
Link kopieren

Bitte nochmal überprüfen, ob fromDB erwartet, dass der Name in LowerCase ist, und wenn das der Fall ist, vorher bitte einmal .toLowerCase().

Bitte nochmal überprüfen, ob fromDB erwartet, dass der Name in LowerCase ist, und wenn das der Fall ist, vorher bitte einmal .toLowerCase().
YoyoNow hat 2023-12-01 17:59:26 +01:00 kommentiert
Veraltet
Review
Link kopieren

Du machst es bei dem code da drüber auch nicht mit toLowercase, soll ich es einfach bei beiden dann einfügen? Ich glaube nämlich das es benötigt wird.

Du machst es bei dem code da drüber auch nicht mit toLowercase, soll ich es einfach bei beiden dann einfügen? Ich glaube nämlich das es benötigt wird.
Lixfel hat 2023-12-01 18:02:27 +01:00 kommentiert
Veraltet
Review
Link kopieren

Dann wird es nicht benötigt.

Dann wird es nicht benötigt.
Lixfel hat 2023-12-01 18:04:49 +01:00 kommentiert
Veraltet
Review
Link kopieren

Bitte das toLowerCase wieder rausnehmen (da unnötig)

Bitte das toLowerCase wieder rausnehmen (da unnötig)
IgnorePublicOnly = config.getBoolean("Schematic.IgnorePublicOnly", false); IgnorePublicOnly = config.getBoolean("Schematic.IgnorePublicOnly", false);
boolean rotate = config.getBoolean("Schematic.Rotate", true); boolean rotate = config.getBoolean("Schematic.Rotate", true);
PasteAligned = config.getBoolean("Schematic.PasteAligned", false); PasteAligned = config.getBoolean("Schematic.PasteAligned", false);

2
FightSystem_Core/src/de/steamwar/fightsystem/commands/GUI.java
Datei anzeigen

@ -172,7 +172,7 @@ public class GUI {
SWInventory inv = new SWInventory(p, invSize, msg.parse("SCHEM_TITLE", p, Config.GameName)); SWInventory inv = new SWInventory(p, invSize, msg.parse("SCHEM_TITLE", p, Config.GameName));
setupSchemTypeRow(p, inv, Config.SchematicType, 0); setupSchemTypeRow(p, inv, Config.SchematicType, 0);
for (int i = 0; i < Config.SubTypes.size(); i++) { for (int i = 0; i < Config.SubTypes.size(); i++) {
setupSchemTypeRow(p, inv, Config.SubTypes.get(0), i + 1); setupSchemTypeRow(p, inv, Config.SubTypes.get(i), i + 1);
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Lixfel hat 2023-12-01 17:53:16 +01:00 kommentiert
Veraltet
Review
Link kopieren

Immer .get(0)? Nicht get(i)?

Immer .get(0)? Nicht get(i)?
YoyoNow hat 2023-12-01 17:59:46 +01:00 kommentiert
Veraltet
Review
Link kopieren

Gut gesehen danke.

Gut gesehen danke.
} }
inv.setCallback(-999, (ClickType click) -> p.closeInventory()); inv.setCallback(-999, (ClickType click) -> p.closeInventory());
inv.open(); inv.open();

2
FightSystem_Core/src/de/steamwar/fightsystem/fight/Fight.java
Datei anzeigen

@ -143,7 +143,7 @@ public class Fight {
if (Config.OnlyPublicSchematics) { if (Config.OnlyPublicSchematics) {
return true; return true;
} }
if (Config.IgnorePublicOnly) { if (Config.IgnorePublicOnly || ArenaMode.RankedEvent.contains(Config.mode)) {
YoyoNow markierte diese Unterhaltung als gelöst Veraltet
Lixfel hat 2023-12-01 17:50:31 +01:00 kommentiert
Veraltet
Review
Link kopieren

Hier fehlt wie früher die Prüfung, ob es ein Event ist (wobei dann ebenfalls IgnorePublicOnly gilt). Wenn es ein Public-Only Turnier ist, wird das bereits über OnlyPublicSchematics separat geregelt.

Hier fehlt wie früher die Prüfung, ob es ein Event ist (wobei dann ebenfalls IgnorePublicOnly gilt). Wenn es ein Public-Only Turnier ist, wird das bereits über OnlyPublicSchematics separat geregelt.
return false; return false;
} }
return redTeam.isPublicsOnly() || blueTeam.isPublicsOnly(); return redTeam.isPublicsOnly() || blueTeam.isPublicsOnly();