SteamWar/BauSystem
Archiviert
13
0

Add variable reference support to internal commands #207

Manuell gemergt
YoyoNow hat 7 Commits von ScriptSystemImprovements nach master 2021-03-24 17:30:00 +01:00 zusammengeführt
Besitzer

Add arithmetic commands
Add auto_trace constant

Add arithmetic commands Add auto_trace constant
YoyoNow hat 1 Commit 2021-03-13 14:44:15 +01:00 hinzugefügt
Add arithmetic commands
Add auto_trace constant
YoyoNow hat ein Review von Chaoscaot 2021-03-13 14:44:44 +01:00 angefragt
YoyoNow hat 1 Commit 2021-03-13 15:06:47 +01:00 hinzugefügt
Add arithmetic commands
Add auto_trace constant
Chaoscaot hat 2021-03-16 00:35:36 +01:00 Änderungen angefragt
@ -45,2 +47,4 @@
}
public boolean isAutoTrace() {
return autoTrace;
Besitzer

return this == RECORD_AUTO || this == IDLE_AUTO

return this == RECORD_AUTO || this == IDLE_AUTO
YoyoNow markierte diese Unterhaltung als gelöst
@ -37,6 +37,8 @@ import org.bukkit.inventory.ItemStack;
import org.bukkit.inventory.meta.BookMeta;
import java.util.*;
import java.util.function.Function;
Besitzer

Unused Import

Unused Import
YoyoNow markierte diese Unterhaltung als gelöst
@ -171,3 +182,2 @@
private static String[] generateArgumentArray(String command, String fullCommand) {
return fullCommand.substring(command.length()).trim().split(" ");
private static String[] generateArgumentArray(String command, ScriptExecutor scriptExecutor, String fullCommand) {
Besitzer

Die ganz Function Chain könnte man nicht static machen, dann müsste man auch den ScriptExecutor nicht als dürch geben.

Die ganz Function Chain könnte man nicht static machen, dann müsste man auch den ScriptExecutor nicht als dürch geben.
Autor
Besitzer

Ich würde dies nicht machen. und lieber durchreichen.

Ich würde dies nicht machen. und lieber durchreichen.
YoyoNow markierte diese Unterhaltung als gelöst
@ -269,1 +275,3 @@
scriptExecutor.variables.put(key, parseValue(value));
private static void arithmeticCommand(ScriptExecutor scriptExecutor, String[] args, IntBinaryOperator operation) {
if (args.length < 1) {
scriptExecutor.player.sendMessage(BauSystem.PREFIX + "§cDas erste Argument ist eine Variable");
Besitzer

Wass soll mir das jetzt sagen?

Wass soll mir das jetzt sagen?
YoyoNow markierte diese Unterhaltung als gelöst
@ -270,0 +278,4 @@
return;
}
if (args.length < 2) {
scriptExecutor.player.sendMessage(BauSystem.PREFIX + "§cDas zweite Argument ist eine Zahl oder Variable");
Besitzer

Ja, das zweite Argument ist eine Zahl oder Variable, eventuel

Ja, das zweite Argument ist eine Zahl oder Variable, eventuel
YoyoNow markierte diese Unterhaltung als gelöst
@ -295,6 +338,7 @@ public class ScriptListener implements Listener {
private static boolean isVariable(ScriptExecutor scriptExecutor, String key) {
switch (key) {
case "trace":
case "auto_trace":
Besitzer

mit unterstrichen in lowercase Text fangen wir hier ambesten garnicht an, dass schaut einfach nicht schön aus.

mit unterstrichen in lowercase Text fangen wir hier ambesten garnicht an, dass schaut einfach nicht schön aus.
YoyoNow markierte diese Unterhaltung als gelöst
YoyoNow hat 1 Commit 2021-03-19 22:28:05 +01:00 hinzugefügt
Fix RecordStatus
YoyoNow hat 1 Commit 2021-03-19 22:41:36 +01:00 hinzugefügt
YoyoNow hat 1 Commit 2021-03-20 12:31:22 +01:00 hinzugefügt
Add Input to ScriptListener
YoyoNow hat ein Review von Chaoscaot 2021-03-20 12:38:15 +01:00 angefragt
YoyoNow hat 2 Commits 2021-03-20 12:45:28 +01:00 hinzugefügt
# Conflicts:
#	BauSystem_Main/src/de/steamwar/bausystem/world/ScriptListener.java
Chaoscaot hat das Review verweigert 2021-03-21 12:00:25 +01:00
Chaoscaot hat die Änderungen 2021-03-23 01:41:08 +01:00 genehmigt
YoyoNow hat Commit ab81466a33 in master 2021-03-24 17:30:00 +01:00 manuell gemerged
YoyoNow löschte die Branch ScriptSystemImprovements 2021-03-24 17:30:17 +01:00
Dieses Repo ist archiviert. Du kannst Pull-Requests nicht kommentieren.
Keine Beschreibung angegeben.