C6 Inventory double items in auto pick up

[cardinal]

I have problem, when i kill a mob and have my Inventory opened i see the item i pick up double. If i close it and reopen it its normal.

[G-hamsteR]

Does this happen on auto-loot or on item pickup?

[jamiskater]

Same here, with autoloot. Anything new in inventory will double. Reopening the inventory refreshes it.
If I have a quest and get a QI, then the inventory refreshes.
Just a quality of life issue, since I like to play with my inventory opened

[G-hamsteR]

Find the doPickupItem function inside Player and try to send an itemlist or inventoryUpdate packet after each pickup.

[nasseka]

thought of the itemlist but usually it's a bit hvy on that party. Inventory update packet would be the best but did it exist in c6? xD

[G-hamsteR]

It does 

Code: [Select]

final InventoryUpdate iu = new InventoryUpdate();
iu.addItem(item);
player.sendPacket(iu);


[jamiskater]

Tried this, doesn't work, still duplicates new items in inventory until I re-open it:

(I'm not a coder, just copy-paster trying my luck )

Code: [Select]

/**
* Manage AutoLoot Task.<br>
* <br>
* <b><u>Actions</u>:</b><br>
* <li>Send a System Message to the Player : YOU_PICKED_UP_S1_ADENA or YOU_PICKED_UP_S1_S2</li>
* <li>Add the Item to the Player inventory</li>
* <li>Send a Server->Client packet InventoryUpdate to this Player with NewItem (use a new slot) or ModifiedItem (increase amount)</li>
* <li>Send a Server->Client packet StatusUpdate to this Player with current weight</li><br>
* <font color=#FF0000><b><u>Caution</u>: If a Party is in progress, distribute Items between party members</b></font>
* @param target The Item dropped
* @param item the item
*/
public void doAutoLoot(Attackable target, ItemHolder item)
{
if (isInParty())
{
getParty().distributeItem(this, item, false, target);
// Send inventory update packet
final InventoryUpdate playerIU = new InventoryUpdate();
sendPacket(playerIU);

}
else if (item.getId() == 57)
{
addAdena("AutoLoot", item.getCount(), target, true);
// Send inventory update packet
final InventoryUpdate playerIU = new InventoryUpdate();
sendPacket(playerIU);

}
else
{
addItem("AutoLoot", item.getId(), item.getCount(), target, true);
// Send inventory update packet
final InventoryUpdate playerIU = new InventoryUpdate();
sendPacket(playerIU);

}
}

The original code is only this (lines 4851+):

Code: [Select]

/**
* Manage AutoLoot Task.<br>
* <br>
* <b><u>Actions</u>:</b><br>
* <li>Send a System Message to the Player : YOU_PICKED_UP_S1_ADENA or YOU_PICKED_UP_S1_S2</li>
* <li>Add the Item to the Player inventory</li>
* <li>Send a Server->Client packet InventoryUpdate to this Player with NewItem (use a new slot) or ModifiedItem (increase amount)</li>
* <li>Send a Server->Client packet StatusUpdate to this Player with current weight</li><br>
* <font color=#FF0000><b><u>Caution</u>: If a Party is in progress, distribute Items between party members</b></font>
* @param target The Item dropped
* @param item the item
*/
public void doAutoLoot(Attackable target, ItemHolder item)
{
if (isInParty())
{
getParty().distributeItem(this, item, false, target);
}
else if (item.getId() == 57)
{
addAdena("AutoLoot", item.getCount(), target, true);
}
else
{
addItem("AutoLoot", item.getId(), item.getCount(), target, true);
}
}
}

[G-hamsteR]

Try this:

Code: [Select]

final Item item = addItem("AutoLoot", item.getId(), item.getCount(), target, true);
// Send inventory update packet
final InventoryUpdate playerIU = new InventoryUpdate();
iu.addItem(item);
sendPacket(playerIU);


[dramaa]

tbh i have this problem too, i am just too afraid to ask, cus i have new questions every day and people are tired of me i am sure xd same happening on cp or when opening any box, gives everything with duplicate, but in my case when i close and open  bag, not disappearing :/

[dramaa]

Try this:

Code: [Select]

final Item item = addItem("AutoLoot", item.getId(), item.getCount(), target, true);
// Send inventory update packet
final InventoryUpdate playerIU = new InventoryUpdate();
iu.addItem(item);
sendPacket(playerIU);


does it work? and where it goes, please tell me if not a problem

[ektorasdj]

Hello, i am also experiencing the same problem in my interlude server. I will try your suggestion after i get off work today and get back to you.
Thank you

[ektorasdj]

Hello !
@G-hamsteR I am not a coder (please bare with me) ,

I get some errors on eclipse:



Thank you for your help

[G-hamsteR]

Hello !
@G-hamsteR I am not a coder (please bare with me) ,

I get some errors on eclipse:



Thank you for your help

Change to the following:

Code: [Select]

final Item itemG = addItem("AutoLoot", item.getId(), item.getCount(), target, true);
final InventoryUpdate playerIU = new InventoryUpdate();
playerIU.addItem(itemG);
sendPacket(playerIU);


[ektorasdj]

Hello!
Thank you for your prompt reply,
Still getting a small error: