Skip to content

fix: Prevent crashes caused by client_entergame and other minor bugs.#736

Merged
Nottinghster merged 3 commits intoopentibiabr:mainfrom
kokekanon:main
Apr 3, 2024
Merged

fix: Prevent crashes caused by client_entergame and other minor bugs.#736
Nottinghster merged 3 commits intoopentibiabr:mainfrom
kokekanon:main

Conversation

@kokekanon
Copy link
Contributor

@kokekanon kokekanon commented Mar 31, 2024

1.- fix: crash game_entergame if httpLoginBox is boolean

produced in #647

I think httpLogin is always boolean, but I haven't studied pr 647 enough.

    if type(httpLogin) == "boolean" then
        enterGame:getChildById('httpLoginBox'):setChecked(httpLogin)
    else
        enterGame:getChildById('httpLoginBox'):setChecked(#httpLogin > 0)
    end

Error:

ERROR: Lua exception: /client_entergame/entergame.lua:270: attempt to get length of local 'httpLogin' (a boolean value)
stack traceback:
	[C]: in function '__len'
	/client_entergame/entergame.lua:270: in function 'setHttpLogin'
	/client_serverlist/serverlist.lua:44: in function 'select'
	/client_serverlist/serverlist.lua:81: in function </client_serverlist/serverlist.lua:80>
ERROR: protected lua call failed: LUA ERROR:
/client_entergame/entergame.lua:270: attempt to get length of local 'httpLogin' (a boolean value)
stack traceback:
	[C]: in function '__len'
	/client_entergame/entergame.lua:270: in function 'setHttpLogin'
	/client_serverlist/serverlist.lua:44: in function 'select'
	/client_serverlist/serverlist.lua:81: in function </client_serverlist/serverlist.lua:80>
ERROR: Lua exception: 
ERROR: protected lua call failed: LUA ERROR:

ERROR: Lua exception: 
ERROR: protected lua call failed: LUA ERROR:

ERROR: Lua exception: 
ERROR: protected lua call failed: LUA ERROR:
2024-03-31.13-58-17.mp4

2.- fix: game_entergame if host nil windows is hidden

2024-03-31.13-56-20.mp4

fix:
image

3.- temp fix: game_entergame windows is not centered

2024-03-31.13-59-32.mp4
the original bug is in
EnterGame.toggleAuthenticatorToken
EnterGame.toggleStayLoggedBox

I don't have how to test I don't use token

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Behavior

Actual

Do this and that doesn't happens

Expected

Do this and that happens

Fixes

# (issue)

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested

  • Test A
  • Test B

Test Configuration:

  • Server Version: 860 | 1320
  • Operating System: windows

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I checked the PR checks reports
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works

the original bug is in
I don't have how to test I don't use token
  EnterGame.toggleAuthenticatorToken
  EnterGame.toggleStayLoggedBox
@Nottinghster Nottinghster merged commit 8bf721f into opentibiabr:main Apr 3, 2024
kokekanon added a commit to kokekanon/otclient.readme that referenced this pull request Apr 17, 2024
The modification made to opentibiabr#736 of loadBox was in the wrong place in the code. causes display loadBox in old protocols.
Nottinghster pushed a commit that referenced this pull request Apr 20, 2024
* fix: [game_entergame] loadBox in old protocol

The modification made to #736 of loadBox was in the wrong place in the code. causes display loadBox in old protocols.

* fix: If you try to login without assets.

* fix: indentation Reviewer: luanluciano93
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.

3 participants