SteamWar/SteamWar
15
4
Fork 5
Code Issues 85 Pull Requests 14 Actions Packages Projects 2 Activity

Fix 1.19 material in 1.15 laufbau #487

Open
Hut512 wants to merge 3 commits from Hut512/SteamWar:version/legacy into version/legacy
pull from: Hut512/SteamWar:version/legacy
merge into: :version/legacy
Conversation 5 Commits 3 Files Changed 5
+13 -4
Hut512 commented 2026-09-04 00:16:24 +02:00
Contributor
Copy Link
Copy Source

BlockBoundingBox uses Material.MUD, which does not exist in 1.15. As a result, the plugin crashes on this version. @MinVersion(19) in LaufbauCommand is not sufficient because linked classes like BellBoundingBox also require BlockBoundingBox.
Potential fix for #467 #471 #472

`BlockBoundingBox` uses `Material.MUD`, which does not exist in 1.15. As a result, the plugin crashes on this version. `@MinVersion(19)` in `LaufbauCommand` is not sufficient because linked classes like `BellBoundingBox` also require `BlockBoundingBox`. Potential fix for #467 #471 #472
BauSystem/BauSystem_Main/src/de/steamwar/bausystem/features/slaves/laufbau/BlockBoundingBox.java Outdated
@@ -102,3 +102,3 @@
addPixel(NMSWrapper.impl.pathMaterial().createBlockData(), 0, 0, 0, 16, 15, 16, createItem("LAUFBAU_BLOCK_GRASS_PATH", NMSWrapper.impl.pathMaterial()));
addPixel(Material.MUD.createBlockData(), 0, 0, 0, 16, 14, 16, createItem("LAUFBAU_BLOCK_SOUL_SAND", Material.SOUL_SAND));
addPixel(Material.SOUL_SAND.createBlockData(), 0, 0, 0, 16, 14, 16, createItem("LAUFBAU_BLOCK_SOUL_SAND", Material.SOUL_SAND));
YoyoNow commented 2026-09-05 10:43:33 +02:00
Owner
Copy Link
Copy Source

SOUL_SAND sollte dann aber nur genutzt werden wenn wir nicht 1.19 oder neuer haben

SOUL_SAND sollte dann aber nur genutzt werden wenn wir nicht 1.19 oder neuer haben
Hut512 commented 2026-09-07 15:55:34 +02:00
Author
Contributor
Copy Link
Copy Source

Bei END_ROD und LIGHTNING_ROD wird es genauso gemacht. Der Laufbau sortiert dann ggf. nach Härte.

Bei END_ROD und LIGHTNING_ROD wird es genauso gemacht. Der Laufbau sortiert dann ggf. nach Härte.
Hut512 added 1 commit 2026-09-07 15:51:11 +02:00
Fix 1.19 material in 1.15 laufbau
Pull Request Build / Build (pull_request) Failing after 1m28s
Details
564a1ceaab
Hut512 force-pushed version/legacy from 1dc25e94af to 564a1ceaab 2026-09-07 15:51:11 +02:00 Compare
Hut512 added 1 commit 2026-09-11 14:12:11 +02:00
Fix Laufbau MUD SWItem
Pull Request Build / Build (pull_request) Failing after 4m51s
Details
fbdf2c888f
YoyoNow commented 2026-09-14 13:16:31 +02:00
Owner
Copy Link
Copy Source

Problem hier ist, dass die Pipeline für die Version es wohl nicht mehr bauen kann. Da muss erstmal was für gemacht werden, damit das wieder tut. Ich sehe aber noch nicht genau wie ich das anstellen könnte.

Problem hier ist, dass die Pipeline für die Version es wohl nicht mehr bauen kann. Da muss erstmal was für gemacht werden, damit das wieder tut. Ich sehe aber noch nicht genau wie ich das anstellen könnte.
Hut512 commented 2026-09-14 20:34:30 +02:00
Author
Contributor
Copy Link
Copy Source

Hättest du da einen Build Log? Ich würde dem Fehler gerne auf den Grund gehen. Der CI build von diesem PR scheitert ja leider daran, dass Fork-PRs auf Gitea keine Secrets bekommen. Und lokal ohne die privaten Dependencies kann ich das nicht reproduzieren.

Hättest du da einen Build Log? Ich würde dem Fehler gerne auf den Grund gehen. Der CI build von diesem PR scheitert ja leider daran, dass Fork-PRs auf Gitea keine Secrets bekommen. Und lokal ohne die privaten Dependencies kann ich das nicht reproduzieren.
YoyoNow commented 2026-09-14 22:49:15 +02:00
Owner
Copy Link
Copy Source

Hier kannst du es dir anschauen (glaube ich):
https://git.steamwar.de/SteamWar/SteamWar/src/branch/version/legacy/
BZW.: https://git.steamwar.de/SteamWar/SteamWar/actions/runs/507/jobs/711

Bauen tut es ja auch local. Also die Dependencies sind da definitiv nicht das Problem.

Hier kannst du es dir anschauen (glaube ich): https://git.steamwar.de/SteamWar/SteamWar/src/branch/version/legacy/ BZW.: https://git.steamwar.de/SteamWar/SteamWar/actions/runs/507/jobs/711 Bauen tut es ja auch local. Also die Dependencies sind da definitiv nicht das Problem.
Hut512 added 1 commit 2026-09-21 22:48:52 +02:00
Fix deploy with temporary directory
Pull Request Build / Build (pull_request) Failing after 11m38s
Details
f8fbe4c39b
Hut512 commented 2026-09-21 22:53:45 +02:00
Author
Contributor
Copy Link
Copy Source

Das ist der Fix aus main. Passt das so?

Das ist der Fix aus main. Passt das so?
Some required checks failed
Pull Request Build / Build (pull_request) Failing after 11m38s
Required
Details
Some required checks were not successful.
This pull request doesn't have enough required approvals yet. 0 of 1 official approvals granted.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u https://git.steamwar.de/Hut512/SteamWar version/legacy:Hut512-version/legacy
git checkout Hut512-version/legacy
Sign in to join this conversation.
Reviewers
No Reviewers
YoyoNow
Labels
Clear labels
Version/1.12
Version/1.15
Version/1.19
Version/1.20
Version/1.21
no-backport
Disable automatic CommonCore backporting for this pull request.
System
Backend
System
BauSystem
System
CommandFramework
System
CommonCore
System
Discord
System
FightSystem
System
MissleWars
System
Multi
Changing multiple Systems
System
SchematicSystem
System
SpigotCore
System
TNTLeague
System
Teamserver
System
TowerRun
System
TutorialSystem
System
VelocityCore
Typ
Bug
Typ
Idee
Typ
Maintanance
Wartung
Typ
Translation
sev
1
1
sev
2
2
sev
3
3
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
Chaoscaot D4rkr34lm DreamJxnas (der.coronas) Haylim (haylim) JajaKings (fncbeispielkind07) Lixfel PsiRobot Sakziea (sakziea) steamwar-ci-bot (SteamWar CI Bot) Website YoyoNow zOnlyKroks
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SteamWar/SteamWar#487