The first code snippet should be in a regular server script parented to your ClickDetector. The second one is what you should replace your current LocalScript with. I’m not sure where you put yours, but I ran mine from StarterPlayerScripts.
Is the model what you click to give the tool? If so, then no. What I meant was put the tool you what to give the Player in ServerStorage. The tool should only be in ServerStorage, and in workspace the model. You can make it a tool if you want, but you don’t have to.
Quick note: Your ClickDetector should not be parented directly under your model but rather inside of a part as a child of said model.
I’m starting to think that. It’s weird, though, just about every topic I reply to, the creator goes inactive after I say something. Just a coincidence, but strange.