I would just split it up:

````
text_facts = request.options[:facts_data] || request.options[:facts]
return if text_facts.nil?
````

Another interesting thing I just noticed about this whole method is that text_facts is a misnomer. It is expected that sometimes those are already an instance of `Puppet::Node::Facts` (!)