13
0

Closes #31 schematic_search #39

Zusammengeführt
Lixfel hat 4 Commits von schematic_search nach master 2020-01-08 17:45:31 +01:00 zusammengeführt
2 geänderte Dateien mit 2 neuen und 1 gelöschten Zeilen
Nur Änderungen aus Commit 727c74a35b werden angezeigt - Alle Commits anzeigen

Datei anzeigen

@ -184,7 +184,7 @@ class GUI {
List<Pair<SWItem, Schematic>> schemList = new LinkedList<>(); List<Pair<SWItem, Schematic>> schemList = new LinkedList<>();
List<Schematic> filteredSchems = new ArrayList<>(); List<Schematic> filteredSchems = new ArrayList<>();
for(Schematic currentSchem : filteredSchems) { for(Schematic currentSchem : Schematic.getSchemsAccessibleByUser(player.getUniqueId())) {
Veraltet
Review

Wird nie durchlaufen, da filteredSchems keine Elemente enthält.

Wird nie durchlaufen, da filteredSchems keine Elemente enthält.
Veraltet
Review

gefixt

gefixt
if(currentSchem.getSchemName().toLowerCase().contains(search.toLowerCase())) if(currentSchem.getSchemName().toLowerCase().contains(search.toLowerCase()))
filteredSchems.add(currentSchem); filteredSchems.add(currentSchem);
Review

Information: Du darfst keine Elemente hinzufügen/löschen aus der Liste, durch die du gerade durchiterierst.

Information: Du darfst keine Elemente hinzufügen/löschen aus der Liste, durch die du gerade durchiterierst.
Review

ja stimmt, wird hier aber nie eintreffen, da es sich um einen Programmierfehler handelt

ja stimmt, wird hier aber nie eintreffen, da es sich um einen Programmierfehler handelt
} }

Datei anzeigen

@ -438,6 +438,7 @@ public class SchematicCommand implements CommandExecutor {
player.sendMessage("§8/§eschem gui §8- §7Öffnet die Schematic-GUI"); player.sendMessage("§8/§eschem gui §8- §7Öffnet die Schematic-GUI");
player.sendMessage("§8/§eschem list §8- §7Zeigt dir deine Schematics an"); player.sendMessage("§8/§eschem list §8- §7Zeigt dir deine Schematics an");
player.sendMessage("§8/§eschem list public §8- §7Zeigt alle Public-Schematics"); player.sendMessage("§8/§eschem list public §8- §7Zeigt alle Public-Schematics");
player.sendMessage("§8/§eschem search §8[§7Stichwort§8] - §7Sucht nach passenden Schematics");
player.sendMessage("§8/§eschem load §8[§7Schematic§8] - §7Lädt eine Schematic"); player.sendMessage("§8/§eschem load §8[§7Schematic§8] - §7Lädt eine Schematic");
player.sendMessage("§8/§eschem save §8[§7Schematic§8] - §7Speichert dein Clipboard als Schematic"); player.sendMessage("§8/§eschem save §8[§7Schematic§8] - §7Speichert dein Clipboard als Schematic");
player.sendMessage("§8/§eschem changetype §8[§7Schematic§8] - §7Ändert die Art deiner Schematic"); player.sendMessage("§8/§eschem changetype §8[§7Schematic§8] - §7Ändert die Art deiner Schematic");