You're killing me Rami
Why are we prefixing vars inside function?
Use yoda conditions we must
Make arrays readable
Let's not return twice here.
Removed duplicate array

Can’t see any reason to duplicate this array. Also don’t see any reason
for the else statements here.
Removed extract, improved insult building
Cleaned up plugin header
Removed function exists check

This was failing under certain situations and causing the raw short
code to be displayed. I’ve prefixed the function and named it so that
it’s fairly unlikely to run into conflicts. You might want to wrap the
whole plugin in a class and break up this function a bit.
Merge pull request #1 from robneu/master

If We're Insulting People, Let's Do It Right