Skip to content

Fix WorldSession constructor parameter in AuctionHouseBot#153

Merged
Kitzunu merged 1 commit into
masterfrom
awd
Jul 27, 2025
Merged

Fix WorldSession constructor parameter in AuctionHouseBot#153
Kitzunu merged 1 commit into
masterfrom
awd

Conversation

@Kitzunu
Copy link
Copy Markdown
Member

@Kitzunu Kitzunu commented Jul 25, 2025

Added missing parameter '0' to the WorldSession constructor call in AuctionHouseBot::Update to match the updated constructor signature.

Changes Proposed:

Issues Addressed:

  • Closes

SOURCE:

Tests Performed:

How to Test the Changes:

Added missing parameter '0' to the WorldSession constructor call in AuctionHouseBot::Update to match the updated constructor signature.
@Helias
Copy link
Copy Markdown
Member

Helias commented Jul 26, 2025

/home/runner/work/mod-ah-bot/mod-ah-bot/modules/mod-ah-bot/src/AuctionHouseBot.cpp:1031:18: fatal error: no matching constructor for initialization of 'WorldSession'
 1031 |     WorldSession _session(_account, std::move(accountName), 0, nullptr, SEC_PLAYER, sWorld->getIntConfig(CONFIG_EXPANSION), 0, LOCALE_enUS, 0, false, false, 0);
      |                  ^        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/runner/work/mod-ah-bot/mod-ah-bot/src/server/game/Server/WorldSession.h:332:5: note: candidate constructor not viable: requires 11 arguments, but 12 were provided
  332 |     WorldSession(uint32 id, std::string&& name, std::shared_ptr<WorldSocket> sock, AccountTypes sec, uint8 expansion, time_t mute_time, LocaleConstant locale, uint32 recruiter, bool isARecruiter, bool skipQueue, uint32 TotalTime);
      |     ^            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/runner/work/mod-ah-bot/mod-ah-bot/src/server/game/Server/WorldSession.h:1203:5: note: candidate constructor not viable: requires single argument 'right', but 12 arguments were provided
 1203 |     WorldSession(WorldSession const& right) = delete;
      |     ^            ~~~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.
gmake[2]: *** [modules/CMakeFiles/modules.dir/build.make:93: modules/CMakeFiles/modules.dir/mod-ah-bot/src/AuctionHouseBot.cpp.o] Error 1
gmake[1]: *** [CMakeFiles/Makefile2:1473: modules/CMakeFiles/modules.dir/all] Error 2
gmake[1]: *** Waiting for unfinished jobs....

@Kitzunu
Copy link
Copy Markdown
Member Author

Kitzunu commented Jul 26, 2025

Well yes because the AC pr isnt merged

@Kitzunu Kitzunu merged commit 80b08a5 into master Jul 27, 2025
1 check failed
@Kitzunu Kitzunu deleted the awd branch July 27, 2025 06:54
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

Successfully merging this pull request may close these issues.

3 participants