Don't include milliseconds in JSON API responses (#19354).

git-svn-id: http://svn.redmine.org/redmine/trunk@14101 e93f8b46-1217-0410-a6f0-8f06a7374b81

diff --git a/lib/redmine/views/builders/structure.rb b/lib/redmine/views/builders/structure.rb
index a420dc497732844698a318ce3c23805584dc682e..b11e255ec507181b6ae1ca0b602bc94570bb51e9 100644 (file)
--- a/lib/redmine/views/builders/structure.rb
+++ b/lib/redmine/views/builders/structure.rb
@@ -37,6 +37,16 @@ module Redmine
           @struct.last.merge!(options) if options
         end
 
+        def encode_value(value)
+          if value.is_a?(Time)
+            # Rails uses a global setting to format JSON times
+            # Don't rely on it for the API as it could have been changed
+            value.xmlschema(0)
+          else
+            value
+          end
+        end
+
         def method_missing(sym, *args, &block)
           if args.any?
             if args.first.is_a?(Hash)
@@ -46,14 +56,15 @@ module Redmine
                 @struct.last[sym] = args.first
               end
             else
+              value = encode_value(args.first)
               if @struct.last.is_a?(Array)
                 if args.size == 1 && !block_given?
-                  @struct.last << args.first
+                  @struct.last << value
                 else
-                  @struct.last << (args.last || {}).merge(:value => args.first)
+                  @struct.last << (args.last || {}).merge(:value => value)
                 end
               else
-                @struct.last[sym] = args.first
+                @struct.last[sym] = value
               end
             end
           end

diff --git a/test/integration/api_test/api_test.rb b/test/integration/api_test/api_test.rb
index 1387fa9058a131fdde2ac3f66a96135719b28676..b81f8e0ad16c972976b588feb298f5523ae2f5e9 100644 (file)
--- a/test/integration/api_test/api_test.rb
+++ b/test/integration/api_test/api_test.rb
@@ -34,4 +34,14 @@ class Redmine::ApiTest::ApiTest < Redmine::ApiTest::Base
   ensure
     ActionController::Base.allow_forgery_protection = false
   end
-end
\ No newline at end of file
+
+  def test_json_datetime_format
+    get '/users/1.json', {}, credentials('admin')
+    assert_include '"created_on":"2006-07-19T17:12:21Z"', response.body
+  end
+
+  def test_xml_datetime_format
+    get '/users/1.xml', {}, credentials('admin')
+    assert_include '<created_on>2006-07-19T17:12:21Z</created_on>', response.body
+  end
+end