my script is firing a function when it shouldn't?

Sorry for the bad title but i didn't know exactly what to write. i am currently making a script which allows the player to pick up a model by pressing a key if the model is currently not being held by some one else and they are not currently holding a model. i have that part of the script sorted but i need to track if the player should be grabbing or dropping a model when they press "E" i have a system in place changing the string name of an index to determine if the player should send a request to the server to drop the model or pick it up but the code picks up the model and then drops it with one single key press, no errors are displayed and i think its a matter of variable placement and timing, please help. (i haven't made the player physically pick up the models yet i'm currently just working on ownership and making sure things work properly)

this is what get printed out:

 Model successfully claimed!
  canDrop
  Model successfully unclaimed!
  canGrab

Client:

-- Remote variables
local replicatedStorage = game:GetService("ReplicatedStorage")
local remoteEvents = replicatedStorage.RemoteEvents
local remoteFunctions = replicatedStorage.RemoteFunctions
local reqGrab = remoteFunctions.RequestGrab
local stpGrab = remoteEvents.StopGrab
-- Variables
local player = game:GetService("Players").LocalPlayer
local mouse = player:GetMouse()
local Uis = game:GetService("UserInputService")
local character = player.Character or player.CharacterAdded:Wait()
local root = character:FindFirstChild("HumanoidRootPart")
local canRequestGrab = {["grabOrDrop"] = "canGrab"}
local part = nil
-- Requesting to grab or stop grab
Uis.InputBegan:Connect(function(inputObject)
    if inputObject.KeyCode == Enum.KeyCode.E then
        -- If grab
        if canRequestGrab["grabOrDrop"] == "canGrab" then
            if mouse.Target ~= nil then 
                local model = mouse.Target:FindFirstAncestorWhichIsA("Model")
                if model then
                    local primaryPart = model:FindFirstChild("PrimaryPart")
                    if primaryPart then
                        local targetFilter = primaryPart
                        if mouse.Target ~= primaryPart then 
                            part = mouse.Target
                            reqGrab:InvokeServer(part)
                            wait(1)                         
                            canRequestGrab["grabOrDrop"] = "canDrop"
                            print(canRequestGrab["grabOrDrop"])                 
                        end
                    end 
                end 
            end
        end
        -- If Drop
        if canRequestGrab["grabOrDrop"] == "canDrop" then
            stpGrab:FireServer(part)
            part = nil
            wait(1)
            canRequestGrab["grabOrDrop"] = "canGrab"
            print(canRequestGrab["grabOrDrop"])
        end 
    end
end)

Server:

-- Remote variables
local replicatedStorage = game:GetService("ReplicatedStorage")
local remoteEvents = replicatedStorage.RemoteEvents
local remoteFunctions = replicatedStorage.RemoteFunctions
local reqGrab = remoteFunctions.RequestGrab
local stpGrab = remoteEvents.StopGrab
-- Variables
local playerParts = {}
local collectionService = game:GetService("CollectionService")
local tag = "PickUp"
-- Grabs part
function requestGrab(player, part)
    if playerParts[part] ~= nil then return false end
    local model = part:FindFirstAncestorWhichIsA("Model")
    if model then
        if collectionService:HasTag(model, tag) then
            playerParts[part] = player
            print("Model successfully claimed!")
        end
    end 
end
-- Stops grab
function stopGrab(player, part)
    if playerParts[part] ~= player then return false end
    playerParts[part] = nil 
    print("Model successfully unclaimed!")
end
-- Requests a grab
reqGrab.OnServerInvoke = (function(player, part)
    requestGrab(player, part)
end)
-- Requests to stop a grab
stpGrab.OnServerEvent:Connect(function(player, part)
    stopGrab(player, part)
end)