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

Fix for the new esx_identity (Already registered PED keeps getting new registration form) #34

Open
wants to merge 33 commits into
base: master
Choose a base branch
from

Conversation

Klay4
Copy link

@Klay4 Klay4 commented May 21, 2020

My request is to add the fix for this problem in the README.md file.

You just need to replace in the server mail.lua file the two:

for k,v in ipairs(GetPlayerIdentifiers(playerId)) do if string.match(v, 'license:') then identifier = string.sub(v, 9) break end end

whit:

for k,v in ipairs(GetPlayerIdentifiers(playerId)) do if string.match(v, 'license:') then identifier = v break end end

@RikoDEV
Copy link

RikoDEV commented May 21, 2020

Have you tested this version after your patches and do not generate errors? Is it ready for merge?

@RikoDEV RikoDEV added the enhancement New feature or request label May 21, 2020
@Klay4
Copy link
Author

Klay4 commented May 23, 2020

There is only a but that i'm trying to fix. But for now i have discovered another problem: in the esx_ambulancejob fix the IsDead variable is wrong, the correct name is isDead.

So, to make it works you have to replace IsDead = false with isDead = false

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants