'LivingDropsEvent'Problem ,How to set the drops twice as long?

[themaw]

@SubscribeEvent
public void DoubleDrop(LivingDropsEvent e){
	if(e.source.getEntity() instanceof EntityPlayer){
		EntityPlayer p =(EntityPlayer)e.source.getEntity(); 
		if(p.isEntityAlive()){
		if(p.getActivePotionEffect(org.MawManager.MawStory.Potion.Register.doubledrop)!=null){
		if(e.drops.size()!=0){
			e.drops.addAll(e.drops);
			p.addChatComponentMessage(new ChatComponentText(StatCollector.translateToLocal("doubledrop.text1")));
		}
		}
		}
		}

I try to do like this,but it didn't work.

How to set the drops twice as long,is there any other way?

[Ugdhar]

Where are you registering this event on the event bus? What does it do when you run it? Do you ever make it to that inside condition where you're attempting to add the drops? Does the fml-client-latest.log say anything?

[Alexiy]

 

e.drops.addAll(e.drops);

I don't think this will have any effect.

[themaw]

  On 11/7/2017 at 11:19 AM, Ugdhar said:

Where are you registering this event on the event bus? What does it do when you run it? Do you ever make it to that inside condition where you're attempting to add the drops? Does the fml-client-latest.log say anything?

Expand  

Yes,I had registered this  event on the event bus,and Item drops number has not changed (I say this, because  their number is odd number)

And if I try to add something,like "apple":

SubscribeEvent
public void DoubleDrop(LivingDropsEvent e){
	if(e.source.getEntity() instanceof EntityPlayer){
		EntityPlayer p =(EntityPlayer)e.source.getEntity(); 
		if(p.isEntityAlive()){
		if(p.getActivePotionEffect(org.MawManager.MawStory.Potion.Register.doubledrop)!=null){
		if(e.drops.size()!=0){
			e.drops.addAll(e.drops);
			//Look here
			e.drops.add(1, new EntityItem(e.entity.worldObj,e.entity.posX,e.entity.posY,e.entity.posZ,new ItemStack(Items.apple,1)));
			//
			p.addChatComponentMessage(new ChatComponentText(StatCollector.translateToLocal("doubledrop.text1")));
		}
		}
		}
		}
		}

 

And it is working,but the normal drops did not change.(three apple,theree chicken)

 

Thank you for answer my problem.

[themaw]

  On 11/7/2017 at 12:51 PM, Alexiy said:

 

e.drops.addAll(e.drops);

I don't think this will have any effect.

Expand  

What should I do?

Thank you .

[Draco18s]

(or you can set each stack's stacksize to double: virtually all drops are done via 1-size stacks, even in the case of dropping 3 or 5 items)

Edited November 7, 2017 by Draco18s

[Alexiy]

  On 11/7/2017 at 1:04 PM, themaw said:

What should I do?

Thank you .

Expand  

You can copy the itemstacks from the entity items in the list, then create new entity items for copied drops, then add those to the list.

[themaw]

Thank you for your attention