no need in "no null" check. the lines below check the custom field content anyway, so you are essentially checking the same thing twice.