Fix lantern not working

[TheLimeGlass]

Fix lantern not working. on place of lantern does not work, you had to use any lantern to fix that issue.

[AyhamAl-Ali]

Does this require optional brackets as well?

[APickledWalrus]

Does this not conflict with regular lanterns? Also, there are two instances of any lantern(so if you're making changes, you'll need to change both)

[TheBentoBox]

Agree with @APickledWalrus, seems like this will conflict with the normal (non-soul) lantern alias. There seems to be a different issue at play here if on place of lantern doesn't work.

As for this, Pickle...

Also, there are two instances of any lantern

I assume you mean the ones in "nether update" and "nether update part 2"? If so, the 2nd one is overriding the first one to include the {waterloggable} overrides for lanterns from lines 670 and 671.

[TheLimeGlass]

Does this not conflict with regular lanterns? Also, there are two instances of any lantern(so if you're making changes, you'll need to change both)

Point out where it's defined twice.

[APickledWalrus]

It's defined here

skript-aliases/decoration.sk

Line 404 in fb9c304

{lantern hanging} lantern = minecraft:lantern

and again here

skript-aliases/decoration.sk

Line 670 in fb9c304

{waterloggable} {lantern hanging} lantern = minecraft:lantern

due to some changes with the data the block supports

[TheLimeGlass]

It's defined here

skript-aliases/decoration.sk

Line 404 in fb9c304

{lantern hanging} lantern = minecraft:lantern

and again here

skript-aliases/decoration.sk

Line 670 in fb9c304

{waterloggable} {lantern hanging} lantern = minecraft:lantern

due to some changes with the data the block supports

It will redefine it

[APickledWalrus]

My main concern here is that this overrides the existing alias for just regular lanterns. There is then no good way to just give the user or check for a "normal" lantern early, and I do not feel that requiring normal lantern is an ideal approach.