fixed build

MUnique · Sep 24, 2024 · c4770eb · c4770eb
1 parent 94b44e8
commit c4770eb
Show file tree

Hide file tree

Showing 4 changed files with 33 additions and 4 deletions.
diff --git a/docs/Packets/C3-16-ExperienceGainedExtended_by-server.md b/docs/Packets/C3-16-ExperienceGainedExtended_by-server.md
@@ -27,6 +27,7 @@ Defines the result and type of experience which is added.
 
 | Value | Name | Description |
 |-------|------|-------------|
+| 0 | Undefined | Undefined, no experience is added. |
 | 1 | Normal | The normal experience is added. |
 | 2 | Master | The master experience is added. |
 | 0x10 | MaxLevelReached | The maximum level has been reached, no experience is added. |

diff --git a/src/GameServer/RemoteView/Character/AddExperienceExtendedPlugIn.cs b/src/GameServer/RemoteView/Character/AddExperienceExtendedPlugIn.cs
@@ -1,13 +1,17 @@
-using System.Runtime.InteropServices;
+// <copyright file="AddExperienceExtendedPlugIn.cs" company="MUnique">
+// Licensed under the MIT License. See LICENSE file in the project root for full license information.
+// </copyright>
+
+namespace MUnique.OpenMU.GameServer.RemoteView.Character;
+
+using System.Runtime.InteropServices;
 using MUnique.OpenMU.GameLogic;
 using MUnique.OpenMU.GameLogic.Views;
 using MUnique.OpenMU.GameLogic.Views.Character;
 using MUnique.OpenMU.Network.Packets.ServerToClient;
 using MUnique.OpenMU.Network.PlugIns;
 using MUnique.OpenMU.PlugIns;
 
-namespace MUnique.OpenMU.GameServer.RemoteView.Character;
-
 /// <summary>
 /// The extended implementation of the <see cref="IAddExperiencePlugIn"/> which is forwarding everything to the game client with specific data packets.
 /// </summary>
@@ -41,11 +45,25 @@ public async ValueTask AddExperienceAsync(int exp, IAttackable? obj, ExperienceT
             killerId = ViewExtensions.ConstantPlayerId;
         }
 
-        await this._player.Connection.SendExperienceGainedExtendedAsync((byte)experienceType,
+        await this._player.Connection.SendExperienceGainedExtendedAsync(
+                Convert(experienceType),
                 (uint)exp,
                 (uint)damage,
                 killedId,
                 killerId)
             .ConfigureAwait(false);
     }
+
+    private static ExperienceGainedExtended.AddResult Convert(ExperienceType experienceType)
+    {
+        return experienceType switch
+        {
+            ExperienceType.Normal => ExperienceGainedExtended.AddResult.Normal,
+            ExperienceType.Master => ExperienceGainedExtended.AddResult.Master,
+            ExperienceType.MaxLevelReached => ExperienceGainedExtended.AddResult.MaxLevelReached,
+            ExperienceType.MaxMasterLevelReached => ExperienceGainedExtended.AddResult.MaxMasterLevelReached,
+            ExperienceType.MonsterLevelTooLowForMasterExperience => ExperienceGainedExtended.AddResult.MonsterLevelTooLowForMasterExperience,
+            _ => ExperienceGainedExtended.AddResult.Undefined
+        };
+    }
 }
diff --git a/src/Network/Packets/ServerToClient/ServerToClientPackets.cs b/src/Network/Packets/ServerToClient/ServerToClientPackets.cs
@@ -6579,6 +6579,11 @@ public readonly struct ExperienceGainedExtended
     /// </summary>
     public enum AddResult
     {
+        /// <summary>
+        /// Undefined, no experience is added.
+        /// </summary>
+            Undefined = 0,
+
         /// <summary>
         /// The normal experience is added.
         /// </summary>

diff --git a/src/Network/Packets/ServerToClient/ServerToClientPackets.xml b/src/Network/Packets/ServerToClient/ServerToClientPackets.xml
@@ -2392,6 +2392,11 @@
           <Name>AddResult</Name>
           <Description>Defines the result and type of experience which is added.</Description>
           <Values>
+            <EnumValue>
+              <Name>Undefined</Name>
+              <Description>Undefined, no experience is added.</Description>
+              <Value>0</Value>
+            </EnumValue>
             <EnumValue>
               <Name>Normal</Name>
               <Description>The normal experience is added.</Description>