Hmm. skumt fel du har, eftersom det inte ar non skillnad mellan Login och Logout i princip.
Har du testat att skriva if satserna i olika ordning? D.v.s. if (action == 'logout') forst.
Jag foreslar att du anvender return false; och return true;.
Vilken ordning det ska vara och sa, vet jag inte, men testa dig fram.