<p>In app/controllers/changeset_controller.rb:</p>
<pre style='color:#555'>> @@ -36,8 +41,10 @@ def create
>    # Return XML giving the basic info about the changeset. Does not
>    # return anything about the nodes, ways and relations in the changeset.
>    def read
> -    changeset = Changeset.find(params[:id])
> -    render :text => changeset.to_xml.to_s, :content_type => "text/xml"
> +    @changeset = Changeset.find(params[:id])
> +    @comments = params['include_discussion'].presence ? @changeset.comments.includes(:author) : false
</pre>
<p>That looks bogus - you have a ternary operator with one branch returning an array and the other returning a boolean. Suggest the else should return an empty array, or perhaps more likely <code>nil</code>. In fact I'd suggest writing it in a much more readable way as:</p>

<pre><code>@comments = @changeset.comments.includes(:author) if params[:include_discussion].presence
</code></pre>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href="https://github.com/openstreetmap/openstreetmap-website/pull/772/files#r15401889">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/1419053__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcyMTkxNjM1OCwiZGF0YSI6eyJpZCI6MzU4NjU3NjJ9fQ==--d027569730981123d32bf29a19e58ec403be80dd.gif" width="1" /></p>