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
StatusEffect -> MobEffect/CreatureEffect/Effect #584
Comments
This was a name from the Minecraft wiki. I will go ask why "Status Effect". |
It's an effect that tells the status of an entity. Seems better than MobEffect or CreatureEffect to me. Effect is just very vague. |
I agree that Effect is very vague. But adding Status to it doesn't make it less vague. It doesn't mention it's the status of an entity, which is why I'm suggesting MobEffect/CreatureEffect/EntityEffect (or MobStatusEffect/CreatureStatusEffect/EntityStatusEffect if you prefer). Mob is the Mojang name, which is why I'd vote for that, but Creature (or LivingEntity, but that's long) would be more precise since it applies to living entities. |
I'm ok with MobEffect if we rename LivingEntity to Mob as mojang has. |
Mojang only calls non-player living entities mobs. I remember seeing an |
I don't really see the point. Status effect is a common name for it, so I doubt anyone would be really confused. |
And how does adding Status help? By the definition of the word "effect", all effects modify some status. It would be like naming it Also, it's not true it doesn't mean anything. The command is named
I've only heard players call it "effect". If you're saying it's a common name, then you mean it's a name commonly used by modders because of the MCP name, which we shouldn't care about, the same way we don't care about |
StatusEffect is definitely not an MCP name. In fact I am glad to see that the MCP name hasn't been brought up once in this discussion (because it's not a good one). I am however referring to the official minecraft wiki, which names effects like that: https://minecraft.gamepedia.com/Status_effect. Given that a lot of players get their information from the wiki, I think it is in fact fair to say that it is common. |
well...whatever. I'm okay with MobEffect I guess just to go with the mojang name. But I also believe in the Menu change to align with mojang |
The "Status" prefix adds no useful information to the class name. All it tells you is that it affects the "status" of some thing (which is what any effect does, by definition).
I suggest changing it to either:
MobEffect
(it's slightly wrong since players are probably not considered mobs, but it's the Mojang name)CreatureEffect
Effect
The text was updated successfully, but these errors were encountered: