[#5698] Removed unneeded invalidate_bin_counts

We don't need to invalidate the bin counts when a bin is deleted,
because that bin will no longer be queried, and won't affect other
bin counts.  (The Bin instance will be deleted, and those are what
control what data is pulled from Globals.bin_data, so leaving the
count data for that bin in Globals won't cause any harm.)
[#5698] Moved calls to invalidate_bin_counts to bottom of methods to ensure they don't happen until update is successful

Signed-off-by: Cory Johns <johnsca@geek.net>