Refs #3809 - Remove rubocop TODOs

Removed the following TODOs so that cops for these will run from now on:

Lint/AmbiguousOperator, DefEndAlignment, DeprecatedClassMethods
EnsureReturn, RequireParentheses, Void, BlockAlignment, EndAlignment,
UselessAccessModifier,

Style/IndentArray, IndentHash, Tab, UnlessElse, UnneededPercentQ, UnneededPercentX, RedundantReturn

Rails/Validation
shouldnt it be written as:
```ruby
conditions =  if user.admin?
               { }
   else
               sanitize_sql_for_conditions([" (taxonomies.id in (?))", user.location_and_child_ids])
   end