It's ok, I guess, but I'll move it to the links section. What system did you test on? Only a very few shells will allow like:
if [ this == that ]
most shells that support the bourne syntax do allow:
if [ this = that ]