CraftAttack/spawn
5
0
Fork 0
Code Issues 3 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

bot

bot
Pupsi added 2 commits 2024-09-29 16:47:45 +00:00
MineTec reviewed 2024-09-29 16:49:45 +00:00
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

?

?
Pupsi marked this conversation as resolved
MineTec reviewed 2024-09-29 16:51:00 +00:00
MineTec reviewed 2024-09-29 16:52:19 +00:00
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

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

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
MineTec reviewed 2024-09-29 16:53:08 +00:00
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

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
MineTec reviewed 2024-09-29 16:56:36 +00:00
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

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
MineTec reviewed 2024-09-29 16:57:35 +00:00
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

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

sieht gut aus :)

sieht gut aus :)
Pupsi added 1 commit 2024-10-05 16:37:13 +00:00
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 Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
bytedream goldbaja MineTec olischma Pupsi
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: CraftAttack/spawn#4
No description provided.
Delete Branch "master-customAdvancements"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?