chromiecraft / chromiecraft

ChromieCraft Discussions and Bug Tracker
https://www.chromiecraft.com
135 stars 31 forks source link

Epic gems BoP do not require jewelcrafting #6050

Open novi69 opened 1 year ago

novi69 commented 1 year ago

What client do you play on?

enUS

Faction

Both

Content Phase:

70

Current Behaviour

Epic gems BoP do not require jewelcrafting so if I unlearn jc but I save epic gems in bag or bank those are still usable without profession. https://wowgaming.altervista.org/aowow/?item=33131 https://wowgaming.altervista.org/aowow/?item=33133 https://wowgaming.altervista.org/aowow/?item=33134 https://wowgaming.altervista.org/aowow/?item=33135 https://wowgaming.altervista.org/aowow/?item=33140 https://wowgaming.altervista.org/aowow/?item=33143 https://wowgaming.altervista.org/aowow/?item=33144

Expected Blizzlike Behaviour

Epic gems BoP should not be usable without 360 jewelcrafting

Source

https://www.wowhead.com/wotlk/item=33131/crimson-sun https://www.wowhead.com/wotlk/item=33133/don-julios-heart https://www.wowhead.com/wotlk/item=33134/kailees-rose https://www.wowhead.com/wotlk/item=33135/falling-star https://www.wowhead.com/wotlk/item=33140/blood-of-amber https://www.wowhead.com/wotlk/item=33143/stone-of-blades https://www.wowhead.com/wotlk/item=33144/facet-of-eternity

image

Steps to reproduce the problem

  1. use /s .setskill 755 375
  2. use .additem 33131 2
  3. unlearn jewelcrafting
  4. match epic gem in socket (this shouldn't be possible)

Extra Notes

No response

AC rev. hash/commit

https://github.com/chromiecraft/azerothcore-wotlk/commit/86bfef8874883a1726966326368918567c5531cd

Operating system

Ubuntu 20.04

Modules

Customizations

None

Server

ChromieCraft

balleny commented 1 year ago

can confirm, wowhead clearly states the jewelcrafting requirement

Gultask commented 1 year ago

image

https://www.wowhead.com/wotlk/item=33131/crimson-sun#comments:id=225584

https://web.archive.org/web/20110128042900/http://www.wowhead.com:80/item=33131

https://wowwiki-archive.fandom.com/wiki/Crimson_Sun?oldid=1816786

I believe this is a Classic change but we could add this feature as well.