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

[Bug] - es_extended #1248

Closed
CzlowiekNinja2021 opened this issue Oct 22, 2023 · 4 comments
Closed

[Bug] - es_extended #1248

CzlowiekNinja2021 opened this issue Oct 22, 2023 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@CzlowiekNinja2021
Copy link

Describe the bug
When we register the command and the argument type we select "merge" ESX returns a problem with argument validation
example kick 1 blah blah blah

@CzlowiekNinja2021 CzlowiekNinja2021 added the bug Something isn't working label Oct 22, 2023
@github-project-automation github-project-automation bot moved this to 🆕 New in ESX Roadmap Oct 22, 2023
@thesysadmindev
Copy link
Contributor

@CzlowiekNinja2021 gonna need a lot more than that. Code samples, screenshots, "blah blah blah".

@Gellipapa
Copy link
Contributor

Gellipapa commented Oct 22, 2023

@CzlowiekNinja2021 Hi! Please write down how to reproduce this issue and we will can fix.

@CzlowiekNinja2021
Copy link
Author

CzlowiekNinja2021 commented Oct 22, 2023

example command:
`ESX.RegisterCommand("kick", "support", function(xPlayer,args)

if xPlayer.source == args.playerId.source then
	return xPlayer.showNotification("Nie możesz wyrzucić samego siebie", "error")
end

args.playerId.kick(args.reason)

end, true, {
help = "Wyrzuca gracza z serwera",
validate = true,
arguments = {
{name = "playerId", help = "ID gracza którego chcesz wyrzucić z serwera", type = "player"},
{name = "reason", help = 'Podaj powód wyrzucenia, pamiętaj aby powód podać w cudzyslowach', type = "merge"}
}
})`

dont work:
kick 1 reason reason

work command, dont return validation
kick 1 "reason reson"
`

Sory my english bad :/

@Gellipapa Gellipapa moved this from 🆕 New to 🔧need test in ESX Roadmap Oct 29, 2023
@Gellipapa
Copy link
Contributor

Gellipapa commented Oct 29, 2023

@CzlowiekNinja2021 Hi! If you use merge type please change validate to false and works perfectly. I take picture i tested.
image

@github-project-automation github-project-automation bot moved this from 🔧need test to ✅ Done in ESX Roadmap Oct 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

No branches or pull requests

4 participants