SteamWar/BauSystem2.0
Archiviert
12
0

TestBlockPaste-Without-Water #146

Zusammengeführt
YoyoNow hat 25 Commits von TestBlockPaste-Without-Water nach master 2023-08-19 23:29:20 +02:00 zusammengeführt
Besitzer
Keine Beschreibung angegeben.
YoyoNow hat 3 Commits 2023-02-04 11:57:32 +01:00 hinzugefügt
Prevent Water from being pasted using /tb <schem>
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
8a425cc435
fixed mistake
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
3d8a6e95fa
fixed mistake 2
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
cdc0dbea86
Hytralium hat 2 Commits 2023-02-04 12:15:55 +01:00 hinzugefügt
Lixfel hat 2023-02-04 19:57:47 +01:00 Änderungen angefragt
@ -175,0 +159,4 @@
public boolean test(BlockVector3 blockVector3) {
BaseBlock block = clipboard.getFullBlock(blockVector3);
String blockName = block.toString().toLowerCase();
if (pasteOptions.isTestBlock() && blockName.equals("minecraft.water")) return true;
Besitzer

Waterlogged-Blöcke werden ebenso Probleme machen. Da könnte man auf ein contains waterlogged=true machen oder ggf. statt über Strings auch direkt über die WorldEdit-Materials/-Blocks, aber das wäre ein größerer Umbau.

Waterlogged-Blöcke werden ebenso Probleme machen. Da könnte man auf ein contains waterlogged=true machen oder ggf. statt über Strings auch direkt über die WorldEdit-Materials/-Blocks, aber das wäre ein größerer Umbau.
zOnlyKroks markierte diese Unterhaltung als gelöst
Hytralium hat 1 Commit 2023-02-06 09:37:42 +01:00 hinzugefügt
finally working waterremover when doing /tb idk
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
4bae6b8d43
Hytralium hat 1 Commit 2023-02-06 21:38:18 +01:00 hinzugefügt
Lixfel hat 2023-02-08 22:43:31 +01:00 Änderungen angefragt
Lixfel hat einen Kommentar hinterlassen
Besitzer

Es werden immer noch keine Waterlogged-Blöcke erkannt.

Zudem: Wenn das Feature als fertig gilt, sollte man das WIP vom PR-Namen entfernen.

Es werden immer noch keine Waterlogged-Blöcke erkannt. Zudem: Wenn das Feature als fertig gilt, sollte man das WIP vom PR-Namen entfernen.
Autor
Besitzer

Hytralium muss diesen PR neu aufmachen weil ich das nur haben wollte um den diff zu sehen. Und ansonsten ist er noch nicht fertig weil genau water logged blöcke fehlen.

Hytralium muss diesen PR neu aufmachen weil ich das nur haben wollte um den diff zu sehen. Und ansonsten ist er noch nicht fertig weil genau water logged blöcke fehlen.
Hytralium hat 1 Commit 2023-02-16 21:54:01 +01:00 hinzugefügt
sollte jetzt auch mit waterlogged blocks gehen
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
5d55784f79
YoyoNow hat 2023-02-18 11:12:41 +01:00 überprüft
@ -42,10 +42,12 @@ import com.sk89q.worldedit.regions.selector.CuboidRegionSelector;
import com.sk89q.worldedit.session.ClipboardHolder;
import com.sk89q.worldedit.world.World;
import com.sk89q.worldedit.world.block.BaseBlock;
import com.sk89q.worldedit.world.block.BlockState;
Autor
Besitzer

Scheint mir nicht genutzt zu sein.

Scheint mir nicht genutzt zu sein.
zOnlyKroks markierte diese Unterhaltung als gelöst
@ -46,3 +47,4 @@
import de.steamwar.bausystem.region.Color;
import de.steamwar.bausystem.region.PasteOptions;
import de.steamwar.bausystem.region.Point;
import net.minecraft.server.v1_15_R1.AreaFactory;
Autor
Besitzer

Dieser Import bitte raus!

Dieser Import bitte raus!
zOnlyKroks markierte diese Unterhaltung als gelöst
@ -147,0 +146,4 @@
Set<String> blocks = new HashSet<>();
{
Autor
Besitzer

Warum schreibst das hier immer rein und machst es nicht optional anhand von dem color flag?

Warum schreibst das hier immer rein und machst es nicht optional anhand von dem color flag?
zOnlyKroks markierte diese Unterhaltung als gelöst
@ -162,0 +169,4 @@
if (blocks.contains(blockName) && pasteOptions.isOnlyColors()) return true;
if (pasteOptions.isTestBlock()) {
if (blockName.startsWith("minecraft:water")) return false;
if (block.toString().toLowerCase().contains("waterlogged=true")) return false;
Autor
Besitzer

Hier vllt nicht sinnvoll das man einfach die Blöcke nicht pastet, eher dann einfach nicht waterlogged pasted.

Hier vllt nicht sinnvoll das man einfach die Blöcke nicht pastet, eher dann einfach nicht waterlogged pasted.
zOnlyKroks markierte diese Unterhaltung als gelöst
@ -58,3 +58,3 @@
if (region == null) return;
try {
region.reset(RegionType.TESTBLOCK, regionExtensionType);
region.reset(RegionType.TESTBLOCK, regionExtensionType, true);
Autor
Besitzer

Hier die Daten durchzureichen ist nicht sinnvoll, da RegionType.TESTBLOCK ja doch schon sagt das es der Testblock ist.

Hier die Daten durchzureichen ist nicht sinnvoll, da RegionType.TESTBLOCK ja doch schon sagt das es der Testblock ist.
zOnlyKroks markierte diese Unterhaltung als gelöst
@ -87,3 +87,3 @@
try {
region.reset(node, RegionType.TESTBLOCK, regionExtensionType);
region.reset(node, RegionType.TESTBLOCK, regionExtensionType, true);
Autor
Besitzer

Siehe oben

Siehe oben
zOnlyKroks markierte diese Unterhaltung als gelöst
@ -68,2 +68,4 @@
*/
private final int waterLevel;
private final boolean testBlock;
Autor
Besitzer

Hier könntest du die ganzen Used in 1.15 Kommentare gleich mit löschen.

Hier könntest du die ganzen `Used in 1.15` Kommentare gleich mit löschen.
zOnlyKroks markierte diese Unterhaltung als gelöst
@ -423,3 +423,3 @@
public void reset(RegionType regionType, RegionExtensionType regionExtensionType) throws IOException {
reset(null, regionType, regionExtensionType, false);
reset(null, regionType, regionExtensionType);
Autor
Besitzer

Muss diese Änderung eigentlich wirklich sein?

Muss diese Änderung eigentlich wirklich sein?
zOnlyKroks markierte diese Unterhaltung als gelöst
@ -448,3 +448,3 @@
}
PasteOptions pasteOptions = new PasteOptions((schematic != null && (schematic.getSchemtype().fightType() || schematic.getSchemtype().check())), ignoreAir, getPlain(Flag.COLOR, ColorMode.class).getColor(), onlyColors, regionExtensionType == RegionExtensionType.EXTENSION, getMinPoint(regionType, regionExtensionType), getMaxPoint(regionType, regionExtensionType), waterLevel);
PasteOptions pasteOptions = new PasteOptions((schematic != null && (schematic.getSchemtype().fightType() || schematic.getSchemtype().check())), ignoreAir, getPlain(Flag.COLOR, ColorMode.class).getColor(), onlyColors, regionExtensionType == RegionExtensionType.EXTENSION, getMinPoint(regionType, regionExtensionType), getMaxPoint(regionType, regionExtensionType), waterLevel, isTestBlock);
Autor
Besitzer

hier dann einfach den isTextBlock Boolean zu einem regionType == RegionType.TESTBLOCK ersetzten, dann brauchst du die ganzen booleans nicht.

hier dann einfach den `isTextBlock` Boolean zu einem `regionType == RegionType.TESTBLOCK` ersetzten, dann brauchst du die ganzen booleans nicht.
zOnlyKroks markierte diese Unterhaltung als gelöst
Hytralium hat 1 Commit 2023-02-18 13:41:08 +01:00 hinzugefügt
zOnlyKroks hat 1 Commit 2023-07-27 01:09:00 +02:00 hinzugefügt
Add option for tnt less pasting
Einige Prüfungen sind fehlgeschlagen
SteamWarCI Build failed
1fe2394e01
zOnlyKroks hat 1 Commit 2023-07-27 01:09:54 +02:00 hinzugefügt
fix logic flaw
Einige Prüfungen sind fehlgeschlagen
SteamWarCI Build failed
9ad455ea2d
zOnlyKroks hat 1 Commit 2023-07-27 15:43:59 +02:00 hinzugefügt
Use parameters
Einige Prüfungen sind fehlgeschlagen
SteamWarCI Build failed
edfc331d03
zOnlyKroks hat 1 Commit 2023-07-28 21:42:03 +02:00 hinzugefügt
Implement Testblock Command
Einige Prüfungen sind fehlgeschlagen
SteamWarCI Build failed
d8b93e976f
zOnlyKroks hat 2 Commits 2023-07-28 21:52:11 +02:00 hinzugefügt
# Conflicts:
#	BauSystem_Main/src/de/steamwar/bausystem/features/region/TestblockCommand.java
Make things actually build
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
8a01394092
YoyoNow hat 1 Commit 2023-07-28 22:16:59 +02:00 hinzugefügt
Fix TestblockCommand
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
ad55b25cf0
Signed-off-by: yoyosource <yoyosource@nidido.de>
YoyoNow hat 1 Commit 2023-07-28 22:54:23 +02:00 hinzugefügt
Fix TestblockCommand
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
b02eda9fe8
Signed-off-by: yoyosource <yoyosource@nidido.de>
YoyoNow hat 1 Commit 2023-07-29 12:47:52 +02:00 hinzugefügt
Add PasteBuilder as the new alternative to pasting via region.reset
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
814a5f4364
Signed-off-by: yoyosource <yoyosource@nidido.de>
YoyoNow hat 1 Commit 2023-07-30 17:39:55 +02:00 hinzugefügt
Merge branch 'master' into TestBlockPaste-Without-Water
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
c398ad239d
YoyoNow hat 1 Commit 2023-07-30 21:02:38 +02:00 hinzugefügt
Update PasteBuilder and usages
Einige Prüfungen sind fehlgeschlagen
SteamWarCI Build failed
3e27a0b366
Signed-off-by: yoyosource <yoyosource@nidido.de>
YoyoNow hat 1 Commit 2023-07-30 21:04:10 +02:00 hinzugefügt
Hotfix ResetCommand
Einige Prüfungen sind fehlgeschlagen
SteamWarCI Build failed
fc03ba67e5
Signed-off-by: yoyosource <yoyosource@nidido.de>
YoyoNow hat 1 Commit 2023-07-30 21:14:19 +02:00 hinzugefügt
Fix stuff
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
7f3c0d2fd0
Signed-off-by: yoyosource <yoyosource@nidido.de>
YoyoNow hat 1 Commit 2023-08-16 21:19:20 +02:00 hinzugefügt
Remove unused api and cleanup rest
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
ca907a7478
Signed-off-by: yoyosource <yoyosource@nidido.de>
YoyoNow hat 1 Commit 2023-08-19 23:25:58 +02:00 hinzugefügt
Fix paste problems
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
ed862ba6b2
Signed-off-by: yoyosource <yoyosource@nidido.de>
YoyoNow hat den Titel von WIP: TestBlockPaste-Without-Water zu TestBlockPaste-Without-Water 2023-08-19 23:27:45 +02:00 geändert
YoyoNow hat 1 Commit 2023-08-19 23:29:11 +02:00 hinzugefügt
Merge branch 'master' into TestBlockPaste-Without-Water
Alle Prüfungen waren erfolgreich
SteamWarCI Build successful
eec5e97842
YoyoNow hat Commit dce679afa2 in master 2023-08-19 23:29:20 +02:00 gemerged
YoyoNow löschte die Branch TestBlockPaste-Without-Water 2023-08-19 23:29:20 +02:00
Dieses Repo ist archiviert. Du kannst Pull-Requests nicht kommentieren.
Keine Beschreibung angegeben.