Skip to content

fix: packet Cyclopedia house auction 13.40#970

Merged
majestyotbr merged 24 commits intoopentibiabr:mainfrom
kokekanon:fix-warning
Jan 3, 2025
Merged

fix: packet Cyclopedia house auction 13.40#970
majestyotbr merged 24 commits intoopentibiabr:mainfrom
kokekanon:fix-warning

Conversation

@kokekanon
Copy link
Copy Markdown
Contributor

@kokekanon kokekanon commented Nov 18, 2024

fix packets , NO features (lua.otui) [I don't know what it is]

SEND Packet test lua - otui
sendCyclopediaHouseAuction 0xAD (173)
GET Packet test lua - otui
GameServerCyclopediaHouseAuctionMessage 0xC3 (195)
GameServerCyclopediaHousesInfo 0xC6 (198)
GameServerCyclopediaHouseList 0xC7 (199)

Description

I was test. this pr opentibiabr/canary#3022

PR sent packets when logging

ERROR: ProtocolGame parse message exception (12336 bytes, 5967 unread, last opcode is 0xc6 (198), prev opcode is 0x9f (159)): unhandled opcode 198
Packet has been saved to packet.log, you can use it to find what was wrong. (Protocol: 1340)

I solved that problem, the rest is theoretical, I don't know how to test that.

last commits

26a2d60
"Merge branch 'main' into test-house"
on Nov 12, 2024, 02:28 PM

Behavior

Actual

Packet error

Expected

Do this and that happens

Fixes

Type of change

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

How Has This Been Tested

untest , until the end of the pr

Test Configuration:

  • Server Version: 13.40 26a2d60
  • Client: this PR
  • Operating System: 11

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

ONLY APPROVE IF PR CANARY # 3022 IS murge. if not close xd

kokekanon and others added 5 commits November 18, 2024 19:14
last commits  26a2d60 "Merge branch 'main' into test-house"

 on Nov 12, 2024, 02:28 PM
… the if statement.)

-Use the init-statement to declare "isOwner" inside the if statement.
kokekanon and others added 16 commits November 25, 2024 13:10
Co-authored-by: Luan Luciano <luanluciano@outlook.com>
Co-authored-by: Luan Luciano <luanluciano@outlook.com>
Co-authored-by: Luan Luciano <luanluciano@outlook.com>
Co-authored-by: Luan Luciano <luanluciano@outlook.com>
Co-authored-by: Luan Luciano <luanluciano@outlook.com>
Co-authored-by: Luan Luciano <luanluciano@outlook.com>
Co-authored-by: Luan Luciano <luanluciano@outlook.com>
Co-authored-by: Luan Luciano <luanluciano@outlook.com>
Co-authored-by: Luan Luciano <luanluciano@outlook.com>
Co-authored-by: Luan Luciano <luanluciano@outlook.com>
@kokekanon kokekanon changed the title fix: packet Canary PR : "cyclopedia house auction" # 3022 fix: packet Cyclopedia house auction 13.40 Dec 31, 2024
@kokekanon kokekanon marked this pull request as ready for review December 31, 2024 18:44
Co-authored-by: Luan Luciano <luanluciano@outlook.com>
@kokekanon
Copy link
Copy Markdown
Contributor Author

canary's pr # 3022 was approved today.

this pr can now be merged.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Jan 2, 2025

@majestyotbr majestyotbr merged commit 9ee23ba into opentibiabr:main Jan 3, 2025
vllsystems pushed a commit to vllsystems/otclient that referenced this pull request Jan 10, 2025
@Dysodium Dysodium mentioned this pull request Jun 18, 2025
5 tasks
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.

5 participants