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

[CLI] Console menu validation fixes #3413

Merged
merged 1 commit into from Jun 17, 2023
Merged

[CLI] Console menu validation fixes #3413

merged 1 commit into from Jun 17, 2023

Conversation

Akkadius
Copy link
Member

Related to work in #2857

Currently in database:dump when you run help, the root CLI menu --help takes over and shows the command menu instead of printing the help for the command itself.

Before

eqemu@332a3650bfe4:~/server$ ./bin/world database:dump --help

> EQEmulator [World] CLI Menu

character
  character:copy-character     Copies a character into a destination account
database
  database:dump                Dumps server database tables
  database:schema              Displays server database schema
  database:set-account-status  Sets account status by account name
  database:version             Shows database version
world
  world:version                Shows server version

After

eqemu@332a3650bfe4:~/server$ ./bin/world database:dump --help

Command

database:dump

Options

  --all
  --content-tables
  --login-tables
  --player-tables
  --bot-tables
  --merc-tables
  --state-tables
  --system-tables
  --query-serv-tables
  --table-structure-only
  --table-lock
  --dump-path=
  --dump-output-to-console
  --drop-table-syntax-only
  --compress

@Akkadius Akkadius merged commit 3200145 into master Jun 17, 2023
1 of 2 checks passed
@Akkadius Akkadius deleted the akkadius/cli-fixes branch June 17, 2023 23:19
@Akkadius Akkadius mentioned this pull request Jun 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants