{ :joins => :fact_values, :conditions => conditions } unless ufs.empty? 

would be a bit better, IMO.