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: game_outfit custom compatible with canary (familiar, outfit/mount in store) #962

Merged
merged 40 commits into from
Jan 23, 2025

Conversation

kokekanon
Copy link
Collaborator

@kokekanon kokekanon commented Nov 7, 2024

Description

feat: familiar (need test)

8.6 player sorry
caacca

feat: Only mine

qcaca

fix game_outfit : outfit/mount in store

image
image

fix: minor error cyclopedia 13.40

feat: server send feature to enable / disable

server:
image
opentibiabr/canary#3061

commits server :
opentibiabr/canary@74ea55f

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)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Test A
  • Test B

Test Configuration:

  • Server Version:
  • Client:
  • Operating System:

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

src/client/protocolgameparse.cpp Outdated Show resolved Hide resolved
src/client/protocolgameparse.cpp Outdated Show resolved Hide resolved
@kokekanon kokekanon changed the title fix: game_outfit custom compatible with canary fix: game_outfit custom compatible with canary ( Familiar, outfit/mount in store) Nov 8, 2024
@kokekanon kokekanon marked this pull request as ready for review January 5, 2025 02:01
@majestyotbr majestyotbr changed the title fix: game_outfit custom compatible with canary ( Familiar, outfit/mount in store) fix: game_outfit custom compatible with canary (familiar, outfit/mount in store) Jan 10, 2025
modules/game_outfit/outfit.lua Outdated Show resolved Hide resolved
modules/game_outfit/outfit.lua Outdated Show resolved Hide resolved
modules/game_outfit/outfit.lua Outdated Show resolved Hide resolved
}
}

if (g_game.getClientVersion() >= 1281) {
std::vector<std::tuple<uint16_t, std::string> > familiarList;
Copy link
Collaborator

Choose a reason for hiding this comment

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

familiarMode?

Copy link
Collaborator Author

@kokekanon kokekanon Jan 22, 2025

Choose a reason for hiding this comment

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

that is used here

g_game.processOpenOutfitWindow(currentOutfit, outfitList, mountList, **familiarList**, wingList, auraList, effectList, shaderList);

if I put it inside the conditional

    Severity	Code	Description	Project	File	Line	Suppression State	Details
Error (active)	E0020	identifier "familiarList" is undefined	otclient	\src\client\protocolgameparse.cpp	2689		

Copy link
Collaborator

Choose a reason for hiding this comment

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

I wanted to ask why you added the mode to outfits and mounts, and not to family members

Copy link
Collaborator Author

@kokekanon kokekanon Jan 22, 2025

Choose a reason for hiding this comment

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

I don't use canary, I don't know many things about familiar, but at the time I test, I didn't see in cipsof that it was used(ui/ux).

image

could you share a picture of a familiar in cipsoft (outfit) ?

@mehah mehah merged commit ac3deac into mehah:main Jan 23, 2025
12 checks passed
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.

4 participants