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

Add missing Minecraft commands #13

Closed
47 of 58 tasks
deathcap opened this issue Apr 9, 2015 · 20 comments
Closed
47 of 58 tasks

Add missing Minecraft commands #13

deathcap opened this issue Apr 9, 2015 · 20 comments

Comments

@deathcap
Copy link
Contributor

deathcap commented Apr 9, 2015

Bukkit added server commands up to 1.7, but Spigot's Bukkit update for 1.8 removed all commands from the API because they instead pass through to the official Minecraft server. Glowstone can't do this obviously, so it relies on the commands in the API project, but they aren't updated for any new 1.8 features. Glowkit may have already added some; need to check. Full command list:

http://minecraft.gamepedia.com/Commands

  • /achievement
  • /ban
  • /ban-ip
  • /banlist
  • /blockdata
  • /clear
  • /clone
  • /debug
  • /defaultgamemode
  • /deop
  • /difficulty
  • /effect
  • /enchant
  • /entitydata
  • /execute
  • /fill
  • /gamemode
  • /gamerule
  • /give
  • /help
  • /kick
  • /kill
  • /list
  • /me
  • /op
  • /pardon
  • /particle
  • /playsound
  • /publish
  • /replaceitem
  • /save-all
  • /save-off
  • /save-on
  • /say
  • /scoreboard
  • /seed
  • /setblock
  • /setidletimeout
  • /setworldspawn
  • /spawnpoint
  • /spreadplayers
  • /stats
  • /stop
  • /summon
  • /tell
  • /tellraw
  • /testfor
  • /testforblock
  • /testforblocks
  • /time
  • /title
  • /toggledownfall
  • /tp
  • /trigger
  • /weather
  • /whitelist
  • /worldborder
  • /xp

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@mastercoms
Copy link
Member

Here is a list of Glowkit commands: https://github.com/GlowstoneMC/Glowkit/tree/master/src/main/java/org/bukkit/command/defaults

I don't think they have added anything.

@deathcap
Copy link
Contributor Author

There's also two in the implementation, /color and /tellraw:

https://github.com/deathcap/GlowstonePlusPlus/tree/master/src/main/java/net/glowstone/command

@mastercoms
Copy link
Member

Oh yes, I missed those, sorry. But still, nothing from 1.8 from what I can tell.

@mastercoms
Copy link
Member

AFAIK commands shouldn't be in Glowstone.

@mastercoms
Copy link
Member

I have figured out the difference. You put commands that extend VanillaCommand in Glowkit and commands that extend BukkitCommand in Glowstone.

@Johni0702
Copy link
Contributor

Every command that isn't on a vanilla server is a BukkitCommand.

@Johni0702
Copy link
Contributor

/tellraw is probably in Glowstone because at the time it was added there wasn't any way of implementing it in Glowkit. I'm not sure about /title, might be the same reason.

Edit: For /tellraw see here

@mastercoms
Copy link
Member

Yeah, I guess we should port them at some point.

@mastercoms
Copy link
Member

Yeah, it seems like Bukkit/Bukkit#1111 seems to be the way to go for tellraw.

@ST-DDT
Copy link

ST-DDT commented Sep 2, 2015

There is a better/newer implementation in SpongeAPI, that you can use here as well.
https://github.com/SpongePowered/SpongeAPI/tree/master/src/main/java/org/spongepowered/api/text

@mastercoms
Copy link
Member

Alright, thanks!

@mastercoms
Copy link
Member

md-5's BaseComponent seems like the thing we should be using for supporting all of this JSON text stuff.

@GreenBeard
Copy link
Contributor

@deathcap you should uncheck /playsound it has a major update in 1.9

@gdude2002
Copy link
Contributor

@GreenBeard This is a ticket about 1.8, though.

Speaking of which, has there been any more movement on this?

@mastercoms mastercoms changed the title Add new 1.8 server commands Add missing Minecraft commands May 7, 2016
@gdude2002 gdude2002 added this to Backlog in March 2017 Feb 27, 2017
@gdude2002 gdude2002 moved this from Backlog to Stalled in March 2017 Feb 27, 2017
@gdude2002 gdude2002 moved this from Stalled (Little activity since '16) to Backlog in March 2017 Mar 10, 2017
@gdude2002 gdude2002 added this to Backlog in April 2017 Apr 1, 2017
@gdude2002 gdude2002 removed this from Backlog in March 2017 Apr 1, 2017
@gdude2002 gdude2002 removed this from Backlog in April 2017 May 4, 2017
@gdude2002 gdude2002 added this to Backlog in May 2017 May 4, 2017
@robotman3000
Copy link

What version of minecraft should the behavior for the remaining commands match?

@mastercoms
Copy link
Member

Glowstone targets the latest Minecraft version, so 1.12

@mastercoms
Copy link
Member

Though, this issue is better: #499

@robotman3000
Copy link

Cool, thanks.

@robotman3000
Copy link

Is there a specific reason why this issue is still open?

@gdude2002
Copy link
Contributor

I was just about to close it in favour of #499 tbh

@ghost ghost removed the help wanted label Jun 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
May 2017
Backlog
Development

No branches or pull requests

7 participants