Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Thrown error when check if an event-entity is equal to a type of entity #3499

Closed
Anarchick opened this issue Oct 28, 2020 · 3 comments
Closed
Labels
bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. completed The issue has been fully resolved and the change will be in the next Skript update. priority: low Issues that are not harmful to the experience but are related to useful changes or additions.

Comments

@Anarchick
Copy link

Description

If we miss to write the "a" from "a spider" , an error is thrown. It's not only for spider but all types of entities.
No skript parsing error.

Steps to Reproduce

on spawn:
    type of event-entity is a zombie or a skeleton or a creeper or a zombie villager or a drowned or a husk or spider
    # Rest of the code

This thrown a console error :
type of event-entity is a zombie or a skeleton or a creeper or a zombie villager or a drowned or a husk or spider
This works :
type of event-entity is a zombie or a skeleton or a creeper or a zombie villager or a drowned or a husk or a spider

Expected Behavior

With previous Skript version, we do not need to write the "a" or "an"

Errors / Screenshots

https://hastebin.com/vojexixame.md

Server Information

  • Server version/platform: PAPER-249 (1.16.3)
  • Skript version: 2.5.1
@Romitou
Copy link
Member

Romitou commented Oct 29, 2020

Oops, I made a mistake in my last comment which I deleted.
In any case, I can reproduce this issue with Skript 2.5.1 and git-Paper-182 (MC: 1.16.2) : https://gist.github.com/Romitou/6895be87f61831dc49ddf28d6d1b7741.

@ShaneBeee
Copy link
Contributor

ShaneBeee commented Oct 30, 2020

This isn't actually a bug, but rather user error

spider refers to an actual entity, ie: the spider.
a spider refers to an entity type.

On that note, it shouldn't throw the error it is throwing, it should throw a Skript error stating the syntax is incorrectly being used.

I'm marking this as a low priority because the error only comes up when using incorrect code, therefor is not truly harmful.

@ShaneBeee ShaneBeee added bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. priority: low Issues that are not harmful to the experience but are related to useful changes or additions. labels Oct 30, 2020
@TheLimeGlass
Copy link
Collaborator

TheLimeGlass commented Dec 11, 2020

Are you using an addon? Does your addon list include Umbaska? There is a "bug" that requires you to put the letter a before an entity type. Seen this happen lots through the years, it has always been because of Umbaska, not so much anymore because Umbaska isn't used anymore. It's really weird, can you attempt the same issue with no add-ons please and thank you.

Also hastebin link has expired.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. completed The issue has been fully resolved and the change will be in the next Skript update. priority: low Issues that are not harmful to the experience but are related to useful changes or additions.
Projects
None yet
Development

No branches or pull requests

6 participants