SteamWar/BungeeCore
Archiviert
13
2

WIP: Add SchemSearch #467

Entwurf
Chaoscaot möchte 14 Commits von schemsearch nach master mergen
Besitzer
Keine Beschreibung angegeben.
Chaoscaot hat 9 Commits 2023-04-12 02:07:32 +02:00 hinzugefügt
Basic SchemSearch
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
06d3aeb9d1
Add Softreload cancel
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
81dbcc691a
Add Softreload cancel
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
edb108b7b3
Add catch for IOException
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
f624d2893d
Stop the dog
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
badb5e5e1e
idk
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
b7d42a767a
Merge branch 'master' into schemsearch
Einige Prüfungen sind fehlgeschlagen
SteamWarCI Build failed
afece3008c
Chaoscaot hat 1 Commit 2023-04-12 02:08:13 +02:00 hinzugefügt
Update CC
Einige Prüfungen sind fehlgeschlagen
SteamWarCI Build failed
3bd18531f5
Chaoscaot hat 1 Commit 2023-04-12 02:09:02 +02:00 hinzugefügt
Remove TestCommand
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
ec0c06227f
Lixfel hat 2023-04-13 18:56:28 +02:00 Änderungen angefragt
Lixfel hat einen Kommentar hinterlassen
Besitzer

Muss es mir nochmal im Detail anschauen, konnte bislang nur darüberfliegen.

Muss es mir nochmal im Detail anschauen, konnte bislang nur darüberfliegen.
@ -0,0 +1,22 @@
package de.steamwar.bungeecore.listeners;
Besitzer

License header

License header
Lixfel markierte diese Unterhaltung als gelöst
@ -0,0 +1,28 @@
package de.steamwar.bungeecore.network.handlers;
Besitzer

.

.
Lixfel markierte diese Unterhaltung als gelöst
@ -0,0 +1,303 @@
package de.steamwar.bungeecore.util;
Besitzer

.

.
Lixfel markierte diese Unterhaltung als gelöst
Chaoscaot hat 1 Commit 2023-04-13 19:02:52 +02:00 hinzugefügt
Add Copyright
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
1c54b1c60f
Lixfel hat 2023-05-01 08:03:53 +02:00 Änderungen angefragt
CommonCore Veraltet
@ -1 +1 @@
Subproject commit 595df40e1e9078c310cbbb4bbded07744c361f15
Subproject commit e8d964d8333f353d9995e8319694077e0f8e063b
Besitzer

Der Commit ist schon im CommonCore-Master?

Der Commit ist schon im CommonCore-Master?
@ -0,0 +49,4 @@
public class SchematicSearch {
private static final Gson gson = new Gson();
private static final String searchBinary = "/home/chaoscaot/schemsearch/target/release/schemsearch-cli";
Besitzer

Falscher Pfad.

Falscher Pfad.
@ -0,0 +55,4 @@
private static ScheduledTask watchdog;
private static void startQueueWatchdog() {
watchdog = BungeeCore.get().getProxy().getScheduler().schedule(BungeeCore.get(), () -> {
Besitzer

Warum den Task ständig starten und stoppen und nicht den Thread ständig offen lassen und einfach eventbasiert auf notifyAll() warten lassen?

Warum den Task ständig starten und stoppen und nicht den Thread ständig offen lassen und einfach eventbasiert auf notifyAll() warten lassen?
@ -0,0 +74,4 @@
public static void queueSearch(ProxiedPlayer player, SchematicNode node, SchematicSearchBehavior behavior) {
SteamwarUser user = SteamwarUser.get(player.getUniqueId());
synchronized (searchQueue) {
if(user.getUserGroup().isAdminGroup()) {
Besitzer

Lass solche Spezialberechtigungen raus.

Lass solche Spezialberechtigungen raus.
Lixfel markierte diese Unterhaltung als gelöst
@ -0,0 +135,4 @@
return removed;
}
private static File schematicNodeToTempFile(NodeData node) {
Besitzer

Der Code in der Funktion (und den folgenden Methoden) wirkt sehr C-mäßig geschrieben. Evtl. kann man das nochmal etwas eleganter formulieren.

Der Code in der Funktion (und den folgenden Methoden) wirkt sehr C-mäßig geschrieben. Evtl. kann man das nochmal etwas eleganter formulieren.
@ -0,0 +169,4 @@
this.behavior = behavior;
}
private static String readInputStream(InputStream is) throws IOException {
Besitzer

BufferedInputStream.readline()?

BufferedInputStream.readline()?
Lixfel markierte diese Unterhaltung als gelöst
@ -0,0 +187,4 @@
}
private void start() {
task = BungeeCore.get().getProxy().getScheduler().runAsync(BungeeCore.get(), () -> {
Besitzer

Warum hier zwei verschiedene Tasks und nicht diesen Task mit dem "watchdog" konsolidieren?

Warum hier zwei verschiedene Tasks und nicht diesen Task mit dem "watchdog" konsolidieren?
@ -0,0 +191,4 @@
try {
NodeData data = NodeData.get(node);
if (!data.getNodeFormat()) {
Message.send("SCHEMATIC_SEARCH_NOT_SUPPORTED", player);
Besitzer

Man könnte auch den Grund (altes Format) mit angeben, um dem User zu helfen.

Man könnte auch den Grund (altes Format) mit angeben, um dem User zu helfen.
Lixfel markierte diese Unterhaltung als gelöst
@ -0,0 +203,4 @@
InputStream stderr = process.getErrorStream();
String bar = "";
while (!process.waitFor(200, TimeUnit.MILLISECONDS)) {
Besitzer

Ich weiß nicht, was das harte Timing hier soll, und wie zuverlässig das hier so ist... (und ob das überhaupt nötig ist)

Ich weiß nicht, was das harte Timing hier soll, und wie zuverlässig das hier so ist... (und ob das überhaupt nötig ist)
@ -0,0 +274,4 @@
} catch (IOException | InterruptedException e) {
end();
} catch (Exception e) {
end();
Besitzer

Wie wäre es mit einem finally { end(); }? Und welche ungenannten Exception können da noch auftreten, dass die nochmal separat in einer SecurityException gewrappt werden müssen?

Wie wäre es mit einem finally { end(); }? Und welche ungenannten Exception können da noch auftreten, dass die nochmal separat in einer SecurityException gewrappt werden müssen?
Lixfel markierte diese Unterhaltung als gelöst
@ -121,6 +121,7 @@ MOD_USE_MODSENDER=§cPlease use the §c§lFabricModSender§c (https://steamwar.d
#Various commands
Besitzer

Deutsche Fassung fehlt noch.

Deutsche Fassung fehlt noch.
Lixfel markierte diese Unterhaltung als gelöst
@ -690,0 +693,4 @@
SCHEMATIC_SEARCH_QUEUED=§7Your search has been queued and will be executed shortly.
SCHEMATIC_SEARCH_STARTED=§7Your search for "§e{0}§7" has started.
SCHEMATIC_SEARCH_NO_RESULTS=§cNo results found.
SCHEMATIC_SEARCH_RESULT_HEADER=§7Searched in §e{0} §7schematics in§e{1}.
Besitzer

Fehlt ein Leerzeichen? Und (ka was dabei gesucht wird) ist es grammatikalisch korrekt? (Searched {0} schematics in {1})?

Fehlt ein Leerzeichen? Und (ka was dabei gesucht wird) ist es grammatikalisch korrekt? (Searched {0} schematics in {1})?
Lixfel markierte diese Unterhaltung als gelöst
@ -690,0 +695,4 @@
SCHEMATIC_SEARCH_NO_RESULTS=§cNo results found.
SCHEMATIC_SEARCH_RESULT_HEADER=§7Searched in §e{0} §7schematics in§e{1}.
SCHEMATIC_SEARCH_RESULT=§7{0}: §e{1}§7% §8(§e{2}§7,§e{3},§e{4}§8)
SCHEMATIC_SEARCH_RESULT_HOVER=§7Click to get more info about {0}.
Besitzer

for more info?

for more info?
Lixfel markierte diese Unterhaltung als gelöst
@ -690,0 +698,4 @@
SCHEMATIC_SEARCH_RESULT_HOVER=§7Click to get more info about {0}.
SCHEMATIC_SEARCH_NOT_SUPPORTED=§cThis schematic is not supported by the schematic search.
SCHEMATIC_SEARCH_REMOVED_FROM_QUEUE=§cYour search has been removed from the queue because you switched servers.
SCHEMATIC_SEARCH_REMOVED_FROM_QUEUE_SOFTRELOAD=§cYour search has been removed from the queue because of an update.
Besitzer

software update

software update
Lixfel markierte diese Unterhaltung als gelöst
@ -690,0 +699,4 @@
SCHEMATIC_SEARCH_NOT_SUPPORTED=§cThis schematic is not supported by the schematic search.
SCHEMATIC_SEARCH_REMOVED_FROM_QUEUE=§cYour search has been removed from the queue because you switched servers.
SCHEMATIC_SEARCH_REMOVED_FROM_QUEUE_SOFTRELOAD=§cYour search has been removed from the queue because of an update.
SCHEMATIC_SEARCH_CANCELED_SOFTRELOAD=§cYour search has been canceled because of an update.
Besitzer

cancelled?

cancelled?
Lixfel markierte diese Unterhaltung als gelöst
Chaoscaot hat 1 Commit 2023-05-01 13:53:57 +02:00 hinzugefügt
Merge branch 'master' into schemsearch
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
d9ad5bd14a
Chaoscaot hat 1 Commit 2023-05-01 16:12:53 +02:00 hinzugefügt
Fix SchematicSearch
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
3b0e4aa5aa
Lixfel hat 2023-05-23 21:46:17 +02:00 überprüft
Lixfel hat einen Kommentar hinterlassen
Besitzer

Muss ich mir nochmal in Ruhe genauer ansehen.

Muss ich mir nochmal in Ruhe genauer ansehen.
@ -651,0 +657,4 @@
SCHEMATIC_SEARCH_RESULT_HOVER=§7Klicke für mehr Informationen
SCHEMATIC_SEARCH_NOT_SUPPORTED=§cDiese Schematic ist in einem alten Format und kann nicht genutzt werden.
SCHEMATIC_SEARCH_REMOVED_FROM_QUEUE=§cDeine Suche wurde aus der Warteschlange entfernt, weil du den Server gewechselt hast.
SCHEMATIC_SEARCH_REMOVED_FROM_QUEUE_SOFTRELOAD=§cWegen eines Software Updates wurde deine Suche aus der Warteschlange entfernt.
Besitzer

Deutsch Softwareupdate statt Software Update

Deutsch Softwareupdate statt Software Update
Chaoscaot hat den Titel von Add SchemSearch zu WIP: Add SchemSearch 2023-09-18 20:27:41 +02:00 geändert
Dieses Repo ist archiviert. Du kannst Pull-Requests nicht kommentieren.
Keine Beschreibung angegeben.