diff --git a/Francesco.Recipes.World/Services/Instruction/IInstructionService.cs b/Francesco.Recipes.World/Services/Instruction/IInstructionService.cs
index fa514a1..41bfe0f 100644
--- a/Francesco.Recipes.World/Services/Instruction/IInstructionService.cs
+++ b/Francesco.Recipes.World/Services/Instruction/IInstructionService.cs
@@ -2,8 +2,8 @@
 {
     public interface IInstructionService
     {
-        Task MoveInstructionUpAsync(Guid instructionId);
+        Task MoveInstructionUpAsync(Guid recipeId, Guid instructionId);
 
-        Task MoveInstructionDownAsync(Guid instructionId);
+        Task MoveInstructionDownAsync(Guid recipeId, Guid instructionId);
     }
 }
diff --git a/Francesco.Recipes.World/Services/Instruction/InstructionService.cs b/Francesco.Recipes.World/Services/Instruction/InstructionService.cs
index c7a4acc..ed95792 100644
--- a/Francesco.Recipes.World/Services/Instruction/InstructionService.cs
+++ b/Francesco.Recipes.World/Services/Instruction/InstructionService.cs
@@ -11,15 +11,15 @@ namespace Francesco.Recipes.World.Services.Instruction
             _instructionRepository = instructionRepository;
         }
 
-        public Task MoveInstructionUpAsync(Guid instructionId)
-          => MoveInstructionAsync(instructionId, moveUp: true);
+        public Task MoveInstructionUpAsync(Guid recipeId, Guid instructionId)
+          => MoveInstructionAsync(recipeId, instructionId, moveUp: true);
 
-        public Task MoveInstructionDownAsync(Guid instructionId)
-            => MoveInstructionAsync(instructionId, moveUp: false);
+        public Task MoveInstructionDownAsync(Guid recipeId, Guid instructionId)
+            => MoveInstructionAsync(recipeId, instructionId, moveUp: false);
 
-        private async Task MoveInstructionAsync(Guid instructionId, bool moveUp)
+        private async Task MoveInstructionAsync(Guid recipeId, Guid instructionId, bool moveUp)
         {
-            var instructions = await _instructionRepository.GetInstructionsOfRecipeAsync(instructionId);
+            var instructions = await _instructionRepository.GetInstructionsOfRecipeAsync(recipeId);
 
             var instruction = instructions.FirstOrDefault(i => i.Id == instructionId);
 
@@ -36,7 +36,7 @@ namespace Francesco.Recipes.World.Services.Instruction
                 return;
             }
 
-            var targetNumber = moveUp ? instruction.Number + 1 : instruction.Number - 1;
+            var targetNumber = moveUp ? instruction.Number - 1 : instruction.Number + 1;
             var neighbor = instructions.FirstOrDefault(i => i.Number == targetNumber);
 
             if (neighbor != null)