Why would you say that?
I don't think I made any mistakes. ^^^
!%all is 0
if ( !%len = getWordCount( %opened ) )// (sets %len to a value, then checks it's opposite boolean variant)
if ( !( %len = getWordCount( %opened ) ) )
You are comparing a client object ID and a player object ID, you are also using a string comparison for two integers, which isn't really a big deal but using "!=" instead would be faster ;D