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

Enchant stacking #52

Open
lingxDEV opened this issue May 4, 2022 · 15 comments
Open

Enchant stacking #52

lingxDEV opened this issue May 4, 2022 · 15 comments

Comments

@lingxDEV
Copy link

lingxDEV commented May 4, 2022

It's possible to stack enchants with engineering tinkers, you can get unlimited stats.

To reproduce:

  1. Skill engineering to 600/600
  2. .ad 38940 20
  3. .ad 90146 20
  4. Enchant Cloak with Greater Agility
  5. Enchant Cloak with Flexweave Underlay (engineering tinker)
  6. Enchant Cloak with Greater Agility
  7. Enchant Cloak with Flexweave Underlay (engineering tinker)
  8. Keep repeating and you will see your agility keeps increasing. You can keep doing this and get unlimited stats.
@Hughcoyan
Copy link

fact

@creativemindx
Copy link

This is ruined my server. Players are abusing and I can't stop them...

@welder1976
Copy link

spell_bonus_data exploit maybe

@welder1976
Copy link

INSERT INTO spell_bonus_data (entry,direct_bonus,dot_bonus,ap_bonus,ap_dot_bonus,comments) VALUES
(34004,0,0,-1,-1, 'Enchant - Greater Agility');

@welder1976
Copy link

need test

@alexkulya
Copy link
Owner

INSERT INTO spell_bonus_data (entry,direct_bonus,dot_bonus,ap_bonus,ap_dot_bonus,comments) VALUES
(34004,0,0,-1,-1, 'Enchant - Greater Agility');

Not working

freadblangks referenced this issue in freadblangks/pandaria_5.4.8 Dec 8, 2022
[Core/System]Merge NpcText System from TC
@rodrigoolvs
Copy link

has this already been fixed? I want to open my server, but I need to know if this serious problem is fixed.

@davidwow1024
Copy link
Contributor

Free not

@mikefernandz
Copy link

Hack the spell in SpellMgr.cpp and try

		case 34004: // Enchant Cloak - Greater Agility (Level 90)
				spellInfo->Attributes& SPELL_ATTR0_NOT_SHAPESHIFT;
				spellInfo->AttributesEx |= SPELL_ATTR1_NO_THREAT;

@mikefernandz
Copy link

Revisa si tienes el Spell en el grupo
INSERT INTO spell_group(id, spell_id, comment) VALUES (1, '34004 ', 'Enchant Cloak - Greater Agility ');

@rodrigoolvs
Copy link

I will test and come back with a feedback.

@GrenderG
Copy link

I will test and come back with a feedback.

Any update on this @rodrigoolvs?

@lingxDEV
Copy link
Author

Still bugged. Its because Engineering tinkers are not handled the right way. You should be able to have both a normal enchant and a tinker attached to the same item.

Because there is somewhat support for it, it will apply the tinker and remove the enchant visually but not actually remove it. Just visually.

@lingxDEV
Copy link
Author

@GrenderG

@Cagzwow
Copy link

Cagzwow commented Oct 9, 2023

Still bugged. Its because Engineering tinkers are not handled the right way. You should be able to have both a normal enchant and a tinker attached to the same item.

Because there is somewhat support for it, it will apply the tinker and remove the enchant visually but not actually remove it. Just visually.

Damn that's horrible.
I tried to fix this via db and core no success, tricky one

ctmwow added a commit to ctmwow/pandaria_5.4.8 that referenced this issue Apr 9, 2024
Engineering enchantments and regular enchantments should coexist

From search engines
ctmwow added a commit to ctmwow/pandaria_5.4.8 that referenced this issue Apr 9, 2024
Engineering enchantments and regular enchantments should coexist
From searcher
alexkulya added a commit that referenced this issue May 31, 2024
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

No branches or pull requests

10 participants