From 0fb725ea611e7788a05bcb39bbf65ccdfe7ae505 Mon Sep 17 00:00:00 2001
From: mmd-osm <mmd.osm@gmail.com>
Date: Sun, 6 Sep 2020 22:11:23 +0200
Subject: [PATCH] API User details JSON format

---
 app/controllers/api/users_controller.rb       | 20 ++++-
 app/views/api/users/_user.json.jbuilder       | 73 +++++++++++++++++++
 .../{_user.builder => _user.xml.builder}      |  0
 app/views/api/users/index.json.jbuilder       |  3 +
 .../{index.builder => index.xml.builder}      |  0
 app/views/api/users/show.json.jbuilder        |  3 +
 .../users/{show.builder => show.xml.builder}  |  0
 7 files changed, 96 insertions(+), 3 deletions(-)
 create mode 100644 app/views/api/users/_user.json.jbuilder
 rename app/views/api/users/{_user.builder => _user.xml.builder} (100%)
 create mode 100644 app/views/api/users/index.json.jbuilder
 rename app/views/api/users/{index.builder => index.xml.builder} (100%)
 create mode 100644 app/views/api/users/show.json.jbuilder
 rename app/views/api/users/{show.builder => show.xml.builder} (100%)

diff --git a/app/controllers/api/users_controller.rb b/app/controllers/api/users_controller.rb
index d97e8774e..b4a2efc7c 100644
--- a/app/controllers/api/users_controller.rb
+++ b/app/controllers/api/users_controller.rb
@@ -9,9 +9,15 @@ module Api
     around_action :api_call_handle_error
     before_action :lookup_user_by_id, :only => [:show]
 
+    before_action :set_request_formats, :except => [:gpx_files]
+
     def show
       if @user.visible?
-        render :content_type => "text/xml"
+        # Render the result
+        respond_to do |format|
+          format.xml
+          format.json
+        end
       else
         head :gone
       end
@@ -19,7 +25,11 @@ module Api
 
     def details
       @user = current_user
-      render :action => :show, :content_type => "text/xml"
+      # Render the result
+      respond_to do |format|
+        format.xml { render :show }
+        format.json { render :show }
+      end
     end
 
     def index
@@ -31,7 +41,11 @@ module Api
 
       @users = User.visible.find(ids)
 
-      render :content_type => "text/xml"
+      # Render the result
+      respond_to do |format|
+        format.xml
+        format.json
+      end
     end
 
     def gpx_files
diff --git a/app/views/api/users/_user.json.jbuilder b/app/views/api/users/_user.json.jbuilder
new file mode 100644
index 000000000..9629e8fa5
--- /dev/null
+++ b/app/views/api/users/_user.json.jbuilder
@@ -0,0 +1,73 @@
+json.user do
+  json.id user.id
+  json.display_name user.display_name
+  json.account_created user.creation_time.xmlschema
+  json.description user.description if user.description
+
+  if current_user && current_user == user
+    json.contributor_terms do
+      json.agreed user.terms_agreed.present?
+      json.pd user.consider_pd
+    end
+  else
+    json.contributor_terms do
+      json.agreed user.terms_agreed.present?
+    end
+  end
+
+  json.img do
+    json.href user_image_url(user) if user.avatar.attached? || user.image_use_gravatar
+  end
+
+  json.roles do
+    json.array! user.roles.map(&:role)
+  end
+
+  json.changesets do
+    json.count user.changesets.size
+  end
+
+  json.traces do
+    json.count user.traces.size
+  end
+
+  json.blocks do
+    json.received do
+      json.count user.blocks.size
+      json.active user.blocks.active.size
+    end
+
+    if user.moderator?
+      json.issued do
+        json.count user.blocks_created.size
+        json.active user.blocks_created.active.size
+      end
+    end
+  end
+
+  if current_user && current_user == user
+    if user.home_lat && user.home_lon
+      json.home do
+        json.lat user.home_lat
+        json.lon user.home_lon
+        json.zoom user.home_zoom
+      end
+    end
+
+    if user.languages
+      json.languages do
+        json.array! user.languages.split(",")
+      end
+    end
+
+    json.messages do
+      json.received do
+        json.count user.messages.size
+        json.unread user.new_messages.size
+      end
+      json.sent do
+        json.count user.sent_messages.size
+      end
+    end
+  end
+end
diff --git a/app/views/api/users/_user.builder b/app/views/api/users/_user.xml.builder
similarity index 100%
rename from app/views/api/users/_user.builder
rename to app/views/api/users/_user.xml.builder
diff --git a/app/views/api/users/index.json.jbuilder b/app/views/api/users/index.json.jbuilder
new file mode 100644
index 000000000..efc905cfe
--- /dev/null
+++ b/app/views/api/users/index.json.jbuilder
@@ -0,0 +1,3 @@
+json.users(@users) do |user|
+  json.partial! user
+end
diff --git a/app/views/api/users/index.builder b/app/views/api/users/index.xml.builder
similarity index 100%
rename from app/views/api/users/index.builder
rename to app/views/api/users/index.xml.builder
diff --git a/app/views/api/users/show.json.jbuilder b/app/views/api/users/show.json.jbuilder
new file mode 100644
index 000000000..8ef92752f
--- /dev/null
+++ b/app/views/api/users/show.json.jbuilder
@@ -0,0 +1,3 @@
+json.users([@user]) do |user|
+  json.partial! user
+end
diff --git a/app/views/api/users/show.builder b/app/views/api/users/show.xml.builder
similarity index 100%
rename from app/views/api/users/show.builder
rename to app/views/api/users/show.xml.builder
-- 
2.39.5