The problem isn't that the rest of the For...Next loop isn't running. The problem is that variables only live as long as the procedures they live in. Specifically "intjdoecount = intjdoecount + 1" doesn't do anything because the variable is destroyed at "End Sub" of that procedure. So, each iteration of the For...Next loop means that the John Doe row is being copied to row 1, overwriting the previous values copied.
The code beclow should work for you. Notice that I got rid of "Activate" commands. There is rarely a need to activate something to work with it in code. You could run this code with any sheet activated and it would still work as desired.
A further note:
If you click "Debug" -> "Step Into" (or just press F8), you can watch the code process one line at a time. You will see that your code does, in fact, go back to the For...Next loop after running through the jdoe() subroutine. Walking through the steps like this is a great way to debug your code and see what is actually happening.
Bookmarks