MagicPlugin icon indicating copy to clipboard operation
MagicPlugin copied to clipboard

The Dropable Blocks Could NOT be Filled All While Casting for Many Times

Open pianwan opened this issue 6 years ago • 6 comments

If someone cast spells like Nuke in the same place for many times while the dropable block is filling,the blocks will not be filled all. This caused some blocks missing. I'm on the newest version of Magic. Please fix it,Thank you! After doing it: https://i.loli.net/2018/12/11/5c0f459fcae79.png

pianwan avatar Dec 11 '18 05:12 pianwan

I can't reproduce this. I test these spells quite a lot for these kinds of issues.

https://youtu.be/jl4fxNAyrFc

May not be able to fix if I can't reproduce, sorry.

Players shouldn't be able to cast nuke many times in the same place anyway, it has a very long cooldown.

NathanWolf avatar Dec 11 '18 18:12 NathanWolf

But It is a multiplayer game,one casted and the other casted after. Another important thing is that I'm on 1.13.2. Other spells have this bug too. If you still can't reproduce it,I'll give more pictures and detailed steps about how to do it in recent future.

pianwan avatar Dec 12 '18 04:12 pianwan

That would be helpful. If you have a way to reproduce it I can probably fix it. It'd be nice if it did not involve multiple players since that is more difficult to reproduce in a dev environment, though doable.

It does not seem to be as simple as casting it several times while overlapping, as you can see in the video I made.

NathanWolf avatar Dec 12 '18 15:12 NathanWolf

Enviroment:

  • paperclip last 1.13.2
  • paperconfig: https://paste2.org/HGm2L94k
  • Other important plugins: Heroes,React,CMI Detailed Steps: 1.Go to somewhere with sand. img: 2.Cast the Skill Cluster Bomb twice, img: https://i.loli.net/2018/12/14/5c13d02dbc4fe.png https://i.loli.net/2018/12/14/5c13d02de1bfe.png 3.Look!it start filling! img: https://i.loli.net/2018/12/14/5c13d02ccedd4.png https://i.loli.net/2018/12/14/5c13d02c793c8.png 4.Cast Cluster Bomb while filling: img: https://i.loli.net/2018/12/14/5c13d02acaf0a.png 5.Cast Nuke there img: https://i.loli.net/2018/12/14/5c13cfb3a73d6.png 6.Wait... and Oh.... Something is missing. img: https://i.loli.net/2018/12/14/5c13d0388e042.png https://i.loli.net/2018/12/14/5c13cfe07d08f.png

I also find that Cluster Bomb can break chest door & more.

pianwan avatar Dec 14 '18 15:12 pianwan

Thanks, I will give this a try when I can. Sand is tricky.

Spells that use vanilla TNT and explosions will blow up whatever TNT and explosions normally would.

NathanWolf avatar Dec 14 '18 15:12 NathanWolf

That's great. Hoping for solution! We really need this~

pianwan avatar Dec 18 '18 14:12 pianwan