I don't think hiding an assignment inside a ternary operator makes this very readable. This can be rewritten as

````
facts_data = request.options[:facts_data] ? request.options[:facts_data] : CGI.unescape(request.options[:facts])
````

This also removes the need for the assignment-in-passing to `text_facts` above.