azerothcore / azerothcore-wotlk

Complete Open Source and Modular solution for MMO
http://www.azerothcore.org
GNU Affero General Public License v3.0
6.55k stars 2.63k forks source link

Report on ICC 10H Access Requirements Issue #19576

Open kambire opened 3 months ago

kambire commented 3 months ago

Current Behaviour

There is an issue concerning the access requirements for Icecrown Citadel (ICC) 10-man heroic (10H) mode. Currently, if the raid leader has defeated the Lich King (LK) in 10-man normal (10N) mode and has the corresponding title, the expectation is that the entire raid group should be allowed entry into ICC 10H. However, the system incorrectly requires every member of the raid to have also defeated LK in 10N mode and possess the title.

Expected Blizzlike Behaviour

Only the raid leader should be required to have defeated LK in 10N mode and hold the title for the entire raid group to be granted access to ICC 10H. Other raid members should not need to meet this requirement.

Source

The system currently requires every individual member of the raid group to have defeated LK in 10N mode and have the title to enter ICC 10H, regardless of the raid leader's qualifications.

Steps to reproduce the problem

1 Form a raid group with a raid leader who has defeated LK in 10N mode and has the title. 2 Include raid members who have not defeated LK in 10N mode and do not have the title. 3 Attempt to enter ICC 10H.

Extra Notes

No response

AC rev. hash/commit

AC> AzerothCore rev. fdd8ff6e0481+ 2024-08-07 18:13:31 +0200 (master branch) (Unix, Release, Static) Connected players: 56. Characters in world: 54. Connection peak: 62. Server uptime: 4 hour(s) 12 minute(s) 29 second(s) Update time diff: 2ms. Last 500 diffs summary: |- Mean: 3ms |- Median: 2ms |- Percentiles (95, 99, max): 9ms, 9ms, 10ms Using SSL version: OpenSSL 3.0.2 15 Mar 2022 (library: OpenSSL 3.0.2 15 Mar 2022) Using Boost version: 1.74.0 Using CMake version: 3.22.1 Using MySQL version: 80039 Found MySQL Executable: /usr/bin/mysql Compiled on: Linux 5.15.0-117-generic Worldserver listening connections on port %u Realmlist (Realm Id: 2) configured in port 8086 VMAPs status: Enabled. LineOfSight: true, getHeight: true, indoorCheck: true MMAPs status: Enabled maps directory located in ./maps. Total size: 291014951 bytes vmaps directory located in ./vmaps. Total size: 658130721 bytes mmaps directory located in ./mmaps. Total size: 2192910844 bytes Default DBC locale: enUS. All available DBC locales: enUS Using World DB: ACDB 335.12-dev Latest LoginDatabase update: 2024_01_20_00.sql Latest CharacterDatabase update: trasmorg.sql Latest WorldDatabase update: trasm_world_texts.sql LoginDatabase queue size: 0 CharacterDatabase queue size: 0 WorldDatabase queue size: 0 List of enabled modules: |- mod-anticheat |- mod-breaking-news-override |- mod-cfbg |- mod-transmog

Operating system

Ubuntu 24.04 LTS

Custom changes or Modules

No response

xinhochen commented 2 months ago

It's a feature, and I love it.

kambire commented 2 months ago

It's a feature, and I love it.

I don't think it's necessarily a bad change, it's just that users weren't accustomed to it, and many noticed the difference. If there's a way to revert to the old format temporarily, we could inform users about the change, making the transition smoother and less abrupt.

YggdrasilWotLK commented 1 month ago

@kambire Try running this in your world database to revert back to the old leader check system :)

UPDATE dungeon_access_requirements
SET leader_only = 1
WHERE dungeon_access_id IN (104, 105);
kambire commented 1 month ago

UPDATE dungeon_access_requirements SET leader_only = 1 WHERE dungeon_access_id IN (104, 105);

you save my life xD