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

Map Property Adjustment. #3125

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

Conversation

MrKeiKun
Copy link
Contributor

@MrKeiKun MrKeiKun commented Mar 16, 2022

Pull Request Prelude

Changes Proposed

  • Dropped clif_maptypeproperty2() and merged it with clif_map_property().
  • Renamed packet_maptypeproperty2 to packet_maptypeproperty and added support for 0x99b(ZC_MAPPROPERTY_R2).
  • clif_map_property_mapall() now redirects its clif_send() job to clif_map_property() which has dummy bl.

Thanks to @Kenpachi2k13

Issues addressed: #2892

@4144
Copy link
Contributor

4144 commented Mar 17, 2022

i know in all emulators map property partially broken, but even if this packets have similar names, they different.
i not sure is it good idea to drop one of them
0x199 change only one property.
0x99b change all properties for given type at same time.

look like exists atleast some flags what can change only 0x199

@MishimaHaruna MishimaHaruna added this to the Release v2022.07.06 milestone Jun 2, 2022
* Dropped clif_maptypeproperty2() and merged it with clif_map_property().
* Renamed packet_maptypeproperty2 to packet_maptypeproperty and added support for 0x99b(ZC_MAPPROPERTY_R2).
* clif_map_property_mapall() now redirects its clif_send() job to clif_map_property() which has dummy bl.
Signed-off-by: Kei <the.keikun@gmail.com>
@csnv
Copy link
Contributor

csnv commented Sep 19, 2023

@MrKeiKun @4144 @MishimaHaruna what's the status on this, can it be merged?

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

4 participants