Again, you don't need an else here... Also, MCsessionid could be nil I think... This needs to be checked against the original unrefactored code.