refactor: Expose guild actions

planetarium · Dec 18, 2024 · 82204ca · 82204ca
1 parent a85e174
commit 82204ca
Show file tree

Hide file tree

Showing 8 changed files with 17 additions and 22 deletions.
diff --git a/Lib9c/Action/Guild/BanGuildMember.cs b/Lib9c/Action/Guild/BanGuildMember.cs
@@ -10,7 +10,7 @@
 namespace Nekoyume.Action.Guild
 {
     // TODO(GUILD-FEATURE): Enable again when Guild features are enabled.
-    // [ActionType(TypeIdentifier)]
+    [ActionType(TypeIdentifier)]
     public class BanGuildMember : ActionBase
     {
         public const string TypeIdentifier = "ban_guild_member";

diff --git a/Lib9c/Action/Guild/JoinGuild.cs b/Lib9c/Action/Guild/JoinGuild.cs
@@ -10,7 +10,7 @@
 namespace Nekoyume.Action.Guild
 {
     // TODO(GUILD-FEATURE): Enable again when Guild features are enabled.
-    // [ActionType(TypeIdentifier)]
+    [ActionType(TypeIdentifier)]
     public class JoinGuild : ActionBase
     {
         public const string TypeIdentifier = "join_guild";

diff --git a/Lib9c/Action/Guild/MakeGuild.cs b/Lib9c/Action/Guild/MakeGuild.cs
@@ -6,7 +6,9 @@
 using Nekoyume.Extensions;
 using Nekoyume.Model.Guild;
 using Nekoyume.Module.Guild;
+using Nekoyume.Module.ValidatorDelegation;
 using Nekoyume.TypedAddress;
+using Nekoyume.ValidatorDelegation;
 
 namespace Nekoyume.Action.Guild
 {
@@ -46,30 +48,23 @@ rawValues is not Dictionary values ||
 
         // TODO: Replace this with ExecutePublic when to deliver features to users.
         public override IWorld Execute(IActionContext context)
-        {
-            var world = ExecutePublic(context);
-
-            if (context.Signer != GuildConfig.PlanetariumGuildOwner)
-            {
-                throw new InvalidOperationException(
-                    $"This action is not allowed for {context.Signer}.");
-            }
-
-            return world;
-        }
-
-        public IWorld ExecutePublic(IActionContext context)
         {
             GasTracer.UseGas(1);
 
             var world = context.PreviousState;
             var random = context.GetRandom();
-
             var repository = new GuildRepository(world, context);
             var guildAddress = new GuildAddress(random.GenerateAddress());
             var validatorAddress = ValidatorAddress;
 
             repository.MakeGuild(guildAddress, validatorAddress);
+
+            // if (context.Signer != GuildConfig.PlanetariumGuildOwner)
+            // {
+            //     throw new InvalidOperationException(
+            //         $"This action is not allowed for {context.Signer}.");
+            // }
+
             return repository.World;
         }
     }

diff --git a/Lib9c/Action/Guild/MoveGuild.cs b/Lib9c/Action/Guild/MoveGuild.cs
@@ -10,7 +10,7 @@
 namespace Nekoyume.Action.Guild
 {
     // TODO(GUILD-FEATURE): Enable again when Guild features are enabled.
-    // [ActionType(TypeIdentifier)]
+    [ActionType(TypeIdentifier)]
     public class MoveGuild : ActionBase
     {
         public const string TypeIdentifier = "move_guild";

diff --git a/Lib9c/Action/Guild/QuitGuild.cs b/Lib9c/Action/Guild/QuitGuild.cs
@@ -9,7 +9,7 @@
 namespace Nekoyume.Action.Guild
 {
     // TODO(GUILD-FEATURE): Enable again when Guild features are enabled.
-    // [ActionType(TypeIdentifier)]
+    [ActionType(TypeIdentifier)]
     public class QuitGuild : ActionBase
     {
         public const string TypeIdentifier = "quit_guild";

diff --git a/Lib9c/Action/Guild/RemoveGuild.cs b/Lib9c/Action/Guild/RemoveGuild.cs
@@ -12,7 +12,7 @@ namespace Nekoyume.Action.Guild
     /// An action to remove the guild.
     /// </summary>
     // TODO(GUILD-FEATURE): Enable again when Guild features are enabled.
-    // [ActionType(TypeIdentifier)]
+    [ActionType(TypeIdentifier)]
     public class RemoveGuild : ActionBase
     {
         public const string TypeIdentifier = "remove_guild";

diff --git a/Lib9c/Action/Guild/UnbanGuildMember.cs b/Lib9c/Action/Guild/UnbanGuildMember.cs
@@ -11,7 +11,7 @@
 namespace Nekoyume.Action.Guild
 {
     // TODO(GUILD-FEATURE): Enable again when Guild features are enabled.
-    // [ActionType(TypeIdentifier)]
+    [ActionType(TypeIdentifier)]
     public class UnbanGuildMember : ActionBase
     {
         public const string TypeIdentifier = "unban_guild_member";

diff --git a/Lib9c/Model/Guild/GuildRejoinCooldown.cs b/Lib9c/Model/Guild/GuildRejoinCooldown.cs
@@ -16,7 +16,7 @@ public GuildRejoinCooldown(AgentAddress agentAddress, long quitHeight)
         }
 
         public GuildRejoinCooldown(AgentAddress agentAddress, IValue bencoded)
-            : this(agentAddress, ((Integer)bencoded))
+            : this(agentAddress, (Integer)bencoded)
         {
         }
 
@@ -40,7 +40,7 @@ public long Cooldown(long currentHeight)
         }
 
         public bool Equals(GuildRejoinCooldown? other)
-        { 
+        {
             if (ReferenceEquals(null, other)) return false;
             if (ReferenceEquals(this, other)) return true;
             return AgentAddress.Equals(other.AgentAddress)