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

improve: Outfit(addons) & Mounts database changes #2404

Open
wants to merge 58 commits into
base: main
Choose a base branch
from

Conversation

luanluciano93
Copy link
Contributor

@luanluciano93 luanluciano93 commented Mar 9, 2024

We are moving Outfits(addons) & Mounts from storagevalues into their own respective database tables.
Increased the mount id size from uint8_t to uint16_t as we already reached 200 mounts and we are soon hitting a wall.

Credits: @EvilHero90

@luanluciano93 luanluciano93 marked this pull request as draft March 9, 2024 10:25
@luanluciano93 luanluciano93 changed the title improve: outfit(addons) & Mounts database changes improve: Outfit(addons) & Mounts database changes Mar 9, 2024
@Aerwix
Copy link
Contributor

Aerwix commented Mar 13, 2024

Will this fix that bug where the GOD is wearing exclusive outfits(GM Outfits) that can't have mount and you select Change Outfit and it crashes the client?

Someone made an issue but I don't get it well.

@omarcopires
Copy link
Contributor

Will this fix that bug where the GOD is wearing exclusive outfits(GM Outfits) that can't have mount and you select Change Outfit and it crashes the client?

Someone made an issue but I don't get it well.

These outfits have been removed as they are discontinued.
The problem occurs because there is no compatibility with the mounts and therefore the client crashes. The solution is to remove administrator clothing.

@odisk777
Copy link
Contributor

Will this fix that bug where the GOD is wearing exclusive outfits(GM Outfits) that can't have mount and you select Change Outfit and it crashes the client?

Someone made an issue but I don't get it well.

it is not possible in any customer that the GOD/GM has a mount. it is not designed for that.

@Aerwix
Copy link
Contributor

Aerwix commented Mar 13, 2024

Will this fix that bug where the GOD is wearing exclusive outfits(GM Outfits) that can't have mount and you select Change Outfit and it crashes the client?
Someone made an issue but I don't get it well.

it is not possible in any customer that the GOD/GM has a mount. it is not designed for that.

Not sure how works here of if it's possible to add a try catch option to AVOID the debug.

Back in time during the Panfestival you were able to wear GM outfit being player and of course client didn't crash with mount option.
Check minute 14, he wears the GM outfit and mount is removed instantly (this is exactly what we have to do to avoid the debug)
https://youtu.be/uLqSYAffQ-g?feature=shared&t=840

@odisk777
Copy link
Contributor

Will this fix that bug where the GOD is wearing exclusive outfits(GM Outfits) that can't have mount and you select Change Outfit and it crashes the client?
Someone made an issue but I don't get it well.

it is not possible in any customer that the GOD/GM has a mount. it is not designed for that.

Not sure how works here of if it's possible to add a try catch option to AVOID the debug.

Back in time during the Panfestival you were able to wear GM outfit being player and of course client didn't crash with mount option. Check minute 14, he wears the GM outfit and mount is removed instantly (this is exactly what we have to do to avoid the debug) https://youtu.be/uLqSYAffQ-g?feature=shared&t=840

Mmm I see your point, but I think that's only for event occasions, I don't think you give everyone access to GM's outfits.

@luanluciano93 luanluciano93 marked this pull request as ready for review March 14, 2024 03:59
@luanluciano93 luanluciano93 marked this pull request as ready for review June 14, 2024 17:16
@luanluciano93 luanluciano93 marked this pull request as draft June 15, 2024 12:30
@luanluciano93 luanluciano93 marked this pull request as ready for review June 16, 2024 02:09
@dudantas
Copy link
Member

We have a problem with this pull request; many sites use this form of storage to fetch mounts and outfits, if I'm not mistaken. Have you checked how this will be affected?

@luanluciano93
Copy link
Contributor Author

We have a problem with this pull request; many sites use this form of storage to fetch mounts and outfits, if I'm not mistaken. Have you checked how this will be affected?

I looked in the most used AAC:

none of them pull the outfits from the storage table binary

@phacUFPE
Copy link
Contributor

Is this PR ready for testing?

@luanluciano93
Copy link
Contributor Author

Is this PR ready for testing?

yes

Copy link
Contributor

This PR is stale because it has been open 45 days with no activity.

@github-actions github-actions bot added the Stale No activity label Nov 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Stale No activity
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants