Merge remote-tracking branch 'upstream/pull/1704'

[rails.git] / app / views / user / api_read.builder

diff --git a/app/views/user/api_read.builder b/app/views/user/api_read.builder
index 71bf292081e0dd2f3dfdd01cdc04cce7c07b70ab..fe5af4bcf6cfa547c108fec36eb9270a4cdfe0a4 100644 (file)
--- a/app/views/user/api_read.builder
+++ b/app/views/user/api_read.builder
@@ -3,18 +3,14 @@ xml.osm("version" => API_VERSION, "generator" => GENERATOR) do
   xml.tag! "user", :id => @this_user.id,
                    :display_name => @this_user.display_name,
                    :account_created => @this_user.creation_time.xmlschema do
   xml.tag! "user", :id => @this_user.id,
                    :display_name => @this_user.display_name,
                    :account_created => @this_user.creation_time.xmlschema do

-    if @this_user.description
-      xml.tag! "description", @this_user.description
-    end
-    if @user && @user == @this_user
-      xml.tag! "contributor-terms", :agreed => !!@this_user.terms_agreed,
-                                    :pd => !!@this_user.consider_pd
+    xml.tag! "description", @this_user.description if @this_user.description
+    if current_user && current_user == @this_user
+      xml.tag! "contributor-terms", :agreed => @this_user.terms_agreed.present?,
+                                    :pd => @this_user.consider_pd

     else
     else

-      xml.tag! "contributor-terms", :agreed => !!@this_user.terms_agreed
-    end
-    if @this_user.image.file? or @this_user.image_use_gravatar
-      xml.tag! "img", :href => user_image_url(@this_user, :size => 256)
+      xml.tag! "contributor-terms", :agreed => @this_user.terms_agreed.present?

     end
     end

+    xml.tag! "img", :href => user_image_url(@this_user, :size => 256) if @this_user.image.file? || @this_user.image_use_gravatar

     xml.tag! "roles" do
       @this_user.roles.each do |role|
         xml.tag! role.role
     xml.tag! "roles" do
       @this_user.roles.each do |role|
         xml.tag! role.role


@@ -30,8 +26,8 @@ xml.osm("version" => API_VERSION, "generator" => GENERATOR) do
                            :active => @this_user.blocks_created.active.size
       end
     end
                            :active => @this_user.blocks_created.active.size
       end
     end

-    if @user && @user == @this_user
-      if @this_user.home_lat and @this_user.home_lon
+    if current_user && current_user == @this_user
+      if @this_user.home_lat && @this_user.home_lon

         xml.tag! "home", :lat => @this_user.home_lat,
                          :lon => @this_user.home_lon,
                          :zoom => @this_user.home_zoom
         xml.tag! "home", :lat => @this_user.home_lat,
                          :lon => @this_user.home_lon,
                          :zoom => @this_user.home_zoom


@@ -41,6 +37,11 @@ xml.osm("version" => API_VERSION, "generator" => GENERATOR) do
           @this_user.languages.split(",") { |lang| xml.tag! "lang", lang }
         end
       end
           @this_user.languages.split(",") { |lang| xml.tag! "lang", lang }
         end
       end

+      xml.tag! "messages" do
+        xml.tag! "received", :count => @this_user.messages.size,
+                             :unread => @this_user.new_messages.size
+        xml.tag! "sent", :count => @this_user.sent_messages.size
+      end

     end
   end
 end
     end
   end
 end