From 35e453dec3a9044bb2087c97a051453eb84f9f8e Mon Sep 17 00:00:00 2001
From: David Cernat <davidcernat@gmail.com>
Date: Tue, 24 Jan 2017 19:32:25 +0200
Subject: [PATCH] [General] Sketch out most of functionality for journal
 saving/loading

---
 apps/openmw-mp/CMakeLists.txt                 |  5 +-
 apps/openmw-mp/Player.hpp                     |  1 +
 apps/openmw-mp/Script/Functions/Quests.cpp    | 79 +++++++++++++++++++
 apps/openmw-mp/Script/Functions/Quests.hpp    | 33 ++++++++
 apps/openmw-mp/Script/ScriptFunctions.hpp     |  2 +
 apps/openmw/mwmp/LocalPlayer.cpp              | 56 ++++++++++++-
 apps/openmw/mwmp/LocalPlayer.hpp              |  1 +
 apps/openmw/mwmp/Networking.cpp               | 12 +++
 components/openmw-mp/Base/BasePlayer.hpp      |  5 ++
 .../Packets/Player/PacketJournal.cpp          | 32 ++++++++
 10 files changed, 222 insertions(+), 4 deletions(-)
 create mode 100644 apps/openmw-mp/Script/Functions/Quests.cpp
 create mode 100644 apps/openmw-mp/Script/Functions/Quests.hpp

diff --git a/apps/openmw-mp/CMakeLists.txt b/apps/openmw-mp/CMakeLists.txt
index 9dace0cba..12bab0186 100644
--- a/apps/openmw-mp/CMakeLists.txt
+++ b/apps/openmw-mp/CMakeLists.txt
@@ -73,8 +73,9 @@ set(SERVER
     Script/ScriptFunctions.cpp
 
     Script/Functions/CharClass.cpp Script/Functions/Chat.cpp Script/Functions/GUI.cpp
-    Script/Functions/Items.cpp Script/Functions/Stats.cpp Script/Functions/Spells.cpp
-    Script/Functions/Timer.cpp Script/Functions/Translocations.cpp Script/Functions/World.cpp
+    Script/Functions/Items.cpp Script/Functions/Quests.cpp Script/Functions/Stats.cpp
+    Script/Functions/Spells.cpp Script/Functions/Timer.cpp Script/Functions/Translocations.cpp
+    Script/Functions/World.cpp
 
     Script/API/TimerAPI.cpp Script/API/PublicFnAPI.cpp
         ${PawnScript_Sources}
diff --git a/apps/openmw-mp/Player.hpp b/apps/openmw-mp/Player.hpp
index 6d30f1b2e..3fdc0b2e3 100644
--- a/apps/openmw-mp/Player.hpp
+++ b/apps/openmw-mp/Player.hpp
@@ -70,6 +70,7 @@ public:
 public:
     mwmp::InventoryChanges inventoryChangesBuffer;
     mwmp::SpellbookChanges spellbookChangesBuffer;
+    mwmp::JournalChanges journalChangesBuffer;
 
 private:
     bool handshakeState;
diff --git a/apps/openmw-mp/Script/Functions/Quests.cpp b/apps/openmw-mp/Script/Functions/Quests.cpp
new file mode 100644
index 000000000..18e829b12
--- /dev/null
+++ b/apps/openmw-mp/Script/Functions/Quests.cpp
@@ -0,0 +1,79 @@
+#include "Quests.hpp"
+#include <apps/openmw-mp/Script/ScriptFunctions.hpp>
+#include <components/openmw-mp/NetworkMessages.hpp>
+#include <apps/openmw-mp/Networking.hpp>
+#include <components/misc/stringops.hpp>
+
+using namespace mwmp;
+
+unsigned int QuestFunctions::GetJournalChangesSize(unsigned short pid) noexcept
+{
+    Player *player;
+    GET_PLAYER(pid, player, 0);
+
+    return player->journalChanges.count;
+}
+
+void QuestFunctions::AddJournalEntry(unsigned short pid, const char* quest, unsigned int index) noexcept
+{
+    Player *player;
+    GET_PLAYER(pid, player, );
+
+    mwmp::JournalItem journalItem;
+    journalItem.type = JournalItem::ENTRY;
+    journalItem.quest = quest;
+    journalItem.index = index;
+
+    player->journalChangesBuffer.journalItems.push_back(journalItem);
+}
+
+void QuestFunctions::AddJournalIndex(unsigned short pid, const char* quest, unsigned int index) noexcept
+{
+    Player *player;
+    GET_PLAYER(pid, player, );
+
+    mwmp::JournalItem journalItem;
+    journalItem.type = JournalItem::INDEX;
+    journalItem.quest = quest;
+    journalItem.index = index;
+
+    player->journalChangesBuffer.journalItems.push_back(journalItem);
+}
+
+const char *QuestFunctions::GetJournalItemQuest(unsigned short pid, unsigned int i) noexcept
+{
+    Player *player;
+    GET_PLAYER(pid, player, "");
+
+    if (i >= player->journalChanges.count)
+        return "invalid";
+
+    return player->journalChanges.journalItems.at(i).quest.c_str();
+}
+
+int QuestFunctions::GetJournalItemIndex(unsigned short pid, unsigned int i) noexcept
+{
+    Player *player;
+    GET_PLAYER(pid, player, 0);
+
+    return player->journalChanges.journalItems.at(i).index;
+}
+
+int QuestFunctions::GetJournalItemType(unsigned short pid, unsigned int i) noexcept
+{
+    Player *player;
+    GET_PLAYER(pid, player, 0);
+
+    return player->journalChanges.journalItems.at(i).type;
+}
+
+void QuestFunctions::SendJournalChanges(unsigned short pid) noexcept
+{
+    Player *player;
+    GET_PLAYER(pid, player, );
+
+    std::swap(player->journalChanges, player->journalChangesBuffer);
+    mwmp::Networking::get().getPlayerController()->GetPacket(ID_GAME_JOURNAL)->Send(player, false);
+    player->journalChanges = std::move(player->journalChangesBuffer);
+    player->journalChangesBuffer.journalItems.clear();
+}
diff --git a/apps/openmw-mp/Script/Functions/Quests.hpp b/apps/openmw-mp/Script/Functions/Quests.hpp
new file mode 100644
index 000000000..656bfb83d
--- /dev/null
+++ b/apps/openmw-mp/Script/Functions/Quests.hpp
@@ -0,0 +1,33 @@
+#ifndef OPENMW_QUESTS_HPP
+#define OPENMW_QUESTS_HPP
+
+#define QUESTAPI \
+    {"GetJournalChangesSize",   QuestFunctions::GetJournalChangesSize},\
+    \
+    {"AddJournalIndex",         QuestFunctions::AddJournalIndex},\
+    \
+    {"GetJournalItemQuest",     QuestFunctions::GetJournalItemQuest},\
+    {"GetJournalItemIndex",     QuestFunctions::GetJournalItemIndex},\
+    {"GetJournalItemType",      QuestFunctions::GetJournalItemType},\
+    \
+    {"SendJournalChanges",      QuestFunctions::SendJournalChanges}
+
+class QuestFunctions
+{
+public:
+
+    static unsigned int GetJournalChangesSize(unsigned short pid) noexcept;
+
+    static void AddJournalEntry(unsigned short pid, const char* quest, unsigned int index) noexcept;
+    static void AddJournalIndex(unsigned short pid, const char* quest, unsigned int index) noexcept;
+
+    static const char *GetJournalItemQuest(unsigned short pid, unsigned int i) noexcept;
+    static int GetJournalItemIndex(unsigned short pid, unsigned int i) noexcept;
+    static int GetJournalItemType(unsigned short pid, unsigned int i) noexcept;
+
+    static void SendJournalChanges(unsigned short pid) noexcept;
+private:
+
+};
+
+#endif //OPENMW_QUESTS_HPP
diff --git a/apps/openmw-mp/Script/ScriptFunctions.hpp b/apps/openmw-mp/Script/ScriptFunctions.hpp
index c3428e7f4..17a26d24c 100644
--- a/apps/openmw-mp/Script/ScriptFunctions.hpp
+++ b/apps/openmw-mp/Script/ScriptFunctions.hpp
@@ -10,6 +10,7 @@
 #include <Script/Functions/GUI.hpp>
 #include <Script/Functions/Stats.hpp>
 #include <Script/Functions/Items.hpp>
+#include <Script/Functions/Quests.hpp>
 #include <Script/Functions/Spells.hpp>
 #include <Script/Functions/World.hpp>
 #include <RakNetTypes.h>
@@ -90,6 +91,7 @@ public:
             TRANSLOCATIONFUNCTIONS,
             STATSFUNCTIONS,
             ITEMAPI,
+            QUESTAPI,
             SPELLAPI,
             GUIFUNCTIONS,
             CHARCLASSFUNCTIONS,
diff --git a/apps/openmw/mwmp/LocalPlayer.cpp b/apps/openmw/mwmp/LocalPlayer.cpp
index a1d9e1fcb..ddebe977c 100644
--- a/apps/openmw/mwmp/LocalPlayer.cpp
+++ b/apps/openmw/mwmp/LocalPlayer.cpp
@@ -6,6 +6,7 @@
 #include "../mwmechanics/aitravel.hpp"
 #include <components/esm/esmwriter.hpp>
 #include "../mwbase/environment.hpp"
+#include "../mwbase/journal.hpp"
 #include "../mwstate/statemanagerimp.hpp"
 #include "../mwinput/inputmanagerimp.hpp"
 #include "../mwscript/scriptmanagerimp.hpp"
@@ -27,6 +28,7 @@
 
 #include "LocalPlayer.hpp"
 #include "Networking.hpp"
+#include "WorldController.hpp"
 #include "Main.hpp"
 
 using namespace mwmp;
@@ -643,6 +645,34 @@ void LocalPlayer::addSpells()
         
 }
 
+void LocalPlayer::addJournalItems()
+{
+    for (unsigned int i = 0; i < journalChanges.count; i++)
+    {
+        mwmp::JournalItem journalItem = journalChanges.journalItems[i];
+        
+        if (journalItem.type == JournalItem::ENTRY)
+        {
+            MWWorld::CellStore *ptrCellStore = Main::get().getWorldController()->getCell(journalItem.actorCell);
+
+            if (!ptrCellStore) continue;
+
+            MWWorld::Ptr ptrFound = ptrCellStore->searchExact(journalItem.actorCellRef.mRefID, journalItem.actorCellRef.mRefNum.mIndex);
+
+            if (!ptrFound)
+            {
+                ptrFound = getPlayerPtr();
+            }
+
+            MWBase::Environment::get().getJournal()->addEntry(journalItem.quest, journalItem.index, ptrFound);
+        }
+        else
+        {
+            MWBase::Environment::get().getJournal()->setJournalIndex(journalItem.quest, journalItem.index);
+        }
+    }
+}
+
 void LocalPlayer::removeItems()
 {
     MWWorld::Ptr ptrPlayer = getPlayerPtr();
@@ -998,13 +1028,35 @@ void LocalPlayer::sendSpellRemoval(const ESM::Spell &spell)
     LOG_MESSAGE_SIMPLE(Log::LOG_INFO, "Not implemented.");
 }
 
-void LocalPlayer::sendJournalEntry(const std::string& id, int index, const MWWorld::Ptr& actor)
+void LocalPlayer::sendJournalEntry(const std::string& quest, int index, const MWWorld::Ptr& actor)
 {
+    journalChanges.journalItems.clear();
+
+    mwmp::JournalItem journalItem;
+    journalItem.type = JournalItem::ENTRY;
+    journalItem.quest = quest;
+    journalItem.index = index;
+
+    journalItem.actorCell = *actor.getCell()->getCell();
+    journalItem.actorCellRef.mRefID = actor.getCellRef().getRefId();
+    journalItem.actorCellRef.mRefNum = actor.getCellRef().getRefNum();
+
+    journalChanges.journalItems.push_back(journalItem);
+
     Main::get().getNetworking()->getPlayerPacket(ID_GAME_JOURNAL)->Send(this);
 }
 
-void LocalPlayer::sendJournalIndex(const std::string& id, int index)
+void LocalPlayer::sendJournalIndex(const std::string& quest, int index)
 {
+    journalChanges.journalItems.clear();
+
+    mwmp::JournalItem journalItem;
+    journalItem.type = JournalItem::INDEX;
+    journalItem.quest = quest;
+    journalItem.index = index;
+
+    journalChanges.journalItems.push_back(journalItem);
+
     Main::get().getNetworking()->getPlayerPacket(ID_GAME_JOURNAL)->Send(this);
 }
 
diff --git a/apps/openmw/mwmp/LocalPlayer.hpp b/apps/openmw/mwmp/LocalPlayer.hpp
index a25ee76f1..2627d05ee 100644
--- a/apps/openmw/mwmp/LocalPlayer.hpp
+++ b/apps/openmw/mwmp/LocalPlayer.hpp
@@ -38,6 +38,7 @@ namespace mwmp
 
         void addItems();
         void addSpells();
+        void addJournalItems();
 
         void removeItems();
         void removeSpells();
diff --git a/apps/openmw/mwmp/Networking.cpp b/apps/openmw/mwmp/Networking.cpp
index 590f32609..82bdfeeaf 100644
--- a/apps/openmw/mwmp/Networking.cpp
+++ b/apps/openmw/mwmp/Networking.cpp
@@ -348,6 +348,18 @@ void Networking::processPlayerPacket(RakNet::Packet *packet)
     }
     case ID_GAME_JOURNAL:
     {
+        if (guid == myGuid)
+        {
+            if (packet->length == myPacket->headerSize())
+            {
+                // Entire journal cannot currently be requested from players
+            }
+            else
+            {
+                myPacket->Packet(&bsIn, getLocalPlayer(), false);
+                getLocalPlayer()->addJournalItems();
+            }
+        }
         break;
     }
     case ID_GAME_ATTACK:
diff --git a/components/openmw-mp/Base/BasePlayer.hpp b/components/openmw-mp/Base/BasePlayer.hpp
index ac13689a8..94ce473be 100644
--- a/components/openmw-mp/Base/BasePlayer.hpp
+++ b/components/openmw-mp/Base/BasePlayer.hpp
@@ -55,6 +55,10 @@ namespace mwmp
             ENTRY = 0,
             INDEX = 1
         };
+
+        ESM::Cell actorCell;
+        ESM::CellRef actorCellRef;
+
         int type; // 0 - An entire entry, 1 - An index
     };
 
@@ -209,6 +213,7 @@ namespace mwmp
         double hour;
         InventoryChanges inventoryChanges;
         SpellbookChanges spellbookChanges;
+        JournalChanges journalChanges;
         bool consoleAllowed;
         bool ignorePosPacket;
         ESM::ActiveSpells activeSpells;
diff --git a/components/openmw-mp/Packets/Player/PacketJournal.cpp b/components/openmw-mp/Packets/Player/PacketJournal.cpp
index b24e4dcd3..8d737839a 100644
--- a/components/openmw-mp/Packets/Player/PacketJournal.cpp
+++ b/components/openmw-mp/Packets/Player/PacketJournal.cpp
@@ -12,4 +12,36 @@ PacketJournal::PacketJournal(RakNet::RakPeerInterface *peer) : PlayerPacket(peer
 void PacketJournal::Packet(RakNet::BitStream *bs, BasePlayer *player, bool send)
 {
     PlayerPacket::Packet(bs, player, send);
+
+    if (!send)
+        player->journalChanges.journalItems.clear();
+    else
+        player->journalChanges.count = (unsigned int)(player->journalChanges.journalItems.size());
+
+    RW(player->journalChanges.count, send);
+
+    for (unsigned int i = 0; i < player->journalChanges.count; i++)
+    {
+        JournalItem journalItem;
+
+        if (send)
+            journalItem = player->journalChanges.journalItems[i];
+
+        RW(journalItem.type, send);
+        RW(journalItem.quest, send);
+        RW(journalItem.index, send);
+
+        if (journalItem.type == JournalItem::ENTRY)
+        {
+            RW(journalItem.actorCellRef.mRefID, send);
+            RW(journalItem.actorCellRef.mRefNum.mIndex, send);
+            RW(journalItem.actorCell.mData.mFlags, send);
+            RW(journalItem.actorCell.mData.mX, send);
+            RW(journalItem.actorCell.mData.mY, send);
+            RW(journalItem.actorCell.mName, send);
+        }
+
+        if (!send)
+            player->journalChanges.journalItems.push_back(journalItem);
+    }
 }