Would this be better written as

    remove|upgrade|deconfigure)
      blah
      blah
      if [[ $1 != "uprade" ]]
      then
        remove_logging
      fi
      blah
    ;;

?