I think the first if statement should be && and not ||. If user is op AND you are op, versus If user is op OR you are op.. otherwise it still tries to work if you are not op.
There are a few other things beyond what napa and jethro are suggesting (which are very valid), but not relevant to functionality.
Ask away if you need help adding what they suggested. Keep up the work!