SteamWar/FightSystem
Archiviert
13
1

MultiSchemTypes #403

Zusammengeführt
YoyoNow hat 4 Commits von MultiSchemTypes nach master 2023-12-05 15:21:23 +01:00 zusammengeführt
Besitzer
Keine Beschreibung angegeben.
YoyoNow hat 2 Commits 2023-12-01 16:20:21 +01:00 hinzugefügt
Add MultiSchemType selection for GUI.preSchemDialog
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
b3ad960973
Fix Fight.publicOnly method
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
8d77f4dcf4
Lixfel wurde von YoyoNow 2023-12-01 16:39:53 +01:00 zugewiesen
Lixfel hat 2023-12-01 17:55:51 +01:00 Änderungen angefragt
@ -61,6 +61,7 @@ Schematic:
z: 0
# The schematic type that can be chosen in this arena
Type: Normal # defaults to Normal if missing
SubTypes: [] # defaults to empty List
Besitzer

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.
YoyoNow markierte diese Unterhaltung als gelöst
@ -181,6 +179,7 @@ public class Config {
int schemsizeZ = config.getInt("Schematic.Size.z");
RanksEnabled = !config.getStringList("Ranks").isEmpty();
SchematicType = de.steamwar.sql.SchematicType.fromDB(Objects.requireNonNull(config.getString("Schematic.Type", "normal")));
SubTypes = config.getStringList("Schematic.SubTypes").stream().map(de.steamwar.sql.SchematicType::fromDB).collect(Collectors.toList());
Besitzer

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().
Autor
Besitzer

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.
Besitzer

Dann wird es nicht benötigt.

Dann wird es nicht benötigt.
Lixfel markierte diese Unterhaltung als gelöst
@ -193,0 +172,4 @@
SWInventory inv = new SWInventory(p, invSize, msg.parse("SCHEM_TITLE", p, Config.GameName));
setupSchemTypeRow(p, inv, Config.SchematicType, 0);
for (int i = 0; i < Config.SubTypes.size(); i++) {
setupSchemTypeRow(p, inv, Config.SubTypes.get(0), i + 1);
Besitzer

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

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

Gut gesehen danke.

Gut gesehen danke.
YoyoNow markierte diese Unterhaltung als gelöst
@ -147,3 +146,1 @@
if(Config.IgnorePublicOnly || ArenaMode.RankedEvent.contains(Config.mode)){
return 1000;
if (Config.IgnorePublicOnly) {
Besitzer

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.
YoyoNow markierte diese Unterhaltung als gelöst
YoyoNow hat 1 Commit 2023-12-01 18:03:23 +01:00 hinzugefügt
Fix non lower case SchematicType fromDB lookup
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
c0665b1868
Fix RankedEvent not being considered für ignore publics only
Fix using index 0 instead of i for creating of gui
Fix config.yml missing comment
Lixfel hat 2023-12-01 18:07:04 +01:00 Änderungen angefragt
@ -182,2 +180,3 @@
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(String::toLowerCase).map(de.steamwar.sql.SchematicType::fromDB).collect(Collectors.toList());
Besitzer

Bitte das toLowerCase wieder rausnehmen (da unnötig)

Bitte das toLowerCase wieder rausnehmen (da unnötig)
YoyoNow markierte diese Unterhaltung als gelöst
@ -200,0 +196,4 @@
});
}
if (SchematicNode.getAllAccessibleSchematicsOfType(SteamwarUser.get(p.getUniqueId()).getId(), Config.SchematicType.toDB()).isEmpty() && !Config.test()) {
Besitzer

Hier nutzt du noch fälschlicherweise Config.SchematicType

Hier nutzt du noch fälschlicherweise Config.SchematicType
YoyoNow markierte diese Unterhaltung als gelöst
@ -150,4 +148,1 @@
}
if(redTeam.getLeader() == null || blueTeam.getLeader() == null){
Besitzer

Ich weiß nicht, ob wir diesen Überprüfungscode einfach so rauswerfen sollten (publicOnly, solange noch einer der Gegner fehlt).

Ich weiß nicht, ob wir diesen Überprüfungscode einfach so rauswerfen sollten (publicOnly, solange noch einer der Gegner fehlt).
YoyoNow markierte diese Unterhaltung als gelöst
YoyoNow hat 1 Commit 2023-12-01 18:15:32 +01:00 hinzugefügt
Revert lowercase
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
3ee7411591
Fix SchematicType not used for checking available schematics
Fix leader not present to enable public only schem selection
Lixfel hat die Änderungen 2023-12-02 20:09:11 +01:00 genehmigt
Besitzer

Erst mergen wenn WarGears in die 1.20 wechseln (und 1.15 & 1.19 Rank-System nicht mehr benötigt wird)

Erst mergen wenn WarGears in die 1.20 wechseln (und 1.15 & 1.19 Rank-System nicht mehr benötigt wird)
YoyoNow hat Commit ad7a42a685 in master 2023-12-05 15:21:23 +01:00 gemerged
YoyoNow löschte die Branch MultiSchemTypes 2023-12-05 15:21:24 +01:00
Dieses Repo ist archiviert. Du kannst Pull-Requests nicht kommentieren.
Keine Beschreibung angegeben.