Still can't see your code.
However, your functions are doing too much. ReadEncoder() should do exactly that and NOTHING more.
ReadEncoderAndDoABunchOfOtherCrap() would read the encoder and do a bunch of other crap. Of course, if you really had a function like that, I'd just hit the back button and move onto the next post.
You have very few Serial.print() statements to learn what the code is really doing.
Besides, it's hard to see the function of an encoder in a post titled "Kitchen Timer with Relay and Magnetic Switch", so some comments in the code would be helpful.