iBlitzkriegi / Vixio

Vixio, The Skript Addon made to fit all of your Discord needs.
http://www.vixio.space
40 stars 28 forks source link

Skript's Local vars returns <none> when used in vixio #307

Open AyhamAl-Ali opened 3 years ago

AyhamAl-Ali commented 3 years ago

Describe the bug I tried sending messages to my discord using the bot and the messages had local vars and they all returned <none>, I tried testing it and it gives the same results

To Reproduce Steps to reproduce the behavior:

on chat:
    set {_test} to "test1"
    set {test} to "test2"
    send "> %{_test}%" to channel with id "ID" with "NAME" # <none>
    send "> %{test}%" to channel with id "ID" with "NAME" # test2

where ID is your bot's id and NAME is your bot's name.

Information Skript version: 2.5.3 Vixio version: 2.0.7 Minecraft Version: 1.16.5 Server software name and version: Paper 1.16.5

Expected behavior Send the correct values of local vars

Screenshots image image

Additional context None.

mashmarvin commented 3 years ago

Duplicate of https://github.com/iBlitzkriegi/Vixio/issues/272 but it won't be fixed anyways...........

AyhamAl-Ali commented 3 years ago

@iBlitzkriegi Could you take a look at this?

HardstylesDev commented 3 years ago

@iBlitzkriegi Could you take a look at this?

It's an issue with newer skript versions.

mashmarvin commented 3 years ago

It's an issue with newer skript versions.

As mentioned in my issue, it's not. I tested down to 2.2-dev36 and it still failed

AyhamAl-Ali commented 3 years ago

It's an issue with newer skript versions.

Even if it is, we can't downgrade Skript because it is the main thing that this addon depends on. I think Blitz is inactive and not gonna fix this soon which is a bad thing for Vixio's users.

We hope he fix it as soon as possible it is very important.

AyhamAl-Ali commented 3 years ago

Check this reply

HardstylesDev commented 3 years ago

It's an issue with newer skript versions.

Even if it is, we can't downgrade Skript because it is the main thing that this addon depends on. I think Blitz is inactive and not gonna fix this soon which is a bad thing for Vixio's users.

We hope he fix it as soon as possible it is very important.

When I used Vixio I used dev36 without any issues, so idk.

AyhamAl-Ali commented 3 years ago

When I used Vixio I used dev36 without any issues, so idk.

As some have said and as Skript devs have said themselves, it is due to the new variables update they made and vixio haven't updated to support that update yet so currently vixio needs to update.

LowScarlet commented 3 years ago

Please fix T_T

AyhamAl-Ali commented 3 years ago

Please fix T_T

Use Disky it is an updated version of Vixio with fixes and new features

AyhamAl-Ali commented 3 years ago

But seems Blitz is back, so I think he will fix it.

ItsTheSky commented 3 years ago

I send the method to fix it to Blitz in discord, but look likes he doesn't answer me yet.

LowScarlet commented 3 years ago

I send the method to fix it to Blitz in discord, but look likes he doesn't answer me yet.

sad

carlodrift commented 3 years ago

Seems to be fixed in 2.0.9