CraftAttack/spawn
6
0
Fork 0
Code Issues 5 Pull Requests Packages Projects Releases Wiki Activity

master-customAdvancements #4

Merged
MineTec merged 3 commits from master-customAdvancements into master 2024-10-05 16:40:52 +00:00
Conversation 8 Commits 3 Files Changed 8
+133 -25
Pupsi commented 2024-09-29 16:47:44 +00:00
Member
Copy Link
Copy Source

bot

bot
Pupsi added 2 commits 2024-09-29 16:47:45 +00:00
added fleischerchest and pixelblock advancements 1aebae6cd5
added more custom advancements, added pending advancements da33e6e592
src/main/java/eu/mhsl/craftattack/spawn/appliances/customAdvancements/listener/CustomAdvancementsListener.java Outdated
@@ -0,0 +30,4 @@
if(!(event.getView().getPlayer() instanceof Player player)) return;
if(result.getType() == Material.RED_SHULKER_BOX) {
// getAppliance().grantAdvancement(Advancements.fleischerchest, player.getUniqueId());
MineTec commented 2024-09-29 16:49:45 +00:00
Owner
Copy Link
Copy Source

?

?
Pupsi marked this conversation as resolved
src/main/java/eu/mhsl/craftattack/spawn/appliances/customAdvancements/CustomAdvancements.java Outdated
@@ -16,0 +34,4 @@
Advancement advancement = Objects.requireNonNull(Bukkit.getAdvancement(namespacedKey));
player.getAdvancementProgress(advancement).awardCriteria("criteria");
} catch (Exception e) {
Main.logger().info("Custom Advancements Datapack not found!");
MineTec commented 2024-09-29 16:52:19 +00:00
Owner
Copy Link
Copy Source

die Fehlermeldung kann irreführend sein. Es sollte zumindest im Fehler ersichtlich sein welches advancement fehlt.

die Fehlermeldung kann irreführend sein. Es sollte zumindest im Fehler ersichtlich sein welches advancement fehlt.
MineTec commented 2024-09-29 17:05:45 +00:00
Owner
Copy Link
Copy Source

mein Vorschlag hier wäre du throwst nach dem custom fehler einfach nochmal die exception throw e und packst in die Objects.requireNonNull das angefragte rein. Dann ist alles immer ersichtlich.

mein Vorschlag hier wäre du throwst nach dem custom fehler einfach nochmal die exception `throw e` und packst in die Objects.requireNonNull das angefragte rein. Dann ist alles immer ersichtlich.
Pupsi marked this conversation as resolved
src/main/java/eu/mhsl/craftattack/spawn/appliances/customAdvancements/CustomAdvancements.java
@@ -16,0 +46,4 @@
grantedAdvancements.forEach(pendingAdvancement -> grantAdvancement(pendingAdvancement.advancement(), player.getUniqueId()));
}
public void addPendingAdvancement(UUID receiver, String advancement) {
MineTec commented 2024-09-29 16:53:08 +00:00
Owner
Copy Link
Copy Source

sollte private sein, wenn grantAdvancement entscheidet ob es pending ist oder nicht

sollte private sein, wenn grantAdvancement entscheidet ob es pending ist oder nicht
Pupsi marked this conversation as resolved
src/main/java/eu/mhsl/craftattack/spawn/appliances/customAdvancements/listener/CustomAdvancementsListener.java Outdated
@@ -0,0 +34,4 @@
getAppliance().addPendingAdvancement(player.getUniqueId(), Advancements.fleischerchest);
} else if(
result.getItemMeta().itemName().equals(Component.text("98fdf0ae-c3ab-4ef7-ae25-efd518d600de"))
&& result.getItemMeta().getEnchantmentGlintOverride()
MineTec commented 2024-09-29 16:56:36 +00:00
Owner
Copy Link
Copy Source

der check nach enchantmentGlintOverride raus, itemName reicht aus

(ggf kommt das glint raus, da zurzeit ein bugs mit overrideGlint auf köpfen besteht)

der check nach enchantmentGlintOverride raus, itemName reicht aus (ggf kommt das glint raus, da zurzeit ein bugs mit overrideGlint auf köpfen besteht)
Pupsi marked this conversation as resolved
src/main/java/eu/mhsl/craftattack/spawn/appliances/customAdvancements/listener/CustomAdvancementsListener.java Outdated
@@ -0,0 +32,4 @@
if(result.getType() == Material.RED_SHULKER_BOX) {
// getAppliance().grantAdvancement(Advancements.fleischerchest, player.getUniqueId());
getAppliance().addPendingAdvancement(player.getUniqueId(), Advancements.fleischerchest);
} else if(
MineTec commented 2024-09-29 16:57:35 +00:00
Owner
Copy Link
Copy Source

das else if konstrukt ist nicht schön...

ich tendiere sogar für jeweils einen eigenen handler pro advancement...

vielleicht einfach zwei einzelstehnede ifs mit quasi early return?

das else if konstrukt ist nicht schön... ich tendiere sogar für jeweils einen eigenen handler pro advancement... vielleicht einfach zwei einzelstehnede ifs mit quasi early return?
Pupsi marked this conversation as resolved
MineTec requested changes 2024-09-29 17:03:36 +00:00
Dismissed
MineTec left a comment
Owner
Copy Link
Copy Source

sieht gut aus :)

sieht gut aus :)
Pupsi added 1 commit 2024-10-05 16:37:13 +00:00
solved pr comments e3b07aa62f
MineTec approved these changes 2024-10-05 16:40:39 +00:00
MineTec merged commit 346847d2b2 into master 2024-10-05 16:40:52 +00:00
MineTec deleted branch master-customAdvancements 2024-10-05 16:40:52 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
MineTec
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
besiece bytedream (Ruben Otto) goldbaja (Jannik Goldbach) jannis MineTec (Elias Müller) Pupsi (Lars Neuhaus)
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: CraftAttack/spawn#4