From d7006574ac0e22d7f1b0613cf3b2f7b39865f41d Mon Sep 17 00:00:00 2001 From: Adam Date: Tue, 20 Mar 2018 20:30:05 +1100 Subject: [PATCH 1/8] proper festival handling (woah) --- src/common/Common.h | 2 +- .../sapphire_zone/Actor/PlayerEvent.cpp | 1 - .../DebugCommand/DebugCommandHandler.cpp | 27 +++++++++---------- .../sapphire_zone/Zone/TerritoryMgr.cpp | 20 ++++++++++++++ src/servers/sapphire_zone/Zone/TerritoryMgr.h | 7 +++++ src/servers/sapphire_zone/Zone/Zone.cpp | 10 ++++++- 6 files changed, 49 insertions(+), 18 deletions(-) diff --git a/src/common/Common.h b/src/common/Common.h index 03be85bc..124e318d 100644 --- a/src/common/Common.h +++ b/src/common/Common.h @@ -582,7 +582,7 @@ namespace Common { GearSetEquipMsg = 0x321, - DisableCurrentFestival = 0x386, + SetFestival = 0x386, // param1: festival.exd index ToggleOrchestrionUnlock = 0x396, Dismount = 0x3A0, diff --git a/src/servers/sapphire_zone/Actor/PlayerEvent.cpp b/src/servers/sapphire_zone/Actor/PlayerEvent.cpp index 7aff9966..60761ef2 100644 --- a/src/servers/sapphire_zone/Actor/PlayerEvent.cpp +++ b/src/servers/sapphire_zone/Actor/PlayerEvent.cpp @@ -16,7 +16,6 @@ #include "Action/EventAction.h" #include "Action/EventItemAction.h" -#include "Event/EventHandler.h" #include "Event/EventHandler.h" #include "Zone/Zone.h" diff --git a/src/servers/sapphire_zone/DebugCommand/DebugCommandHandler.cpp b/src/servers/sapphire_zone/DebugCommand/DebugCommandHandler.cpp index 1d1d6512..71d8d8fe 100644 --- a/src/servers/sapphire_zone/DebugCommand/DebugCommandHandler.cpp +++ b/src/servers/sapphire_zone/DebugCommand/DebugCommandHandler.cpp @@ -134,6 +134,7 @@ void Core::DebugCommandHandler::help( char* data, Entity::Player& player, boost: void Core::DebugCommandHandler::set( char * data, Entity::Player& player, boost::shared_ptr< DebugCommand > command ) { auto pLog = g_fw.get< Logger >(); + auto pTerriMgr = g_fw.get< TerritoryMgr >(); auto pDb = g_fw.get< Db::DbWorkerPool< Db::CharaDbConnection > >(); std::string subCommand = ""; std::string params = ""; @@ -309,6 +310,17 @@ void Core::DebugCommandHandler::set( char * data, Entity::Player& player, boost: player.getCurrentZone()->setWeatherOverride( static_cast< Common::Weather >( weatherId ) ); } + else if( subCommand == "festival" ) + { + uint16_t festivalId; + sscanf( params.c_str(), "%hu", &festivalId ); + + pTerriMgr->setCurrentFestival( festivalId ); + } + else if( subCommand == "festivaldisable" ) + { + pTerriMgr->disableCurrentFestival(); + } else { player.sendUrgent( subCommand + " is not a valid SET command." ); @@ -829,21 +841,6 @@ void Core::DebugCommandHandler::instance( char* data, Entity::Player &player, bo instance->setBranch( branch ); } - else if( subCommand == "festival" ) - { - uint32_t festivalId; - sscanf( params.c_str(), "%d", &festivalId ); - - player.getCurrentZone()->setCurrentFestival( static_cast< uint16_t >( festivalId ) ); - } - else if( subCommand == "disablefestival" ) - { - Network::Packets::ZoneChannelPacket< Network::Packets::Server::FFXIVIpcActorControl143 > actorControl( player.getId() ); - actorControl.data().category = Core::Common::ActorControlType::DisableCurrentFestival; - player.queuePacket( actorControl ); - - player.getCurrentZone()->setCurrentFestival( 0 ); - } else if ( subCommand == "qte_start" ) { auto instance = boost::dynamic_pointer_cast< InstanceContent >( player.getCurrentZone() ); diff --git a/src/servers/sapphire_zone/Zone/TerritoryMgr.cpp b/src/servers/sapphire_zone/Zone/TerritoryMgr.cpp index 500f2d93..b6ee10f4 100644 --- a/src/servers/sapphire_zone/Zone/TerritoryMgr.cpp +++ b/src/servers/sapphire_zone/Zone/TerritoryMgr.cpp @@ -356,5 +356,25 @@ Core::ZonePtr Core::TerritoryMgr::getLinkedInstance( uint32_t playerId ) const return nullptr; } +const uint16_t Core::TerritoryMgr::getCurrentFestival() const +{ + return m_currentFestival; +} + +void Core::TerritoryMgr::setCurrentFestival( uint16_t festivalId ) +{ + m_currentFestival = festivalId; + + for( const auto& zone : m_zoneSet ) + { + zone->setCurrentFestival( m_currentFestival ); + } +} + +void Core::TerritoryMgr::disableCurrentFestival() +{ + setCurrentFestival( 0 ); +} + diff --git a/src/servers/sapphire_zone/Zone/TerritoryMgr.h b/src/servers/sapphire_zone/Zone/TerritoryMgr.h index c6bcf796..c4602e5e 100644 --- a/src/servers/sapphire_zone/Zone/TerritoryMgr.h +++ b/src/servers/sapphire_zone/Zone/TerritoryMgr.h @@ -118,6 +118,10 @@ namespace Core /*! returns an instancePtr if the player is still bound to an isntance */ ZonePtr getLinkedInstance( uint32_t playerId ) const; + void setCurrentFestival( uint16_t festivalId ); + void disableCurrentFestival(); + const uint16_t getCurrentFestival() const; + private: using TerritoryTypeDetailCache = std::unordered_map< uint16_t, Data::TerritoryTypePtr >; using InstanceIdToZonePtrMap = std::unordered_map< uint32_t, ZonePtr >; @@ -154,6 +158,9 @@ namespace Core /*! set of ZonePtrs for quick iteration*/ std::set< ZonePtr > m_instanceZoneSet; + /*! id of current festival to set for public zones from festival.exd */ + uint16_t m_currentFestival; + public: /*! returns a list of instanceContent InstanceIds currently active */ InstanceIdList getInstanceContentIdList( uint16_t instanceContentId ) const; diff --git a/src/servers/sapphire_zone/Zone/Zone.cpp b/src/servers/sapphire_zone/Zone/Zone.cpp index ba5d5869..570de342 100644 --- a/src/servers/sapphire_zone/Zone/Zone.cpp +++ b/src/servers/sapphire_zone/Zone/Zone.cpp @@ -12,6 +12,7 @@ #include #include #include +#include #include "Zone.h" #include "InstanceContent.h" @@ -136,8 +137,15 @@ uint16_t Core::Zone::getCurrentFestival() const void Core::Zone::setCurrentFestival( uint16_t festivalId ) { m_currentFestivalId = festivalId; -} + for( const auto& playerEntry : m_playerMap ) + { + auto player = playerEntry.second; + + ActorControlPacket143 enableFestival( player->getId(), SetFestival, m_currentFestivalId ); + playerEntry.second->queuePacket( enableFestival ); + } +} void Core::Zone::loadCellCache() { From dd8ed62430a1c1a00567ecfa3b4290100f636704 Mon Sep 17 00:00:00 2001 From: Adam Date: Tue, 20 Mar 2018 22:08:25 +1100 Subject: [PATCH 2/8] eventPlay -> playScene, added playSceneChain --- src/servers/Scripts/common/Aethernet.cpp | 8 +- src/servers/Scripts/common/Aetheryte.cpp | 12 +- .../Scripts/common/CmnDefCutSceneReplay.cpp | 4 +- src/servers/Scripts/common/CmnDefInnBed.cpp | 8 +- .../Scripts/common/CmnDefLinkShell.cpp | 8 +- .../Scripts/common/HouFurOrchestrion.cpp | 2 +- .../Scripts/opening/OpeningGridania.cpp | 14 +- src/servers/Scripts/opening/OpeningLimsa.cpp | 14 +- src/servers/Scripts/opening/OpeningUldah.cpp | 14 +- src/servers/Scripts/quest/ManFst001.cpp | 12 +- src/servers/Scripts/quest/ManFst002.cpp | 30 ++-- src/servers/Scripts/quest/ManFst004.cpp | 129 ++++++++---------- src/servers/Scripts/quest/ManSea001.cpp | 30 ++-- src/servers/Scripts/quest/ManSea002.cpp | 20 +-- src/servers/Scripts/quest/ManWil001.cpp | 30 ++-- src/servers/Scripts/quest/ManWil002.cpp | 20 +-- .../quest/subquest/gridania/SubFst001.cpp | 8 +- .../quest/subquest/gridania/SubFst002.cpp | 4 +- .../quest/subquest/gridania/SubFst010.cpp | 4 +- .../quest/subquest/gridania/SubFst013.cpp | 26 ++-- src/servers/sapphire_zone/Actor/Player.h | 23 +++- .../sapphire_zone/Actor/PlayerEvent.cpp | 84 +++++++----- .../sapphire_zone/Event/EventHandler.cpp | 16 ++- .../sapphire_zone/Event/EventHandler.h | 10 +- .../Network/Handlers/EventHandlers.cpp | 3 + 25 files changed, 276 insertions(+), 257 deletions(-) diff --git a/src/servers/Scripts/common/Aethernet.cpp b/src/servers/Scripts/common/Aethernet.cpp index 6d4bcc17..8c31b874 100644 --- a/src/servers/Scripts/common/Aethernet.cpp +++ b/src/servers/Scripts/common/Aethernet.cpp @@ -21,10 +21,8 @@ public: { if( player.isAetheryteRegistered( eventId & 0xFFFF ) ) { - player.eventPlay( eventId, 2, 0, []( Entity::Player& player, const Event::SceneResult& result ) - { - if( result.param1 == 256 ) - { + player.playScene( eventId, 2, 0, []( Entity::Player& player, const Event::SceneResult& result ) { + if( result.param1 == 256 ) { player.teleport( result.param2, 2 ); } } ); @@ -34,7 +32,7 @@ public: player.eventActionStart( eventId, ACTION_ATTUNE, []( Entity::Player& player, uint32_t eventId, uint64_t additional ) { player.registerAetheryte( eventId & 0xFFFF ); - player.eventPlay( eventId, 3, 0, 0, 0 ); + player.playScene( eventId, 3, 0, 0, 0 ); }, [] ( Entity::Player& ply, uint32_t evntId, uint64_t additional ) { diff --git a/src/servers/Scripts/common/Aetheryte.cpp b/src/servers/Scripts/common/Aetheryte.cpp index 95ba67db..93fa20af 100644 --- a/src/servers/Scripts/common/Aetheryte.cpp +++ b/src/servers/Scripts/common/Aetheryte.cpp @@ -23,20 +23,16 @@ public: { if( player.isAetheryteRegistered( eventId & 0xFFFF ) ) { - player.eventPlay( eventId, 0, 1, []( Entity::Player& player, const Event::SceneResult& result ) - { + player.playScene( eventId, 0, 1, []( Entity::Player& player, const Event::SceneResult& result ) { if( result.param1 == 256 ) // set homepoint { player.setHomepoint( result.eventId & 0xFFFF ); player.sendQuestMessage( result.eventId, 2, 0xEA, 0, 0 ); - } - else if( result.param1 == 512 ) // aethernet access + } else if( result.param1 == 512 ) // aethernet access { - if( result.param2 == 4 ) - { + if( result.param2 == 4 ) { player.teleport( result.param3, 2 ); - } - else if( result.param2 == 2 ) // register favored destination + } else if( result.param2 == 2 ) // register favored destination { } diff --git a/src/servers/Scripts/common/CmnDefCutSceneReplay.cpp b/src/servers/Scripts/common/CmnDefCutSceneReplay.cpp index ebe216b3..5b853704 100644 --- a/src/servers/Scripts/common/CmnDefCutSceneReplay.cpp +++ b/src/servers/Scripts/common/CmnDefCutSceneReplay.cpp @@ -18,7 +18,7 @@ public: } }; - player.eventPlay( getId(), 0, 0x2000, 0, 1, callback ); + player.playScene( getId( ), 0, 0x2000, 0, 1, callback ); } void Scene00001( Entity::Player& player, uint16_t returnScene ) @@ -28,7 +28,7 @@ public: // todo: this is fucked }; - player.eventPlay( getId(), 1, 0xFB2EC8F8, 0, 1, returnScene, callback ); + player.playScene( getId( ), 1, 0xFB2EC8F8, 0, 1, returnScene, callback ); } void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override diff --git a/src/servers/Scripts/common/CmnDefInnBed.cpp b/src/servers/Scripts/common/CmnDefInnBed.cpp index 8a88d3fd..95ee3492 100644 --- a/src/servers/Scripts/common/CmnDefInnBed.cpp +++ b/src/servers/Scripts/common/CmnDefInnBed.cpp @@ -17,7 +17,7 @@ public: Scene00001( player, result.param2 ); }; - player.eventPlay( getId(), 0, 0x2000, 0, 1, callback ); + player.playScene( getId( ), 0, 0x2000, 0, 1, callback ); } // lay down @@ -28,13 +28,13 @@ public: Scene00002( player, result.param2 ); }; - player.eventPlay( getId(), 1, 0xF32E48F8, 0, 1, param, callback ); + player.playScene( getId( ), 1, 0xF32E48F8, 0, 1, param, callback ); } // logout void Scene00002( Entity::Player& player, uint16_t param ) { - player.eventPlay( getId(), 2, 0xF32E48F8, 0, 1, param, nullptr ); + player.playScene( getId( ), 2, 0xF32E48F8, 0, 1, param, nullptr ); } // wake up @@ -43,7 +43,7 @@ public: auto callback = []( Entity::Player& player, const Event::SceneResult& result ) {}; - player.eventPlay( getId(), 100, 0x800, 0, 0, callback ); + player.playScene( getId( ), 100, 0x800, 0, 0, callback ); } void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override diff --git a/src/servers/Scripts/common/CmnDefLinkShell.cpp b/src/servers/Scripts/common/CmnDefLinkShell.cpp index 45c2a64b..c1b831fb 100644 --- a/src/servers/Scripts/common/CmnDefLinkShell.cpp +++ b/src/servers/Scripts/common/CmnDefLinkShell.cpp @@ -32,25 +32,25 @@ public: } }; - player.eventPlay( getId(), 1, 0, 0, 0, callback ); + player.playScene( getId( ), 1, 0, 0, 0, callback ); } // create linkshell void Scene00002( Entity::Player& player ) { - player.eventPlay( getId(), 2, 0, 0, 0 ); + player.playScene( getId( ), 2, 0, 0, 0 ); } // rename linkshell void Scene00003( Entity::Player& player ) { - player.eventPlay( getId(), 3, 0, 0, 0 ); + player.playScene( getId( ), 3, 0, 0, 0 ); } // remove linkshell void Scene00004( Entity::Player& player ) { - player.eventPlay( getId(), 4, 0, 0, 0 ); + player.playScene( getId( ), 4, 0, 0, 0 ); } void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override diff --git a/src/servers/Scripts/common/HouFurOrchestrion.cpp b/src/servers/Scripts/common/HouFurOrchestrion.cpp index 6340bc9d..b3aabcc8 100644 --- a/src/servers/Scripts/common/HouFurOrchestrion.cpp +++ b/src/servers/Scripts/common/HouFurOrchestrion.cpp @@ -10,7 +10,7 @@ public: void Scene00000( Entity::Player& player ) { - player.eventPlay( getId(), 0, 0x2000, 0, 1 ); + player.playScene( getId( ), 0, 0x2000, 0, 1 ); } void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override diff --git a/src/servers/Scripts/opening/OpeningGridania.cpp b/src/servers/Scripts/opening/OpeningGridania.cpp index 0d13df54..7f123e01 100644 --- a/src/servers/Scripts/opening/OpeningGridania.cpp +++ b/src/servers/Scripts/opening/OpeningGridania.cpp @@ -34,24 +34,24 @@ private: Scene00001( player ); }; - player.eventPlay( getId(), 0, NO_DEFAULT_CAMERA | INVIS_ENPC | - CONDITION_CUTSCENE | HIDE_UI | - HIDE_HOTBAR | SILENT_ENTER_TERRI_ENV, 0, 1, callback ); + player.playScene( getId( ), 0, NO_DEFAULT_CAMERA | INVIS_ENPC | + CONDITION_CUTSCENE | HIDE_UI | + HIDE_HOTBAR | SILENT_ENTER_TERRI_ENV, 0, 1, callback ); } void Scene00001( Entity::Player& player ) { - player.eventPlay( getId(), 1, NO_DEFAULT_CAMERA | HIDE_HOTBAR, 1, 0x32 ); + player.playScene( getId( ), 1, NO_DEFAULT_CAMERA | HIDE_HOTBAR, 1, 0x32 ); } void Scene00020( Entity::Player& player ) { - player.eventPlay( getId(), 20, NO_DEFAULT_CAMERA | HIDE_HOTBAR, 0, 1 ); + player.playScene( getId( ), 20, NO_DEFAULT_CAMERA | HIDE_HOTBAR, 0, 1 ); } void Scene00030( Entity::Player& player ) { - player.eventPlay( getId(), 30, NO_DEFAULT_CAMERA | HIDE_HOTBAR, 0, 0 ); + player.playScene( getId( ), 30, NO_DEFAULT_CAMERA | HIDE_HOTBAR, 0, 0 ); } void Scene00040( Entity::Player& player ) @@ -65,7 +65,7 @@ private: } }; - player.eventPlay( getId(), 40, 1, 2, 1, callback ); + player.playScene( getId( ), 40, 1, 2, 1, callback ); } public: diff --git a/src/servers/Scripts/opening/OpeningLimsa.cpp b/src/servers/Scripts/opening/OpeningLimsa.cpp index 8011377f..0d57d41f 100644 --- a/src/servers/Scripts/opening/OpeningLimsa.cpp +++ b/src/servers/Scripts/opening/OpeningLimsa.cpp @@ -49,24 +49,24 @@ private: Scene00001( player ); }; - player.eventPlay( getId(), 0, NO_DEFAULT_CAMERA | INVIS_ENPC | - CONDITION_CUTSCENE | HIDE_UI | - HIDE_HOTBAR | SILENT_ENTER_TERRI_ENV, 0, 1, callback ); + player.playScene( getId( ), 0, NO_DEFAULT_CAMERA | INVIS_ENPC | + CONDITION_CUTSCENE | HIDE_UI | + HIDE_HOTBAR | SILENT_ENTER_TERRI_ENV, 0, 1, callback ); } void Scene00001( Entity::Player& player ) { - player.eventPlay( getId(), 1, NO_DEFAULT_CAMERA | HIDE_HOTBAR, 1, 0x32 ); + player.playScene( getId( ), 1, NO_DEFAULT_CAMERA | HIDE_HOTBAR, 1, 0x32 ); } void Scene00020( Entity::Player& player ) { - player.eventPlay( getId(), 20, NO_DEFAULT_CAMERA | HIDE_HOTBAR, 0, 1 ); + player.playScene( getId( ), 20, NO_DEFAULT_CAMERA | HIDE_HOTBAR, 0, 1 ); } void Scene00030( Entity::Player& player ) { - player.eventPlay( getId(), 30, NO_DEFAULT_CAMERA | HIDE_HOTBAR, 0, 0 ); + player.playScene( getId( ), 30, NO_DEFAULT_CAMERA | HIDE_HOTBAR, 0, 0 ); } void Scene00040( Entity::Player& player ) @@ -80,7 +80,7 @@ private: } }; - player.eventPlay( getId(), 40, NO_DEFAULT_CAMERA, 2, 1, callback ); + player.playScene( getId( ), 40, NO_DEFAULT_CAMERA, 2, 1, callback ); } diff --git a/src/servers/Scripts/opening/OpeningUldah.cpp b/src/servers/Scripts/opening/OpeningUldah.cpp index 437b66c1..22b2bb92 100644 --- a/src/servers/Scripts/opening/OpeningUldah.cpp +++ b/src/servers/Scripts/opening/OpeningUldah.cpp @@ -35,24 +35,24 @@ private: Scene00001( player ); }; - player.eventPlay( getId(), 0, NO_DEFAULT_CAMERA | INVIS_ENPC | - CONDITION_CUTSCENE | HIDE_UI | - HIDE_HOTBAR | SILENT_ENTER_TERRI_ENV, 0, 1, callback ); + player.playScene( getId( ), 0, NO_DEFAULT_CAMERA | INVIS_ENPC | + CONDITION_CUTSCENE | HIDE_UI | + HIDE_HOTBAR | SILENT_ENTER_TERRI_ENV, 0, 1, callback ); } void Scene00001( Entity::Player& player ) { - player.eventPlay( getId(), 1, HIDE_HOTBAR | NO_DEFAULT_CAMERA, 1, 0x32 ); + player.playScene( getId( ), 1, HIDE_HOTBAR | NO_DEFAULT_CAMERA, 1, 0x32 ); } void Scene00020( Entity::Player& player ) { - player.eventPlay( getId(), 20, HIDE_HOTBAR | NO_DEFAULT_CAMERA, 0, 1 ); + player.playScene( getId( ), 20, HIDE_HOTBAR | NO_DEFAULT_CAMERA, 0, 1 ); } void Scene00030( Entity::Player& player ) { - player.eventPlay( getId(), 30, HIDE_HOTBAR | NO_DEFAULT_CAMERA, 0, 0 ); + player.playScene( getId( ), 30, HIDE_HOTBAR | NO_DEFAULT_CAMERA, 0, 0 ); } void Scene00040( Entity::Player& player ) @@ -63,7 +63,7 @@ private: Scene00030( player ); }; - player.eventPlay( getId(), 40, NO_DEFAULT_CAMERA, 2, 1, callback ); + player.playScene( getId( ), 40, NO_DEFAULT_CAMERA, 2, 1, callback ); } diff --git a/src/servers/Scripts/quest/ManFst001.cpp b/src/servers/Scripts/quest/ManFst001.cpp index 41907eaa..33049e36 100644 --- a/src/servers/Scripts/quest/ManFst001.cpp +++ b/src/servers/Scripts/quest/ManFst001.cpp @@ -44,7 +44,7 @@ private: } }; - player.eventPlay( getId(), 0, HIDE_HOTBAR, 0, 0, callback ); + player.playScene( getId( ), 0, HIDE_HOTBAR, 0, 0, callback ); } void Scene00001( Entity::Player& player ) @@ -54,7 +54,7 @@ private: Scene00002( player ); }; - player.eventPlay( getId(), 1, DISABLE_SKIP | HIDE_HOTBAR | SET_BASE, 0, 0, callback ); + player.playScene( getId( ), 1, DISABLE_SKIP | HIDE_HOTBAR | SET_BASE, 0, 0, callback ); } void Scene00002( Entity::Player& player ) @@ -63,10 +63,10 @@ private: { player.updateQuest( getId(), SEQ_FINISH ); - player.eventPlay( OPENING_EVENT_HANDLER, 0x1E, HIDE_HOTBAR | NO_DEFAULT_CAMERA, 0, 0 ); + player.playScene( OPENING_EVENT_HANDLER, 0x1E, HIDE_HOTBAR | NO_DEFAULT_CAMERA, 0, 0 ); }; - player.eventPlay( getId(), 2, 0, 0, 0, callback ); + player.playScene( getId( ), 2, 0, 0, 0, callback ); } void Scene00004( Entity::Player& player ) @@ -76,7 +76,7 @@ private: Scene00005( player ); }; - player.eventPlay( getId(), 4, FADE_OUT | HIDE_HOTBAR | CONDITION_CUTSCENE | HIDE_UI , 0, 0, callback ); + player.playScene( getId( ), 4, FADE_OUT | HIDE_HOTBAR | CONDITION_CUTSCENE | HIDE_UI, 0, 0, callback ); } void Scene00005( Entity::Player& player ) @@ -90,7 +90,7 @@ private: } }; - player.eventPlay( getId(), 5, INVIS_OTHER_PC, 0, 0, callback ); + player.playScene( getId( ), 5, INVIS_OTHER_PC, 0, 0, callback ); } public: diff --git a/src/servers/Scripts/quest/ManFst002.cpp b/src/servers/Scripts/quest/ManFst002.cpp index ba71c173..04fe588f 100644 --- a/src/servers/Scripts/quest/ManFst002.cpp +++ b/src/servers/Scripts/quest/ManFst002.cpp @@ -88,7 +88,7 @@ private: Scene00050( player ); }; - player.eventPlay( getId(), SEQ_0_ACTOR0, HIDE_HOTBAR, 0, 0, callback ); + player.playScene( getId( ), SEQ_0_ACTOR0, HIDE_HOTBAR, 0, 0, callback ); } void Scene00001( Entity::Player& player ) @@ -99,7 +99,7 @@ private: checkQuestCompletion( player, 0 ); }; - player.eventPlay( getId(), SEQ_1_ACTOR1, 0x0EFB, 0, 0, callback ); + player.playScene( getId( ), SEQ_1_ACTOR1, 0x0EFB, 0, 0, callback ); } void Scene00002( Entity::Player& player ) @@ -110,7 +110,7 @@ private: checkQuestCompletion( player, 3 ); }; - player.eventPlay( getId(), SEQ_1_ACTOR2, NONE, 0, 0, callback ); + player.playScene( getId( ), SEQ_1_ACTOR2, NONE, 0, 0, callback ); } void Scene00003( Entity::Player& player ) @@ -123,12 +123,12 @@ private: Scene00099( player ); }; - player.eventPlay( getId(), SEQ_1_ACTOR3, NONE, 0, 0, callback ); + player.playScene( getId( ), SEQ_1_ACTOR3, NONE, 0, 0, callback ); } void Scene00004( Entity::Player& player ) { - player.eventPlay( getId(), SEQ_1_ACTOR0, NONE, 0, 0 ); + player.playScene( getId( ), SEQ_1_ACTOR0, NONE, 0, 0 ); } void Scene00005( Entity::Player& player ) @@ -142,7 +142,7 @@ private: } }; - player.eventPlay( getId(), SEQ_2_ACTOR4, FADE_OUT | CONDITION_CUTSCENE | HIDE_UI, 0, 0, callback ); + player.playScene( getId( ), SEQ_2_ACTOR4, FADE_OUT | CONDITION_CUTSCENE | HIDE_UI, 0, 0, callback ); } void Scene00050( Entity::Player& player ) @@ -157,27 +157,17 @@ private: player.forceZoneing( TERRITORYTYPE0 ); }; - player.eventPlay( getId(), SEQ_0_ACTOR0_LQ, FADE_OUT | CONDITION_CUTSCENE | HIDE_UI, 0, 0, callback ); + player.playScene( getId( ), SEQ_0_ACTOR0_LQ, FADE_OUT | CONDITION_CUTSCENE | HIDE_UI, 0, 0, callback ); } void Scene00051( Entity::Player& player ) { - auto callback = [&]( Entity::Player& player, const Event::SceneResult& result ) - { - Scene00001( player ); - }; - - player.eventPlay( getId(), SEQ_1_ACTOR1_WAIT, NONE, 0, 0, callback ); + player.playSceneChain( getId(), SEQ_1_ACTOR1_WAIT, NONE, Scene00001 ); } void Scene00099( Entity::Player& player ) { - auto callback = [&]( Entity::Player& player, const Event::SceneResult& result ) - { - Scene00004( player ); - }; - - player.eventPlay( getId(), SEQ_1_ACTOR3_NPCTRADENO, NONE, 0, 0, callback ); + player.playSceneChain( getId(), SEQ_1_ACTOR3_NPCTRADENO, NONE, Scene00004 ); } void Scene00100( Entity::Player& player ) @@ -190,7 +180,7 @@ private: checkQuestCompletion( player, 2 ); }; - player.eventPlay( getId(), SEQ_1_ACTOR3_NPCTRADEOK, 0x0EFB, 0, 0, callback ); + player.playScene( getId( ), SEQ_1_ACTOR3_NPCTRADEOK, 0x0EFB, 0, 0, callback ); } public: diff --git a/src/servers/Scripts/quest/ManFst004.cpp b/src/servers/Scripts/quest/ManFst004.cpp index e76c16d8..bb2514bd 100644 --- a/src/servers/Scripts/quest/ManFst004.cpp +++ b/src/servers/Scripts/quest/ManFst004.cpp @@ -143,113 +143,100 @@ class ManFst004 : public EventScript // Available Scenes in this quest, not necessarly all are used void Scene00000( Entity::Player& player ) { - player.eventPlay( m_id, 0, 0x2000, - [&]( Entity::Player& player, const Event::SceneResult& result ) - { - if( result.param2 == 1 ) // accept quest - { - Scene00050( player ); - } - }); + player.playScene( m_id, 0, 0x2000, + [&]( Entity::Player& player, const Event::SceneResult& result ) { + if( result.param2 == 1 ) // accept quest + { + Scene00050( player ); + } + } ); } void Scene00001( Entity::Player& player ) { - player.eventPlay( m_id, 1, 0, - [&]( Entity::Player& player, const Event::SceneResult& result ) - { - player.setQuestUI8AL( m_id, 1 ); - checkQuestCompletion( player, 0 ); - }); + player.playScene( m_id, 1, 0, + [&]( Entity::Player& player, const Event::SceneResult& result ) { + player.setQuestUI8AL( m_id, 1 ); + checkQuestCompletion( player, 0 ); + } ); } void Scene00002( Entity::Player& player ) { - player.eventPlay( m_id, 2, 0, - [&]( Entity::Player& player, const Event::SceneResult& result ) - { - player.setQuestUI8BH( m_id, 1 ); - checkQuestCompletion( player, 3 ); - }); + player.playScene( m_id, 2, 0, + [&]( Entity::Player& player, const Event::SceneResult& result ) { + player.setQuestUI8BH( m_id, 1 ); + checkQuestCompletion( player, 3 ); + } ); } void Scene00003( Entity::Player& player ) { - player.eventPlay( m_id, 3, 0, - [&]( Entity::Player& player, const Event::SceneResult& result ) - { - if( result.param2 == 1 ) - { - Scene00100( player ); - } - else - { - Scene00099( player ); - } - }); + player.playScene( m_id, 3, 0, + [&]( Entity::Player& player, const Event::SceneResult& result ) { + if( result.param2 == 1 ) { + Scene00100( player ); + } else { + Scene00099( player ); + } + } ); } void Scene00004( Entity::Player& player ) { - player.eventPlay( m_id, 4, 0, 0, 0 ); + player.playScene( m_id, 4, 0, 0, 0 ); } void Scene00005( Entity::Player& player ) { - player.eventPlay( m_id, 5, FADE_OUT | CONDITION_CUTSCENE | HIDE_UI, 0, 0, - [&]( Entity::Player& player, const Event::SceneResult& result ) - { - if( result.param2 == 1 ) // finish quest - { - if( player.giveQuestRewards( m_id, 0 ) ) - { - player.finishQuest( m_id ); - } - } - }); + player.playScene( m_id, 5, FADE_OUT | CONDITION_CUTSCENE | HIDE_UI, 0, 0, + [&]( Entity::Player& player, const Event::SceneResult& result ) { + if( result.param2 == 1 ) // finish quest + { + if( player.giveQuestRewards( m_id, 0 )) { + player.finishQuest( m_id ); + } + } + } ); } void Scene00050( Entity::Player& player ) { - player.eventPlay( m_id, 50, FADE_OUT | CONDITION_CUTSCENE | HIDE_UI, 0/*unk*/, 0/*unk*/, - [&]( Entity::Player& player, const Event::SceneResult& result ) - { - // accepting quest "close to home" - player.updateQuest( m_id, 1 ); - player.setQuestUI8CH( m_id, 1 ); // receive key item - // event is done, need to teleport to real zone. - player.setZone( 132 ); - //player.setZone(183); back to starting griania for debug purpose - }); + player.playScene( m_id, 50, FADE_OUT | CONDITION_CUTSCENE | HIDE_UI, 0/*unk*/, 0/*unk*/, + [&]( Entity::Player& player, const Event::SceneResult& result ) { + // accepting quest "close to home" + player.updateQuest( m_id, 1 ); + player.setQuestUI8CH( m_id, 1 ); // receive key item + // event is done, need to teleport to real zone. + player.setZone( 132 ); + //player.setZone(183); back to starting griania for debug purpose + } ); } void Scene00051( Entity::Player& player ) { - player.eventPlay( m_id, 51, NONE, - [&]( Entity::Player& player, const Event::SceneResult& result ) - { - Scene00001( player ); - }); + player.playScene( m_id, 51, NONE, + [&]( Entity::Player& player, const Event::SceneResult& result ) { + Scene00001( player ); + } ); } void Scene00099( Entity::Player& player ) { - player.eventPlay( m_id, 99, NONE, - [&]( Entity::Player& player, const Event::SceneResult& result ) - { - Scene00004( player ); - }); + player.playScene( m_id, 99, NONE, + [&]( Entity::Player& player, const Event::SceneResult& result ) { + Scene00004( player ); + } ); } void Scene00100( Entity::Player& player ) { - player.eventPlay( m_id, 100, NONE, - [&]( Entity::Player& player, const Event::SceneResult& result ) - { - player.setQuestUI8CH( m_id, 0 ); // remove key item, since we have just traded it - player.setQuestUI8BL( m_id, 1 ); - checkQuestCompletion( player, 2 ); - }); + player.playScene( m_id, 100, NONE, + [&]( Entity::Player& player, const Event::SceneResult& result ) { + player.setQuestUI8CH( m_id, 0 ); // remove key item, since we have just traded it + player.setQuestUI8BL( m_id, 1 ); + checkQuestCompletion( player, 2 ); + } ); } }; \ No newline at end of file diff --git a/src/servers/Scripts/quest/ManSea001.cpp b/src/servers/Scripts/quest/ManSea001.cpp index 19a9a890..aca4e5c3 100644 --- a/src/servers/Scripts/quest/ManSea001.cpp +++ b/src/servers/Scripts/quest/ManSea001.cpp @@ -36,7 +36,7 @@ private: } }; - player.eventPlay( getId(), 0, HIDE_HOTBAR, 0, 0, callback ); + player.playScene( getId( ), 0, HIDE_HOTBAR, 0, 0, callback ); } void Scene00001( Entity::Player& player ) @@ -46,7 +46,7 @@ private: Scene00002( player ); }; - player.eventPlay( getId(), 1, DISABLE_SKIP | HIDE_HOTBAR | SET_BASE, 0, 0, callback ); + player.playScene( getId( ), 1, DISABLE_SKIP | HIDE_HOTBAR | SET_BASE, 0, 0, callback ); } void Scene00002( Entity::Player& player ) @@ -56,22 +56,22 @@ private: Scene00003( player ); }; - player.eventPlay( getId(), 2, NONE, 0, 0, callback ); + player.playScene( getId( ), 2, NONE, 0, 0, callback ); } void Scene00003( Entity::Player& player ) { auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) { - player.eventPlay( OPENING_EVENT_HANDLER, 0x1E, HIDE_HOTBAR | NO_DEFAULT_CAMERA, 1, 0 ); + player.playScene( OPENING_EVENT_HANDLER, 0x1E, HIDE_HOTBAR | NO_DEFAULT_CAMERA, 1, 0 ); }; - player.eventPlay( getId(), 3, NONE, 0, 0, callback ); + player.playScene( getId( ), 3, NONE, 0, 0, callback ); } void Scene00004( Entity::Player& player ) { - player.eventPlay( getId(), 4, NONE, 0, 0 ); + player.playScene( getId( ), 4, NONE, 0, 0 ); } void Scene00005( Entity::Player& player ) @@ -81,7 +81,7 @@ private: Scene00006( player ); }; - player.eventPlay( getId(), 5, HIDE_HOTBAR, 0, 0, callback ); + player.playScene( getId( ), 5, HIDE_HOTBAR, 0, 0, callback ); } void Scene00006( Entity::Player& player ) @@ -93,27 +93,27 @@ private: player.changePosition( 9, 40, 14, 2 ); }; - player.eventPlay( getId(), 6, INVIS_OTHER_PC, 0, 0, callback ); + player.playScene( getId( ), 6, INVIS_OTHER_PC, 0, 0, callback ); } void Scene00007( Entity::Player& player ) { - player.eventPlay( getId(), 7, NONE, 0, 0 ); + player.playScene( getId( ), 7, NONE, 0, 0 ); } void Scene00008( Entity::Player& player ) { - player.eventPlay( getId(), 8, NONE, 0, 0 ); + player.playScene( getId( ), 8, NONE, 0, 0 ); } void Scene00009( Entity::Player& player ) { - player.eventPlay( getId(), 9, NONE, 0, 0 ); + player.playScene( getId( ), 9, NONE, 0, 0 ); } void Scene00010( Entity::Player& player ) { - player.eventPlay( getId(), 10, NONE, 0, 0 ); + player.playScene( getId( ), 10, NONE, 0, 0 ); } void Scene00011( Entity::Player& player ) @@ -123,7 +123,7 @@ private: Scene00012( player ); }; - player.eventPlay( getId(), 11, 0x2c02, 0, 0, callback ); + player.playScene( getId( ), 11, 0x2c02, 0, 0, callback ); } void Scene00012( Entity::Player& player ) @@ -137,12 +137,12 @@ private: } }; - player.eventPlay( getId(), 12, INVIS_OTHER_PC, 0, 0, callback ); + player.playScene( getId( ), 12, INVIS_OTHER_PC, 0, 0, callback ); } void Scene00013( Entity::Player& player ) { - player.eventPlay( getId(), 13, NONE, 0, 0 ); + player.playScene( getId( ), 13, NONE, 0, 0 ); } public: diff --git a/src/servers/Scripts/quest/ManSea002.cpp b/src/servers/Scripts/quest/ManSea002.cpp index d97e343e..135bbdfd 100644 --- a/src/servers/Scripts/quest/ManSea002.cpp +++ b/src/servers/Scripts/quest/ManSea002.cpp @@ -41,7 +41,7 @@ private: Scene00050( player ); }; - player.eventPlay( getId(), 0, HIDE_HOTBAR, 0, 0, callback ); + player.playScene( getId( ), 0, HIDE_HOTBAR, 0, 0, callback ); } void Scene00001( Entity::Player& player ) @@ -51,7 +51,7 @@ private: }; - player.eventPlay( getId(), 1, NONE, callback ); + player.playScene( getId( ), 1, NONE, callback ); } void Scene00002( Entity::Player& player ) @@ -61,7 +61,7 @@ private: }; - player.eventPlay( getId(), 2, NONE, callback ); + player.playScene( getId( ), 2, NONE, callback ); } void Scene00003( Entity::Player& player ) @@ -71,7 +71,7 @@ private: }; - player.eventPlay( getId(), 3, NONE, callback ); + player.playScene( getId( ), 3, NONE, callback ); } void Scene00004( Entity::Player& player ) @@ -81,7 +81,7 @@ private: }; - player.eventPlay( getId(), 4, NONE, callback ); + player.playScene( getId( ), 4, NONE, callback ); } void Scene00005( Entity::Player& player ) @@ -91,7 +91,7 @@ private: }; - player.eventPlay( getId(), 5, NONE, callback ); + player.playScene( getId( ), 5, NONE, callback ); } void Scene00006( Entity::Player& player ) @@ -101,7 +101,7 @@ private: }; - player.eventPlay( getId(), 6, NONE, callback ); + player.playScene( getId( ), 6, NONE, callback ); } void Scene00007( Entity::Player& player ) @@ -111,7 +111,7 @@ private: }; - player.eventPlay( getId(), 7, NONE, callback ); + player.playScene( getId( ), 7, NONE, callback ); } void Scene00008( Entity::Player& player ) @@ -121,7 +121,7 @@ private: }; - player.eventPlay( getId(), 8, NONE, callback ); + player.playScene( getId( ), 8, NONE, callback ); } void Scene00050( Entity::Player& player ) @@ -136,7 +136,7 @@ private: player.forceZoneing( 128 ); }; - player.eventPlay( getId(), 50, FADE_OUT | CONDITION_CUTSCENE | HIDE_UI, 0, 0, callback ); + player.playScene( getId( ), 50, FADE_OUT | CONDITION_CUTSCENE | HIDE_UI, 0, 0, callback ); } public: ManSea002() : EventScript( 65644 ) {} diff --git a/src/servers/Scripts/quest/ManWil001.cpp b/src/servers/Scripts/quest/ManWil001.cpp index 247c7255..bc0770f8 100644 --- a/src/servers/Scripts/quest/ManWil001.cpp +++ b/src/servers/Scripts/quest/ManWil001.cpp @@ -37,7 +37,7 @@ private: } }; - player.eventPlay( getId(), 0, HIDE_HOTBAR, 0, 0, callback ); + player.playScene( getId( ), 0, HIDE_HOTBAR, 0, 0, callback ); } void Scene00001( Entity::Player& player ) @@ -47,22 +47,22 @@ private: Scene00002( player ); }; - player.eventPlay( getId(), 1, DISABLE_SKIP | HIDE_HOTBAR | SET_BASE, 0, 0, callback ); + player.playScene( getId( ), 1, DISABLE_SKIP | HIDE_HOTBAR | SET_BASE, 0, 0, callback ); } void Scene00002( Entity::Player& player ) { auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) { - player.eventPlay( OPENING_EVENT_HANDLER, 0x1E, HIDE_HOTBAR | NO_DEFAULT_CAMERA, 0, 0 ); + player.playScene( OPENING_EVENT_HANDLER, 0x1E, HIDE_HOTBAR | NO_DEFAULT_CAMERA, 0, 0 ); }; - player.eventPlay( getId(), 2, NONE, 0, 0, callback ); + player.playScene( getId( ), 2, NONE, 0, 0, callback ); } void Scene00003( Entity::Player& player ) { - player.eventPlay( getId(), 3, NONE, 0, 0 ); + player.playScene( getId( ), 3, NONE, 0, 0 ); } void Scene00004( Entity::Player& player ) @@ -72,7 +72,7 @@ private: Scene00005( player ); }; - player.eventPlay( getId(), 4, 0x2c02, 0, 0, callback ); + player.playScene( getId( ), 4, 0x2c02, 0, 0, callback ); } void Scene00005( Entity::Player& player ) @@ -86,47 +86,47 @@ private: } }; - player.eventPlay( getId(), 5, INVIS_OTHER_PC, 0, 0, callback ); + player.playScene( getId( ), 5, INVIS_OTHER_PC, 0, 0, callback ); } void Scene00006( Entity::Player& player ) { - player.eventPlay( getId(), 6, 0, 0, 0 ); + player.playScene( getId( ), 6, 0, 0, 0 ); } void Scene00007( Entity::Player& player ) { - player.eventPlay( getId(), 7, 0, 0, 0 ); + player.playScene( getId( ), 7, 0, 0, 0 ); } void Scene00008( Entity::Player& player ) { - player.eventPlay( getId(), 8, 0, 0, 0 ); + player.playScene( getId( ), 8, 0, 0, 0 ); } void Scene00009( Entity::Player& player ) { - player.eventPlay( getId(), 9, 0, 0, 0 ); + player.playScene( getId( ), 9, 0, 0, 0 ); } void Scene00010( Entity::Player& player ) { - player.eventPlay( getId(), 10, 0, 0, 0 ); + player.playScene( getId( ), 10, 0, 0, 0 ); } void Scene00011( Entity::Player& player ) { - player.eventPlay( getId(), 11, 0, 0, 0 ); + player.playScene( getId( ), 11, 0, 0, 0 ); } void Scene00012( Entity::Player& player ) { - player.eventPlay( getId(), 12, 0, 0, 0 ); + player.playScene( getId( ), 12, 0, 0, 0 ); } void Scene00013( Entity::Player& player ) { - player.eventPlay( getId(), 13, 0, 0, 0 ); + player.playScene( getId( ), 13, 0, 0, 0 ); } diff --git a/src/servers/Scripts/quest/ManWil002.cpp b/src/servers/Scripts/quest/ManWil002.cpp index 3fb0e48a..1349ee7f 100644 --- a/src/servers/Scripts/quest/ManWil002.cpp +++ b/src/servers/Scripts/quest/ManWil002.cpp @@ -47,7 +47,7 @@ private: Scene00050( player ); }; - player.eventPlay( getId(), 0, HIDE_HOTBAR, 0, 0, callback ); + player.playScene( getId( ), 0, HIDE_HOTBAR, 0, 0, callback ); } void Scene00001( Entity::Player& player ) @@ -57,7 +57,7 @@ private: }; - player.eventPlay( getId(), 1, NONE, callback ); + player.playScene( getId( ), 1, NONE, callback ); } void Scene00002( Entity::Player& player ) @@ -67,7 +67,7 @@ private: }; - player.eventPlay( getId(), 2, NONE, callback ); + player.playScene( getId( ), 2, NONE, callback ); } void Scene00003( Entity::Player& player ) @@ -77,7 +77,7 @@ private: }; - player.eventPlay( getId(), 3, NONE, callback ); + player.playScene( getId( ), 3, NONE, callback ); } void Scene00004( Entity::Player& player ) @@ -87,7 +87,7 @@ private: }; - player.eventPlay( getId(), 4, NONE, callback ); + player.playScene( getId( ), 4, NONE, callback ); } void Scene00005( Entity::Player& player ) @@ -97,7 +97,7 @@ private: }; - player.eventPlay( getId(), 5, NONE, callback ); + player.playScene( getId( ), 5, NONE, callback ); } void Scene00006( Entity::Player& player ) @@ -107,7 +107,7 @@ private: }; - player.eventPlay( getId(), 6, NONE, callback ); + player.playScene( getId( ), 6, NONE, callback ); } void Scene00007( Entity::Player& player ) @@ -117,7 +117,7 @@ private: }; - player.eventPlay( getId(), 7, NONE, callback ); + player.playScene( getId( ), 7, NONE, callback ); } void Scene00008( Entity::Player& player ) @@ -127,7 +127,7 @@ private: }; - player.eventPlay( getId(), 8, NONE, callback ); + player.playScene( getId( ), 8, NONE, callback ); } void Scene00050( Entity::Player& player ) @@ -142,7 +142,7 @@ private: player.forceZoneing( 130 ); }; - player.eventPlay( getId(), 50, FADE_OUT | CONDITION_CUTSCENE | HIDE_UI, 0, 0, callback ); + player.playScene( getId( ), 50, FADE_OUT | CONDITION_CUTSCENE | HIDE_UI, 0, 0, callback ); } diff --git a/src/servers/Scripts/quest/subquest/gridania/SubFst001.cpp b/src/servers/Scripts/quest/subquest/gridania/SubFst001.cpp index 113d9e6c..1e9b9059 100644 --- a/src/servers/Scripts/quest/subquest/gridania/SubFst001.cpp +++ b/src/servers/Scripts/quest/subquest/gridania/SubFst001.cpp @@ -34,17 +34,17 @@ private: } }; - player.eventPlay( getId(), 0, NONE, callback ); + player.playScene( getId( ), 0, NONE, callback ); } void Scene00001( Entity::Player& player ) { - player.eventPlay( getId(), 1, NONE ); + player.playScene( getId( ), 1, NONE ); } void Scene00099( Entity::Player& player ) { - player.eventPlay( getId(), 99, NONE ); + player.playScene( getId( ), 99, NONE ); } void Scene00100( Entity::Player& player ) @@ -58,7 +58,7 @@ private: } }; - player.eventPlay( getId(), 100, NONE, callback ); + player.playScene( getId( ), 100, NONE, callback ); } diff --git a/src/servers/Scripts/quest/subquest/gridania/SubFst002.cpp b/src/servers/Scripts/quest/subquest/gridania/SubFst002.cpp index 17696ab8..32382e0b 100644 --- a/src/servers/Scripts/quest/subquest/gridania/SubFst002.cpp +++ b/src/servers/Scripts/quest/subquest/gridania/SubFst002.cpp @@ -31,7 +31,7 @@ private: } }; - player.eventPlay( getId (), 0, NONE, callback ); + player.playScene( getId( ), 0, NONE, callback ); } void Scene00001(Entity::Player& player) @@ -45,7 +45,7 @@ private: } }; - player.eventPlay( getId(), 1, NONE, callback ); + player.playScene( getId( ), 1, NONE, callback ); } public: diff --git a/src/servers/Scripts/quest/subquest/gridania/SubFst010.cpp b/src/servers/Scripts/quest/subquest/gridania/SubFst010.cpp index 3b135781..c8ba532d 100644 --- a/src/servers/Scripts/quest/subquest/gridania/SubFst010.cpp +++ b/src/servers/Scripts/quest/subquest/gridania/SubFst010.cpp @@ -27,7 +27,7 @@ private: player.updateQuest( getId(), SEQ_FINISH ); }; - player.eventPlay( getId(), 0, NONE, callback ); + player.playScene( getId( ), 0, NONE, callback ); } void Scene00001( Entity::Player& player ) @@ -41,7 +41,7 @@ private: } }; - player.eventPlay( getId(), 1, NONE, callback ); + player.playScene( getId( ), 1, NONE, callback ); } public: diff --git a/src/servers/Scripts/quest/subquest/gridania/SubFst013.cpp b/src/servers/Scripts/quest/subquest/gridania/SubFst013.cpp index d82a4432..baf08fbf 100644 --- a/src/servers/Scripts/quest/subquest/gridania/SubFst013.cpp +++ b/src/servers/Scripts/quest/subquest/gridania/SubFst013.cpp @@ -49,12 +49,12 @@ private: } }; - player.eventPlay( getId(), 0, NONE, callback ); + player.playScene( getId( ), 0, NONE, callback ); } void Scene00001( Entity::Player& player ) { - player.eventPlay( getId(), 2, NONE ); + player.playScene( getId( ), 2, NONE ); } void Scene00002( Entity::Player& player ) @@ -64,12 +64,12 @@ private: player.updateQuest( getId(), SEQ_3 ); }; - player.eventPlay( getId(), 2, NONE, callback ); + player.playScene( getId( ), 2, NONE, callback ); } void Scene00003( Entity::Player& player ) { - player.eventPlay( getId(), 3, NONE ); + player.playScene( getId( ), 3, NONE ); } void Scene00004( Entity::Player& player ) @@ -79,12 +79,12 @@ private: player.updateQuest( getId(), SEQ_5 ); }; - player.eventPlay( getId(), 4, NONE, callback ); + player.playScene( getId( ), 4, NONE, callback ); } void Scene00005( Entity::Player& player ) { - player.eventPlay( getId(), 5, NONE ); + player.playScene( getId( ), 5, NONE ); } void Scene00006( Entity::Player& player ) @@ -98,12 +98,12 @@ private: } }; - player.eventPlay( getId(), 6, NONE, callback ); + player.playScene( getId( ), 6, NONE, callback ); } void Scene00095( Entity::Player& player ) { - player.eventPlay( getId(), 95, NONE ); + player.playScene( getId( ), 95, NONE ); } void Scene00096( Entity::Player& player ) @@ -114,12 +114,12 @@ private: player.sendQuestMessage( getId(), 4, 2, 0, 0 ); }; - player.eventPlay( getId(), 96, NONE, callback ); + player.playScene( getId( ), 96, NONE, callback ); } void Scene00097( Entity::Player& player ) { - player.eventPlay( getId(), 97, NONE ); + player.playScene( getId( ), 97, NONE ); } void Scene00098( Entity::Player& player ) @@ -130,12 +130,12 @@ private: player.sendQuestMessage( getId(), 2, 2, 0, 0 ); }; - player.eventPlay( getId(), 98, NONE, callback ); + player.playScene( getId( ), 98, NONE, callback ); } void Scene00099( Entity::Player& player ) { - player.eventPlay( getId(), 99, NONE ); + player.playScene( getId( ), 99, NONE ); } void Scene00100( Entity::Player& player ) @@ -146,7 +146,7 @@ private: player.sendQuestMessage( getId(), 0, 2, 0, 0 ); }; - player.eventPlay( getId(), 100, NONE, callback ); + player.playScene( getId( ), 100, NONE, callback ); } diff --git a/src/servers/sapphire_zone/Actor/Player.h b/src/servers/sapphire_zone/Actor/Player.h index 1bd8f2e6..8a6da096 100644 --- a/src/servers/sapphire_zone/Actor/Player.h +++ b/src/servers/sapphire_zone/Actor/Player.h @@ -56,22 +56,35 @@ public: /*! start/register a normal event */ void eventStart( uint64_t actorId, uint32_t eventId, Event::EventHandler::EventType eventParam, uint8_t eventParam1, uint32_t eventParam2 ); /*! play a subevent */ - void eventPlay( uint32_t eventId, uint32_t scene, uint32_t flags, uint32_t eventParam2, uint32_t eventParam3 ); + void playScene( uint32_t eventId, uint32_t scene, uint32_t flags, uint32_t eventParam2, uint32_t eventParam3 ); void directorPlayScene( uint32_t eventId, uint32_t scene, uint32_t flags, uint32_t eventParam3, uint32_t eventParam4, uint32_t eventParam5 = 0 ); /*! play a subevent */ - void eventPlay( uint32_t eventId, uint32_t scene, uint32_t flags, + void playScene( uint32_t eventId, uint32_t scene, uint32_t flags, uint32_t eventParam2, uint32_t eventParam3, Event::EventHandler::SceneReturnCallback eventReturnCallback ); /*! play a subevent */ - void eventPlay( uint32_t eventId, uint32_t scene, uint32_t flags, + void playScene( uint32_t eventId, uint32_t scene, uint32_t flags, uint32_t eventParam2, uint32_t eventParam3, uint32_t eventParam4, Event::EventHandler::SceneReturnCallback eventReturnCallback ); /*! play a subevent */ - void eventPlay( uint32_t eventId, uint32_t scene, uint32_t flags, + void playScene( uint32_t eventId, uint32_t scene, uint32_t flags, Event::EventHandler::SceneReturnCallback eventReturnCallback ); /*! play a subevent */ - void eventPlay( uint32_t eventId, uint32_t scene, uint32_t flags ); + void playScene( uint32_t eventId, uint32_t scene, uint32_t flags ); + /*! play a subevent */ + void playSceneChain( uint32_t eventId, uint32_t scene, uint32_t flags, + uint32_t eventParam2, uint32_t eventParam3, uint32_t eventParam4, + Event::EventHandler::SceneChainCallback sceneChainCallback ); + /*! play a subevent */ + void playSceneChain( uint32_t eventId, uint32_t scene, uint32_t flags, + uint32_t eventParam2, uint32_t eventParam3, + Event::EventHandler::SceneChainCallback sceneChainCallback ); + /*! play a subevent */ + void playSceneChain( uint32_t eventId, uint32_t scene, uint32_t flags, + Event::EventHandler::SceneChainCallback sceneChainCallback ); + /*! setup the event and return a ptr to it */ + Event::EventHandlerPtr bootstrapSceneEvent( uint32_t eventId, uint32_t flags ); /*! finish / unregister an event */ void eventFinish( uint32_t eventId, uint32_t freePlayer ); /*! add an event to the event array */ diff --git a/src/servers/sapphire_zone/Actor/PlayerEvent.cpp b/src/servers/sapphire_zone/Actor/PlayerEvent.cpp index 60761ef2..b8115329 100644 --- a/src/servers/sapphire_zone/Actor/PlayerEvent.cpp +++ b/src/servers/sapphire_zone/Actor/PlayerEvent.cpp @@ -113,56 +113,32 @@ void Core::Entity::Player::eventStart( uint64_t actorId, uint32_t eventId, } -void Core::Entity::Player::eventPlay( uint32_t eventId, uint32_t scene, +void Core::Entity::Player::playScene( uint32_t eventId, uint32_t scene, uint32_t flags, uint32_t eventParam2, uint32_t eventParam3 ) { - eventPlay( eventId, scene, flags, eventParam2, eventParam3, nullptr ); + playScene( eventId, scene, flags, eventParam2, eventParam3, nullptr ); } -void Core::Entity::Player::eventPlay( uint32_t eventId, uint32_t scene, +void Core::Entity::Player::playScene( uint32_t eventId, uint32_t scene, uint32_t flags, Event::EventHandler::SceneReturnCallback eventCallback ) { - eventPlay( eventId, scene, flags, 0, 0, eventCallback ); + playScene( eventId, scene, flags, 0, 0, eventCallback ); } -void Core::Entity::Player::eventPlay( uint32_t eventId, uint32_t scene, uint32_t flags ) +void Core::Entity::Player::playScene( uint32_t eventId, uint32_t scene, uint32_t flags ) { - eventPlay( eventId, scene, flags, 0, 0, nullptr ); + playScene( eventId, scene, flags, 0, 0, nullptr ); } -void Core::Entity::Player::eventPlay( uint32_t eventId, uint32_t scene, +void Core::Entity::Player::playScene( uint32_t eventId, uint32_t scene, uint32_t flags, uint32_t eventParam2, uint32_t eventParam3, Event::EventHandler::SceneReturnCallback eventCallback ) { - if( flags & 0x02 ) - setStateFlag( PlayerStateFlag::WatchingCutscene ); - - auto pEvent = getEvent( eventId ); - if( !pEvent && getEventCount() ) - { - // We're trying to play a nested event, need to start it first. - eventStart( getId(), eventId, Event::EventHandler::Nest, 0, 0 ); - pEvent = getEvent( eventId ); - } - else if( !pEvent ) - { - auto pLog = g_fw.get< Logger >(); - pLog->error( "Could not find event " + std::to_string( eventId ) + ", event has not been started!" ); - return; - } - - pEvent->setPlayedScene( true ); - pEvent->setEventReturnCallback( eventCallback ); - EventPlayPacket eventPlay( getId(), pEvent->getActorId(), pEvent->getId(), - scene, flags, eventParam2, eventParam3 ); - - queuePacket( eventPlay ); + playScene( eventId, scene, flags, eventParam2, eventParam3, 0, eventCallback ); } -void Core::Entity::Player::eventPlay( uint32_t eventId, uint32_t scene, - uint32_t flags, uint32_t eventParam2, - uint32_t eventParam3, uint32_t eventParam4, Event::EventHandler::SceneReturnCallback eventCallback ) +Core::Event::EventHandlerPtr Core::Entity::Player::bootstrapSceneEvent( uint32_t eventId, uint32_t flags ) { if( flags & 0x02 ) setStateFlag( PlayerStateFlag::WatchingCutscene ); @@ -178,9 +154,20 @@ void Core::Entity::Player::eventPlay( uint32_t eventId, uint32_t scene, { auto pLog = g_fw.get< Logger >(); pLog->error( "Could not find event " + std::to_string( eventId ) + ", event has not been started!" ); - return; + return nullptr; } + return pEvent; +} + +void Core::Entity::Player::playScene( uint32_t eventId, uint32_t scene, + uint32_t flags, uint32_t eventParam2, + uint32_t eventParam3, uint32_t eventParam4, Event::EventHandler::SceneReturnCallback eventCallback ) +{ + auto pEvent = bootstrapSceneEvent( eventId, flags ); + if( !pEvent ) + return; + pEvent->setPlayedScene( true ); pEvent->setEventReturnCallback( eventCallback ); EventPlayPacket eventPlay( getId(), pEvent->getActorId(), pEvent->getId(), @@ -189,6 +176,35 @@ void Core::Entity::Player::eventPlay( uint32_t eventId, uint32_t scene, queuePacket( eventPlay ); } +void Core::Entity::Player::playSceneChain( uint32_t eventId, uint32_t scene, uint32_t flags, + uint32_t eventParam2, uint32_t eventParam3, uint32_t eventParam4, + Core::Event::EventHandler::SceneChainCallback sceneChainCallback ) +{ + auto pEvent = bootstrapSceneEvent( eventId, flags ); + if( !pEvent ) + return; + + pEvent->setPlayedScene( true ); + pEvent->setSceneChainCallback( sceneChainCallback ); + EventPlayPacket eventPlay( getId(), pEvent->getActorId(), pEvent->getId(), + scene, flags, eventParam2, eventParam3, eventParam4 ); + + queuePacket( eventPlay ); +} + +void Core::Entity::Player::playSceneChain( uint32_t eventId, uint32_t scene, uint32_t flags, + uint32_t eventParam2, uint32_t eventParam3, + Core::Event::EventHandler::SceneChainCallback sceneChainCallback ) +{ + playSceneChain( eventId, scene, flags, eventParam2, eventParam3, 0, sceneChainCallback ); +} + +void Core::Entity::Player::playSceneChain( uint32_t eventId, uint32_t scene, uint32_t flags, + Core::Event::EventHandler::SceneChainCallback sceneChainCallback ) +{ + playSceneChain( eventId, scene, flags, 0, 0, 0, sceneChainCallback ); +} + void Core::Entity::Player::eventFinish( uint32_t eventId, uint32_t freePlayer ) { auto pEvent = getEvent( eventId ); diff --git a/src/servers/sapphire_zone/Event/EventHandler.cpp b/src/servers/sapphire_zone/Event/EventHandler.cpp index 1968f59b..322e8c3e 100644 --- a/src/servers/sapphire_zone/Event/EventHandler.cpp +++ b/src/servers/sapphire_zone/Event/EventHandler.cpp @@ -10,7 +10,7 @@ Core::Event::EventHandler::EventHandler( Entity::Player* pOwner, uint64_t actorI m_entryId = static_cast< uint16_t >( eventId ); m_type = static_cast< uint16_t >( eventId >> 16 ); m_eventParam = eventParam; - m_callback = nullptr; + m_returnCallback = nullptr; } uint64_t Core::Event::EventHandler::getActorId() const @@ -45,12 +45,22 @@ uint32_t Core::Event::EventHandler::getEventParam() const Core::Event::EventHandler::SceneReturnCallback Core::Event::EventHandler::getEventReturnCallback() const { - return m_callback; + return m_returnCallback; } void Core::Event::EventHandler::setEventReturnCallback( SceneReturnCallback callback ) { - m_callback = callback; + m_returnCallback = callback; +} + +Core::Event::EventHandler::SceneChainCallback Core::Event::EventHandler::getSceneChainCallback() const +{ + return m_chainCallback; +} + +void Core::Event::EventHandler::setSceneChainCallback( Core::Event::EventHandler::SceneChainCallback callback ) +{ + m_chainCallback = callback; } bool Core::Event::EventHandler::hasPlayedScene() const diff --git a/src/servers/sapphire_zone/Event/EventHandler.h b/src/servers/sapphire_zone/Event/EventHandler.h index 1b4f6b22..acdaf592 100644 --- a/src/servers/sapphire_zone/Event/EventHandler.h +++ b/src/servers/sapphire_zone/Event/EventHandler.h @@ -72,7 +72,8 @@ namespace Core { ICDirector = 0x8003, }; - using SceneReturnCallback = std::function< void( Entity::Player&, const SceneResult& ) > ; + using SceneReturnCallback = std::function< void( Entity::Player&, const SceneResult& ) >; + using SceneChainCallback = std::function< void( Entity::Player& ) >; EventHandler( Entity::Player* pOwner, uint64_t actorId, uint32_t eventId, EventType eventType, uint32_t eventParam ); @@ -98,6 +99,10 @@ namespace Core { void setEventReturnCallback( SceneReturnCallback callback ); + SceneChainCallback getSceneChainCallback() const; + + void setSceneChainCallback( SceneChainCallback callback ); + bool hasNestedEvent() const; void removeNestedEvent(); @@ -114,7 +119,8 @@ namespace Core { uint32_t m_eventParam; EventHandlerPtr m_pNestedEvent; bool m_playedScene; - SceneReturnCallback m_callback; + SceneReturnCallback m_returnCallback; + SceneChainCallback m_chainCallback; }; } diff --git a/src/servers/sapphire_zone/Network/Handlers/EventHandlers.cpp b/src/servers/sapphire_zone/Network/Handlers/EventHandlers.cpp index 120e1935..f8a4763c 100644 --- a/src/servers/sapphire_zone/Network/Handlers/EventHandlers.cpp +++ b/src/servers/sapphire_zone/Network/Handlers/EventHandlers.cpp @@ -218,6 +218,9 @@ void Core::Network::GameConnection::eventHandlerReturn( const Packets::GamePacke result.param3 = param3; eventCallback( player, result ); } + // we might have a scene chain callback instead so check for that too + else if( auto chainCallback = pEvent->getSceneChainCallback() ) + chainCallback( player ); } From 66fbc9931b30f53e3468dbede73b63dab49d4e16 Mon Sep 17 00:00:00 2001 From: Adam Date: Tue, 20 Mar 2018 22:33:51 +1100 Subject: [PATCH 3/8] fix formatting --- src/servers/Scripts/ScriptObject.h | 2 + src/servers/Scripts/common/Aethernet.cpp | 6 +- src/servers/Scripts/common/Aetheryte.cpp | 9 +- .../Scripts/common/CmnDefCutSceneReplay.cpp | 8 +- src/servers/Scripts/common/CmnDefInnBed.cpp | 18 +- .../Scripts/common/CmnDefLinkShell.cpp | 12 +- .../Scripts/common/HouFurOrchestrion.cpp | 4 +- .../Scripts/opening/OpeningGridania.cpp | 25 +- src/servers/Scripts/opening/OpeningLimsa.cpp | 23 +- src/servers/Scripts/opening/OpeningUldah.cpp | 24 +- src/servers/Scripts/quest/ManFst001.cpp | 19 +- src/servers/Scripts/quest/ManFst002.cpp | 38 +-- src/servers/Scripts/quest/ManFst004.cpp | 265 +++++++++--------- src/servers/Scripts/quest/ManSea001.cpp | 53 ++-- src/servers/Scripts/quest/ManSea002.cpp | 26 +- src/servers/Scripts/quest/ManWil001.cpp | 47 ++-- src/servers/Scripts/quest/ManWil002.cpp | 25 +- .../quest/subquest/gridania/SubFst001.cpp | 24 +- .../quest/subquest/gridania/SubFst002.cpp | 21 +- .../quest/subquest/gridania/SubFst010.cpp | 13 +- .../quest/subquest/gridania/SubFst013.cpp | 51 ++-- 21 files changed, 370 insertions(+), 343 deletions(-) diff --git a/src/servers/Scripts/ScriptObject.h b/src/servers/Scripts/ScriptObject.h index 914bfa75..216ef1f7 100644 --- a/src/servers/Scripts/ScriptObject.h +++ b/src/servers/Scripts/ScriptObject.h @@ -6,5 +6,7 @@ #include +#define BIND_SCENE( x ) std::bind( x, *this, std::placeholders::_1 ) + #endif //SAPPHIRE_SCRIPTOBJECT_H diff --git a/src/servers/Scripts/common/Aethernet.cpp b/src/servers/Scripts/common/Aethernet.cpp index 8c31b874..78e22b8b 100644 --- a/src/servers/Scripts/common/Aethernet.cpp +++ b/src/servers/Scripts/common/Aethernet.cpp @@ -21,8 +21,10 @@ public: { if( player.isAetheryteRegistered( eventId & 0xFFFF ) ) { - player.playScene( eventId, 2, 0, []( Entity::Player& player, const Event::SceneResult& result ) { - if( result.param1 == 256 ) { + player.playScene( eventId, 2, 0, []( Entity::Player& player, const Event::SceneResult& result ) + { + if( result.param1 == 256 ) + { player.teleport( result.param2, 2 ); } } ); diff --git a/src/servers/Scripts/common/Aetheryte.cpp b/src/servers/Scripts/common/Aetheryte.cpp index 93fa20af..800d5c8f 100644 --- a/src/servers/Scripts/common/Aetheryte.cpp +++ b/src/servers/Scripts/common/Aetheryte.cpp @@ -23,16 +23,19 @@ public: { if( player.isAetheryteRegistered( eventId & 0xFFFF ) ) { - player.playScene( eventId, 0, 1, []( Entity::Player& player, const Event::SceneResult& result ) { + player.playScene( eventId, 0, 1, []( Entity::Player& player, const Event::SceneResult& result ) + { if( result.param1 == 256 ) // set homepoint { player.setHomepoint( result.eventId & 0xFFFF ); player.sendQuestMessage( result.eventId, 2, 0xEA, 0, 0 ); } else if( result.param1 == 512 ) // aethernet access { - if( result.param2 == 4 ) { + if( result.param2 == 4 ) + { player.teleport( result.param3, 2 ); - } else if( result.param2 == 2 ) // register favored destination + } + else if( result.param2 == 2 ) // register favored destination { } diff --git a/src/servers/Scripts/common/CmnDefCutSceneReplay.cpp b/src/servers/Scripts/common/CmnDefCutSceneReplay.cpp index 5b853704..f286e95a 100644 --- a/src/servers/Scripts/common/CmnDefCutSceneReplay.cpp +++ b/src/servers/Scripts/common/CmnDefCutSceneReplay.cpp @@ -6,11 +6,11 @@ class CmnDefCutSceneReplay : public EventScript { public: CmnDefCutSceneReplay() : EventScript( 721028 ) - {} + { } void Scene00000( Entity::Player& player ) { - auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) + auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) { if( result.param2 != 0 ) { @@ -18,7 +18,7 @@ public: } }; - player.playScene( getId( ), 0, 0x2000, 0, 1, callback ); + player.playScene( getId(), 0, 0x2000, 0, 1, callback ); } void Scene00001( Entity::Player& player, uint16_t returnScene ) @@ -28,7 +28,7 @@ public: // todo: this is fucked }; - player.playScene( getId( ), 1, 0xFB2EC8F8, 0, 1, returnScene, callback ); + player.playScene( getId(), 1, 0xFB2EC8F8, 0, 1, returnScene, callback ); } void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override diff --git a/src/servers/Scripts/common/CmnDefInnBed.cpp b/src/servers/Scripts/common/CmnDefInnBed.cpp index 95ee3492..e8d7964e 100644 --- a/src/servers/Scripts/common/CmnDefInnBed.cpp +++ b/src/servers/Scripts/common/CmnDefInnBed.cpp @@ -6,44 +6,44 @@ class CmnDefInnBed : public EventScript { public: CmnDefInnBed() : EventScript( 720916 ) - {} + { } // menu void Scene00000( Entity::Player& player ) { - auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) + auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) { if( result.param2 > 0 ) Scene00001( player, result.param2 ); }; - player.playScene( getId( ), 0, 0x2000, 0, 1, callback ); + player.playScene( getId(), 0, 0x2000, 0, 1, callback ); } // lay down void Scene00001( Entity::Player& player, uint16_t param ) { - auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) + auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) { Scene00002( player, result.param2 ); }; - player.playScene( getId( ), 1, 0xF32E48F8, 0, 1, param, callback ); + player.playScene( getId(), 1, 0xF32E48F8, 0, 1, param, callback ); } // logout void Scene00002( Entity::Player& player, uint16_t param ) { - player.playScene( getId( ), 2, 0xF32E48F8, 0, 1, param, nullptr ); + player.playScene( getId(), 2, 0xF32E48F8, 0, 1, param, nullptr ); } // wake up void Scene00100( Entity::Player& player ) { auto callback = []( Entity::Player& player, const Event::SceneResult& result ) - {}; + { }; - player.playScene( getId( ), 100, 0x800, 0, 0, callback ); + player.playScene( getId(), 100, 0x800, 0, 0, callback ); } void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override @@ -51,7 +51,7 @@ public: Scene00000( player ); } - void onEnterTerritory( Entity::Player &player, uint32_t eventId, uint16_t param1, uint16_t param2 ) override + void onEnterTerritory( Entity::Player& player, uint32_t eventId, uint16_t param1, uint16_t param2 ) override { Scene00100( player ); } diff --git a/src/servers/Scripts/common/CmnDefLinkShell.cpp b/src/servers/Scripts/common/CmnDefLinkShell.cpp index c1b831fb..10bc9dc7 100644 --- a/src/servers/Scripts/common/CmnDefLinkShell.cpp +++ b/src/servers/Scripts/common/CmnDefLinkShell.cpp @@ -10,11 +10,11 @@ class CmnDefLinkShell : public EventScript { public: CmnDefLinkShell() : EventScript( 0xB0006 ) - {} + { } void Scene00001( Entity::Player& player ) { - auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) + auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) { switch( result.param2 ) { @@ -32,25 +32,25 @@ public: } }; - player.playScene( getId( ), 1, 0, 0, 0, callback ); + player.playScene( getId(), 1, 0, 0, 0, callback ); } // create linkshell void Scene00002( Entity::Player& player ) { - player.playScene( getId( ), 2, 0, 0, 0 ); + player.playScene( getId(), 2, 0, 0, 0 ); } // rename linkshell void Scene00003( Entity::Player& player ) { - player.playScene( getId( ), 3, 0, 0, 0 ); + player.playScene( getId(), 3, 0, 0, 0 ); } // remove linkshell void Scene00004( Entity::Player& player ) { - player.playScene( getId( ), 4, 0, 0, 0 ); + player.playScene( getId(), 4, 0, 0, 0 ); } void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override diff --git a/src/servers/Scripts/common/HouFurOrchestrion.cpp b/src/servers/Scripts/common/HouFurOrchestrion.cpp index b3aabcc8..2df8a762 100644 --- a/src/servers/Scripts/common/HouFurOrchestrion.cpp +++ b/src/servers/Scripts/common/HouFurOrchestrion.cpp @@ -6,11 +6,11 @@ class HouFurOrchestrion : public EventScript { public: HouFurOrchestrion() : EventScript( 721226 ) - {} + { } void Scene00000( Entity::Player& player ) { - player.playScene( getId( ), 0, 0x2000, 0, 1 ); + player.playScene( getId(), 0, 0x2000, 0, 1 ); } void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override diff --git a/src/servers/Scripts/opening/OpeningGridania.cpp b/src/servers/Scripts/opening/OpeningGridania.cpp index 7f123e01..71dc22b9 100644 --- a/src/servers/Scripts/opening/OpeningGridania.cpp +++ b/src/servers/Scripts/opening/OpeningGridania.cpp @@ -28,35 +28,35 @@ private: void Scene00000( Entity::Player& player ) { - auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) + auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) { player.setOpeningSequence( 1 ); Scene00001( player ); }; - player.playScene( getId( ), 0, NO_DEFAULT_CAMERA | INVIS_ENPC | - CONDITION_CUTSCENE | HIDE_UI | - HIDE_HOTBAR | SILENT_ENTER_TERRI_ENV, 0, 1, callback ); + player.playScene( getId(), 0, NO_DEFAULT_CAMERA | INVIS_ENPC | + CONDITION_CUTSCENE | HIDE_UI | + HIDE_HOTBAR | SILENT_ENTER_TERRI_ENV, 0, 1, callback ); } void Scene00001( Entity::Player& player ) { - player.playScene( getId( ), 1, NO_DEFAULT_CAMERA | HIDE_HOTBAR, 1, 0x32 ); + player.playScene( getId(), 1, NO_DEFAULT_CAMERA | HIDE_HOTBAR, 1, 0x32 ); } void Scene00020( Entity::Player& player ) { - player.playScene( getId( ), 20, NO_DEFAULT_CAMERA | HIDE_HOTBAR, 0, 1 ); + player.playScene( getId(), 20, NO_DEFAULT_CAMERA | HIDE_HOTBAR, 0, 1 ); } void Scene00030( Entity::Player& player ) { - player.playScene( getId( ), 30, NO_DEFAULT_CAMERA | HIDE_HOTBAR, 0, 0 ); + player.playScene( getId(), 30, NO_DEFAULT_CAMERA | HIDE_HOTBAR, 0, 0 ); } void Scene00040( Entity::Player& player ) { - auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) + auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) { if( player.getOpeningSequence() == 2 ) { @@ -65,20 +65,19 @@ private: } }; - player.playScene( getId( ), 40, 1, 2, 1, callback ); + player.playScene( getId(), 40, 1, 2, 1, callback ); } public: OpeningGridania() : EventScript( 1245186 ) - {} + { } - void onEnterTerritory( Entity::Player &player, uint32_t eventId, uint16_t param1, uint16_t param2 ) override + void onEnterTerritory( Entity::Player& player, uint32_t eventId, uint16_t param1, uint16_t param2 ) override { if( player.getOpeningSequence() == 0 ) { Scene00000( player ); - } - else + } else { Scene00040( player ); } diff --git a/src/servers/Scripts/opening/OpeningLimsa.cpp b/src/servers/Scripts/opening/OpeningLimsa.cpp index 0d57d41f..9613149e 100644 --- a/src/servers/Scripts/opening/OpeningLimsa.cpp +++ b/src/servers/Scripts/opening/OpeningLimsa.cpp @@ -43,35 +43,35 @@ private: void Scene00000( Entity::Player& player ) { - auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) + auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) { player.setOpeningSequence( 1 ); Scene00001( player ); }; - player.playScene( getId( ), 0, NO_DEFAULT_CAMERA | INVIS_ENPC | - CONDITION_CUTSCENE | HIDE_UI | - HIDE_HOTBAR | SILENT_ENTER_TERRI_ENV, 0, 1, callback ); + player.playScene( getId(), 0, NO_DEFAULT_CAMERA | INVIS_ENPC | + CONDITION_CUTSCENE | HIDE_UI | + HIDE_HOTBAR | SILENT_ENTER_TERRI_ENV, 0, 1, callback ); } void Scene00001( Entity::Player& player ) { - player.playScene( getId( ), 1, NO_DEFAULT_CAMERA | HIDE_HOTBAR, 1, 0x32 ); + player.playScene( getId(), 1, NO_DEFAULT_CAMERA | HIDE_HOTBAR, 1, 0x32 ); } void Scene00020( Entity::Player& player ) { - player.playScene( getId( ), 20, NO_DEFAULT_CAMERA | HIDE_HOTBAR, 0, 1 ); + player.playScene( getId(), 20, NO_DEFAULT_CAMERA | HIDE_HOTBAR, 0, 1 ); } void Scene00030( Entity::Player& player ) { - player.playScene( getId( ), 30, NO_DEFAULT_CAMERA | HIDE_HOTBAR, 0, 0 ); + player.playScene( getId(), 30, NO_DEFAULT_CAMERA | HIDE_HOTBAR, 0, 0 ); } void Scene00040( Entity::Player& player ) { - auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) + auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) { if( player.getOpeningSequence() == 2 ) { @@ -80,14 +80,15 @@ private: } }; - player.playScene( getId( ), 40, NO_DEFAULT_CAMERA, 2, 1, callback ); + player.playScene( getId(), 40, NO_DEFAULT_CAMERA, 2, 1, callback ); } public: - OpeningLimsa() : EventScript( 1245185 ) {} + OpeningLimsa() : EventScript( 1245185 ) + { } - void onEnterTerritory( Entity::Player &player, uint32_t eventId, uint16_t param1, uint16_t param2 ) override + void onEnterTerritory( Entity::Player& player, uint32_t eventId, uint16_t param1, uint16_t param2 ) override { if( player.getOpeningSequence() == 0 ) Scene00000( player ); diff --git a/src/servers/Scripts/opening/OpeningUldah.cpp b/src/servers/Scripts/opening/OpeningUldah.cpp index 22b2bb92..dfc0bd62 100644 --- a/src/servers/Scripts/opening/OpeningUldah.cpp +++ b/src/servers/Scripts/opening/OpeningUldah.cpp @@ -29,49 +29,49 @@ private: void Scene00000( Entity::Player& player ) { - auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) + auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) { player.setOpeningSequence( 1 ); Scene00001( player ); }; - player.playScene( getId( ), 0, NO_DEFAULT_CAMERA | INVIS_ENPC | - CONDITION_CUTSCENE | HIDE_UI | - HIDE_HOTBAR | SILENT_ENTER_TERRI_ENV, 0, 1, callback ); + player.playScene( getId(), 0, NO_DEFAULT_CAMERA | INVIS_ENPC | + CONDITION_CUTSCENE | HIDE_UI | + HIDE_HOTBAR | SILENT_ENTER_TERRI_ENV, 0, 1, callback ); } void Scene00001( Entity::Player& player ) { - player.playScene( getId( ), 1, HIDE_HOTBAR | NO_DEFAULT_CAMERA, 1, 0x32 ); + player.playScene( getId(), 1, HIDE_HOTBAR | NO_DEFAULT_CAMERA, 1, 0x32 ); } void Scene00020( Entity::Player& player ) { - player.playScene( getId( ), 20, HIDE_HOTBAR | NO_DEFAULT_CAMERA, 0, 1 ); + player.playScene( getId(), 20, HIDE_HOTBAR | NO_DEFAULT_CAMERA, 0, 1 ); } void Scene00030( Entity::Player& player ) { - player.playScene( getId( ), 30, HIDE_HOTBAR | NO_DEFAULT_CAMERA, 0, 0 ); + player.playScene( getId(), 30, HIDE_HOTBAR | NO_DEFAULT_CAMERA, 0, 0 ); } void Scene00040( Entity::Player& player ) { - auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) + auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) { if( player.getOpeningSequence() == 2 ) Scene00030( player ); }; - player.playScene( getId( ), 40, NO_DEFAULT_CAMERA, 2, 1, callback ); + player.playScene( getId(), 40, NO_DEFAULT_CAMERA, 2, 1, callback ); } - public: - OpeningUldah() : EventScript( 1245187 ) {} + OpeningUldah() : EventScript( 1245187 ) + { } - void onEnterTerritory( Entity::Player &player, uint32_t eventId, uint16_t param1, uint16_t param2 ) override + void onEnterTerritory( Entity::Player& player, uint32_t eventId, uint16_t param1, uint16_t param2 ) override { if( player.getOpeningSequence() == 0 ) Scene00000( player ); diff --git a/src/servers/Scripts/quest/ManFst001.cpp b/src/servers/Scripts/quest/ManFst001.cpp index 33049e36..578ca55b 100644 --- a/src/servers/Scripts/quest/ManFst001.cpp +++ b/src/servers/Scripts/quest/ManFst001.cpp @@ -44,7 +44,7 @@ private: } }; - player.playScene( getId( ), 0, HIDE_HOTBAR, 0, 0, callback ); + player.playScene( getId(), 0, HIDE_HOTBAR, 0, 0, callback ); } void Scene00001( Entity::Player& player ) @@ -54,7 +54,7 @@ private: Scene00002( player ); }; - player.playScene( getId( ), 1, DISABLE_SKIP | HIDE_HOTBAR | SET_BASE, 0, 0, callback ); + player.playScene( getId(), 1, DISABLE_SKIP | HIDE_HOTBAR | SET_BASE, 0, 0, callback ); } void Scene00002( Entity::Player& player ) @@ -66,7 +66,7 @@ private: player.playScene( OPENING_EVENT_HANDLER, 0x1E, HIDE_HOTBAR | NO_DEFAULT_CAMERA, 0, 0 ); }; - player.playScene( getId( ), 2, 0, 0, 0, callback ); + player.playScene( getId(), 2, 0, 0, 0, callback ); } void Scene00004( Entity::Player& player ) @@ -76,7 +76,7 @@ private: Scene00005( player ); }; - player.playScene( getId( ), 4, FADE_OUT | HIDE_HOTBAR | CONDITION_CUTSCENE | HIDE_UI, 0, 0, callback ); + player.playScene( getId(), 4, FADE_OUT | HIDE_HOTBAR | CONDITION_CUTSCENE | HIDE_UI, 0, 0, callback ); } void Scene00005( Entity::Player& player ) @@ -85,20 +85,21 @@ private: { if( result.param2 == 1 ) { - if( player.giveQuestRewards( getId(), 0 ) ) - player.finishQuest( getId() ); + if( player.giveQuestRewards( getId(), 0 )) + player.finishQuest( getId()); } }; - player.playScene( getId( ), 5, INVIS_OTHER_PC, 0, 0, callback ); + player.playScene( getId(), 5, INVIS_OTHER_PC, 0, 0, callback ); } public: - ManFst001() : EventScript( 65575 ) {} + ManFst001() : EventScript( 65575 ) + { } void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override { - auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId ) ); + auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId )); if( actor == ACTOR0 ) Scene00000( player ); diff --git a/src/servers/Scripts/quest/ManFst002.cpp b/src/servers/Scripts/quest/ManFst002.cpp index 04fe588f..b2824886 100644 --- a/src/servers/Scripts/quest/ManFst002.cpp +++ b/src/servers/Scripts/quest/ManFst002.cpp @@ -72,9 +72,9 @@ private: else player.sendQuestMessage( getId(), 0, 0, 0, 0 ); - auto var_attuned = player.getQuestUI8AL( static_cast< uint16_t >( getId() ) ); - auto var_class = player.getQuestUI8BH( static_cast< uint16_t >( getId() ) ); - auto var_trade = player.getQuestUI8BL( static_cast< uint16_t >( getId() ) ); + auto var_attuned = player.getQuestUI8AL( static_cast< uint16_t >( getId())); + auto var_class = player.getQuestUI8BH( static_cast< uint16_t >( getId())); + auto var_trade = player.getQuestUI8BL( static_cast< uint16_t >( getId())); if( var_attuned == 1 && var_class == 1 && var_trade == 1 ) player.updateQuest( getId(), SEQ_FINISH ); @@ -88,7 +88,7 @@ private: Scene00050( player ); }; - player.playScene( getId( ), SEQ_0_ACTOR0, HIDE_HOTBAR, 0, 0, callback ); + player.playScene( getId(), SEQ_0_ACTOR0, HIDE_HOTBAR, 0, 0, callback ); } void Scene00001( Entity::Player& player ) @@ -99,7 +99,7 @@ private: checkQuestCompletion( player, 0 ); }; - player.playScene( getId( ), SEQ_1_ACTOR1, 0x0EFB, 0, 0, callback ); + player.playScene( getId(), SEQ_1_ACTOR1, 0x0EFB, 0, 0, callback ); } void Scene00002( Entity::Player& player ) @@ -110,7 +110,7 @@ private: checkQuestCompletion( player, 3 ); }; - player.playScene( getId( ), SEQ_1_ACTOR2, NONE, 0, 0, callback ); + player.playScene( getId(), SEQ_1_ACTOR2, NONE, 0, 0, callback ); } void Scene00003( Entity::Player& player ) @@ -123,12 +123,12 @@ private: Scene00099( player ); }; - player.playScene( getId( ), SEQ_1_ACTOR3, NONE, 0, 0, callback ); + player.playScene( getId(), SEQ_1_ACTOR3, NONE, 0, 0, callback ); } void Scene00004( Entity::Player& player ) { - player.playScene( getId( ), SEQ_1_ACTOR0, NONE, 0, 0 ); + player.playScene( getId(), SEQ_1_ACTOR0, NONE, 0, 0 ); } void Scene00005( Entity::Player& player ) @@ -137,12 +137,12 @@ private: { if( result.param2 == 1 ) // finish quest { - if( player.giveQuestRewards( getId(), 0 ) ) - player.finishQuest( getId() ); + if( player.giveQuestRewards( getId(), 0 )) + player.finishQuest( getId()); } }; - player.playScene( getId( ), SEQ_2_ACTOR4, FADE_OUT | CONDITION_CUTSCENE | HIDE_UI, 0, 0, callback ); + player.playScene( getId(), SEQ_2_ACTOR4, FADE_OUT | CONDITION_CUTSCENE | HIDE_UI, 0, 0, callback ); } void Scene00050( Entity::Player& player ) @@ -157,17 +157,17 @@ private: player.forceZoneing( TERRITORYTYPE0 ); }; - player.playScene( getId( ), SEQ_0_ACTOR0_LQ, FADE_OUT | CONDITION_CUTSCENE | HIDE_UI, 0, 0, callback ); + player.playScene( getId(), SEQ_0_ACTOR0_LQ, FADE_OUT | CONDITION_CUTSCENE | HIDE_UI, 0, 0, callback ); } void Scene00051( Entity::Player& player ) { - player.playSceneChain( getId(), SEQ_1_ACTOR1_WAIT, NONE, Scene00001 ); + player.playSceneChain( getId(), SEQ_1_ACTOR1_WAIT, NONE, BIND_SCENE( &ManFst002::Scene00001 )); } void Scene00099( Entity::Player& player ) { - player.playSceneChain( getId(), SEQ_1_ACTOR3_NPCTRADENO, NONE, Scene00004 ); + player.playSceneChain( getId(), SEQ_1_ACTOR3_NPCTRADENO, NONE, BIND_SCENE( &ManFst002::Scene00005 )); } void Scene00100( Entity::Player& player ) @@ -180,15 +180,16 @@ private: checkQuestCompletion( player, 2 ); }; - player.playScene( getId( ), SEQ_1_ACTOR3_NPCTRADEOK, 0x0EFB, 0, 0, callback ); + player.playScene( getId(), SEQ_1_ACTOR3_NPCTRADEOK, 0x0EFB, 0, 0, callback ); } public: - ManFst002() : EventScript( 65621 ) {} + ManFst002() : EventScript( 65621 ) + { } void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override { - auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId ) ); + auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId )); if( actor == ACTOR0 ) Scene00000( player ); @@ -206,8 +207,7 @@ public: player.eventActionStart( 0x050002, 0x13, event, nullptr, 0x050002 ); - } - else if( actor == ACTOR2 ) + } else if( actor == ACTOR2 ) Scene00002( player ); else if( actor == ACTOR3 ) Scene00003( player ); diff --git a/src/servers/Scripts/quest/ManFst004.cpp b/src/servers/Scripts/quest/ManFst004.cpp index bb2514bd..c6832811 100644 --- a/src/servers/Scripts/quest/ManFst004.cpp +++ b/src/servers/Scripts/quest/ManFst004.cpp @@ -11,119 +11,117 @@ class ManFst004 : public EventScript { - private: - // Basic quest information - // Quest vars / flags used - // GetQuestBitFlag8 - // GetQuestUI8AL - // GetQuestUI8BH - // GetQuestUI8BL - // GetQuestUI8CH +private: + // Basic quest information + // Quest vars / flags used + // GetQuestBitFlag8 + // GetQuestUI8AL + // GetQuestUI8BH + // GetQuestUI8BL + // GetQuestUI8CH - // Steps in this quest ( 0 is before accepting, - // 1 is first, 255 means ready for turning it in - enum Sequence : uint8_t - { - Seq0 = 0, - Seq1 = 1, - SeqFinish = 255, - }; + // Steps in this quest ( 0 is before accepting, + // 1 is first, 255 means ready for turning it in + enum Sequence : uint8_t + { + Seq0 = 0, + Seq1 = 1, + SeqFinish = 255, + }; - // Quest rewards - static constexpr auto RewardExpFactor = 100; - static constexpr auto RewardGil = 107; + // Quest rewards + static constexpr auto RewardExpFactor = 100; + static constexpr auto RewardGil = 107; - // Entities found in the script data of the quest - static constexpr auto Actor0 = 1001140; - static constexpr auto Actor1 = 2; - static constexpr auto Actor2 = 1000323; - static constexpr auto Actor20 = 1000159; - static constexpr auto Actor3 = 1000768; - static constexpr auto Actor4 = 1000100; - static constexpr auto BindActor0 = 6229224; - static constexpr auto Item0 = 2000120; - static constexpr auto LocActor0 = 1003159; - static constexpr auto LocMarker01 = 2153091; - static constexpr auto LocMarker02 = 2153104; - static constexpr auto LocMarker03 = 2153111; - static constexpr auto LocMarker04 = 2154539; - static constexpr auto LocMarker05 = 2154540; - static constexpr auto LocMarker06 = 2154541; - static constexpr auto LocMarker07 = 2210446; - static constexpr auto LocMarker08 = 2210454; - static constexpr auto LocMarker09 = 2210461; - static constexpr auto LocMotion0 = 799; - static constexpr auto Poprange0 = 2280858; - static constexpr auto RewardDesion = 1; - static constexpr auto Seq0Actor0 = 0; - static constexpr auto Seq0Actor0Lq = 50; - static constexpr auto Seq1Actor0 = 4; - static constexpr auto Seq1Actor1 = 1; - static constexpr auto Seq1Actor1Wait = 51; - static constexpr auto Seq1Actor2 = 2; - static constexpr auto Seq1Actor3 = 3; - static constexpr auto Seq1Actor3Npctradeno = 99; - static constexpr auto Seq1Actor3Npctradeok = 100; - static constexpr auto Seq2Actor4 = 5; - static constexpr auto Territorytype0 = 132; - static constexpr auto UnlockDesion = 14; + // Entities found in the script data of the quest + static constexpr auto Actor0 = 1001140; + static constexpr auto Actor1 = 2; + static constexpr auto Actor2 = 1000323; + static constexpr auto Actor20 = 1000159; + static constexpr auto Actor3 = 1000768; + static constexpr auto Actor4 = 1000100; + static constexpr auto BindActor0 = 6229224; + static constexpr auto Item0 = 2000120; + static constexpr auto LocActor0 = 1003159; + static constexpr auto LocMarker01 = 2153091; + static constexpr auto LocMarker02 = 2153104; + static constexpr auto LocMarker03 = 2153111; + static constexpr auto LocMarker04 = 2154539; + static constexpr auto LocMarker05 = 2154540; + static constexpr auto LocMarker06 = 2154541; + static constexpr auto LocMarker07 = 2210446; + static constexpr auto LocMarker08 = 2210454; + static constexpr auto LocMarker09 = 2210461; + static constexpr auto LocMotion0 = 799; + static constexpr auto Poprange0 = 2280858; + static constexpr auto RewardDesion = 1; + static constexpr auto Seq0Actor0 = 0; + static constexpr auto Seq0Actor0Lq = 50; + static constexpr auto Seq1Actor0 = 4; + static constexpr auto Seq1Actor1 = 1; + static constexpr auto Seq1Actor1Wait = 51; + static constexpr auto Seq1Actor2 = 2; + static constexpr auto Seq1Actor3 = 3; + static constexpr auto Seq1Actor3Npctradeno = 99; + static constexpr auto Seq1Actor3Npctradeok = 100; + static constexpr auto Seq2Actor4 = 5; + static constexpr auto Territorytype0 = 132; + static constexpr auto UnlockDesion = 14; - public: - ManFst004() : EventScript( 65660 ){}; - ~ManFst004(){}; +public: + ManFst004() : EventScript( 65660 ) + { }; + + ~ManFst004() + { }; ////////////////////////////////////////////////////////////////////// // Event Handlers void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override { - auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId ) ); + auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId )); if( actor == ManFst004::Actor0 ) { Scene00000( player ); - } - else if( actor == ManFst004::Actor1 ) + } else if( actor == ManFst004::Actor1 ) { // starting the Aetheryte eventaction // player.eventStart( actorId, 0x050002, 7, 0, 0); // starting the eventaction 0x13 ( attuning ) player.eventActionStart( 0x050002, 0x13, - [&]( Entity::Player& player, uint32_t eventId, uint64_t additional ) - { - player.sendQuestMessage( 0x050002, 0, 1, 0, 0 ); - player.registerAetheryte( 2 ); - player.learnAction( 1 ); - Scene00051( player ); - }, - [&]( Entity::Player& player, uint32_t eventId, uint64_t additional ) {}, - eventId ); - } - else if( actor == ManFst004::Actor2 ) + [&]( Entity::Player& player, uint32_t eventId, uint64_t additional ) + { + player.sendQuestMessage( 0x050002, 0, 1, 0, 0 ); + player.registerAetheryte( 2 ); + player.learnAction( 1 ); + Scene00051( player ); + }, + [&]( Entity::Player& player, uint32_t eventId, uint64_t additional ) + { }, + eventId ); + } else if( actor == ManFst004::Actor2 ) { Scene00002( player ); - } - else if( actor == ManFst004::Actor3 ) + } else if( actor == ManFst004::Actor3 ) { Scene00003( player ); - } - else if( actor == ManFst004::Actor4 ) + } else if( actor == ManFst004::Actor4 ) { Scene00005( player ); } } - private: +private: void checkQuestCompletion( Entity::Player& player, uint32_t varIdx ) { if( varIdx == 3 ) { player.sendQuestMessage( m_id, 1, 0, 0, 0 ); - } - else if( varIdx == 2 ) + } else if( varIdx == 2 ) { player.sendQuestMessage( m_id, 2, 0, 0, 0 ); - } - else + } else { player.sendQuestMessage( m_id, 0, 0, 0, 0 ); } @@ -139,47 +137,54 @@ class ManFst004 : public EventScript player.updateQuest( questId, SeqFinish ); } } + ////////////////////////////////////////////////////////////////////// // Available Scenes in this quest, not necessarly all are used void Scene00000( Entity::Player& player ) { player.playScene( m_id, 0, 0x2000, - [&]( Entity::Player& player, const Event::SceneResult& result ) { - if( result.param2 == 1 ) // accept quest - { - Scene00050( player ); - } - } ); + [&]( Entity::Player& player, const Event::SceneResult& result ) + { + if( result.param2 == 1 ) // accept quest + { + Scene00050( player ); + } + } ); } void Scene00001( Entity::Player& player ) { player.playScene( m_id, 1, 0, - [&]( Entity::Player& player, const Event::SceneResult& result ) { - player.setQuestUI8AL( m_id, 1 ); - checkQuestCompletion( player, 0 ); - } ); + [&]( Entity::Player& player, const Event::SceneResult& result ) + { + player.setQuestUI8AL( m_id, 1 ); + checkQuestCompletion( player, 0 ); + } ); } void Scene00002( Entity::Player& player ) { player.playScene( m_id, 2, 0, - [&]( Entity::Player& player, const Event::SceneResult& result ) { - player.setQuestUI8BH( m_id, 1 ); - checkQuestCompletion( player, 3 ); - } ); + [&]( Entity::Player& player, const Event::SceneResult& result ) + { + player.setQuestUI8BH( m_id, 1 ); + checkQuestCompletion( player, 3 ); + } ); } void Scene00003( Entity::Player& player ) { player.playScene( m_id, 3, 0, - [&]( Entity::Player& player, const Event::SceneResult& result ) { - if( result.param2 == 1 ) { - Scene00100( player ); - } else { - Scene00099( player ); - } - } ); + [&]( Entity::Player& player, const Event::SceneResult& result ) + { + if( result.param2 == 1 ) + { + Scene00100( player ); + } else + { + Scene00099( player ); + } + } ); } void Scene00004( Entity::Player& player ) @@ -190,53 +195,59 @@ class ManFst004 : public EventScript void Scene00005( Entity::Player& player ) { player.playScene( m_id, 5, FADE_OUT | CONDITION_CUTSCENE | HIDE_UI, 0, 0, - [&]( Entity::Player& player, const Event::SceneResult& result ) { - if( result.param2 == 1 ) // finish quest - { - if( player.giveQuestRewards( m_id, 0 )) { - player.finishQuest( m_id ); - } - } - } ); + [&]( Entity::Player& player, const Event::SceneResult& result ) + { + if( result.param2 == 1 ) // finish quest + { + if( player.giveQuestRewards( m_id, 0 )) + { + player.finishQuest( m_id ); + } + } + } ); } void Scene00050( Entity::Player& player ) { player.playScene( m_id, 50, FADE_OUT | CONDITION_CUTSCENE | HIDE_UI, 0/*unk*/, 0/*unk*/, - [&]( Entity::Player& player, const Event::SceneResult& result ) { - // accepting quest "close to home" - player.updateQuest( m_id, 1 ); - player.setQuestUI8CH( m_id, 1 ); // receive key item - // event is done, need to teleport to real zone. - player.setZone( 132 ); - //player.setZone(183); back to starting griania for debug purpose - } ); + [&]( Entity::Player& player, const Event::SceneResult& result ) + { + // accepting quest "close to home" + player.updateQuest( m_id, 1 ); + player.setQuestUI8CH( m_id, 1 ); // receive key item + // event is done, need to teleport to real zone. + player.setZone( 132 ); + //player.setZone(183); back to starting griania for debug purpose + } ); } void Scene00051( Entity::Player& player ) { player.playScene( m_id, 51, NONE, - [&]( Entity::Player& player, const Event::SceneResult& result ) { - Scene00001( player ); - } ); + [&]( Entity::Player& player, const Event::SceneResult& result ) + { + Scene00001( player ); + } ); } void Scene00099( Entity::Player& player ) { player.playScene( m_id, 99, NONE, - [&]( Entity::Player& player, const Event::SceneResult& result ) { - Scene00004( player ); - } ); + [&]( Entity::Player& player, const Event::SceneResult& result ) + { + Scene00004( player ); + } ); } void Scene00100( Entity::Player& player ) { player.playScene( m_id, 100, NONE, - [&]( Entity::Player& player, const Event::SceneResult& result ) { - player.setQuestUI8CH( m_id, 0 ); // remove key item, since we have just traded it - player.setQuestUI8BL( m_id, 1 ); - checkQuestCompletion( player, 2 ); - } ); + [&]( Entity::Player& player, const Event::SceneResult& result ) + { + player.setQuestUI8CH( m_id, 0 ); // remove key item, since we have just traded it + player.setQuestUI8BL( m_id, 1 ); + checkQuestCompletion( player, 2 ); + } ); } }; \ No newline at end of file diff --git a/src/servers/Scripts/quest/ManSea001.cpp b/src/servers/Scripts/quest/ManSea001.cpp index aca4e5c3..3b9bc45e 100644 --- a/src/servers/Scripts/quest/ManSea001.cpp +++ b/src/servers/Scripts/quest/ManSea001.cpp @@ -27,7 +27,7 @@ private: void Scene00000( Entity::Player& player ) { - auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) + auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) { if( result.param2 == 1 ) { @@ -36,121 +36,122 @@ private: } }; - player.playScene( getId( ), 0, HIDE_HOTBAR, 0, 0, callback ); + player.playScene( getId(), 0, HIDE_HOTBAR, 0, 0, callback ); } void Scene00001( Entity::Player& player ) { - auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) + auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) { Scene00002( player ); }; - player.playScene( getId( ), 1, DISABLE_SKIP | HIDE_HOTBAR | SET_BASE, 0, 0, callback ); + player.playScene( getId(), 1, DISABLE_SKIP | HIDE_HOTBAR | SET_BASE, 0, 0, callback ); } void Scene00002( Entity::Player& player ) { - auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) + auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) { Scene00003( player ); }; - player.playScene( getId( ), 2, NONE, 0, 0, callback ); + player.playScene( getId(), 2, NONE, 0, 0, callback ); } void Scene00003( Entity::Player& player ) { - auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) + auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) { player.playScene( OPENING_EVENT_HANDLER, 0x1E, HIDE_HOTBAR | NO_DEFAULT_CAMERA, 1, 0 ); }; - player.playScene( getId( ), 3, NONE, 0, 0, callback ); + player.playScene( getId(), 3, NONE, 0, 0, callback ); } void Scene00004( Entity::Player& player ) { - player.playScene( getId( ), 4, NONE, 0, 0 ); + player.playScene( getId(), 4, NONE, 0, 0 ); } void Scene00005( Entity::Player& player ) { - auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) + auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) { Scene00006( player ); }; - player.playScene( getId( ), 5, HIDE_HOTBAR, 0, 0, callback ); + player.playScene( getId(), 5, HIDE_HOTBAR, 0, 0, callback ); } void Scene00006( Entity::Player& player ) { - auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) + auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) { player.updateQuest( getId(), SEQ_FINISH ); player.prepareZoning( player.getZoneId(), true, 1, 0 ); player.changePosition( 9, 40, 14, 2 ); }; - player.playScene( getId( ), 6, INVIS_OTHER_PC, 0, 0, callback ); + player.playScene( getId(), 6, INVIS_OTHER_PC, 0, 0, callback ); } void Scene00007( Entity::Player& player ) { - player.playScene( getId( ), 7, NONE, 0, 0 ); + player.playScene( getId(), 7, NONE, 0, 0 ); } void Scene00008( Entity::Player& player ) { - player.playScene( getId( ), 8, NONE, 0, 0 ); + player.playScene( getId(), 8, NONE, 0, 0 ); } void Scene00009( Entity::Player& player ) { - player.playScene( getId( ), 9, NONE, 0, 0 ); + player.playScene( getId(), 9, NONE, 0, 0 ); } void Scene00010( Entity::Player& player ) { - player.playScene( getId( ), 10, NONE, 0, 0 ); + player.playScene( getId(), 10, NONE, 0, 0 ); } void Scene00011( Entity::Player& player ) { - auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) + auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) { Scene00012( player ); }; - player.playScene( getId( ), 11, 0x2c02, 0, 0, callback ); + player.playScene( getId(), 11, 0x2c02, 0, 0, callback ); } void Scene00012( Entity::Player& player ) { - auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) + auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) { if( result.param2 == 1 ) // finish quest { - if( player.giveQuestRewards( getId(), 0 ) ) - player.finishQuest( getId() ); + if( player.giveQuestRewards( getId(), 0 )) + player.finishQuest( getId()); } }; - player.playScene( getId( ), 12, INVIS_OTHER_PC, 0, 0, callback ); + player.playScene( getId(), 12, INVIS_OTHER_PC, 0, 0, callback ); } void Scene00013( Entity::Player& player ) { - player.playScene( getId( ), 13, NONE, 0, 0 ); + player.playScene( getId(), 13, NONE, 0, 0 ); } public: - ManSea001() : EventScript( 65643 ) {} + ManSea001() : EventScript( 65643 ) + { } void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override { - auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId ) ); + auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId )); if( actor == ACTOR0 ) Scene00000( player ); diff --git a/src/servers/Scripts/quest/ManSea002.cpp b/src/servers/Scripts/quest/ManSea002.cpp index 135bbdfd..268fc643 100644 --- a/src/servers/Scripts/quest/ManSea002.cpp +++ b/src/servers/Scripts/quest/ManSea002.cpp @@ -41,7 +41,7 @@ private: Scene00050( player ); }; - player.playScene( getId( ), 0, HIDE_HOTBAR, 0, 0, callback ); + player.playScene( getId(), 0, HIDE_HOTBAR, 0, 0, callback ); } void Scene00001( Entity::Player& player ) @@ -51,7 +51,7 @@ private: }; - player.playScene( getId( ), 1, NONE, callback ); + player.playScene( getId(), 1, NONE, callback ); } void Scene00002( Entity::Player& player ) @@ -61,7 +61,7 @@ private: }; - player.playScene( getId( ), 2, NONE, callback ); + player.playScene( getId(), 2, NONE, callback ); } void Scene00003( Entity::Player& player ) @@ -71,7 +71,7 @@ private: }; - player.playScene( getId( ), 3, NONE, callback ); + player.playScene( getId(), 3, NONE, callback ); } void Scene00004( Entity::Player& player ) @@ -81,7 +81,7 @@ private: }; - player.playScene( getId( ), 4, NONE, callback ); + player.playScene( getId(), 4, NONE, callback ); } void Scene00005( Entity::Player& player ) @@ -91,7 +91,7 @@ private: }; - player.playScene( getId( ), 5, NONE, callback ); + player.playScene( getId(), 5, NONE, callback ); } void Scene00006( Entity::Player& player ) @@ -101,7 +101,7 @@ private: }; - player.playScene( getId( ), 6, NONE, callback ); + player.playScene( getId(), 6, NONE, callback ); } void Scene00007( Entity::Player& player ) @@ -111,7 +111,7 @@ private: }; - player.playScene( getId( ), 7, NONE, callback ); + player.playScene( getId(), 7, NONE, callback ); } void Scene00008( Entity::Player& player ) @@ -121,7 +121,7 @@ private: }; - player.playScene( getId( ), 8, NONE, callback ); + player.playScene( getId(), 8, NONE, callback ); } void Scene00050( Entity::Player& player ) @@ -136,14 +136,16 @@ private: player.forceZoneing( 128 ); }; - player.playScene( getId( ), 50, FADE_OUT | CONDITION_CUTSCENE | HIDE_UI, 0, 0, callback ); + player.playScene( getId(), 50, FADE_OUT | CONDITION_CUTSCENE | HIDE_UI, 0, 0, callback ); } + public: - ManSea002() : EventScript( 65644 ) {} + ManSea002() : EventScript( 65644 ) + { } void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override { - auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId ) ); + auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId )); // todo: this doesn't actually play a cutscene after accepting the quest if( actor == ACTOR0 ) diff --git a/src/servers/Scripts/quest/ManWil001.cpp b/src/servers/Scripts/quest/ManWil001.cpp index bc0770f8..123551cc 100644 --- a/src/servers/Scripts/quest/ManWil001.cpp +++ b/src/servers/Scripts/quest/ManWil001.cpp @@ -28,7 +28,7 @@ private: void Scene00000( Entity::Player& player ) { - auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) + auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) { if( result.param2 == 1 ) { @@ -37,105 +37,106 @@ private: } }; - player.playScene( getId( ), 0, HIDE_HOTBAR, 0, 0, callback ); + player.playScene( getId(), 0, HIDE_HOTBAR, 0, 0, callback ); } void Scene00001( Entity::Player& player ) { - auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) + auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) { Scene00002( player ); }; - player.playScene( getId( ), 1, DISABLE_SKIP | HIDE_HOTBAR | SET_BASE, 0, 0, callback ); + player.playScene( getId(), 1, DISABLE_SKIP | HIDE_HOTBAR | SET_BASE, 0, 0, callback ); } void Scene00002( Entity::Player& player ) { - auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) + auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) { player.playScene( OPENING_EVENT_HANDLER, 0x1E, HIDE_HOTBAR | NO_DEFAULT_CAMERA, 0, 0 ); }; - player.playScene( getId( ), 2, NONE, 0, 0, callback ); + player.playScene( getId(), 2, NONE, 0, 0, callback ); } void Scene00003( Entity::Player& player ) { - player.playScene( getId( ), 3, NONE, 0, 0 ); + player.playScene( getId(), 3, NONE, 0, 0 ); } void Scene00004( Entity::Player& player ) { - auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) + auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) { Scene00005( player ); }; - player.playScene( getId( ), 4, 0x2c02, 0, 0, callback ); + player.playScene( getId(), 4, 0x2c02, 0, 0, callback ); } void Scene00005( Entity::Player& player ) { - auto callback = [ this ]( Entity::Player& player, const Event::SceneResult& result ) + auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) { if( result.param2 == 1 ) { - if( player.giveQuestRewards( getId(), 0 ) ) - player.finishQuest( getId() ); + if( player.giveQuestRewards( getId(), 0 )) + player.finishQuest( getId()); } }; - player.playScene( getId( ), 5, INVIS_OTHER_PC, 0, 0, callback ); + player.playScene( getId(), 5, INVIS_OTHER_PC, 0, 0, callback ); } void Scene00006( Entity::Player& player ) { - player.playScene( getId( ), 6, 0, 0, 0 ); + player.playScene( getId(), 6, 0, 0, 0 ); } void Scene00007( Entity::Player& player ) { - player.playScene( getId( ), 7, 0, 0, 0 ); + player.playScene( getId(), 7, 0, 0, 0 ); } void Scene00008( Entity::Player& player ) { - player.playScene( getId( ), 8, 0, 0, 0 ); + player.playScene( getId(), 8, 0, 0, 0 ); } void Scene00009( Entity::Player& player ) { - player.playScene( getId( ), 9, 0, 0, 0 ); + player.playScene( getId(), 9, 0, 0, 0 ); } void Scene00010( Entity::Player& player ) { - player.playScene( getId( ), 10, 0, 0, 0 ); + player.playScene( getId(), 10, 0, 0, 0 ); } void Scene00011( Entity::Player& player ) { - player.playScene( getId( ), 11, 0, 0, 0 ); + player.playScene( getId(), 11, 0, 0, 0 ); } void Scene00012( Entity::Player& player ) { - player.playScene( getId( ), 12, 0, 0, 0 ); + player.playScene( getId(), 12, 0, 0, 0 ); } void Scene00013( Entity::Player& player ) { - player.playScene( getId( ), 13, 0, 0, 0 ); + player.playScene( getId(), 13, 0, 0, 0 ); } public: - ManWil001() : EventScript( 66130 ) {} + ManWil001() : EventScript( 66130 ) + { } void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override { - auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId ) ); + auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId )); if( actor == ACTOR0 ) Scene00000( player ); diff --git a/src/servers/Scripts/quest/ManWil002.cpp b/src/servers/Scripts/quest/ManWil002.cpp index 1349ee7f..ef99b871 100644 --- a/src/servers/Scripts/quest/ManWil002.cpp +++ b/src/servers/Scripts/quest/ManWil002.cpp @@ -47,7 +47,7 @@ private: Scene00050( player ); }; - player.playScene( getId( ), 0, HIDE_HOTBAR, 0, 0, callback ); + player.playScene( getId(), 0, HIDE_HOTBAR, 0, 0, callback ); } void Scene00001( Entity::Player& player ) @@ -57,7 +57,7 @@ private: }; - player.playScene( getId( ), 1, NONE, callback ); + player.playScene( getId(), 1, NONE, callback ); } void Scene00002( Entity::Player& player ) @@ -67,7 +67,7 @@ private: }; - player.playScene( getId( ), 2, NONE, callback ); + player.playScene( getId(), 2, NONE, callback ); } void Scene00003( Entity::Player& player ) @@ -77,7 +77,7 @@ private: }; - player.playScene( getId( ), 3, NONE, callback ); + player.playScene( getId(), 3, NONE, callback ); } void Scene00004( Entity::Player& player ) @@ -87,7 +87,7 @@ private: }; - player.playScene( getId( ), 4, NONE, callback ); + player.playScene( getId(), 4, NONE, callback ); } void Scene00005( Entity::Player& player ) @@ -97,7 +97,7 @@ private: }; - player.playScene( getId( ), 5, NONE, callback ); + player.playScene( getId(), 5, NONE, callback ); } void Scene00006( Entity::Player& player ) @@ -107,7 +107,7 @@ private: }; - player.playScene( getId( ), 6, NONE, callback ); + player.playScene( getId(), 6, NONE, callback ); } void Scene00007( Entity::Player& player ) @@ -117,7 +117,7 @@ private: }; - player.playScene( getId( ), 7, NONE, callback ); + player.playScene( getId(), 7, NONE, callback ); } void Scene00008( Entity::Player& player ) @@ -127,7 +127,7 @@ private: }; - player.playScene( getId( ), 8, NONE, callback ); + player.playScene( getId(), 8, NONE, callback ); } void Scene00050( Entity::Player& player ) @@ -142,16 +142,17 @@ private: player.forceZoneing( 130 ); }; - player.playScene( getId( ), 50, FADE_OUT | CONDITION_CUTSCENE | HIDE_UI, 0, 0, callback ); + player.playScene( getId(), 50, FADE_OUT | CONDITION_CUTSCENE | HIDE_UI, 0, 0, callback ); } public: - ManWil002() : EventScript( 66104 ) { } + ManWil002() : EventScript( 66104 ) + { } void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override { - auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId ) ); + auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId )); if( actor == ACTOR0 ) Scene00000( player ); diff --git a/src/servers/Scripts/quest/subquest/gridania/SubFst001.cpp b/src/servers/Scripts/quest/subquest/gridania/SubFst001.cpp index 1e9b9059..f5558a74 100644 --- a/src/servers/Scripts/quest/subquest/gridania/SubFst001.cpp +++ b/src/servers/Scripts/quest/subquest/gridania/SubFst001.cpp @@ -34,17 +34,17 @@ private: } }; - player.playScene( getId( ), 0, NONE, callback ); + player.playScene( getId(), 0, NONE, callback ); } void Scene00001( Entity::Player& player ) { - player.playScene( getId( ), 1, NONE ); + player.playScene( getId(), 1, NONE ); } void Scene00099( Entity::Player& player ) { - player.playScene( getId( ), 99, NONE ); + player.playScene( getId(), 99, NONE ); } void Scene00100( Entity::Player& player ) @@ -53,32 +53,32 @@ private: { if( result.param2 == 1 ) // accept quest { - if( player.giveQuestRewards( getId(), 0 ) ) - player.finishQuest( getId() ); + if( player.giveQuestRewards( getId(), 0 )) + player.finishQuest( getId()); } }; - player.playScene( getId( ), 100, NONE, callback ); + player.playScene( getId(), 100, NONE, callback ); } public: - SubFst001() : EventScript( 65560 ) {} + SubFst001() : EventScript( 65560 ) + { } void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override { - auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId ) ); + auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId )); if( actor == ACTOR0 ) { - if( !player.hasQuest( getId() ) ) + if( !player.hasQuest( getId())) Scene00000( player ); else Scene00001( player ); - } - else if( actor == ACTOR1 ) + } else if( actor == ACTOR1 ) { - if( !player.hasQuest( getId() ) ) + if( !player.hasQuest( getId())) Scene00099( player ); else Scene00100( player ); diff --git a/src/servers/Scripts/quest/subquest/gridania/SubFst002.cpp b/src/servers/Scripts/quest/subquest/gridania/SubFst002.cpp index 32382e0b..c3f9ae6e 100644 --- a/src/servers/Scripts/quest/subquest/gridania/SubFst002.cpp +++ b/src/servers/Scripts/quest/subquest/gridania/SubFst002.cpp @@ -31,33 +31,34 @@ private: } }; - player.playScene( getId( ), 0, NONE, callback ); + player.playScene( getId(), 0, NONE, callback ); } - void Scene00001(Entity::Player& player) + void Scene00001( Entity::Player& player ) { auto callback = [&]( Entity::Player& player, const Event::SceneResult& result ) { if( result.param2 == 1 ) // finish quest { - if( player.giveQuestRewards( getId(), 0 ) ) - player.finishQuest( getId() ); + if( player.giveQuestRewards( getId(), 0 )) + player.finishQuest( getId()); } }; - player.playScene( getId( ), 1, NONE, callback ); + player.playScene( getId(), 1, NONE, callback ); } public: - SubFst002() : EventScript( 65561 ) {} + SubFst002() : EventScript( 65561 ) + { } void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override { - auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId ) ); + auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId )); - if( actor == ACTOR0 && !player.hasQuest( getId() ) ) + if( actor == ACTOR0 && !player.hasQuest( getId())) Scene00000( player ); - else if( actor == ACTOR0 && player.getQuestSeq( getId() ) == SEQ_FINISH ) + else if( actor == ACTOR0 && player.getQuestSeq( getId()) == SEQ_FINISH ) Scene00001( player ); } @@ -66,7 +67,7 @@ public: if( npcId != ENEMY0 ) return; - auto currentKC = player.getQuestUI8AL( getId() ) + 1; + auto currentKC = player.getQuestUI8AL( getId()) + 1; if( currentKC >= 6 ) player.updateQuest( getId(), SEQ_FINISH ); diff --git a/src/servers/Scripts/quest/subquest/gridania/SubFst010.cpp b/src/servers/Scripts/quest/subquest/gridania/SubFst010.cpp index c8ba532d..83da85a8 100644 --- a/src/servers/Scripts/quest/subquest/gridania/SubFst010.cpp +++ b/src/servers/Scripts/quest/subquest/gridania/SubFst010.cpp @@ -27,7 +27,7 @@ private: player.updateQuest( getId(), SEQ_FINISH ); }; - player.playScene( getId( ), 0, NONE, callback ); + player.playScene( getId(), 0, NONE, callback ); } void Scene00001( Entity::Player& player ) @@ -36,20 +36,21 @@ private: { if( result.param2 == 1 ) // finish quest { - if( player.giveQuestRewards( getId(), 0 ) ) - player.finishQuest( getId() ); + if( player.giveQuestRewards( getId(), 0 )) + player.finishQuest( getId()); } }; - player.playScene( getId( ), 1, NONE, callback ); + player.playScene( getId(), 1, NONE, callback ); } public: - SubFst010() : EventScript( 65537 ) {} + SubFst010() : EventScript( 65537 ) + { } void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override { - auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId ) ); + auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId )); if( actor == ACTOR0 ) Scene00000( player ); diff --git a/src/servers/Scripts/quest/subquest/gridania/SubFst013.cpp b/src/servers/Scripts/quest/subquest/gridania/SubFst013.cpp index baf08fbf..55e50d2d 100644 --- a/src/servers/Scripts/quest/subquest/gridania/SubFst013.cpp +++ b/src/servers/Scripts/quest/subquest/gridania/SubFst013.cpp @@ -49,12 +49,12 @@ private: } }; - player.playScene( getId( ), 0, NONE, callback ); + player.playScene( getId(), 0, NONE, callback ); } void Scene00001( Entity::Player& player ) { - player.playScene( getId( ), 2, NONE ); + player.playScene( getId(), 2, NONE ); } void Scene00002( Entity::Player& player ) @@ -64,12 +64,12 @@ private: player.updateQuest( getId(), SEQ_3 ); }; - player.playScene( getId( ), 2, NONE, callback ); + player.playScene( getId(), 2, NONE, callback ); } void Scene00003( Entity::Player& player ) { - player.playScene( getId( ), 3, NONE ); + player.playScene( getId(), 3, NONE ); } void Scene00004( Entity::Player& player ) @@ -79,12 +79,12 @@ private: player.updateQuest( getId(), SEQ_5 ); }; - player.playScene( getId( ), 4, NONE, callback ); + player.playScene( getId(), 4, NONE, callback ); } void Scene00005( Entity::Player& player ) { - player.playScene( getId( ), 5, NONE ); + player.playScene( getId(), 5, NONE ); } void Scene00006( Entity::Player& player ) @@ -93,17 +93,17 @@ private: { if( result.param2 == 1 ) // finish quest { - if( player.giveQuestRewards( getId(), 0 ) ) - player.finishQuest( getId() ); + if( player.giveQuestRewards( getId(), 0 )) + player.finishQuest( getId()); } }; - player.playScene( getId( ), 6, NONE, callback ); + player.playScene( getId(), 6, NONE, callback ); } void Scene00095( Entity::Player& player ) { - player.playScene( getId( ), 95, NONE ); + player.playScene( getId(), 95, NONE ); } void Scene00096( Entity::Player& player ) @@ -114,12 +114,12 @@ private: player.sendQuestMessage( getId(), 4, 2, 0, 0 ); }; - player.playScene( getId( ), 96, NONE, callback ); + player.playScene( getId(), 96, NONE, callback ); } void Scene00097( Entity::Player& player ) { - player.playScene( getId( ), 97, NONE ); + player.playScene( getId(), 97, NONE ); } void Scene00098( Entity::Player& player ) @@ -130,12 +130,12 @@ private: player.sendQuestMessage( getId(), 2, 2, 0, 0 ); }; - player.playScene( getId( ), 98, NONE, callback ); + player.playScene( getId(), 98, NONE, callback ); } void Scene00099( Entity::Player& player ) { - player.playScene( getId( ), 99, NONE ); + player.playScene( getId(), 99, NONE ); } void Scene00100( Entity::Player& player ) @@ -146,18 +146,19 @@ private: player.sendQuestMessage( getId(), 0, 2, 0, 0 ); }; - player.playScene( getId( ), 100, NONE, callback ); + player.playScene( getId(), 100, NONE, callback ); } public: - SubFst013() : EventScript( 65576 ) {} + SubFst013() : EventScript( 65576 ) + { } void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override { - auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId ) ); + auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId )); - if( !player.hasQuest( getId() ) ) + if( !player.hasQuest( getId())) { Scene00000( player ); return; @@ -165,23 +166,23 @@ public: if( actor == ACTOR1 ) // talking to Aunillie while on quest Scene00003( player ); - else if( actor == ACTOR0 && player.getQuestSeq( getId() ) == SEQ_2 ) + else if( actor == ACTOR0 && player.getQuestSeq( getId()) == SEQ_2 ) Scene00002( player ); - else if( actor == ACTOR0 && player.getQuestSeq( getId() ) == SEQ_4 ) + else if( actor == ACTOR0 && player.getQuestSeq( getId()) == SEQ_4 ) Scene00004( player ); - else if( actor == ACTOR0 && player.getQuestSeq( getId() ) == SEQ_FINISH ) + else if( actor == ACTOR0 && player.getQuestSeq( getId()) == SEQ_FINISH ) Scene00006( player ); } void onEmote( uint64_t actorId, uint32_t eventId, uint32_t emoteId, Entity::Player& player ) override { - auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId ) ); + auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId )); - if( actor == ACTOR1 && emoteId == 5 && player.getQuestSeq( getId() ) == SEQ_1 ) + if( actor == ACTOR1 && emoteId == 5 && player.getQuestSeq( getId()) == SEQ_1 ) Scene00100( player ); - else if( actor == ACTOR1 && emoteId == 18 && player.getQuestSeq( getId() ) == SEQ_3 ) + else if( actor == ACTOR1 && emoteId == 18 && player.getQuestSeq( getId()) == SEQ_3 ) Scene00098( player ); - else if( actor == ACTOR1 && emoteId == 11 && player.getQuestSeq( getId() ) == SEQ_5 ) + else if( actor == ACTOR1 && emoteId == 11 && player.getQuestSeq( getId()) == SEQ_5 ) Scene00096( player ); } }; \ No newline at end of file From 660709becb4646a8ec6c3b229b188c34f1524f5b Mon Sep 17 00:00:00 2001 From: Adam Date: Tue, 20 Mar 2018 22:43:16 +1100 Subject: [PATCH 4/8] fix formatting again thanks clion --- src/servers/Scripts/quest/ManFst001.cpp | 6 +++--- src/servers/Scripts/quest/ManFst002.cpp | 10 +++++----- src/servers/Scripts/quest/ManFst003.cpp | 3 ++- src/servers/Scripts/quest/ManSea001.cpp | 6 +++--- src/servers/Scripts/quest/ManSea002.cpp | 4 ++-- src/servers/Scripts/quest/ManWil001.cpp | 4 ++-- src/servers/Scripts/quest/ManWil002.cpp | 2 +- .../quest/subquest/gridania/SubFst001.cpp | 6 +++--- .../quest/subquest/gridania/SubFst002.cpp | 12 ++++++------ .../quest/subquest/gridania/SubFst010.cpp | 8 ++++---- .../quest/subquest/gridania/SubFst013.cpp | 16 ++++++++-------- 11 files changed, 39 insertions(+), 38 deletions(-) diff --git a/src/servers/Scripts/quest/ManFst001.cpp b/src/servers/Scripts/quest/ManFst001.cpp index 578ca55b..22e53190 100644 --- a/src/servers/Scripts/quest/ManFst001.cpp +++ b/src/servers/Scripts/quest/ManFst001.cpp @@ -86,7 +86,7 @@ private: if( result.param2 == 1 ) { if( player.giveQuestRewards( getId(), 0 )) - player.finishQuest( getId()); + player.finishQuest( getId() ); } }; @@ -95,11 +95,11 @@ private: public: ManFst001() : EventScript( 65575 ) - { } + {} void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override { - auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId )); + auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId ) ); if( actor == ACTOR0 ) Scene00000( player ); diff --git a/src/servers/Scripts/quest/ManFst002.cpp b/src/servers/Scripts/quest/ManFst002.cpp index b2824886..9f3c6e3f 100644 --- a/src/servers/Scripts/quest/ManFst002.cpp +++ b/src/servers/Scripts/quest/ManFst002.cpp @@ -72,9 +72,9 @@ private: else player.sendQuestMessage( getId(), 0, 0, 0, 0 ); - auto var_attuned = player.getQuestUI8AL( static_cast< uint16_t >( getId())); - auto var_class = player.getQuestUI8BH( static_cast< uint16_t >( getId())); - auto var_trade = player.getQuestUI8BL( static_cast< uint16_t >( getId())); + auto var_attuned = player.getQuestUI8AL( static_cast< uint16_t >( getId() ) ); + auto var_class = player.getQuestUI8BH( static_cast< uint16_t >( getId() ) ); + auto var_trade = player.getQuestUI8BL( static_cast< uint16_t >( getId() ) ); if( var_attuned == 1 && var_class == 1 && var_trade == 1 ) player.updateQuest( getId(), SEQ_FINISH ); @@ -185,11 +185,11 @@ private: public: ManFst002() : EventScript( 65621 ) - { } + {} void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override { - auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId )); + auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId ) ); if( actor == ACTOR0 ) Scene00000( player ); diff --git a/src/servers/Scripts/quest/ManFst003.cpp b/src/servers/Scripts/quest/ManFst003.cpp index 3da79b16..25344bf2 100644 --- a/src/servers/Scripts/quest/ManFst003.cpp +++ b/src/servers/Scripts/quest/ManFst003.cpp @@ -15,7 +15,8 @@ private: public: - ManFst003() : EventScript( 65659 ) {} + ManFst003() : EventScript( 65659 ) + {} void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override { diff --git a/src/servers/Scripts/quest/ManSea001.cpp b/src/servers/Scripts/quest/ManSea001.cpp index 3b9bc45e..3c4600a3 100644 --- a/src/servers/Scripts/quest/ManSea001.cpp +++ b/src/servers/Scripts/quest/ManSea001.cpp @@ -132,7 +132,7 @@ private: { if( result.param2 == 1 ) // finish quest { - if( player.giveQuestRewards( getId(), 0 )) + if( player.giveQuestRewards( getId(), 0 ) ) player.finishQuest( getId()); } }; @@ -147,11 +147,11 @@ private: public: ManSea001() : EventScript( 65643 ) - { } + {} void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override { - auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId )); + auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId ) ); if( actor == ACTOR0 ) Scene00000( player ); diff --git a/src/servers/Scripts/quest/ManSea002.cpp b/src/servers/Scripts/quest/ManSea002.cpp index 268fc643..9817969a 100644 --- a/src/servers/Scripts/quest/ManSea002.cpp +++ b/src/servers/Scripts/quest/ManSea002.cpp @@ -141,11 +141,11 @@ private: public: ManSea002() : EventScript( 65644 ) - { } + {} void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override { - auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId )); + auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId ) ); // todo: this doesn't actually play a cutscene after accepting the quest if( actor == ACTOR0 ) diff --git a/src/servers/Scripts/quest/ManWil001.cpp b/src/servers/Scripts/quest/ManWil001.cpp index 123551cc..8dd296fb 100644 --- a/src/servers/Scripts/quest/ManWil001.cpp +++ b/src/servers/Scripts/quest/ManWil001.cpp @@ -132,11 +132,11 @@ private: public: ManWil001() : EventScript( 66130 ) - { } + {} void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override { - auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId )); + auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId ) ); if( actor == ACTOR0 ) Scene00000( player ); diff --git a/src/servers/Scripts/quest/ManWil002.cpp b/src/servers/Scripts/quest/ManWil002.cpp index ef99b871..55530020 100644 --- a/src/servers/Scripts/quest/ManWil002.cpp +++ b/src/servers/Scripts/quest/ManWil002.cpp @@ -152,7 +152,7 @@ public: void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override { - auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId )); + auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId ) ); if( actor == ACTOR0 ) Scene00000( player ); diff --git a/src/servers/Scripts/quest/subquest/gridania/SubFst001.cpp b/src/servers/Scripts/quest/subquest/gridania/SubFst001.cpp index f5558a74..4c22f807 100644 --- a/src/servers/Scripts/quest/subquest/gridania/SubFst001.cpp +++ b/src/servers/Scripts/quest/subquest/gridania/SubFst001.cpp @@ -64,7 +64,7 @@ private: public: SubFst001() : EventScript( 65560 ) - { } + {} void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override { @@ -72,13 +72,13 @@ public: if( actor == ACTOR0 ) { - if( !player.hasQuest( getId())) + if( !player.hasQuest( getId() ) ) Scene00000( player ); else Scene00001( player ); } else if( actor == ACTOR1 ) { - if( !player.hasQuest( getId())) + if( !player.hasQuest( getId() ) ) Scene00099( player ); else Scene00100( player ); diff --git a/src/servers/Scripts/quest/subquest/gridania/SubFst002.cpp b/src/servers/Scripts/quest/subquest/gridania/SubFst002.cpp index c3f9ae6e..505a601c 100644 --- a/src/servers/Scripts/quest/subquest/gridania/SubFst002.cpp +++ b/src/servers/Scripts/quest/subquest/gridania/SubFst002.cpp @@ -40,8 +40,8 @@ private: { if( result.param2 == 1 ) // finish quest { - if( player.giveQuestRewards( getId(), 0 )) - player.finishQuest( getId()); + if( player.giveQuestRewards( getId(), 0 ) ) + player.finishQuest( getId() ); } }; @@ -50,15 +50,15 @@ private: public: SubFst002() : EventScript( 65561 ) - { } + {} void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override { - auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId )); + auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId ) ); - if( actor == ACTOR0 && !player.hasQuest( getId())) + if( actor == ACTOR0 && !player.hasQuest( getId() ) ) Scene00000( player ); - else if( actor == ACTOR0 && player.getQuestSeq( getId()) == SEQ_FINISH ) + else if( actor == ACTOR0 && player.getQuestSeq( getId() ) == SEQ_FINISH ) Scene00001( player ); } diff --git a/src/servers/Scripts/quest/subquest/gridania/SubFst010.cpp b/src/servers/Scripts/quest/subquest/gridania/SubFst010.cpp index 83da85a8..a7053079 100644 --- a/src/servers/Scripts/quest/subquest/gridania/SubFst010.cpp +++ b/src/servers/Scripts/quest/subquest/gridania/SubFst010.cpp @@ -36,8 +36,8 @@ private: { if( result.param2 == 1 ) // finish quest { - if( player.giveQuestRewards( getId(), 0 )) - player.finishQuest( getId()); + if( player.giveQuestRewards( getId(), 0 ) ) + player.finishQuest( getId() ); } }; @@ -46,11 +46,11 @@ private: public: SubFst010() : EventScript( 65537 ) - { } + {} void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override { - auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId )); + auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId ) ); if( actor == ACTOR0 ) Scene00000( player ); diff --git a/src/servers/Scripts/quest/subquest/gridania/SubFst013.cpp b/src/servers/Scripts/quest/subquest/gridania/SubFst013.cpp index 55e50d2d..6c5e2b99 100644 --- a/src/servers/Scripts/quest/subquest/gridania/SubFst013.cpp +++ b/src/servers/Scripts/quest/subquest/gridania/SubFst013.cpp @@ -152,11 +152,11 @@ private: public: SubFst013() : EventScript( 65576 ) - { } + {} void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override { - auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId )); + auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId ) ); if( !player.hasQuest( getId())) { @@ -166,11 +166,11 @@ public: if( actor == ACTOR1 ) // talking to Aunillie while on quest Scene00003( player ); - else if( actor == ACTOR0 && player.getQuestSeq( getId()) == SEQ_2 ) + else if( actor == ACTOR0 && player.getQuestSeq( getId() ) == SEQ_2 ) Scene00002( player ); - else if( actor == ACTOR0 && player.getQuestSeq( getId()) == SEQ_4 ) + else if( actor == ACTOR0 && player.getQuestSeq( getId() ) == SEQ_4 ) Scene00004( player ); - else if( actor == ACTOR0 && player.getQuestSeq( getId()) == SEQ_FINISH ) + else if( actor == ACTOR0 && player.getQuestSeq( getId() ) == SEQ_FINISH ) Scene00006( player ); } @@ -178,11 +178,11 @@ public: { auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId )); - if( actor == ACTOR1 && emoteId == 5 && player.getQuestSeq( getId()) == SEQ_1 ) + if( actor == ACTOR1 && emoteId == 5 && player.getQuestSeq( getId() ) == SEQ_1 ) Scene00100( player ); - else if( actor == ACTOR1 && emoteId == 18 && player.getQuestSeq( getId()) == SEQ_3 ) + else if( actor == ACTOR1 && emoteId == 18 && player.getQuestSeq( getId() ) == SEQ_3 ) Scene00098( player ); - else if( actor == ACTOR1 && emoteId == 11 && player.getQuestSeq( getId()) == SEQ_5 ) + else if( actor == ACTOR1 && emoteId == 11 && player.getQuestSeq( getId() ) == SEQ_5 ) Scene00096( player ); } }; \ No newline at end of file From a1c531932c3cba0868e02bd13c7810102bf3d6c5 Mon Sep 17 00:00:00 2001 From: Adam Date: Tue, 20 Mar 2018 23:56:06 +1100 Subject: [PATCH 5/8] fix infinite callback loop --- .../Scripts/common/CmnDefCutSceneReplay.cpp | 2 +- src/servers/Scripts/common/CmnDefInnBed.cpp | 2 +- .../Scripts/common/CmnDefLinkShell.cpp | 2 +- .../Scripts/common/HouFurOrchestrion.cpp | 2 +- src/servers/Scripts/quest/ManFst001.cpp | 16 ++--------- src/servers/Scripts/quest/ManFst004.cpp | 17 +++++++---- src/servers/Scripts/quest/ManSea001.cpp | 28 +++---------------- src/servers/Scripts/quest/ManWil001.cpp | 7 +---- .../sapphire_zone/Actor/PlayerEvent.cpp | 2 ++ 9 files changed, 25 insertions(+), 53 deletions(-) diff --git a/src/servers/Scripts/common/CmnDefCutSceneReplay.cpp b/src/servers/Scripts/common/CmnDefCutSceneReplay.cpp index f286e95a..d3730d60 100644 --- a/src/servers/Scripts/common/CmnDefCutSceneReplay.cpp +++ b/src/servers/Scripts/common/CmnDefCutSceneReplay.cpp @@ -6,7 +6,7 @@ class CmnDefCutSceneReplay : public EventScript { public: CmnDefCutSceneReplay() : EventScript( 721028 ) - { } + {} void Scene00000( Entity::Player& player ) { diff --git a/src/servers/Scripts/common/CmnDefInnBed.cpp b/src/servers/Scripts/common/CmnDefInnBed.cpp index e8d7964e..b97f129b 100644 --- a/src/servers/Scripts/common/CmnDefInnBed.cpp +++ b/src/servers/Scripts/common/CmnDefInnBed.cpp @@ -6,7 +6,7 @@ class CmnDefInnBed : public EventScript { public: CmnDefInnBed() : EventScript( 720916 ) - { } + {} // menu void Scene00000( Entity::Player& player ) diff --git a/src/servers/Scripts/common/CmnDefLinkShell.cpp b/src/servers/Scripts/common/CmnDefLinkShell.cpp index 10bc9dc7..c5758211 100644 --- a/src/servers/Scripts/common/CmnDefLinkShell.cpp +++ b/src/servers/Scripts/common/CmnDefLinkShell.cpp @@ -10,7 +10,7 @@ class CmnDefLinkShell : public EventScript { public: CmnDefLinkShell() : EventScript( 0xB0006 ) - { } + {} void Scene00001( Entity::Player& player ) { diff --git a/src/servers/Scripts/common/HouFurOrchestrion.cpp b/src/servers/Scripts/common/HouFurOrchestrion.cpp index 2df8a762..2269789e 100644 --- a/src/servers/Scripts/common/HouFurOrchestrion.cpp +++ b/src/servers/Scripts/common/HouFurOrchestrion.cpp @@ -6,7 +6,7 @@ class HouFurOrchestrion : public EventScript { public: HouFurOrchestrion() : EventScript( 721226 ) - { } + {} void Scene00000( Entity::Player& player ) { diff --git a/src/servers/Scripts/quest/ManFst001.cpp b/src/servers/Scripts/quest/ManFst001.cpp index 22e53190..993c1b98 100644 --- a/src/servers/Scripts/quest/ManFst001.cpp +++ b/src/servers/Scripts/quest/ManFst001.cpp @@ -49,12 +49,7 @@ private: void Scene00001( Entity::Player& player ) { - auto callback = [&]( Entity::Player& player, const Event::SceneResult& result ) - { - Scene00002( player ); - }; - - player.playScene( getId(), 1, DISABLE_SKIP | HIDE_HOTBAR | SET_BASE, 0, 0, callback ); + player.playSceneChain( getId(), 1, DISABLE_SKIP | HIDE_HOTBAR | SET_BASE, BIND_SCENE( &ManFst001::Scene00002 ) ); } void Scene00002( Entity::Player& player ) @@ -71,12 +66,7 @@ private: void Scene00004( Entity::Player& player ) { - auto callback = [&]( Entity::Player& player, const Event::SceneResult& result ) - { - Scene00005( player ); - }; - - player.playScene( getId(), 4, FADE_OUT | HIDE_HOTBAR | CONDITION_CUTSCENE | HIDE_UI, 0, 0, callback ); + player.playSceneChain( getId(), 4, FADE_OUT | HIDE_HOTBAR | CONDITION_CUTSCENE | HIDE_UI, BIND_SCENE( &ManFst001::Scene00005 ) ); } void Scene00005( Entity::Player& player ) @@ -85,7 +75,7 @@ private: { if( result.param2 == 1 ) { - if( player.giveQuestRewards( getId(), 0 )) + if( player.giveQuestRewards( getId(), 0 ) ) player.finishQuest( getId() ); } }; diff --git a/src/servers/Scripts/quest/ManFst004.cpp b/src/servers/Scripts/quest/ManFst004.cpp index c6832811..da1fb49b 100644 --- a/src/servers/Scripts/quest/ManFst004.cpp +++ b/src/servers/Scripts/quest/ManFst004.cpp @@ -79,11 +79,13 @@ public: // Event Handlers void onTalk( uint32_t eventId, Entity::Player& player, uint64_t actorId ) override { - auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId )); + auto actor = Event::mapEventActorToRealActor( static_cast< uint32_t >( actorId ) ); + if( actor == ManFst004::Actor0 ) { Scene00000( player ); - } else if( actor == ManFst004::Actor1 ) + } + else if( actor == ManFst004::Actor1 ) { // starting the Aetheryte eventaction // player.eventStart( actorId, 0x050002, 7, 0, 0); @@ -99,13 +101,16 @@ public: [&]( Entity::Player& player, uint32_t eventId, uint64_t additional ) { }, eventId ); - } else if( actor == ManFst004::Actor2 ) + } + else if( actor == ManFst004::Actor2 ) { Scene00002( player ); - } else if( actor == ManFst004::Actor3 ) + } + else if( actor == ManFst004::Actor3 ) { Scene00003( player ); - } else if( actor == ManFst004::Actor4 ) + } + else if( actor == ManFst004::Actor4 ) { Scene00005( player ); } @@ -199,7 +204,7 @@ private: { if( result.param2 == 1 ) // finish quest { - if( player.giveQuestRewards( m_id, 0 )) + if( player.giveQuestRewards( m_id, 0 ) ) { player.finishQuest( m_id ); } diff --git a/src/servers/Scripts/quest/ManSea001.cpp b/src/servers/Scripts/quest/ManSea001.cpp index 3c4600a3..ecc202af 100644 --- a/src/servers/Scripts/quest/ManSea001.cpp +++ b/src/servers/Scripts/quest/ManSea001.cpp @@ -41,22 +41,12 @@ private: void Scene00001( Entity::Player& player ) { - auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) - { - Scene00002( player ); - }; - - player.playScene( getId(), 1, DISABLE_SKIP | HIDE_HOTBAR | SET_BASE, 0, 0, callback ); + player.playSceneChain( getId(), 1, DISABLE_SKIP | HIDE_HOTBAR | SET_BASE, BIND_SCENE( &ManSea001::Scene00002 ) ); } void Scene00002( Entity::Player& player ) { - auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) - { - Scene00003( player ); - }; - - player.playScene( getId(), 2, NONE, 0, 0, callback ); + player.playSceneChain( getId(), 2, NONE, BIND_SCENE( &ManSea001::Scene00003 ) ); } void Scene00003( Entity::Player& player ) @@ -76,12 +66,7 @@ private: void Scene00005( Entity::Player& player ) { - auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) - { - Scene00006( player ); - }; - - player.playScene( getId(), 5, HIDE_HOTBAR, 0, 0, callback ); + player.playSceneChain( getId(), 5, HIDE_HOTBAR, BIND_SCENE( &ManSea001::Scene00006 ) ); } void Scene00006( Entity::Player& player ) @@ -118,12 +103,7 @@ private: void Scene00011( Entity::Player& player ) { - auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) - { - Scene00012( player ); - }; - - player.playScene( getId(), 11, 0x2c02, 0, 0, callback ); + player.playSceneChain( getId(), 11, 0x2c02, 0, 0, BIND_SCENE( &ManSea001::Scene00012 ) ); } void Scene00012( Entity::Player& player ) diff --git a/src/servers/Scripts/quest/ManWil001.cpp b/src/servers/Scripts/quest/ManWil001.cpp index 8dd296fb..667f429a 100644 --- a/src/servers/Scripts/quest/ManWil001.cpp +++ b/src/servers/Scripts/quest/ManWil001.cpp @@ -67,12 +67,7 @@ private: void Scene00004( Entity::Player& player ) { - auto callback = [this]( Entity::Player& player, const Event::SceneResult& result ) - { - Scene00005( player ); - }; - - player.playScene( getId(), 4, 0x2c02, 0, 0, callback ); + player.playSceneChain( getId(), 4, 0x2c02, BIND_SCENE( &ManWil001::Scene00005 ) ); } void Scene00005( Entity::Player& player ) diff --git a/src/servers/sapphire_zone/Actor/PlayerEvent.cpp b/src/servers/sapphire_zone/Actor/PlayerEvent.cpp index b8115329..80e3dbda 100644 --- a/src/servers/sapphire_zone/Actor/PlayerEvent.cpp +++ b/src/servers/sapphire_zone/Actor/PlayerEvent.cpp @@ -170,6 +170,7 @@ void Core::Entity::Player::playScene( uint32_t eventId, uint32_t scene, pEvent->setPlayedScene( true ); pEvent->setEventReturnCallback( eventCallback ); + pEvent->setSceneChainCallback( nullptr ); EventPlayPacket eventPlay( getId(), pEvent->getActorId(), pEvent->getId(), scene, flags, eventParam2, eventParam3, eventParam4 ); @@ -186,6 +187,7 @@ void Core::Entity::Player::playSceneChain( uint32_t eventId, uint32_t scene, uin pEvent->setPlayedScene( true ); pEvent->setSceneChainCallback( sceneChainCallback ); + pEvent->setEventReturnCallback( nullptr ); EventPlayPacket eventPlay( getId(), pEvent->getActorId(), pEvent->getId(), scene, flags, eventParam2, eventParam3, eventParam4 ); From 8984a98b349c929ff9c6048a71360c0a6c289acd Mon Sep 17 00:00:00 2001 From: Adam Date: Wed, 21 Mar 2018 19:07:05 +1100 Subject: [PATCH 6/8] fix zone crash from an invalid class instance ptr --- src/servers/Scripts/ScriptObject.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/servers/Scripts/ScriptObject.h b/src/servers/Scripts/ScriptObject.h index 216ef1f7..b63e2b61 100644 --- a/src/servers/Scripts/ScriptObject.h +++ b/src/servers/Scripts/ScriptObject.h @@ -6,7 +6,7 @@ #include -#define BIND_SCENE( x ) std::bind( x, *this, std::placeholders::_1 ) +#define BIND_SCENE( x ) std::bind( x, this, std::placeholders::_1 ) #endif //SAPPHIRE_SCRIPTOBJECT_H From bda2a513624ba698a3e208ee5d4a517d2e71d9a3 Mon Sep 17 00:00:00 2001 From: Adam Date: Thu, 22 Mar 2018 00:54:00 +1100 Subject: [PATCH 7/8] update submodule --- src/libraries | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/libraries b/src/libraries index 67b949df..61712f8f 160000 --- a/src/libraries +++ b/src/libraries @@ -1 +1 @@ -Subproject commit 67b949dfe3ffbbba7963b0861670ab4bb1819991 +Subproject commit 61712f8f11892d12ad6878a80b9b89b318908558 From 00a7c1c6d3b89ceb21e7ab0ab4bc2ef820d1976d Mon Sep 17 00:00:00 2001 From: Adam Date: Thu, 22 Mar 2018 23:45:17 +1100 Subject: [PATCH 8/8] remove bindscene macro and move to wrapper func --- src/servers/Scripts/ScriptObject.h | 3 --- src/servers/Scripts/quest/ManFst001.cpp | 4 ++-- src/servers/Scripts/quest/ManFst002.cpp | 4 ++-- src/servers/Scripts/quest/ManSea001.cpp | 8 ++++---- src/servers/Scripts/quest/ManWil001.cpp | 2 +- src/servers/sapphire_zone/Script/NativeScriptApi.h | 8 ++++++++ 6 files changed, 17 insertions(+), 12 deletions(-) diff --git a/src/servers/Scripts/ScriptObject.h b/src/servers/Scripts/ScriptObject.h index b63e2b61..1106541c 100644 --- a/src/servers/Scripts/ScriptObject.h +++ b/src/servers/Scripts/ScriptObject.h @@ -6,7 +6,4 @@ #include -#define BIND_SCENE( x ) std::bind( x, this, std::placeholders::_1 ) - - #endif //SAPPHIRE_SCRIPTOBJECT_H diff --git a/src/servers/Scripts/quest/ManFst001.cpp b/src/servers/Scripts/quest/ManFst001.cpp index 993c1b98..a351e61c 100644 --- a/src/servers/Scripts/quest/ManFst001.cpp +++ b/src/servers/Scripts/quest/ManFst001.cpp @@ -49,7 +49,7 @@ private: void Scene00001( Entity::Player& player ) { - player.playSceneChain( getId(), 1, DISABLE_SKIP | HIDE_HOTBAR | SET_BASE, BIND_SCENE( &ManFst001::Scene00002 ) ); + player.playSceneChain( getId(), 1, DISABLE_SKIP | HIDE_HOTBAR | SET_BASE, bindScene( &ManFst001::Scene00002 ) ); } void Scene00002( Entity::Player& player ) @@ -66,7 +66,7 @@ private: void Scene00004( Entity::Player& player ) { - player.playSceneChain( getId(), 4, FADE_OUT | HIDE_HOTBAR | CONDITION_CUTSCENE | HIDE_UI, BIND_SCENE( &ManFst001::Scene00005 ) ); + player.playSceneChain( getId(), 4, FADE_OUT | HIDE_HOTBAR | CONDITION_CUTSCENE | HIDE_UI, bindScene( &ManFst001::Scene00005 ) ); } void Scene00005( Entity::Player& player ) diff --git a/src/servers/Scripts/quest/ManFst002.cpp b/src/servers/Scripts/quest/ManFst002.cpp index 9f3c6e3f..153fb9a0 100644 --- a/src/servers/Scripts/quest/ManFst002.cpp +++ b/src/servers/Scripts/quest/ManFst002.cpp @@ -162,12 +162,12 @@ private: void Scene00051( Entity::Player& player ) { - player.playSceneChain( getId(), SEQ_1_ACTOR1_WAIT, NONE, BIND_SCENE( &ManFst002::Scene00001 )); + player.playSceneChain( getId(), SEQ_1_ACTOR1_WAIT, NONE, bindScene( &ManFst002::Scene00001 )); } void Scene00099( Entity::Player& player ) { - player.playSceneChain( getId(), SEQ_1_ACTOR3_NPCTRADENO, NONE, BIND_SCENE( &ManFst002::Scene00005 )); + player.playSceneChain( getId(), SEQ_1_ACTOR3_NPCTRADENO, NONE, bindScene( &ManFst002::Scene00005 )); } void Scene00100( Entity::Player& player ) diff --git a/src/servers/Scripts/quest/ManSea001.cpp b/src/servers/Scripts/quest/ManSea001.cpp index ecc202af..79d56ecc 100644 --- a/src/servers/Scripts/quest/ManSea001.cpp +++ b/src/servers/Scripts/quest/ManSea001.cpp @@ -41,12 +41,12 @@ private: void Scene00001( Entity::Player& player ) { - player.playSceneChain( getId(), 1, DISABLE_SKIP | HIDE_HOTBAR | SET_BASE, BIND_SCENE( &ManSea001::Scene00002 ) ); + player.playSceneChain( getId(), 1, DISABLE_SKIP | HIDE_HOTBAR | SET_BASE, bindScene( &ManSea001::Scene00002 ) ); } void Scene00002( Entity::Player& player ) { - player.playSceneChain( getId(), 2, NONE, BIND_SCENE( &ManSea001::Scene00003 ) ); + player.playSceneChain( getId(), 2, NONE, bindScene( &ManSea001::Scene00003 ) ); } void Scene00003( Entity::Player& player ) @@ -66,7 +66,7 @@ private: void Scene00005( Entity::Player& player ) { - player.playSceneChain( getId(), 5, HIDE_HOTBAR, BIND_SCENE( &ManSea001::Scene00006 ) ); + player.playSceneChain( getId(), 5, HIDE_HOTBAR, bindScene( &ManSea001::Scene00006 ) ); } void Scene00006( Entity::Player& player ) @@ -103,7 +103,7 @@ private: void Scene00011( Entity::Player& player ) { - player.playSceneChain( getId(), 11, 0x2c02, 0, 0, BIND_SCENE( &ManSea001::Scene00012 ) ); + player.playSceneChain( getId(), 11, 0x2c02, 0, 0, bindScene( &ManSea001::Scene00012 ) ); } void Scene00012( Entity::Player& player ) diff --git a/src/servers/Scripts/quest/ManWil001.cpp b/src/servers/Scripts/quest/ManWil001.cpp index 667f429a..8a3f0dfd 100644 --- a/src/servers/Scripts/quest/ManWil001.cpp +++ b/src/servers/Scripts/quest/ManWil001.cpp @@ -67,7 +67,7 @@ private: void Scene00004( Entity::Player& player ) { - player.playSceneChain( getId(), 4, 0x2c02, BIND_SCENE( &ManWil001::Scene00005 ) ); + player.playSceneChain( getId(), 4, 0x2c02, bindScene( &ManWil001::Scene00005 ) ); } void Scene00005( Entity::Player& player ) diff --git a/src/servers/sapphire_zone/Script/NativeScriptApi.h b/src/servers/sapphire_zone/Script/NativeScriptApi.h index 9e7f9d36..fe1e5244 100644 --- a/src/servers/sapphire_zone/Script/NativeScriptApi.h +++ b/src/servers/sapphire_zone/Script/NativeScriptApi.h @@ -4,6 +4,7 @@ #include #include #include +#include #include "Forwards.h" #ifdef _MSC_VER @@ -75,6 +76,13 @@ public: class EventScript : public ScriptObject { +protected: + template< typename Ret, class Obj > + inline Event::EventHandler::SceneChainCallback bindScene( Ret ( Obj::*f )( Entity::Player& ) ) + { + return std::bind( f, static_cast< Obj* >( this ), std::placeholders::_1 ); + } + public: explicit EventScript( uint32_t questId ) : ScriptObject( questId, typeid( EventScript ).hash_code() )