NPC drops with Ice Barrage

Hey guys,

Something that I am struggling to do on a Ruse Base is with Ice Barrage or actually any Spells that hit inMulti
Right now, I've got an area where there are Chickens that are easy to 1 hit as they've only got 1 hp, but the thing is only the Chicken that I click on when casting the spell will drop loot such as bones.
Every other chicken that dies from the Area of Effect from a Barrage spell doesn't drop anything.

What I've done so far is just played around with CombatAncientSpell.java where in finishCast as it mentions hitting the NPCs in multi.
This is the code that I am working with, it's pretty much in every Ruse base.

Code:

---

Iterator<? extends Character> it = null;
                if (cast.isPlayer() && castOn.isPlayer()) {
                    it = ((Player) cast).getLocalPlayers().iterator();
                } else if (cast.isPlayer() && castOn.isNpc()) {
                    it = ((Player) cast).getLocalNpcs().iterator();
                } else if (cast.isNpc() && castOn.isNpc()) {
                    it = World.getNpcs().iterator();
                } else if (cast.isNpc() && castOn.isPlayer()) {
                    it = World.getPlayers().iterator();
                }

                for (Iterator<? extends Character> $it = it; $it.hasNext();) {
                    Character next = $it.next();

                    if (next == null) {
                        continue;
                    }
                   
                    if(next.isNpc()) {
                            NPC n = (NPC)next;
                            if(!n.getDefinition().isAttackable() || n.isSummoningNpc()) {
                                    continue;
                            }
                    } else {
                            Player p = (Player)next;
                            if(p.getLocation() != Location.WILDERNESS || !Location.inMulti(p)) {
                                    continue;
                            }
                    }
                   

                    if (next.getPosition().isWithinDistance(castOn.getPosition(),
                        spellRadius()) && !next.equals(cast) && !next.equals(castOn) && next.getConstitution() > 0 && next.getConstitution() > 0) {
                        cast.getCurrentlyCasting().endGraphic().ifPresent(next::performGraphic);
                        int calc = Misc.random(damage);
                        next.dealDamage(new Hit(calc, Hitmask.RED, CombatIcon.MAGIC));
                        next.getCombatBuilder().addDamage(cast, calc);
                       
                       
                       
                        spellEffect(cast, next, calc);
                    }
                }

---

Any pointers on what I should be doing, or looking for?
I've just concluded that the inMulti damage isn't considered the players which is why it doesn't drop anything.

Make it so the damage know that the player dealt it instead of just dealing the damage.
Because with your code it seems like it just deal the damage and have no idea who the killer of the npc was to reward anything.

Quote:

---

Originally Posted by arch337 [Only registered and activated users can see links. Click Here To Register...]

Make it so the damage know that the player dealt it instead of just dealing the damage.
Because with your code it seems like it just deal the damage and have no idea who the killer of the npc was to reward anything.

---

Yeah, that's what I thought. Do you know which Java file I would need to look at to do that?
Or I'd just need to add more code in this file to specify the owner of the damage.

Quote:

---

Originally Posted by Rng Rng Ownu [Only registered and activated users can see links. Click Here To Register...]

Yeah, that's what I thought. Do you know which Java file I would need to look at to do that?
Or I'd just need to add more code in this file to specify the owner of the damage.

---

I would advice to check your combat code to see how they deal the damage.
Like take a look at a single target spell to see how it determ the damage as.

Code:

---

        /**
        * Adds damage to the damage map, as long as the argued amount of damage is
        * above 0 and the argued entity is a player.
        *
        * @[Only registered and activated users can see links. Click Here To Register...] entity
        *            the entity to add damage for.
        * @[Only registered and activated users can see links. Click Here To Register...] amount
        *            the amount of damage to add for the argued entity.
        */
        public void addDamage(Character entity, int amount) {

                // No damage below 0, and no npcs can be added to the map.
                if (amount < 1 || entity.isNpc()) {
                        return;
                }

                // Add on to the damage for existing players.
                Player player = (Player) entity;
                if (damageMap.containsKey(player)) {
                        damageMap.get(player).incrementDamage(amount);
                        return;
                }

                // Or add a completely new entry.
                damageMap.put(player, new CombatDamageCache(amount));
        }

---

this should work fine, nothing wrong with the code you posted I think. Maybe the drop code has a problem.

Quote:

---

Originally Posted by knd6060 [Only registered and activated users can see links. Click Here To Register...]

this should work fine, nothing wrong with the code you posted I think. Maybe the drop code has a problem.

---

Yeah I found out that it was fine all along. It only bugs out with no drops because it 1 hits the NPC. I tested it out NPCs with higher HP and I don't have that issue. It's only when that multi hit actually 1 shots something does this happen. Thank you though.

Quote:

---

Originally Posted by Rng Rng Ownu [Only registered and activated users can see links. Click Here To Register...]

Yeah I found out that it was fine all along. It only bugs out with no drops because it 1 hits the NPC. I tested it out NPCs with higher HP and I don't have that issue. It's only when that multi hit actually 1 shots something does this happen. Thank you though.

---

Try swapping around the add damage and deal damage statements. It might execute the NPCDeathTask and try get the killer in the same call before it updates the damage map, which would explain why it works on higher hp but not on one hit. I'd have to check code again to see if that is the case.

EDIT:This code would get executed immediately inside dealDamage sub calls

Code:

---

@Override
        public void appendDeath() {
                if(!isDying && !summoningNpc) {
                        TaskManager.submit(new NPCDeathTask(this));
                        isDying = true;
                }
        }

---

EDIT2: It appears that the NPCDeathTask is not an immediate task so and gets the killer 2 ticks later rather than executing in same call chain. So this isn't the problem either and the swapping won't likely fix it. Can try anyway