Commit graph

60 commits

Author SHA1 Message Date
1e8d21b517 Merge pull request 'Skip playerbots so bot re-rolls don't spam the realm' (#1) from fix/skip-playerbots into main 2026-09-03 11:21:44 -06:00
Claude
0ad5f1d345
fix: skip playerbots so bot re-rolls do not spam the server
Some checks failed
core-build / build (pull_request) Has been cancelled
PlayerbotFactory calls Player::GiveLevel() in six places while randomising a
bot, and GiveLevel fires OnPlayerLevelChanged exactly as a real ding does. On
a Playerbots realm that means every bot re-roll past level 10/20/.../80 sends
a server-wide broadcast plus a raid warning and hands the bot gold and two
reward items. Turning off CongratsPerLevel.Enable does not help: the
reward-level announcements sit outside that gate.

Adds Congrats.SkipBots (default 1). The bot test is guarded by
#ifdef MOD_PLAYERBOTS, which the core defines for every module when
mod-playerbots is present, so a core built without it is unaffected.

Also adds the missing `override` on OnPlayerLogin. The signature already
matched the base, so behaviour is unchanged -- it just stops a future
signature change from silently detaching the hook.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019vTNrJGsprcjdjjjDJD5ZM
2026-09-03 17:21:11 +00:00
sudlud
fed4752e0c
Merge pull request #42 from azerothcore/sudlud-pach-42-4
fix order of language entries
2025-02-28 14:15:30 +01:00
Ludwig
66f9e0a4f5 fix order of language entries 2025-02-28 13:59:52 +01:00
sudlud
0c9c0c9bc8
Merge pull request #41 from jamssismy/patch-2
Update mod_congratsonlevel.cpp
2025-02-28 13:58:09 +01:00
jamssismy
ae5e5d3b28
Update mod_congratsonlevel.cpp
I aligned the code, and actually, our communication is done through AI translation. I initially thought your mention of alignment referred to the unification of the level display function calls. Thank you for the reminder. I'm not a professional programmer and haven't received formal training, so I don't have good code alignment habits. Thanks again for your reminder!~
2025-02-28 11:29:09 +08:00
jamssismy
4fe199845e
Update mod_congratsonlevel.cpp
The calling method for levels has been standardized
<< static_cast<int>(player->GetLevel()) <<
2025-02-27 16:41:14 +08:00
jamssismy
0d233e2c33
Update mod_congratsonlevel.cpp
The software or game now supports user interface prompts in both Simplified Chinese and Traditional Chinese, making it more accessible to a wider range of Chinese-speaking users.
A correction has been made to the on-screen display of currency amounts. Specifically, where small amounts of currency were being shown, the unit was incorrectly displayed as "Gold." This has been corrected so that it now accurately displays "Copper," which is the appropriate unit for smaller denominations in games like World of Warcraft.
2025-02-27 10:02:34 +08:00
sudlud
d756b42db3
Merge pull request #40 from azerothcore/fix-level-messages-1
fix messages not showing level correctly
2025-02-26 20:34:26 +01:00
Ludwig
239bae9d5b fix messages not showing level correctly 2025-02-26 20:25:04 +01:00
sudlud
6f7a3dbbeb
Merge pull request #39 from azerothcore/fix-reward-messages-1
fix reward level up message and CongratsPerLevel.Enable functionality
2025-02-26 12:23:36 +01:00
Ludwig
b78a72d5ce Merge remote-tracking branch 'origin/master' into fix-reward-messages-1 2025-02-26 11:18:50 +01:00
sudlud
eb72cb9a29
Merge pull request #38 from azerothcore/sudlud-pach-42-3
fix build and add enabled hook lists for performance improvement
2025-02-26 11:18:33 +01:00
Ludwig
d133684206 fix for WorldSessionMgr 2025-02-26 10:12:16 +01:00
Ludwig
b9894eea11 fix reward level up message and CongratsPerLevel.Enable functionality 2025-02-26 08:56:16 +01:00
Ludwig
0f8ee189b1 fix unintended changes 2025-02-26 08:50:31 +01:00
Ludwig
793fc6ade3 restore braces 2025-02-26 08:49:55 +01:00
Ludwig
f444ef80ab fix build and add enabled hook lists for performance improvement 2025-02-26 08:39:57 +01:00
Walter Pagani
668b19995b
fix(build) SendNotification is not a member of WorldSession (#35) 2024-10-24 11:12:36 +02:00
cptcl
58652e93a2
Update mod_congratsonlevel.cpp (#33)
changed getLevel to GetLevel
2024-07-25 22:42:47 +02:00
Walter Pagani
494cde182c
feat. Add option to Learn or cast spells (#30)
* feat. Add option to Learn or cast spells

* boolean learn attribute
2023-09-05 05:17:05 -03:00
Walter Pagani
8e4aef41dc
feat. Move awards to database and enable races/classes (#27)
* feat. Move awards to database and enable races/classes

* feat. Allow multiple records in the table

* feat. Check that the values in the register are greater than zero
2023-07-19 07:17:47 -03:00
Walter Pagani
5961c1bd12
feat. (Repository): Add issue and pull request templates (#26) 2023-07-12 06:30:52 -03:00
Stefano Borzì
1494dd61e8
Merge pull request #21 from pangolp/spanish
(Translations/Spanish) Adding Spanish Messages
2022-05-19 22:55:01 +02:00
Pagani Walter
670a554d50 Adding ident in the Switch 2022-05-19 16:57:51 -03:00
Pagani Walter
c3f5966f5e (Translations/Spanish) Adding Spanish Messages 2022-05-16 17:51:36 -03:00
Patrick Lewis
426321584f
chore: switch to reusable module workflow (#20) 2022-04-19 20:15:34 -07:00
Stefano Borzì
3784c4f924
Merge pull request #19 from SoulSeekkor/master
Updated README.
2022-04-10 11:01:08 +02:00
SoulSeekkor
100ffd3087 Updated README. 2022-04-09 14:23:52 -05:00
Stefano Borzì
967fbbdb9b
Merge pull request #18 from SoulSeekkor/master
Fixed compiler deprecation warnings
2022-04-09 20:28:24 +02:00
SoulSeekkor
c0d6a6af9f Updated to clear compiler warnings. 2022-04-09 12:19:15 -05:00
SoulSeekkor
f4e142d9e2
fix bad config value (#17) 2022-03-03 10:43:03 +07:00
SoulSeekkor
3f540b4c91 fix bad config value 2022-03-02 21:08:57 -06:00
BytesGalore
9705cfb8ee
refactor: updated module registration to core rev 4++ (#15)
Co-authored-by: BytesGalore <3903215+BytesGalore@users.noreply.github.com>
2021-09-18 21:45:01 +07:00
Stefano Borzì
d931077c99
fix: build and ci (#14) 2021-05-30 22:57:47 -07:00
Stefano Borzì
b8c313cb5a
Merge pull request #13 from ReyDonovan/master
fix all bug close #9
2021-03-31 22:44:27 +02:00
ReyDonovan
90fad08fa6 fix build 2021-03-31 22:45:06 +03:00
ReyDonovan
754c241066 fix all bug close #9 2021-03-31 22:41:21 +03:00
Stefano Borzì
162bf8a92a
Merge pull request #12 from Mitradis/patch-1
fix for last PR ACE remove
2021-03-03 18:45:18 +01:00
Vitaliy
78e6452eb2
Update mod_congratsonlevel.cpp 2021-03-02 13:16:56 +03:00
Vitaliy
237793f4ae
Delete after_ws_install.cmake 2021-03-02 13:16:30 +03:00
Vitaliy
ec1de1033d
fix for last PR ACE remove
https://github.com/azerothcore/azerothcore-wotlk/pull/4608
2021-03-02 13:16:11 +03:00
Patrick Lewis
be4d582f6c
docs(readme): Update build status badge (#11) 2020-11-21 18:58:16 -08:00
Stefano Borzì
2fc6f67d95
Merge pull request #10 from azerothcore/github-actions
feat(CI): move from Travis to GitHub Actions
2020-08-30 22:44:56 +02:00
Helias
2482ec9ed6 feat(CI): move from Travis to GitHub Actions 2020-08-30 18:45:28 +02:00
Micrah
211294f28e
Merge pull request #8 from milestorme/master
clean up config
2019-08-21 11:43:27 +10:00
Micrah
6b35f444d9
update fix config 2019-08-21 10:49:32 +10:00
Micrah
de8685a119
Merge pull request #7 from milestorme/master
Added new config option disable/enable per level up announcer this closes https://github.com/azerothcore/mod-congrats-on-level/issues/4
2019-06-10 06:10:25 +10:00
Micrah
a034126731
update 2019-06-09 13:24:05 +10:00
Micrah
5b985afcb2
add per level announcer 2019-06-09 13:22:17 +10:00