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

Protocol 12.72 update #3797

Merged
merged 18 commits into from
Nov 28, 2021

Conversation

EPuncker
Copy link
Contributor

@EPuncker EPuncker commented Nov 20, 2021

Pull Request Prelude

Changes Proposed

  • 12.72 support
  • cid being dependent on GUID (this is necessary to save containers)
  • adjust monster/npc generated ids (the cid)
  • standardized creature id bounds (uses variable instead of hardcoded number now)
  • improved code behind mounts
  • updated summon status refresh packet
  • updated talkactions message types (blue console message is deprecated)
  • feature: packet sequencing
  • feature: login with email (you can set @admin as your email btw or if you're running otclient, still use your account name)
  • feature: right click to talk with npc
  • feature: save opened containers
  • feature: mount colors
  • feature: session end packet
  • feature: cid and GUID in onLook (gm/god only)
  • new items
  • new outfits
  • new mounts
  • new NPC bubbles
  • new condition icons (enums only for now)
  • new message types

Credits

99% of the code on this PR is made by @Zbizu !!!

Special thanks to:

  • OTBR community (especially people involved in "Global" and "Canary" projects and their OTClient fork) for providing the code and tools necessary to achieve that
  • SaiyansKing for paving the way for QT clients
  • Arch-Mina for providing version 12 tools to work with client
  • EPuncker - providing spr/dat, help with otb, game content information, testing and help with login.php
  • kokokoko - updating items.otb with new items and data
  • omarcopires - review, testing and help with login.php
  • TFS maintainers for ensuring code quality and helping with project direction

Issues addressed:

EPuncker and others added 10 commits November 19, 2021 23:01
also updated looktype command with new invalid ids and max range
It is not present in the client anymore
Co-Authored-By: Zbizu <Zbizu@users.noreply.github.com>
- sendCreatureType
- sendCreatureHelpers
- buildInventoryList

Co-Authored-By: Zbizu <Zbizu@users.noreply.github.com>
Co-Authored-By: Zbizu <Zbizu@users.noreply.github.com>
nekiro
nekiro previously approved these changes Nov 20, 2021

// String client version
if (version >= 1240) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

as minor versions are not supported, do we still need this check?

src/protocolgame.cpp Show resolved Hide resolved
Copy link
Member

@DSpeichert DSpeichert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Last commit ("fix mount login speed") should be squashed with the earlier one ("Fixed mount speed update")
  • Typo in "Added senClientFeatures function"

@DSpeichert
Copy link
Member

Linked #3771

EPuncker and others added 8 commits November 28, 2021 00:53
Co-Authored-By: Zbizu <Zbizu@users.noreply.github.com>
and added missing ITEM_GROUP_PODIUM
- added podium code placeholder
- added isContainer for loot icon and quiver ammo count
- added sendResourceBalance
- added sendStoreBalance
- changed getClientIcons from 16 to 32

Co-Authored-By: Zbizu <Zbizu@users.noreply.github.com>
Co-Authored-By: Zbizu <Zbizu@users.noreply.github.com>
and fixed hidden health creatures display

Co-Authored-By: Zbizu <Zbizu@users.noreply.github.com>
@EPuncker EPuncker force-pushed the don't-mind-me,-just-a-branch-name branch from 7b384db to 85dc488 Compare November 28, 2021 04:00
@DSpeichert DSpeichert merged commit 00626a7 into otland:master Nov 28, 2021
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

5 participants