From 6cc4f4506b9a6612f57dd33d79e933d181a02713 Mon Sep 17 00:00:00 2001 From: Andy Allan Date: Wed, 7 Sep 2016 11:02:20 +0100 Subject: [PATCH] Use .present? instead of double negative to check if terms_agreed timestamp is set. --- app/views/user/api_read.builder | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/app/views/user/api_read.builder b/app/views/user/api_read.builder index 2b304f53c..55775df9e 100644 --- a/app/views/user/api_read.builder +++ b/app/views/user/api_read.builder @@ -7,10 +7,10 @@ xml.osm("version" => API_VERSION, "generator" => GENERATOR) do xml.tag! "description", @this_user.description end if @user && @user == @this_user - xml.tag! "contributor-terms", :agreed => !!@this_user.terms_agreed, + xml.tag! "contributor-terms", :agreed => @this_user.terms_agreed.present?, :pd => @this_user.consider_pd else - xml.tag! "contributor-terms", :agreed => !!@this_user.terms_agreed + xml.tag! "contributor-terms", :agreed => @this_user.terms_agreed.present? end if @this_user.image.file? or @this_user.image_use_gravatar xml.tag! "img", :href => user_image_url(@this_user, :size => 256) -- 2.39.5