Why does this happen?

[mascat102]

I'm trying to change my multiplier skript to something else and I got this.

on break:
    if player's tool is enchanted with multishot 1:
        add drops of event-block to {blocks::*}
        clear drops
        loop {blocks::*}:
            give loop-value to player
            give loop-value to player
            stop
            

Though whenever I break a block it just uses the first block I broke. So if I broke wood first anytime I use a tool with multishot it will give me wood. I can't see the problem or why this is happening because I recently started learning skript. I am wondering if anyone knows how to fix it.

[gabinun]

The issue is that you're storing the drops in the list {blocks::*} but not resetting it before each break event, leading to it retaining only the first block's drops. You need to clear {blocks::*} at the beginning of the event handler. Here’s the corrected script:

on break:
 clear {blocks::*}
 player's tool is enchanted with multishot 1
 add drops of event-block to {blocks::*}
 clear drops
 loop {blocks::*}:
  give loop-value to player
  give loop-value to player
 stop

Edited October 10 by gabinun

[mascat102]

Thank you for telling me this, but now the problem is it doesn't work with fortune. Do you have any way to fix that?

Edited October 10 by mascat102

[gabinun]

I don't think this is a good way of making fortune, and I will not be trying to make a Skript for fortune. If you want an autopickup with fortune, getting the plugin is the best choice.

[Dead07]

Give the player the amount of that destroyed block equivalent to their fortune level on that item so basically give (fortune level) (destroyed block) to player.

Use this:

https://skripthub.net/docs/?id=929

Edited Wednesday at 06:38 PM by ApexSplat