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

added Anti Player Loot despawn #16

Merged
MineTec merged 6 commits from feature/AntiPlayerLootDespawn into master 2026-10-14 10:31:29 +00:00
Conversation 8 Commits 6 Files Changed 2
+2 -3
2 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 5b7910b697 - Show all commits

No files matched your search

craftattack/src/main/java/eu/mhsl/craftattack/spawn/craftattack/appliances/metaGameplay/antiPlayerLootDespawn/AntiPlayerLootDespawn.java
+1 -2
View File
1
 @@ -24,7 +24,7 @@ public class AntiPlayerLootDespawn extends Appliance {
); );
} }
MineTec marked this conversation as resolved
MineTec commented 2026-10-13 22:43:40 +00:00
Review
Copy link
Copy source

Wenn das Markieren von Items hier als Logik in der Appliance liegt, sollte das testen auf markierte Items auch hier liegen. Dann ist die Logik "was ist ein markiertes Item?" hier zusammen und nicht geteilt.

Sowas wie boolean shouldDelete(ItemStack stack) {}
Dann wird hier auch der counter berechnet und das event bricht dann nur anhand des booleans ab.

Wenn das Markieren von Items hier als Logik in der Appliance liegt, sollte das testen auf markierte Items auch hier liegen. Dann ist die Logik "was ist ein markiertes Item?" hier zusammen und nicht geteilt. Sowas wie `boolean shouldDelete(ItemStack stack) {}` Dann wird hier auch der counter berechnet und das event bricht dann nur anhand des booleans ab.
public boolean AboveMaxAge(Item item) { public boolean isItemAboveMaxAge(Item item) {
MineTec marked this conversation as resolved Outdated
MineTec commented 2026-10-14 10:27:37 +00:00
Outdated
Review
Copy link
Copy source

methoden werden kleingeschrieben
isItemAboveMaxAge
ich hatte nur das Max noch einfügen wollen, nicht das 'isItem' streichen.
Ich war halt am handy.

methoden werden kleingeschrieben `isItemAboveMaxAge` ich hatte nur das Max noch einfügen wollen, nicht das 'isItem' streichen. Ich war halt am handy.
ItemStack itemStack = item.getItemStack(); ItemStack itemStack = item.getItemStack();
int cycles = itemStack.getPersistentDataContainer() int cycles = itemStack.getPersistentDataContainer()
@@ -35,7 +35,6 @@ public class AntiPlayerLootDespawn extends Appliance {
); );
item.setItemStack(itemStack); item.setItemStack(itemStack);
return cycles > 0; return cycles > 0;
} }
craftattack/src/main/java/eu/mhsl/craftattack/spawn/craftattack/appliances/metaGameplay/antiPlayerLootDespawn/AntiPlayerLootDespawnListener.java
+1 -1
View File
@@ -14,6 +14,6 @@ class AntiPlayerLootDespawnListener extends ApplianceListener<AntiPlayerLootDesp
@EventHandler @EventHandler
public void onItemDespawn(ItemDespawnEvent event) { public void onItemDespawn(ItemDespawnEvent event) {
MineTec marked this conversation as resolved Outdated
MineTec commented 2026-10-13 20:09:34 +00:00
Outdated
Review
Copy link
Copy source

die klasse kann package-private sein, dann sieht nur die Appliance den Listener, und nicht das ganze Projekt. Dazu einfach das Public ersatzlos wegnehmen.

die klasse kann package-private sein, dann sieht nur die Appliance den Listener, und nicht das ganze Projekt. Dazu einfach das Public ersatzlos wegnehmen.
event.setCancelled(this.getAppliance().AboveMaxAge(event.getEntity())); event.setCancelled(this.getAppliance().isItemAboveMaxAge(event.getEntity()));
MineTec marked this conversation as resolved Outdated
MineTec commented 2026-10-13 20:08:55 +00:00
Outdated
Review
Copy link
Copy source

alle Konstanten zur appliance, nicht in den listener

alle Konstanten zur appliance, nicht in den listener
} }
} }