event-item bug

[short509]

Description

"event-item" which worked well in version 2.4, does not work.

Steps to Reproduce

on rightclick:
       if player is sneaking:
               message "%type of event-item%" to player

Errors / Screenshots

Server Information

• Server version/platform: 1.15.1
• Skript version: 2.5-alpha

[APickledWalrus]

I did some testing, and this only seems to be an issue with click events. The issue is that the event-item is returning no value, not the type expression.

[iOshawott]

well in my skripts
"set {_y} to number of event-item in player's inventory"
doesn't work too :(

[short509]

well in my skripts
"set {_y} to number of event-item in player's inventory"
doesn't work too :(

"event-item" seems to be the problem.

[Misio12320]

Give there a priority and event-entity, does not work too

[iOshawott]

have a Hope this, 2714 and 2802 will be fixed in next release

[TheLimeGlass]

Duplicate #2736

[bensku]

@TheLimeGlass No, this is separate issue that may be caused by same underlying issue.

[ShaneBeee]

I think the "issue" here is whether or not you consider this an issue.

Prior versions, event-item returned the player's tool.
Whereas now it returns the item type you clicked on.
In this case, you didn't click on anything, hence it being none.
If you right-clicked on grass it would return grass.

Now, in my option, I think this way makes more sense.
event-item = clicked block
player's tool = thing in hand.

Personally I wouldn't classify this as a bug, but maybe a change for the better?
I guess the "issue" here is that it changed, which could potentially break scripts.

[bensku]

Event item used to give the item that was activated by clicking, which could be offhand item too.

[iOshawott]

https://pastebin.com/Dxxpc2MA

With event-item doesn't work porperly you can't get "set {_y} to number of event-item in player's inventory" - changing it to number of player's tool causes issue, that it can't be more than one and breaks this checks system which works good on 2.4

[Misio12320]

?!??? How its still need-testing??? Its a bug, go and fix it.

[iOshawott]

I hope it'll get fixed and then I'll update from 2.4.1 ;|

[ShaneBeee]

?!??? How its still need-testing??? It's a bug, go and fix it.

I don't even know hot to reply to you. Your utter rudeness and sheer ignorance is really bothering me, and some others.
This is an open source project, please feel free to fix it and submit a PR.

[Misio12320]

?!??? How its still need-testing??? It's a bug, go and fix it.

I don't even know hot to reply to you. Your utter rudeness and sheer ignorance is really bothering me, and some others.

I got little rage because Dev of SkQuery has answer me after 2 months of my report bug and said "this is Skript bug". And there is still no update in mean need-testing. Its again my fault because your arguments are not possible to break. Apologies.

This is an open source project, please feel free to fix it and submit a PR.

Good to know.

[APickledWalrus]

This is still a present issue. event-item is returning the wrong value, and this may have broke many scripts.